無限ループに陥ります

質問

プログラム中の以下の部分で無限ループに陥ってしまいます

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]

どなたか分かる方お願いします。

回答No.10

Tacosan · ベストアンサー

全然本題と関係ないんだけど, a は初期化しなくていいの?
あと, EOF は「負の整数」としか定義されていないので, char では表せないかもしれませんね＞#8.

回答No.9

yama5140

＞無限ループに陥ってしまいます

　《これだけ》を解決するには、

☆ a の前の * を取る（なにか意図があって付けたハズ◆だが）。
☆ fgetc の逆手をとる。

　でいいのでは・・（動作確認済み）。

　http://www.bohyoh.com/CandCPP/C/Library/fgetc.html

　　　while( a != EOF ){

　　　　a = (unsigned char)fgetc( fi );
----------------------------------
《◆本題》

・半角空白をデリミタにして、前に読み込んだものを、使いたいみたいだけど、それはムリなんでは・・。

　*a = fgetc( fi ); 同じ場所に上書きされてんじゃないかな。

　下のようにすると、空白に囲まれた数字の下１桁が、ファイルの終了まで得られるよ。

　　int iVal[ 1 ];

　　while( 1 == fscanf( fp, "%d", iVal ) ){

　　　　t = iVal[ 0 ] % 10;
----------------------------------
なお、上のようにしたら関係ないけど、

　if( a == 32 ){ は、（ a が char であることを明示するため）

　if( ' ' == a ){ または、if( 0x20 == a ){

　とコーディングした方が「後々」よろしいかと・・。

補足

ポインタに代入するするたびに勝手に代入位置が一個進むもんだと勘違いしてました。。
他にもいろいろ理解があやふやな部分があるようです。
すこし勉強しなおしてきます。

回答No.8

Oh-Orange

★アドバイス
＞特にオプションなどがない限り、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);
}

回答No.7

titokani

＞　char型は0x00～0xFFまでを扱います。
特にオプションなどがない限り、charの範囲は-128～127ですが。
でも、だとすると動くはずだよな。
本当にcharですか？unsigned charではなくて。

回答No.6

tatsu99

#5です。
前回のは取り消します。
終了条件の判定は、int型のデータで処理する必要があるが、
１文字の処理で、t = *(a-1);　を行っているとことは、
int型にすると、おかしくなるので、だめですね。
最初のところを、以下のようにして下さい。
----------------------------------
int data;
while(1){
data = fgetc(fi);
if (data == EOF) break;
*a = data; /* *a = fgetc(fi); のところです。*/
--------------------------
以下同様

}

補足

ありがとうございます。
試してみます。

回答No.5

tatsu99

char *a を
int *a にして下さい。それで解決します。原因は、#4の方が述べたとおりです。

回答No.4

Oh-Orange

★アドバイス
・記号定数EOFの値は何？
　char型は0x00～0xFFまでを扱います。
　char型ではファイルの最後のEOFを表せません。
　int型ならEOF、１バイトの0x00～0xFFを表せます。
・さて気づきましたか？

回答No.3

koko_u_

>char型です
それもおかしな話だな。
fgetc() のプロトタイプ宣言を見よ。

補足

すみません、色々調べましたがよくわかりませんでした。。

回答No.2

SAYKA

ループ中　どこか何か表示するようにして　何が起きてるか把握するところから始めるのが良いんじゃない？

補足

読み込んだ後すぐに*aを出力してみたところ、ファイルの内容は正しく表示されましたが、ファイルの終わりを認識せずに処理を続けているようでした。
*a!=EOF の部分が問題なのでしょうか

回答No.1

koko_u_

a の型は？

補足

char型です