かぶった文字を消すプログラム

文字列s2中の任意の文字に等しい文字をs1から除外するプログラムを以下の様に作りましたが、ビルトと入力は出来るものの、出力が出ません。何が間違っているのでしょう？



#include <stdio.h>
main()
{
int i,j,c;
char s[100],t[100];

for( i = 0; (c = getchar()) != '\n'; i++){
s[i] = c;
}
for( i = 0; ((c = getchar()) != '\n') && (c != EOF); i++){
t[i] = c;
}
for( i = 0; s[i] != '\0'; i++){
for( j=0; t[j] != '\0'; j++){
if( s[i] == t[j] ){
s[i] = ' ';
}
}
}
for( i = 0; s[i] != '\0'; ++i ){
printf("%s", s[i]);
}
}

No.2 ベストアンサー 回答者： gerugugu84 回答日時： 2011/06/27 17:28

s[]、t[]にはヌル文字をセットしていないので、その後のfor文ループで止まれません。

なので、s[]、t[]の終端にヌル文字をセットしてやります。

for( i = 0; (c = getchar()) != '\n'; i++){
s[i] = c;
}
s[i] = '\0'; // ←追加
for( i = 0; ((c = getchar()) != '\n') && (c != EOF); i++){
t[i] = c;
}
t[i] = '\0'; // ←追加


また最後の出力で%sで指定しているので、for文は不要。
printf("%s", s);の１行でOK。

No.6 回答者： hiroshi09s 回答日時： 2011/06/28 06:45

＞ビルトと入力は出来るものの、出力が出ません。

何が間違っているのでしょう？

質問プログラムの内容をそのままにして直してみました。

１．文字列入力の for()文は、改行（'\n'）コードを読んだ時点でメモリに格納せずに終わっています。したがって、それ以降の文字は不定のままになっています。
　解決方法は for()文の後に「s[i] = '\0';」とか「t[i] = '\0';」とか追加して文字列を区切る必要があります。ここでは static char として文字配列 s[]、t[] を初期化して、これを代用し、プログラムの修正箇所を少なくしています。
２．かぶった文字を消す部分の for()文は、まず、読み込んだ文字列が「'\0'」で区切られていることが大前提です。それは区切られていませんでしたから、動くはずもありません。
　内容については、二重ループになっていることから、かぶる文字は各１文字づつの操作となっています。つまり、「abcdef」の文字について「bd」の文字を消そうとすると「bd」の文字列はないのに「b」と「d」の文字が消されるプログラム内容になっています。この辺りはちょっとヘンかなと思います。
３．出力は、文字列についてのみ行い、改行「"\n"」は格納文字列に含まれていません。よって、別途改行してやる必要があります。また、書式付き出力の printf() よりは、１文字出力の putchar() 関数のほうが合っているように思います。
４．今はCの学習中なのでしょうが、読み込みは fgets()関数が簡単で良いでしょうし、結果の出力についても printf()の１行で事足りてしまいます。




#include <stdio.h>
int main(void)
{
int i,j,c;
static char s[100],t[100];

printf("Input strings: ");
for( i = 0; (c = getchar()) != '\n'; i++){
s[i] = c;
}
printf("Input cutting word: ");
for( i = 0; ((c = getchar()) != '\n') && (c != EOF); i++){
t[i] = c;
}

for( i = 0; s[i] != '\0'; i++){
for( j=0; t[j] != '\0'; j++){
if( s[i] == t[j] ){
s[i] = ' ';
}
}
}

for( i = 0; s[i] != '\0'; ++i ){
printf("%c", s[i]); // １字出力
}
printf("\n"); // 改行して完

return 0;
}

No.5 回答者： asuncion 回答日時： 2011/06/27 20:45

「除外する」の仕様がよくわかりません。

そこで、入力と出力の例を挙げてみてください。

No.4 回答者： kmee 回答日時： 2011/06/27 17:42

> printf("%s", s[i]);

%sに対応する型は char *(あるいは、それと同等のchar [])です。
sがchar[] なら s[i]は何になると思いますか?
コンパイラによっては、書式と型の不一致を警告してくれるものもありますが、エラーではないのでビルドは成功します。また、警告すらしないコンパイラも多いです。

他には細かい点ですが

> main()
こんな書き方をしている参考書をお持ちなら、新しいのを買いましょう。

> for( i = 0; (c = getchar()) != '\n'; i++){
> s[i] = c;
> }

1行読み込むなら、標準関数には fgets , gets という便利なものがあります。

> 等しい文字をs1から除外する

ここで言う「除外」とは、「該当文字をスペースに置き換える」ことでよろしいでしょうか?
そうでないなら、
>for( i = 0; s[i] != '\0'; i++){
> for( j=0; t[j] != '\0'; j++){
(略)
の部分は間違いになります

No.3 回答者： askaaska 回答日時： 2011/06/27 17:36

良く見ると最後は

printf("%c", s[i]);
よね。

No.1 回答者： askaaska 回答日時： 2011/06/27 17:20

s[i] != '\0'

にマッチングするものがない
ってことでしょう？

私の手元には確認できる環境がないのでできないけど
面倒でも各処理の途中経過をprintしてみてはどお？