csvファイルを構造体に読み込みたい

質問

C言語初心者です。

csvファイルのデータを1行ずつ構造体に読み込みたいのですが、うまくできません。

#define MAX_SIZE 100

char filename[] = ″test.csv";
struct ALL_DATA{
int cnt;
char list[10];
char addr[10];
char cmd[10];
char csize[10];
char input[10];
char icmd[10];
char input2[10];
};
for (int lp = 0; lp < 10; lp++){
FILE *fp;
struct ALL_DATA ADATA[MAX_SIZE]; // 構造体配列の宣言
if ((fp = fopen(filename, "r")) == NULL){
printf("%s open error !\n", filename);
exit(1);
}

for (fc = 0; fc < MAX_SIZE; fc++) {
if (feof(fp)){
break;
}
else{
fscanf(fp, "%d,%[^,],%[^,],%[^,],%[^,],%[^,],%[^,],%[^,],\n", &ADATA[fc].cnt, ADATA[fc].list, ADATA[fc].addr,ADATA[fc].cmd,ADATA[fc].csize,ADATA[fc].input,ADATA[fc].icmd,ADATA[fc].input2);
}
}
fclose(fp);

while (i < MAX_SIZE){
printf("ADATA[i].cnt -> %d, ADATA[i].list -> %s, ADATA[i].addr -> %s, ADATA[i].cmd -> %s, ADATA[i].csize -> %s, ADATA[i].input -> %s, ADATA[i].icmd -> %s, ADATA[i].input2 -> %s\n",ADATA[fc].cnt, ADATA[fc].list,ADATA[fc].addr,ADATA[fc].cmd,ADATA[fc].csize,ADATA[fc].input,ADATA[fc].icmd,ADATA[fc].input2);
i++;
}
}

読み込みたいcsvデータは
1 test 0x0012 write 0x001 yes 0xabc yes
2 testa 0x00cd read 0x024 0x1a3

のように、「数字、文字列・・・」と並んでいます。
このデータを取り込んで表示させたところ、
最初の数字は問題ないのですが、ADATA[i].addr から値がおかしくなり、
ADATA[i].addr = 0x0012write
ADATA[i].cmd = write
ADATA[fc].csize = 0x001yes

のように、16進数を文字列として構造体に入れるときに、次の文字(英字)まで一緒に含んでしまう現象が出ております。
ネットで調べて、csvファイルから構造体に取り込む方法はいくつか見たのですが、同じようにしている(と思っている)のにうまくいかないため、質問させていただきました。

原因がわかる方がいらっしゃいましたら、アドバイスをよろしくお願いいたします。

回答No.3

wormhole · ベストアンサー

>fscanfの使い方が間違っているのでしょうか？

構造体のid2からはcharの配列のサイズが、'\0'文字分の計上がされていないように見えますけど。

それと質問に元のソースと異なるソースを書かれるのは構いませんが少なくとも現象が再現できるものにしましょう。

お礼

wormholeさん

ご回答、ありがとうございました。
charの配列サイズを修正したところ、目的の結果が得られることを確認できました。
いろいろと不手際があったにもかかわらず、最後までご対応いただきましてありがとうございました。

>それと質問に元のソースと異なるソースを書かれるのは構いませんが少なくとも現象が再現できるものにしましょう。
こちら、最初に貼り付けたものが動作しないものということだったので、再度現象再現するものを貼り付けさせていただいたのですが、こちらも動作しなかったでしょうか？
何度も不手際が合ったようで、今後質問する際には注意したいと思います。

回答No.2

wormhole

>これはcsvファイルではないのでしょうか？
>拡張子はcsvとなっているのですが・・・

カンマで区切られているならcsvファイルです。
前回の回答にComma Separated Valueと書いてますけど意味調べて見ました？

>最初の数字は問題ないのですが、ADATA[i].addr から値がおかしくなり、

質問に書かれているものが実際の物かどうかわかりませんが(そのままじゃコンパイルできないし)、

