水平ヒストグラムは最初の入力の後に入力を受け付けません（編集済み、印刷入力はありません）

Question

私は入力の単語の長さの水平ヒストグラムを印刷するプログラムを作っています。だから、whileを1に変更したので、1つ以上の入力を受け入れるようになりました。しかし、今回は何が問題なのですか？グラフは印刷されません。

右そこにあなたの問題です

/*Write a program to print a histogram of the lengths of words in its input. 
It is easy to draw the histogram with the bars horizontal*/ 
#include <stdio.h> 
#define MAX 30 
#define IN 1 
#define OUT 0 
int main() 
{ 
int a,c,i,k,state,word[MAX]; 
a=0; 
k=0; 
state=OUT; 
for(i=0;i<MAX;i++) 
    word[i]=0; 
while((c=getchar())!=EOF) 
{ 
    if(c==' '||c=='\t'||c=='\n') 
     state=OUT; 
    else 
     state=IN; 
    if(state==IN) 
     a++; 

    if(state==OUT) 
     { 
     word[i]=a; 
     i++; 
     a=0; 
     } 



} 
/*This part is pissing me off, I don't know how to print X multiple times!*/ 
for(i=0;i<MAX;i++) 
    if(word[i]>0) 
     { 
      for(k=0;k<=word[i];k++) 
      putchar('-'); 
      putchar('\n'); 
     } 


} 

Answer 1

while(state==IN) 
    a++; 

これは基本的に無限ループで、それがループ永遠a

編集インクリメントします： [OK]を、パート2を。

まず、aでやっていることは、あなたがしているとは思わない。

あなたは単語の長さを数えるのにそれを使用していると思いますが、各単語の後に0にリセットしないので、実際に見つかった文字の数をカウントし続けます。ヒストグラムには数字が増えます。

第2に、印刷ループでは、両方のループでiを使用しています。何らかの種類のおいしさにつながります。

編集：パート3

私はちょうどあなたがこれはおそらく仕事に行くされていない条件付き

if((c==getchar())&&c==EOF) 

に入力ループ内のヒストグラムを印刷していることを発見し、私は移動することをお勧めwhileループの外側に印刷して条件を削除します。

while ((c=getchar())!=EOF) 
{ 
    ... 
} 
for(i=0;i<MAX;i++) 
{ 
    for(k=0;k<=word[i];k++) 
     putchar('-'); 
    putchar('\n'); 
} 

あなたはその結果、プログラムが外部のすべてのあるインデックス30-59にワード長を割り当てること、それをゼロに設定することなく、配列のインデックスとしてiを再利用しているパート4

：入力

編集を取って終了アレイがクラッシュしないのは純粋な運だけです。入力ループを入力する前にi = 0を設定すると、プログラムが機能します。

これで、単語の長さが長すぎるのがなぜ読まれるのか、考えてみましょう。 ヒント：比較演算子と関係があります