C言語 区切り分割ができない。

Question

前提・実現したいこと

私は、文字を入力して back space を入力したら文字を削除しようと思っています。
なので、back space を入力したら文字列の未尾を削除しようと思い、strtokで実現しようと思いましたが、コンパイルはできるものの文字が表示されず、すぐウィンドウズが閉じてしまいます。

これは、","で分割し、それをwsprintf()でbに入れてます。そうすると、"a,b,c,d"が"abcd"に
なるのではと思いましたが、文字が表示されず、すぐに閉じてしまいます。

該当のソースコード

#include <windows.h>
LRESULT CALLBACK WndProc(HWND hwnd , UINT msg , WPARAM wp , LPARAM lp) {
	PAINTSTRUCT ps;
	static char *str = "a,b,c,d";
	static char *token;
	static char b[300];
	LPCTSTR a;
	HDC hdc;
	switch (msg) {
	case WM_DESTROY:
		PostQuitMessage(0);
		return 0;
	case WM_PAINT:
		hdc = BeginPaint(hwnd,&ps);
		token = strtok(str,",");
		wsprintf(b,"%s%s",b,token);
		token = strtok(NULL,",");
		wsprintf(b,"%s%s",b,token);
		token = strtok(NULL,",");
		wsprintf(b,"%s%s",b,token);
		token = strtok(NULL,",");
		wsprintf(b,"%s%s",b,token);
		a = b;
		TextOut(hdc,0,0,b,lstrlen(b));
		EndPaint(hwnd,&ps);
		return 0;

	}
	return DefWindowProc(hwnd , msg , wp , lp);
}

int WINAPI WinMain(HINSTANCE hInstance , HINSTANCE hPrevInstance ,
			PSTR lpCmdLine , int nCmdShow ) {
	HWND hwnd;
	MSG msg;
	WNDCLASS winc;

	winc.style		= CS_HREDRAW | CS_VREDRAW;
	winc.lpfnWndProc	= WndProc;
	winc.cbClsExtra	= winc.cbWndExtra	= 0;
	winc.hInstance		= hInstance;
	winc.hIcon		= LoadIcon(NULL , IDI_APPLICATION);
	winc.hCursor		= LoadCursor(NULL , IDC_ARROW);
	winc.hbrBackground	= (HBRUSH)GetStockObject(WHITE_BRUSH);
	winc.lpszMenuName	= NULL;
	winc.lpszClassName	= TEXT("KITTY");

	if (!RegisterClass(&winc)) return -1;

	hwnd = CreateWindow(
			TEXT("KITTY") , TEXT("Kitty on your lap") ,
			WS_OVERLAPPEDWINDOW | WS_VISIBLE ,
			CW_USEDEFAULT , CW_USEDEFAULT ,
			CW_USEDEFAULT , CW_USEDEFAULT ,
			NULL , NULL , hInstance , NULL
	);

	if (hwnd == NULL) return -1;

	while(GetMessage(&msg , NULL , 0 , 0)) {
		DispatchMessage(&msg);
	}
	return msg.wParam;
}

補足情報（FW/ツールのバージョンなど）

MinGW-win32を使っております。
コンパイル方法は、gcc -mwindows filename.cです

Comments

[episteme]

Visual Studio 2019 では再現しません。"abcd"が出力されます。
※ 一か所変更: static char str[] = "a,b,c,d"; // char* -> char[]

[dodox86]

とりあえずback spaceは関係無さそうなコードですが、その前の段階で要望通りの動きにならないので困っているという話でしょうか。

> wsprintf(b,"%s%s",b,token);

で文字列トークンを繋げていこうというつもりなのかもしれませんが、ちょっと使い方が乱暴ですね。

[A_.1]

まず、私はキーボードイベントを使って、入力した値を文字列に代入し、それを表示させています。そして、その数だけint型の変数の値をあげます。
back spaceをおすと文字を削除させるために、strtokを使おうと思い、
まず文字列に値を入れるとき、"," を間にいれて後でstrtokでint型の変数の分だけ分割と連結をして削除しようと思いましたが、少し乱暴ですか？
もっと効率が良い方法があるなら教えてほしいです。
説明下手なので、わからなかったらすいません。

[dodox86]

乱暴、と書いたのはwsprintfという関数の使い方のお話です。wsprintfはWindows APIで提供される関数ですが、
https://docs.microsoft.com/en-us/windows/win32/api/winuser/nf-winuser-wsprintfa

wsprintf(b,"%s%s",b,token);とコードに書いた場合、第3引数でbを指定しているのでbを
参照しつつ同じbと言う文字列バッファに上書きしているかたちです。

上記の公式のリファレンスにはそのような使い方が許されているとは書いていないので、APIの方でうまく処理をしてくれているのだと思います。一般的には文字列を"連結する"ことを意識しつつ、そのようなコードを書きます。まぁ、要望通りに動作しているのでとりあえず良いのではないでしょうか。

[episteme]

wsprintf(b+strlen(b), "%s", token); ではいかがかと。
てか strcat(b, token); でよかろーと。

[A_.1]

ありがとうございます。
その方法でもできました。
少し内容はわかりませんが、
b[b]にtokenをいれてるのでしょうか？

[episteme]

b+strlen(b) : 先頭位置がbで、それに現在のbの長さを足したとこ。
つまり文字列bの末尾に書き込むことになる。

Answer 1

回答になるかどうかわからないけど、間違いを１つ。

static char *str = "a,b,c,d";

このstrの指し先は、定数値となるため、書き換えはできません（できないことになっている）
いったん、変数配列にコピーしてからやってみては

Comments

[A_.1]

回答遅れてすいません。
static char *strをstatic char str[]にしたらできました。
ありがとうございます。