ファイルを読み込んだ後に新しい値で更新する関数で配列を取得できません

Question

私は関数を持ち、その関数の外で初期化された配列を持っています。この関数の中では、私の配列のすべての値をゼロに設定しています。そのため、ゴミが返されません。私はファイルをスキャンしようとしており、ファイル内でスキャンしている番号（たとえば7）を使用して、配列のインデックスを1つ追加して更新します（インデックス7の値は1になります）。 whileループは最初にプログラムをクラッシュさせ、forループを使用するとプログラムは実行されますが、値は更新されません。私のループは、未知の時間の間走らせる必要がありますが、それは別の時間の問題だと思います。

void createVArray (FILE*votes, int ray[], int len){ 

    votes=fopen("votes.txt","r"); 

    int i,k,num; 

    for(i=1;i<len;i++){ 
     ray[i]=0; 
    } 

    while(num!=30){ 
    fscanf(votes,"%d",&num); 
     ray[num]++; 
    } 

} 

問題は何ですか？

Answer 1

while(num!=30){ 
    fscanf(votes,"%d",&num); 
     ray[num]++; 
    } 

ここにはいくつかの問題があります。最初にnumを初期化しないので、初めてnumを30と比較すると、未定義の動作が発生します。マジックナンバー30はどこから来たのですか？

第2に、numを読み取った後に、ray[num]を増やす前に、それが(len-1)より大きくないようにする必要があります。

おそらく、このようなものは動作します：

while(fscanf(votes,"%d",&num) == 1) { 
     if ((num >= 0) && (num < len)) { 
     ray[num]++; 
     } 
    } 

Answer 2

よし男を。問題が解決しました。私はちょうど馬鹿です - 私は "投票"の代わりに "votes.txt"というファイルを付けましたので、ファイルを読むことさえできませんでした。ロックオン。助けを借りてくれてありがとう。スーパーは助けに感謝します。