c言語でファイルから一文字ずつ読み込んでchar配列に代入したい

Question

char name[50];

while(1)
    while(1) {
      c = fgetc(sfp);
      if('A' <= c && c <= 'Z') c += 32;
      if('a' > c || c > 'z') break;
      name[i] = c;
      i++;
    }
    if(c == EOF) break;
    name[i] = '\n';
    tinsert(root, name);

    i = 0;
  }

上のコードで、sfpから一文字読み込み、その文字が大文字なら小文字に変換後name[i]に代入したいのですが、実行するとうまくいかずname[i] = cのところで例外が発生しました。と出てきます。なにが原因で、どのように変更すれば良いのでしょうか？
やりたいことは、英語で書かれたファイルから1単語ずつ読み込んで単語数を数えようとしてます。

"追記"
tinsertはname[50]に入れた単語を2分木に格納する関数です。

試したこと

 int i;
  char name[5] = "abcde";
  char a[5];
  for(i=0; i<5; i++) {
    a[i] = name[i];
  }
  for(i=0; i<5; i++) {
    printf("%c", a[i]);
  }
  puts(" ");
  printf("%s\n", a);

実行結果
abcde
abcdeabcde

試したことは、同じようなコードを書いて文字配列に一文字ずつ代入できるか確認してみました。しかし、char a[5]を%sで一文字ずつ表示すると何故かabcdeが2回表示されました。ここら辺の理由がわかっていません。

Comments

[rubato6809]

上のコードはプログラムの一部しかアップされていません。
プログラムの全体をアップしてください。

Answer 1

配列のインデックス(i)が無限に増え続けるパターンが存在する時点であまりよろしくないプログラムです。
まずは、配列の長さを超える時点でループを抜けるように条件設定するのが前提です。
そのうえで、もし想定している終了条件（ここではif('a' > c || c > 'z') break;）で引っかからない理由を、デバッガなりprintfなりで確認してください。

Answer 2

変数 i を０に初期化しないまま、つまり不定値のまま while ループを始めたのでは。
int i; を int i = 0; に修正する。

Comments

[K.M.PEANUTS]

その通りでした。つまらないミス気をつけます。

Answer 3

一文字づつ読み込むんじゃなくて、
まとめて読み込んで、そのあとで変換していくようにしましょう

Comments

[K.M.PEANUTS]

もう少し詳しく聞かせてほしいです。
name[50]という配列に1単語を読み込みたいのですが、どこまでをまとめて読み込み、1単語を判断してname[50]に代入するのでしょうか？

[y_waiwai]

そのバッファのサイズ、５０バイト読み込めばいいんです。
あとの変換や１単語の判定は、その後にすればいいです

今みたいに全部一緒くたでしようとするから、わけわからんようになるんです。

Answer 4

name[i] = cのところで例外が発生しました。

51文字以上読み込んだのでしょう。
なお、ちょうど51文字目で例外が発生するとは限りません。

if('a' > c && c > 'z')

'a'より小さくて'z'より大きい文字は存在しないので、この条件は常に偽ですが、この行の意図は何でしょう？

%sで一文字ずつ表示すると

%sは「1文字ずつ表示」では無いです。「NUL文字が出てくるまで連続して表示」です。
質問のコードではNUL文字を使ってないので、%sは使えません。
無理矢理使ったので、たまたまNUL文字が入っていたところまで表示しています。

Comments

[K.M.PEANUTS]

＞ 'a'より小さくて'z'より大きい文字は存在しないので、この条件は常に偽ですが、この行の意図は何でしょう？

自分も今そのミスに気づき、修正しました。が、いまだに同じエラー内容が表示されています。。

Answer 5

name[i] = cのところで例外が発生しました。と出てきます。なにが原因で、どのように変更すれば良いのでしょうか？

nameの定義が書かれていないので、なんとも言えません。

しかし、char a[5]を%sで一文字ずつ表示すると何故かabcdeが2回表示されました。

末端のヌル文字が入っていないため、配列の領域を超えて読み込み続けてしまいます。"abcde"の文字列を格納するには、最後のヌル文字を入れて6バイトの領域が必要です。

Comments

[K.M.PEANUTS]

char name[50]です。

%sは文字列の先頭アドレスを受け取ってヌル文字の前までを表示するのでしたね????
後半については理解できました。

[maisumakun]

・iは初期化していますか？
・50文字を超えて処理しようとしていませんか？

[K.M.PEANUTS]

『if('a' > c && c > 'z') break;』これを満たす文字が存在しないため、無限ループになっており、iが初期化される段階まで行っていませんでしたので修正しました。これでiは初期化されています。
しかし同じ内容のエラーが出続けているので原因は他にもあるようです。。

[maisumakun]

> これでiは初期化されています。

提示されたコードにはiの宣言すら見当たらないです。

[K.M.PEANUTS]

編集させていただきました????‍♂️

[maisumakun]

宣言はどこですか？