c言語 構造体配列にファイルから値を読み込みたい

Question

前提・実現したいこと

ファイルから読み込んだ文字を構造体配列に格納した後、それを2分探索で調べ構造体配列に格納されているなら構造体の中身を表示するプログラムを作りたいのですが思うような結果になりません。
原因はファイルからの読み込みにあるようなのですが直し方がわからないので教えて欲しいです。

発生している問題・エラーメッセージ

ファイルから読み込むとすべての構造体配列の値が同じになる
例：２回目の読み込みを行うと
 p[0].english = bear , p[1].english = bear
となってしまう。

実行結果は
英単語を入力して下さい --->wolf
min = 0 , max = 49 , middle = 24
wolfの意味 : 狼
続けますか? (yes/no) --->yes
英単語を入力して下さい --->dog
min = 0 , max = 49 , middle = 24
min = 0 , max = 23 , middle = 11
min = 0 , max = 10 , middle = 5
min = 0 , max = 4 , middle = 2
min = 0 , max = 1 , middle = 0
dogの意味 : 辞書に登録されていません
続けますか? (yes/no) --->no
プログラムを終了します.

該当のソースコード

#include<stdio.h>
#include<string.h>
#include<stdlib.h>

typedef struct DIC_CELL{
  char *english;
  char *mean;
}DIC_CELL;

char *search(DIC_CELL *p , int min, int max, char *name){
  int middle;
  char *not = "NOT";
  while(min <= max){
    middle = (min + max) / 2;
    //確認
    printf("min = %d , max = %d , middle = %d\n" , min,max,middle);

    if(strcmp(p[middle].english,name) == 0){
      return p[middle].mean;
    }
    else if(strcmp(p[middle].english,name) < 0){
      min = middle + 1;
    }
    else{
      max = middle -1;
    }
  }


  return not;

}

int main(int argc , char *argv[]){
  char name[100];
  char *mean;
  char dat[100],zat[100];
  DIC_CELL p[50];
  int i;
  FILE *fp;
  char end[3];

 // プログラム利用方法表示
  if ( argc != 2 ) {
    fprintf( stderr, "Usage: %s filename\n", argv[0] );
    exit( 1 );
  }

  //ファイルオープン
 if (( fp = fopen( argv[1], "r" ) ) == NULL ) {
    fprintf( stderr, "File open error %s\n", argv[1] );
    exit( 1 );
  }

 // データ読み込み&作成
 while( fscanf( fp, "%s %s", dat,zat ) != EOF ) {
   p[i].english = dat;
   p[i].mean = zat;
   i++;
 }

  while(1){
    printf("英単語を入力して下さい --->");
    scanf("%s" , name);
    
    //検索
    mean = search(p , 0 , 49 , name);

    //表示
    if(strcmp(mean,"NOT") == 0){
      printf("%sの意味 : 辞書に登録されていません\n" , name);
    }
    else{
      printf("%sの意味 : %s\n" , name,mean);
    }

    //継続の確認
    printf("続けますか? (yes/no) --->");
    scanf("%s" , end);
    if(strcmp(end,"no") == 0) break;
  }
  printf("プログラムを終了します.\n");

  fclose(fp);
  return 0;
}

###読み込みファイル

alligator   鰐
bear        熊
camel       ラクダ
cat         猫
caterpilar  毛虫
catfish     なまず
cattle      牛
crow        烏
deer        鹿
dog         犬
dolphin     イルカ
duck        アヒル
eagle       鷲
elephant    象
fowl        鶏
frog        蛙
giraffe     麒麟
gorilla     ゴリラ
hawk        鷹
horse       馬
jaguar      豹
kangaroo    カンガルー
koala       コアラ
lion        ライオン
monkey      猿
ostrich     駝鳥
panda       パンダ
peacock     孔雀
penguin     ペンギン
pig         豚
rat         鼠
seal        アザラシ
sheep       羊
snake       蛇
sparrow     雀
tiger       虎
whale       鯨
wolf        狼

