質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

Q&A

3回答

3678閲覧

ポインタでのEOF判定

ba_max009

総合スコア21

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

ポインタ

ポインタはアドレスを用いてメモリに格納された値を"参照する"変数です。

0グッド

0クリップ

投稿2016/11/09 15:28

###前提・実現したいこと
構造体でポインタの配列を使って文字列を読み込むプログラムなのですが、ポインタ配列をEOFで判定できないとエラーが出てわからなくなりました。
どこが間違っているのか教えてほしいです><

###発生している問題・エラーメッセージ
pref3main.c: In function ‘main’:
pref3main.c:12:14: error: ‘EOF’ undeclared (first use in this function)
pref3main.c:12:14: note: each undeclared identifier is reported only once for each function it appears in

エラーメッセージ

###該当のソースコード

#define N 3 #define LINELEN 128 #define data 47 typedef struct { char name[NAMELEN+1]; /* 県名 */ double area; /* 面積 (km2)*/ int population; /* 人口 (人) */ } pref_t; pref_t *pref_new(char *name, double area, int population) ; void pref_print(pref_t* p) ; pref_t *pref_new_scan(); #include <stdlib.h> #include <stdio.h> #include <string.h> #include "pref.h" int main(void) { pref_t *p[data]; int i; int count=0; for(i=0;;i++){ p[i]=pref_new_scan(); if(p[i]==EOF){break;} count++; } for(i=count;i>0;i--){ pref_print(p[i]); } for(i=0;i<count;i++){ free(p[i]); p[i]=NULL; } return 0; } #include <stdio.h> #include <stdlib.h> #include <string.h> #include "pref.h" #define NAMELEN 15 pref_t *pref_new(char *name, double area, int population){ pref_t *x; //動的割り当て x = (pref_t *) malloc(sizeof(pref_t)); /* 値の代入 */ strncpy(x->name, name, NAMELEN); x->area = area; x->population = population; return x; } void pref_print(pref_t* p){ printf("%-10s %9.2f %9d\n", p->name, p->area, p->population); } pref_t *pref_new_scan(){ pref_t *p; char s[LINELEN+1]; //動的割り当て p=(pref_t*)malloc(sizeof(pref_t)); if(scanf("%s",p->name)==EOF){return NULL;} if(scanf("%lf",p->area)==EOF){return NULL;} if(scanf("%d",p->population)==EOF){return NULL;} return p; } ここにご自身が実行したソースコードを書いてください

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

終了をEOFで判定しているがpref_new_scan()で NULLを返しているのだからNULLで判定する。

C

1int main(void) 2{ 3// ・・・略・・・ 4 if(p[i]==NULL){break;}

あと、return NULL; の前に malloc()した領域を解放しないと。

C

1pref_t *pref_new_scan(){ 2// ・・・略・・・ 3 //動的割り当て 4 p=(pref_t*)malloc(sizeof(pref_t)); 5 6 if(scanf("%s",p->name)==EOF){free(p);return NULL;} 7 if(scanf("%lf",p->area)==EOF){free(p);return NULL;} 8 if(scanf("%d",p->population)==EOF){free(p);return NULL;}

投稿2016/11/09 15:41

Y.H.

総合スコア7914

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

p[i]==EOF
左辺は独自定義した構造体ですよ。EOFと比較するのはおかしいはずです。

投稿2016/11/09 15:38

HogeAnimalLover

総合スコア4830

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

エラー内容をよく読みましょう。

error: ‘EOF’ undeclared (first use in this function)

だから’EOF'を宣言(declare)していない。
’#defineでEOFを定義すればこのエラーは無くなるでしょう。

また構造体とEOFを比較はできないですよね。

投稿2016/11/09 15:37

編集2016/11/09 15:39
MasahikoHirata

総合スコア3747

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問