C言語 単語を取得するプログラム

質問

ファイルからアルファベットをスペース等で区切って取得するプログラムを作成しているのですが、デバグがうまくいきません。
どの部分がプログラム的におかしいか、どうすればよいか教えていただけませんでしょうか。

テキスト
I have a pen. if you have a pen, please lend me your pen.
abc, def.ghi jkl mno pqr..,stu vwx .yz

ソースプログラム

#include<stdio.h>
#include<string.h>

int word(char data[],FILE *fp){
char c;
int n=0;

data[0]='\0';

while((c=fgetc(fp))!=EOF){

if(c!=' ' && c!=',' && c!='.'){

n=strlen(data);

data[n]=c;
data[n+1]='\n';

}

else return(n);

}

return(0);
}

int main(){

FILE *fp;
char data[30];

fp=fopen("test.txt","r");
if((fp = fopen("test.txt","r"))==NULL){
printf("no file\n");
exit(1);
}

while((word(data,fp)!=0)){
printf("%s \n",data);
}

fclose(fp);
return(0);
}

回答No.3

hanabutako · ベストアンサー

デバッグというくらいですから、普通にデバッガーを使ってステップ実行すればすぐわかると思います。
この行にブレイクポイントを入れてステップ実行すれば良いでしょう。
> if(c!=' ' && c!=',' && c!='.'){

そして、ループを回るごとにnの値がどう変化するかを観察しましょう。
あと、文字列の終端を示す記号は\nではないですよね。

他の点については既にkmeeさんが指摘しているとおりです。
他には文字列が長いとバッファが溢れそうとか、isspace使えばいいのにとか色々思うところはありますが。

回答No.4

Wr5

>fp=fopen("test.txt","r");
>if((fp = fopen("test.txt","r"))==NULL){

ファイルの多重オープンが許可されている環境なんでしょうか？
2回目のオープンが失敗する可能性と、1回目のオープンが閉じていない状態は問題ありませんか？

ファイルから取得した後、なにをやりたい…んでしょうかね？？

他は既に指摘されているようですので……。

回答No.2

kmee

まずは、「どうおかしいか?」を分析しましょう。
それがわからなければ、プログラムを修正することはできません。

> char c;

fgetcのマニュアルを良くよみましょう。
戻り値は何になっていますか?

原稿用紙等のマス目のある紙を用意してください。
そのマスに鉛筆で適当な文字を書いていってください。
これが、dataの初期状態です。

> data[0]='\0';
最初のマスを'\0'に書き換えます。

> n=strlen(data);

先頭から'\0' までのマスを数えます。これがstrlenの動作です。
一番最初なので0です。

> data[n]=c;

さっき求めたnのマスを書き換えます。先頭ですね。

> data[n+1]='\n';

その次のマスを'\n' に書き換えます。

> n=strlen(data);

さて、次にstrlenを実行したら、このnは何になるでしょうか?
「先頭から'\0' までのマスを数える」strlenです。\0はどこにあるでしょうか?

回答No.1

bin-chan

> デバグがうまくいきません。
どう、デバッグがうまくいかないの？
例示のデータで実行したらどうなったの？