試したこと

gdbコマンドによる値の確認

###追記

#include<stdio.h>
#include<string.h>
#include<stdlib.h>

typedef struct DIC_CELL{
  char *english;
  char *mean;
}DIC_CELL;

char *search(DIC_CELL *p , int min, int max, char *name){
  int middle;
  char *not = "NOT";
  while(min <= max){
    middle = (min + max) / 2;
    //確認
    printf("min = %d , max = %d , middle = %d\n" , min,max,middle);

    if(strcmp(p[middle].english,name) == 0){
      return p[middle].mean;
    }
    else if(strcmp(p[middle].english,name) < 0){
      min = middle + 1;
    }
    else{
      max = middle -1;
    }
  }


  return not;

}

int main(int argc , char *argv[]){
  char name[100];
  char *mean;
  char dat[100],zat[100];
  DIC_CELL p[50];
  int i=0;
  FILE *fp;
  char end[3];

 // プログラム利用方法表示
  if ( argc != 2 ) {
    fprintf( stderr, "Usage: %s filename\n", argv[0] );
    exit( 1 );
  }

  //ファイルオープン
 if (( fp = fopen( argv[1], "r" ) ) == NULL ) {
    fprintf( stderr, "File open error %s\n", argv[1] );
    exit( 1 );
  }

 // データ読み込み&作成
  while( fscanf( fp, "%s %s", dat,zat) != EOF ) {
    strcpy(p[i].english, dat);
    strcpy(p[i].mean, zat);
   i++;
 }

  while(1){
    printf("英単語を入力して下さい --->");
    scanf("%s" , name);
    
    //検索
    mean = search(p , 0 , 49 , name);

    //表示
    if(strcmp(mean,"NOT") == 0){
      printf("%sの意味 : 辞書に登録されていません\n" , name);
    }
    else{
      printf("%sの意味 : %s\n" , name,mean);
    }

    //継続の確認
    printf("続けますか? (yes/no) --->");
    scanf("%s" , end);
    if(strcmp(end,"no") == 0) break;
  }
  printf("プログラムを終了します.\n");

  fclose(fp);
  return 0;
}

###追記2

#include<stdio.h>
#include<string.h>
#include<stdlib.h>

typedef struct DIC_CELL{
  char *english;
  char *mean;
}DIC_CELL;

char *search(DIC_CELL *p , int min, int max, char *name){
  int middle;
  char *not = "NOT";
  while(min <= max){
    middle = (min + max) / 2;
    //確認
    printf("min = %d , max = %d , middle = %d\n" , min,max,middle);

    if(strcmp(p[middle].english,name) == 0){
      return p[middle].mean;
    }
    else if(strcmp(p[middle].english,name) < 0){
      min = middle + 1;
    }
    else{
      max = middle -1;
    }
  }


  return not;

}

int main(int argc , char *argv[]){
  char name[100];
  char *mean;
  char dat[100],zat[100];
  DIC_CELL p[50];
  int i=0;
  FILE *fp;
  char end[3];

 // プログラム利用方法表示
  if ( argc != 2 ) {
    fprintf( stderr, "Usage: %s filename\n", argv[0] );
    exit( 1 );
  }

  //ファイルオープン
 if (( fp = fopen( argv[1], "r" ) ) == NULL ) {
    fprintf( stderr, "File open error %s\n", argv[1] );
    exit( 1 );
  }

 for(i=0;i<50;i++){   
 p[i].english = (char*)malloc(sizeof(char));
 p[i].mean = (char*)malloc(sizeof(char));
 }

 // データ読み込み&作成
  while( fscanf( fp, "%s %s", dat,zat) != EOF ) {
    strcpy(p[i].english, dat);
    strcpy(p[i].mean, zat);
   i++;
 }

  while(1){
    printf("英単語を入力して下さい --->");
    scanf("%s" , name);

    //検索
    mean = search(p , 0 , 49 , name);

    //表示
    if(strcmp(mean,"NOT") == 0){
      printf("%sの意味 : 辞書に登録されていません\n" , name);
    }
    else{
      printf("%sの意味 : %s\n" , name,mean);
    }

    //継続の確認
    printf("続けますか? (yes/no) --->");
    scanf("%s" , end);
    if(strcmp(end,"no") == 0) break;
  }
  printf("プログラムを終了します.\n");

  fclose(fp);
  return 0;
}

