fscanfでエラーがでてしまいます。

Question

fscanfでエラーがでてしまいます。どのように対処すればいいのでしょうか？

  int *num;
  int n;
char **str;
  fscanf(fp, "%d",&n);
  str = (char**)malloc(5 * sizeof(char *) * n); //5文字のchar
  num=(int*)malloc(sizeof(int*)*n);
 
for (int i = 0; i < n; i++) {

    fscanf(fp, "%s%d",str[i],&num[i]);
}

どうぞよろしくお願いいたします。

追記です。

  str = (char**)malloc(sizeof(char *) * n); // ｎ行個分のアドレスを保持する領域を確保
  
  for(int i=0; i<n; i++){
    str[i]=(char*)malloc(5* sizeof(char));
  }

これでもまだ同じエラーが出るのですが、どうすれば良いでしょうか？

Comments

[dodox86]

fscanfでどの様な入力を想定しているのかを詳しく質問文中に示しましょう。

Answer 1

fscanf(fp, "%s%d",name[i],&num[i]);

name ってナニモノでしょうか。

---

どこの場所にデータが読まれるのかをイメージできてるでしょうか。

str = (char**)malloc(5 * sizeof(char *) * n); //5文字のchar

これは5n個のcharポインタの領域が確保されるだけで、ポインタ自体はデタラメの値です
このあとに、それぞれのポインタに、中身のための領域の確保の必要があります

＃まだまだツッコミどころはあるけど

Comments

[alizona]

もうしわけありません。strと書くべきところをnameと書いてしまいました。
訂正します。

[y_waiwai]

それがstrだとしても、領域確保が間違ってるので結果は同じになりますね

[alizona]

for(int i=0; i<n< i++){
str[i]=(char*)malloc(5 * sizeof(char));
}
のような確保方法も試したのですが、同じ結果でした。
また、それと同様に、numもfor ループでmallocしたのですが同じでした。

領域確保でどこが間違えているのかアドバイスいただけないでしょうか？

[y_waiwai]

回答に追記しました

[alizona]

質問に追記したのですが、さらにループをつかってmallocを呼び出しても、同じエラーが出てしまいました。

[y_waiwai]

まだまだツッコミどころです
> str[i]=(char*)malloc(5* sizeof(char));

5文字分しかないけど、これだと４文字以下の文字列しか格納できませんがそんでいいんでしょうか。
ファイルの中に、一つでも４文字以上の文字列があればエラーでますよ

[alizona]

質問に追記したコードのしたで、初期化のつもりで、strにnullを代入していました。
無事解決いたしました。ありがとうございました。

[alizona]

今、返信を確認いたしました。

[alizona]

ありがとうございます。1もじ増やして、６にしました。

[alizona]

他にも問題ありますでしょうか？

[y_waiwai]

まあ、課題とか問題とかで、５文字の文字列しか来ない、ということであればそれでいいんでしょう。

たとえば、あなたが使ってるアプリかなんかで、５文字入れるところを間違って６文字入れたら、エラーで落ちた、となったらどう思うでしょうか。

ということで、何かの問題の回答としては○なんでしょうけど、実務のコードとしてはおはなしになりません

[alizona]

わかりました。ありがとうございました。課題が終わったらそれについて考えてみます。

[alizona]

アドバイスいただきましてありがとうございました。

[dodox86]

＞@質問者さん

ですから、ファイルからの入力内容であるとか、前提条件を詳しく正しく示さないと、第三者にとってはそのプログラムの適切さは分からないのですよ。場合によっては過剰スペックになります。（その故の、「質問への追記・修正、ベストアンサー選択の依頼」での指摘です）

[alizona]

わかりました。今追記します