新明解C言語 演習11-8

[vayacy]

初投稿です。
質問は文字列の中の数字を削除する関数を作れという問題で

コード:

/*文字列中の数字を消去する関数*/
void del_digit(char *str)
{
	int i,j;
	int len=strlen(str);
	for(i=0;i<len;i++){
		if(toupper(*(str+i))==tolower(*(str+i))){
			for(j=i;j<len;j++){
				*(str+j)=*(str+j+1);
				}
				i--;
		}
	}
}

int main(void)
{
	char str[128];
	
	printf("文字列:");
	scanf("%s",str);

	del_digit(str);
	printf("%s",str);
	
	return 0;
}

と書いてcygwinで実行したのですが、文字入力までしか実行できなく止まってしまいます。できれば数字の大文字と小文字が同じであることを使いたいんですけど何か間違いがあればご指摘ください。

ちなみにdel_digit内のifの条件式を"=="→"!="に変えてえ実行すると数字だけ残るように実行されてきっちり実行されます…

[みけCAT]

終端のナル文字も「大文字と小文字が同じ」と判定されるので、
ナル文字を削除しようとし続け、無限ループになってしまうようですね。
文字を削除したら当然文字数が減るので、文字を削除したらlenもデクリメントするようにするといいでしょう。

オフトピック

できれば数字の大文字と小文字が同じであることを使いたいんですけど

vayacyさんが使いたいのであれば無理にやめろとは言いませんが、
どうしてisdigitを使わずにわざわざそんな独創的な方法をとりたいのでしょうか…？
それと、数字以外にも(記号など)「大文字と小文字が同じ」になる文字はありますよ。

[かずま]

数字だったら、その後ろ全部をずらす、というのは無駄ですね。
数字でない文字を 1文字コピーするだけでよいでしょう。

コード:

void del_digit(char *str)
{
	unsigned char c;                   // isdigit の引数は unsigned char
	int i = 0, j = 0;                  // コピー元の位置とコピー先の位置
	while ((c = str[i++]) != '\0')
		if (!isdigit(c)) str[j++] = c; // 数字でなければ、コピー
	str[j] = '\0';
}

[vayacy]

間違いに気づけました。ありがとうございました。
あとこのコードを書いたときは学習が進んでおらずisdigitやisalfaを知らなかったのでこじつけて作ったらこんなややこしくなりました（笑）