c言語でテキストファイル内のデータを一つだけ表示するプログラム

Question

#実現したいプログラム
c言語でテキストファイル内の何人分かのデータ（名前　番号）を配列に格納し、乱数を用いてその中の一人分の名前と番号のみを表示する。
#ソースコード

#include <stdio.h>
#include <stdlib.h>   
#include <time.h>
#define NINZU 3

typedef struct str {
	char name[50];
    char gakuseki[7];      
} data;

int main(void){
   	FILE *fp;   
	char fname[] = "gakuseki.txt";
	char line[256];     //1行の最大文字数
	char name[50];		//名前用文字配列
    char gakuseki[7];      //学籍番号
    int i,j;

    data data[NINZU];      
    srand(1); // 乱数の初期化

 //ファイルオープン動作
	fp = fopen(fname, "r"); // ファイルを開く。失敗するとNULLを返す。
	if(fp == NULL) {
		printf("ファイルが開けん!\n");
		return -1;
	}

    //while開始
	while(fgets(line, 256, fp) != NULL) {
		sscanf(line, "%s %s", name, gakuseki);

        i=rand()%NINZU;        //ランダム関数呼び出し
        for(j = 0; j  < sizeof(data[i].name) / sizeof(data[i].name[0]); j++) {
			data[i].name[j] = name[j];
		}

        for(j = 0; j  < sizeof(data[i].gakuseki) / sizeof(data[i].gakuseki[0]); j++) {
			data[i].gakuseki[j] = gakuseki[j];
		}
	}
    printf("%s %s\n",data[i].name[j],data[i].gakuseki[j]);
	fclose(fp);
		
 return 0;
}

#出力結果
(null) (null)

Comments

[kazuma-s]

そのコードでは出力結果が (null) (null) になりません。
質問を編集して、出力結果が (null) (null) になるコードまたは現在の最新のコードを追記してください。

Answer 1

全体の流れがおかしいです。
作りたいものは、以下の内容ですよね。

テキストファイル内の何人分かのデータ（名前　番号）を配列に格納し、乱数を用いてその中の一人分の名前と番号のみを表示する。

まずは、コーディングに入る前に、ここに書かれている内容をよく理解して、大枠の手順に分解する必要があります。
一番大きな枠としては、こうなります。

1. テキストファイル内の何人分かのデータ（名前　番号）を配列に格納
2. 乱数を用いてその中の一人分の名前と番号のみを表示

この二つの段階に分けて処理を作ります。
今の状態では二つの機能が混ざり合っておかしなことになってしまっているので、
二つを分離させて、片方が終わってから、もう片方に手を付けるようにしてください。

#include <stdio.h>
#include <stdlib.h>   
#include <time.h>
#define NINZU 3

typedef struct str {
    char name[50];
    char gakuseki[7];      
} data;

int main(void){
       FILE *fp;   
    char fname[] = "gakuseki.txt";
    char line[256];     //1行の最大文字数
    char name[50];        //名前用文字配列
    char gakuseki[7];      //学籍番号
    int i,j;

    data data[NINZU];      
    srand(1); // 乱数の初期化

 //ファイルオープン動作
    fp = fopen(fname, "r"); // ファイルを開く。失敗するとNULLを返す。
    if(fp == NULL) {
        printf("ファイルが開けん!\n");
        return -1;
    }

 // テキストファイル内の何人分かのデータ（名前　番号）を配列に格納



 // 乱数を用いてその中の一人分の名前と番号のみを表示



    printf("%s %s\n",data[i].name[j],data[i].gakuseki[j]);
    fclose(fp);

 return 0;
}

という感じに二つの機能を分離して書けば、きちんと書けるはず。

Comments

[yukatii]

アドバイスありがとうございます。
ランダム関数を呼び出す順などを変更し、出力が出るようにはなったのですが、番号のみで名前が表示されません。自分なりに原因を考えてみたのですが、番号と同様の操作を行っているはずなのに片方が出力されない理由がわかりません。
お手数ですが、教えていただけないでしょうか。

以下変更した部分(while文以降)のコードです。
while(fgets(line, 256, fp) != NULL) {
sscanf(line, "%s %s", name, gakuseki);

for(j = 0; j < sizeof(data[i].name) / sizeof(data[i].name[0]); j++) {
data[i].name[j] = name[j];
}

for(j = 0; j < sizeof(data[i].gakuseki) / sizeof(data[i].gakuseki[0]); j++) {
data[i].gakuseki[j] = gakuseki[j];
}
i++;
}
i=rand()%NINZU; //ランダム関数呼び出し
printf("%s %s\n", data[i].name , data[i].gakuseki);
fclose(fp);

return 0;

[amiya]

番号のみで名前が表示されない。というのがよくわからないけど、そもそもの話として、
データはきちんと読めているのか？って問題もありそうなので、
sscanfの前に、lineの内容を出力してみる
sscanfの後に、nameやgakusekiの内容を出力してみる
さらに、forの後に、ほんとにデータが入っているのか、構造体の中身を出力してみる
最後にランダムではなく、0、1，2等の番号を直接入れて出力してみる
とかすると、どこに問題があるか、仕分けできるかと思います。

[yukatii]

なんとかうまくいきました！ありがとうございました！

Answer 2

ファイルからデータを全員分読み込んでから、「0以上(人数-1)以下」の整数値を疑似乱数で生成してその値をインデックスとする必要がある。

細かな問題点としては、

• 疑似乱数列の初期化(シードの指定)は、例えばtime(NULL)を指定する
• ファイルからの読み込み処理が「NINZU分だけデータがファイルに書いてある」という前提でコードを書いているのは危険。少ない場合と多い場合でどうなるか考えよう
• 文字列のコピーはstrncpyなどを使う
• 最後のprintfで表示するのは2つの文字列なので、data[XXX].name[YYY]では型が合わない(これは1文字である)。XXXの値はともかく、形としてはdata[XXX].nameになる(これは文字配列の先頭位置である)

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
#define NINZU 3

typedef struct {
    char name[50];
    char gakuseki[7];
} Data;

int main(void)
{
    FILE *fp;
    char fname[] = "gakuseki.txt";
    char line[256];
    char name[50];
    char gakuseki[50];

    Data data[NINZU];
    srand(time(NULL));

    fp = fopen(fname, "r");
    if (fp == NULL) {
        printf("ファイルが開けん!\n");
        return -1;
    }

    int i = 0;
    while (i < NINZU && fgets(line, sizeof(line), fp) != NULL) {
        sscanf(line, "%s %s", name, gakuseki);

        strncpy(data[i].name, name, sizeof(data[i].name));
        data[i].name[sizeof(data[i].name) - 1] = '\0';

        strncpy(data[i].gakuseki, gakuseki, sizeof(data[i].gakuseki));
        data[i].gakuseki[sizeof(data[i].gakuseki) - 1] = '\0';

        i++;
    }

    int j = rand() % i;
    printf("%s %s\n", data[j].name, data[j].gakuseki);

    fclose(fp);

    return 0;
}