C言語の文字列の処理について

Question

C言語の文字の扱いについてわからないことがあるため質問させてもらいます。
ｔｘｔファイルに以下のような文字列が入っているとします。

Apple 1 2 3 4
Orange 2 4 6 8
Lemon 3 6 9 12

このテキストファイルはスペース区切りで名前、数字4つのように構成されています。
プログラムで扱いやすいように以下のような構造体を定義しました。

struct list{
    char* name;
    int num[4];
};
typedef struct list *listp;

この構造体のポインターの配列を用意してそこにtest.txtから読み込んだ値を入れていきたいのですが、うまくいきません。
プログラムは以下のようです。

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

struct list{
    char* name;
    int num[4];
};
typedef struct list *listp;

int main(int argc, const char * argv[]) {
    FILE *fp;
    char *fname = "test.txt";
    char s[100];
    char *ptr;
    int i = 0, j = 0;
    listp item[3];
    
    fp = fopen(fname, "r" );
    while(fgets(s, 100, fp) != NULL){
        ptr = strtok(s, " ");
        item[i]->name = ptr;
        while(ptr != NULL) {
            ptr = strtok(NULL, " ");
            if(ptr != NULL) {
                item[i]->num[j] = *ptr;
                j += 1;
            }
        }
        i += 1;
    }

とりあえずfgetsでtxtを一行ずつ読み取って、strtokで文字列を分割して処理しているのですが、エラーが出てしまいます。ptrはcharのポインターなので代入するだけではダメだと思うのですが、具体的にどのようにしたら良いのかわかりません。構造体もこれでいいのかよくわかりません。とりあえずtxtをいい感じにプログラムで扱えるようにしたいです。プログラムの間違いや、根本的にこのようなtxtを扱うのに良いやり方があればお教えください。よろしくお願いします。

Answer 1

listp item[3];の各要素はポインタなので、item[i]->name = ～;とその先に代入しようとしても、その先には何もありません。
使用に先立って、item[i]=(listp)malloc(sizeof(struct list));などと、ポインターにその指す変数用領域アドレスの代入が必要です。

また、item[i]->name = ptr;と、s[100]の中のアドレスを代入していますが、s[100]は1行毎に上書きされるので、次の行を読むとそのアドレスに書かれている文字は変わってしまいます。

num[j]に文字を代入していますが、やりたいのは文字列を十進数とみなしてintにしてその値を代入することでしょうから、atoi()等が必要です。

ざっとみて気づいた物を書きましたが、もっと他にも間違いがあるかと思います。

ポインタについての理解はあるようですが、ポインタの使い方の理解が全く出来ていませんので、もっと簡単なプログラムから始めるのが良いと思います。

Comments

[退会済みユーザー]

ご回答ありがとうございます。あまりC言語を書かないのでだいぶ苦労しています。

>また、item[i]->name = ptr;と、s[100]の中のアドレスを代入していますが、s[100]は1行毎に上書きされるので、次の行を読むとそのアドレスに書かれている文字は変わってしまいます。

この問題を回避するには構造体のnameをポインター変数ではなく、普通の変数にしてstrcpyを使えば解消できるでしょうか？

[otn]

> この問題を回避するには構造体のnameをポインター変数ではなく、普通の変数にしてstrcpyを使えば解消できるでしょうか？

それもありですね。配列サイズを決められるのなら、それが簡単です。

[otn]

listp item[3]; についても同様。
struct list item[3]; なら各要素がポインタじゃなくて構造体そのものになるので、item[i].num[j] = ～ とかすればいい。

Answer 2

裏技ですがこんな方法もあります。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
typedef struct{
  int num[4];
  char name[1];
} *listp;

/*
    戻り値 作成したlist構造体へのポインタ 呼び出した側が責任持ってfreeすること
 */
listp createList(const char* name, int num1, int num2, int num3, int num4){
  size_t len = strlen(name);
  listp result = malloc(sizeof(listp[0]) + len);
  result->num[0] = num1;
  result->num[1] = num2;
  result->num[2] = num3;
  result->num[3] = num4;
  strcpy(result->name, name);
  return result;
}
/*
    戻り値 作成したlist構造体へのポインタ 呼び出した側が責任持ってfreeすること
 */
listp createListFromString(const char* str){
  char name[100] = {};
  int num1, num2, num3, num4;
  sscanf(str,"%99s %d %d %d %d", name, &num1, &num2, &num3, &num4);
  return createList(name, num1, num2,num3,num4);
}

int main(){
  listp p = createList("test", 10, 200, 3000, 4);
  printf("[%s]: [%d] [%d] [%d] [%d]\n", p->name, p->num[0], p->num[1],
                                                 p->num[2], p->num[3]);
  free(p);
  p = createListFromString("test2 1 2 3 4");
  printf("[%s]: [%d] [%d] [%d] [%d]\n", p->name, p->num[0], p->num[1],
                                                 p->num[2], p->num[3]);
  free(p);
  return 0;
}

Comments

[退会済みユーザー]

回答有り難うございます。
無事解決できました。

Answer 3

たぶん何かの課題だと思われますが、動的メモリを使うという問題でないのであれば、構造体の定義を見直すべきでしょう。
現在の定義はnameがポインタになっております。
ポインタはあくまでもどこかの領域のアドレス値を持つだけで、今の作りのように変数sの一部のアドレスを格納しても、領域は常に同じなため意味がありません。
nameはポインタはなく実体としましょう。

struct list{
    char name[16];
    int num[4];
};

サイズは仮に16バイト（15文字＋NULL文字）としていますが、文字数の上限の決まりがあるならそれにあわせてください。
あとはstrtokで切り出したあたいをコピーすればよいでしょう。

strcpy(item[i]->name, ptr);

ただし、numに関してはint型なので数値に変換する必要があります。
atoi関数あたりを使うのが妥当です。

item[i]->num[0] = atoi(ptr);

Comments

[退会済みユーザー]

回答有り難うございます。
無事解決できました。