>printf("ADATA[i].cnt -> %d, ADATA[i].list -> %s, ADATA[i].addr -> %s, ADATA[i].cmd -> %s, ADATA[i].csize -> %s, ADATA[i].input -> %s, ADATA[i].icmd -> %s, ADATA[i].input2 -> %s\n",ADATA[fc].cnt, ADATA[fc].list,ADATA[fc].addr,ADATA[fc].cmd,ADATA[fc].csize,ADATA[fc].input,ADATA[fc].icmd,ADATA[fc].input2);

は、本当にADATA[i].cnt,ADATA[i].list,...の値を出力していますか？
よく見てみてください。

補足

wormholeさん

返信が遅くなりました。
説明が悪かったので、少しコードを修正して確認したものを記載します。

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

#define CSV_SIZE 20

int main(int argc, char *argv[]){
char Log[512];

// csvファイルを構造体へ格納
char filename[] = "/home/nvme/ubuntu_test/sample/test/testlist.csv";

struct CSV_DATA{
int id;
char id2[10];
char cmd[5];
char id3[10];
char oaddr[10];
char oaddr2[10];
char format[3];
char size[10];
char format2[3];
};

FILE *fp;
struct CSV_DATA CSVD[CSV_SIZE]; // 構造体配列の宣言

if ((fp = fopen(filename, "r")) == NULL){
printf("%s open error !\n", filename);
exit(1);
}

for (int fc = 0; fc < CSV_SIZE; fc++) {
if (feof(fp)){
break;
}
else{
fscanf(fp, "%d,%[^,],%[^,],%[^,],%[^,],%[^,],%[^,],%[^,],%s\n", &CSVD[fc].id, CSVD[fc].id2, CSVD[fc].cmd, CSVD[fc].id3, CSVD[fc].oaddr, CSVD[fc].oaddr2, CSVD[fc].format, CSVD[fc].size, CSVD[fc].format2);
}
printf("********** id:%d id2:%s cmd:%s id3:%s oaddr:%s oaddr2:%s format:%s size:%s format2:%s *********\n", CSVD[fc].id, CSVD[fc].id2, CSVD[fc].cmd, CSVD[fc].id3, CSVD[fc].oaddr, CSVD[fc].oaddr2, CSVD[fc].format, CSVD[fc].size, CSVD[fc].format2);
}
}

読込むCSVファイルは以下になります。
6,0x00060001,Write,0x00000001,0x005bea30,0x00000000,Yes,0x0000001f,Yes
7,0x00030000,Flush,0x00000001,0x00000000,0x00000000, ,0x00000000, ,
1,0x00340001,Write,0x00000001,0x00600848,0x00000000,Yes,0x00000007,Yes
のような値が入っています。
これを実行すると、
id:6 id2:0x00060001Write0x000000010x005bea300x00000000Yes0x0000001fYes,, cmd:Write0x000000010x005bea300x00000000Yes0x0000001fYes,, id3:0x000000010x005bea300x00000000Yes0x0000001fYes,, oaddr:0x005bea300x00000000Yes0x0000001fYes,, oaddr2:0x00000000Yes0x0000001fYes,, format:Yes0x0000001fYes,, size:0x0000001fYes,, format2:Yes,, *********
のような値になってしまいます。
id2以降に構造体に入っている値が取得したい値から最後のYesまでになっております。
fscanfの使い方が間違っているのでしょうか？

回答No.1

wormhole

読み込みたいcsvデータは本当に

1 test 0x0012 write 0x001 yes 0xabc yes
2 testa 0x00cd read 0x024 0x1a3

になってるんですか？
もしそうならCSV(Comma Separated Value)データではありませんけど。

補足

wormholeさん
回答ありがとうございます。
読み込みたいファイルをメモ帳で開くと、下記のようになっております。

1,test,0x0012,write,0x001,yes,0xabc,yes
2,testa,0x00cd,read,0x024,,0x1a3,

これはcsvファイルではないのでしょうか？
拡張子はcsvとなっているのですが・・・

記載時に「，」が抜けていたためcsvではないと思われたのであれば、私の誤記です。すいませんでした。