- ベストアンサー
無限ループに陥ります
プログラム中の以下の部分で無限ループに陥ってしまいます while(*a != EOF){ *a = fgetc(fi); if(*a == 32){ b++; } if(b == 3){ b = 0; t = *(a-1); if(t == '1'|| t == '3' ||t == '5' || t == '7'|| t == '9'){ c[n] = 1; } else{ c[n] = 2; } n++; } } fiに読み込んでいるファイルの末尾はこんな感じです。 50 49 0 34 34 34 34 34 34 34 34 34 34 34 34 50 49 0 68 68 68 51 51 50 34 34 34 34 34 34 34 34 34 34 34 34 34 34 34 34 34 34 34 34 34 [EOF] どなたか分かる方お願いします。
- みんなの回答 （10）
- 専門家の回答
質問者が選んだベストアンサー
その他の回答 (9)
- yama5140
- ベストアンサー率54% (136/250)
- Oh-Orange
- ベストアンサー率63% (854/1345)
★アドバイス ＞特にオプションなどがない限り、charの範囲は-128～127ですが。 ↑ 処理系によってはcharをunsigned char型として扱う場合があるでしょう。 だからオプションを設定しなくてもcharが-128～+127かどうかは分かりません。 今回はchar型ですとなっています。 charの指定がsigned char型なら『while(*a != EOF){』の部分も抜けるはず。 無限ループするとなっているのできっとchar指定がunsigned char型となる コンパイラかもしれない。と思ったのです。0x00～0xFF。 ・そもそもfgetc()関数の戻り値はint型なのでchar型(signed,unsignedも含む)の 変数に代入しているのが間違いですね。 あと良く考えたら変数aはchar型なのにポインタみたいに使っていますね。 本当に『char a;』なのでしょうか？ 『char s[256], *a=s;』なのか？ どちらでしょうね。 『char a;』が正しいなら『t = *(a-1);』この指定はおかしいです。 ・質問者さんへ全ソースを貼り付けてみて下さい。
補足
以下が全ソースです。 ちょこちょこいじってますが問題は解決していません。 #include<stdio.h> int main(void) { FILE *fi; char fname[100], ss[255], t, ch; int count, b, n, *a, err, c[1000]; printf("file name?\n"); gets(fname); fi = fopen(fname,"r"); fgets(ss,255,fi); puts(ss); printf("are"); if(ss[0]!='P') err = 1; if(ss[1]!='3') err = 1; if(err ==1){ printf("file type is wrong\n"); exit(1); } while(count < 3){ *a = fgetc(fi); printf("%d",*a); if(*a =='\n' ){ count++; } if(*a == '#'){ count--; } } b = 0; while(1){ *a = fgetc(fi); if(*a == EOF){ break; } if(*a == '\x20'){ b++; } if(b == 3){ b = 0; t = *(a-1); if(t == '1'|| t == '3' ||t == '5' || t == '7'|| t == '9'){ c[n] = 1; } else{ c[n] = 2; } n++; } } n = 0; while(c[n] !=0){ count = 0; while(count < 8){ if(c[n] == 1){ ch++; } ch = ch + ch; count++; n++; } printf("%c",ch); } fclose(fi); }
- titokani
- ベストアンサー率19% (341/1726)
- tatsu99
- ベストアンサー率52% (391/751)
#5です。 前回のは取り消します。 終了条件の判定は、int型のデータで処理する必要があるが、 １文字の処理で、t = *(a-1); を行っているとことは、 int型にすると、おかしくなるので、だめですね。 最初のところを、以下のようにして下さい。 ---------------------------------- int data; while(1){ data = fgetc(fi); if (data == EOF) break; *a = data; /* *a = fgetc(fi); のところです。*/ -------------------------- 以下同様 }
補足
ありがとうございます。 試してみます。
- tatsu99
- ベストアンサー率52% (391/751)
- Oh-Orange
- ベストアンサー率63% (854/1345)
- koko_u_
- ベストアンサー率18% (459/2509)
>char型です それもおかしな話だな。 fgetc() のプロトタイプ宣言を見よ。
補足
すみません、色々調べましたがよくわかりませんでした。。
- SAYKA
- ベストアンサー率34% (944/2776)
ループ中 どこか何か表示するようにして 何が起きてるか把握するところから始めるのが良いんじゃない？
補足
読み込んだ後すぐに*aを出力してみたところ、ファイルの内容は正しく表示されましたが、ファイルの終わりを認識せずに処理を続けているようでした。 *a!=EOF の部分が問題なのでしょうか
- koko_u_
- ベストアンサー率18% (459/2509)
a の型は？
補足
char型です
補足
ポインタに代入するするたびに勝手に代入位置が一個進むもんだと勘違いしてました。。 他にもいろいろ理解があやふやな部分があるようです。 すこし勉強しなおしてきます。