Answer 1

質問の追記のコードなら次のようにすれば strcpy で読み込めます。

typedef struct DIC_CELL {
  char english[100];
  char mean[100];
} DIC_CELL;

質問の最初のコードなら、strdup で領域確保とコピーを同時に行えます。

   p[i].english = strdup(dat);
   p[i].mean = strdup(zat);

strdup は多くの処理系に用意されていますが、標準関数ではありません。
その時は、自分で定義すればいいでしょう。

char *strdup(const char *s)
{
	char *p = malloc(strlen(s)+1);
	if (p) strcpy(p, s): 
	return p;
}

Comments

[tamintya]

回答ありがとうございます。
ほしい結果を得ることができました。

[kazuma-s]

ほしい結果を得ることができたのは偶然です。
読み込みファイルの行数は何行ですか？
データが 50個ないのに、mean = search(p , 0 , 49 , name); はダメでしょう。

[tamintya]

確かにその通りですね。
mean = search(p , 0 , i , name)と直したのですがこれで問題ないでしょうか？
またほかに危ない個所はありますか。

[kazuma-s]

問題あるかないか、なぜ分からないのでしょうか？
i の値はいくつですか？
「試したこと　gdbコマンドによる値の確認」と書いていますが本当に確認していますか？
mean = search(p , 0 , i-1 , name); ではないのですか？

[tamintya]

i がデータの数になっていたので問題ないと思ってしまいました。
注意していただきありがとうございます。

Answer 2

    // データ読み込み&作成
    while (fscanf(fp, "%s %s", dat, zat) != EOF) {
        p[i].english = dat;
        p[i].mean = zat;
        i++;
    }

ここがダメ。同じアドレス値(dat, zat) をセットすれば同じ文字列になるのはアタリマエ。

int x = 1;
int* p0 = &x;
x = 2;
int* p1 = &x;
// *p0、*p1 ともに 2 となる。
// あなたはコレと同じことしてる。

Comments

[tamintya]

回答ありがとうございます。
知識不足で理解できなかったのでもう少し詳しい説明をお願いしたいです。

[tamintya]

理解することができました。

[tamintya]

間違いを理解することは出来たのですが、
解決方法が分からないので少し教えてもらいたいです。

[episteme]

datそのものをp[i]にセットするのがマチガイなんだから、"複製"を セットすればいいんだがな。

[tamintya]

複製をセットするとは strcpy を用いて値を代入するということであっていますか？
またそれを行うとセグメンテーション違反が発生してしまうのですがそれはなぜなのでしょうか？

[episteme]

コピー先:p[i] がドコを指してるんです?

[tamintya]

mallcを用いて動的確保を行ったのですがまだセグメンテーション違反が発生してしまいます。
やり方が違うのでしょうか？

[episteme]

コード見せてもらわんことにはなんとも...

[tamintya]

コードは質問文に追記しました。

[pepperleaf]

malloc()で 一バイト分しか確保してないですが、OK ?

[tamintya]

確かにそうですね。　見落としていました。
しかしその部分を p[i].english = (char*)malloc(sizeof(char)*100) に変更しても
セグメンテーション違反が直らないのですがそれはどうしてなのでしょうか？

[pepperleaf]

ここの iの値は ? 多分、その前の for()の終わったあたいなので、 51から?
デバッグできてます?

[episteme]

いや、デバッグしてないよね...

[tamintya]

ほしい結果を得ることができました。
ありがとうございました。