test.csvの内容が以下のように100件位あったとして、

質問

test.csvの内容が以下のように100件位あったとして、
d,4,ddddd
bb,2,bbbbb
a,1,aaa
cc,3,ccccc
以下略

やりたい事としてtest.csvを2列目をキーとして昇順にソートし、新たなファイル「test1.csv」(こちらは全件表示)に書き込み、
また、test1.csvのキリのいい所だけ抜き出したcsvファイル「test2.csv」(1件目、10件目、20件目、…と以下10の倍数毎のデータだけを書き込む)
を作りたいと考えて下の様なプログラムを作っていますが、ソートの段階で詰まっています。
もしよろしければこの続きを教えて頂けないでしょうか？

#include <stdio.h>

#define MBF 256
#define SIZE 32
#define FNAME1 "test.csv"
#define FNAME2 "test1.csv"
#define FNAME3 "test2.csv"
struct tb{
char a[SIZE];
char b[SIZE];
char c[SIZE];
};

int main(void)
{
struct tb tbl[20];
struct tb *tp;
int ntb,itb;
FILE* fi;
FILE* fo;
FILE* fq;
char buff[MBF];

//== 入力CSVファイルからの読み込み処理 ==
fi = fopen( FNAME1, "r" );
if( fi == NULL ){
printf( "ファイル\"%s\"が開けません\n", FNAME1 );
return -1;
}

while( fgets(buff, MBF, fi) != NULL )
{
strcpy(test.a,strtok(buff,",\""));
strcpy(test.b,strtok(NULL,",\""));
strcpy(test.c,strtok(NULL,",\""));
}
fclose( fi );

//== 出力CSV(test2.csv)ファイルへの書き込み処理 ==
fo = fopen( FNAME2, "w" );
if( fo == NULL ){
printf( "ファイル\"%s\"が開けません\n", FNAME2 );
return -1;
}
for ( itb=0; itb<ntb; itb++ ){
//書き込み処理？
}
fclose( fo ); //出力ファイルクローズ

//== 出力CSV(test2.csv)ファイルへの書き込み処理 ==
fq = fopen( FNAME2, "w" );
if( fq == NULL ){
printf( "ファイル\"%s\"が開けません\n", FNAME3 );
return -1;
}
for ( itb=0; itb<ntb; itb++ ){
//書き込み処理？
}
fclose( fq ); //出力ファイルクローズ

return 0;
}

回答No.5

anicicle · ベストアンサー

No.4
>tbpはキャスト必要でしたか…参考にしたHPにはキャストする必要無いみたいな事書いてあったので…
malloc()の戻りは「void *」です。利用には通常キャストが必要です。

>>struct tb tbl[SIZE];
（中略）
>また、他にもwhile文やfprintf文のtblも同様に書き換えなければいけないんですよね？
はい。
malloc()で確保したメモリは「確保しただけ」です。
確保したものを利用するためにはtbl[]の部分は全て変更が必要です。

補足

いつもありがとうございます。

anicicle様の的確なアドバイスのおかげで無事、「Segmentation fault」も出ることなく
大きなデータでもcsvファイルソートが出来る事が出来ました。

一応同じ悩みの方が出てきた時用に、ソースを残しておきます。（コンパイル済み）

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

#define NAME "test.csv"
#define NAME1 "test1.csv"
#define NAME2 "test2.csv"

#define SIZE 1024

struct tb{
char a[SIZE];
char b[SIZE];
char c[SIZE];
};

int comp(const void *, const void *);

int main(void)
{

struct tb *tbp; //構造体のポインタ
FILE *fp,*fq,*fr;

char buff[SIZE], string_buff[SIZE];
int j,n;

tbp =(struct tb *)malloc(sizeof(struct tb)*SIZE); //メモリの確保

if((fp=fopen(NAME,"r")) == NULL){
printf("ファイル%sが開けません\n",NAME);
return -1;
}

if((fq=fopen(NAME1,"w")) == NULL){
printf("ファイル%sが開けません\n",NAME1);
return -1;
}

if((fr=fopen(NAME2,"w")) == NULL){
printf("ファイル%sが開けません\n",NAME2);
return -1;
}

j=0;

while(fgets(buff,SIZE,fp)!=NULL){
if( j>= SIZE ){
printf("buffer Over!");
break;
}
strcpy((tbp+j)->a,strtok(buff,",\""));
strcpy((tbp+j)->b,strtok(NULL,",\""));
strcpy((tbp+j)->c,strtok(NULL,",\""));

j++;
}
fclose(fp);

printf("test.csvの表示");
for(n=0; (n<j) && (n<SIZE); n++){
printf("%s %s %s\n",(tbp+n)->a,(tbp+n)->b,(tbp+n)->c);
}
printf("\n");

qsort(tbp, j, sizeof(struct tb), comp);

printf("ソート後\n(test2.csv?)");
for(n=0; (n<j) && (n<SIZE); n++){
printf("%s %s %s\n",(tbp+n)->a,(tbp+n)->b,(tbp+n)->c);
fprintf(fq,"%s,%s,%s\n",(tbp+n)->a,(tbp+n)->b,(tbp+n)->c);
}
fclose(fq);
printf("\n");

printf("キリのいい所だけ(test3.csv?)\n");

for(n=0; (n<j) && (n<SIZE); n++){
if(n%10==0){ //「%10」ここを変えればキリ番を自由に設定可
printf("%s %s %s\n",(tbp+n)->a,(tbp+n)->b,(tbp+n)->c);
fprintf(fr,"%s,%s,%s\n",(tbp+n)->a,(tbp+n)->b,(tbp+n)->c);
}
}
fclose(fr);

printf("\n");

free(tbp); //メモリの開放

return 0;
}

int comp(const void *_p0, const void *_p1) {
struct tb *p0 = (struct tb *)_p0;
struct tb *p1 = (struct tb *)_p1;

return strcmp(p0->b, p1->b); //ここを変えれば好きなキー(列)でソート可
}

本当にありがとうございました。

回答No.4

anicicle

No.1、3です。
ソースがどうりだとしたら、これコンパイル通りませんよ？
軽くコンパイラを通して文法チェックしただけですが
-----------------------
>int main(main)
int main(void)

>struct tb tbl[SIZE];
mallocで使用するように変更したのなら不要。変数宣言自体がメモリの無駄遣い

>tbp =malloc(sizeof(struct tb)*件数); //メモリの確保？(追加)
tbp = (struct tb *)malloc(sizeof(struct tb)*SIZE); //メモリの確保？(追加)

>if((fq=fopen(NAME1,"w")) == NULL){
>if((fq=fopen(NAME2,"w")) == NULL){
定義がない

>fprintf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
fprintf(fp, "%s %s %s\n", tbl[n].a, tbl[n].b,tbl[n].c);

>fcolse(fq);
fclose(fq);
-----------------------
に問題があります。
ソース通りで、exeファイルを実行したとしたら、それは「コンパイルができていた」時点の古いものです。
よく作成日時を確認してみてください。
それとmalloc()で確保した「tbp」を使用するなら、根本的に「tbl」の部分の置き換えも必要です。

補足

ご指摘ありがとうございます。ご指摘頂いた所はコピペミスでした。
大変失礼致しました。

tbpはキャスト必要でしたか…参考にしたHPにはキャストする必要無いみたいな事書いてあったので…

>>struct tb tbl[SIZE];
>mallocで使用するように変更したのなら不要。変数宣言自体がメモリの無駄遣い
という事ですが、これを消すとqsortエラーが出るのですが、これも

>根本的に「tbl」の部分の置き換えも必要です。
という事で書き換えで何とかなるという事ですよね？
また、他にもwhile文やfprintf文のtblも同様に書き換えなければいけないんですよね？

質問ばかりでほんとすいません。

回答No.3

anicicle

No.1です。
>しかし大きなデータのcsvファイルにしたら「Segmentation fault」に
「Segmentation fault」の原因は「メモリアクセスの不備」によるものです。
確保していない領域をアクセスすることが主な原因です。

目立つところでは、
>struct tb tbl[SIZE];
ですね。
最低でも(SIZE*3)*SIZE、1024なら、3MByte、コレだけの領域をローカル変数で確保することはできないと考えたほうがいいでしょう。
tblはmalloc()で領域を確保してください。

>while(fgets(buff,SIZE,fp)!=NULL)
も危険です。
読み込むデータの件数が超えた場合に問題が発生します。
while(fgets(buff,SIZE,fp)!=NULL)
{
__j++;
__if( j>= SIZE )
__{
____printf("buffer Over!");
____break;
__}
}
のようにフェイルセーフは入れておくべきです。

>for( n = 0; n < j; n++ )
も同様。
for( n = 0; (n < j) && (n < SIZE); n++ )
としておくと危険度が減ります。

補足

チェックありがとうございます。また自分なりに修正してみたのですが(と言ってもアドバイス頂いた所をバカ正直に直してみただけですが…)
まだ、「Segmentation fault」になるという事はまだ、どっかが足りないという事ですよね。
もしかしたら、修正すら上手くいって無いとか…

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

#define NAME "test.csv"
#define SIZE 1024

struct tb{
char a[SIZE];
char b[SIZE];
char c[SIZE];
};

int comp(const void *, const void *);

int main(main)
{
struct tb tbl[SIZE];
struct tb *tbp; //構造体のポインタ(追加)
FILE *fp,*fq,*fr;

char buff[SIZE], string_buff[SIZE];
int j,n;

tbp =malloc(sizeof(struct tb)*件数); //メモリの確保？(追加)

if((fp=fopen(NAME,"r")) == NULL){
printf("ファイル%sが開けません\n",NAME);
return -1;
}

if((fq=fopen(NAME1,"w")) == NULL){
printf("ファイル%sが開けません\n",NAME1);
return -1;
}

if((fr=fopen(NAME2,"w")) == NULL){
printf("ファイル%sが開けません\n",NAME1);
return -1;
}

j=0;

while(fgets(buff,SIZE,fp)!=NULL){
if( j>= SIZE ){
printf("buffer Over!");
break;
}
strcpy(tbl[j].a,strtok(buff,",\""));
strcpy(tbl[j].b,strtok(NULL,",\""));
strcpy(tbl[j].c,strtok(NULL,",\""));

j++;
}
fclose(fp);

printf("test.csvの表示");
for(n=0; (n<j) && (n<SIZE); n++){
printf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
}
printf("\n");

qsort(tbl, j, sizeof(struct tb), comp);
printf("ソート後\n(test2.csv?)");
for(n=0; (n<j) && (n<SIZE); n++) {
printf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
fprintf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
}
fcolse(fq);
printf("\n");

printf("キリのいい所だけ(test3.csv?)\n");

for(n=0; (n<j) && (n<SIZE); n++){
if(n%10==0){
printf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
fprintf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
}
}
fclose(fr);

printf("\n");

free(tbl); //メモリの開放？(追加)

return 0;
}

int comp(const void *_p0, const void *_p1) {
struct tb *p0 = (struct tb *)_p0;
struct tb *p1 = (struct tb *)_p1;

return strcmp(p0->b, p1->b);
}

回答No.2

kmee

えーと、Cじゃないと駄目ですか?

Cのプログラムを作ること自体が目的でないのなら、Perlとかで使うと楽なんですが

補足

>>えーと、Cじゃないと駄目ですか?

Cオーダーです。申し訳ありません。

回答No.1

anicicle

とりあえず、test.csvを一旦全部読みこんで、そのデータをソートしてからtest1.csvに書き出す。
test1.csvから適宜必要な部分を読み込み、test2.csvを作る。

それと、提示の者にはtest2.csvへの書き込み処理ないよ？

お礼

お礼の所ですいません。
printfにfprintfも追加したら対象ファイルに書き込めました。
しかし大きなデータのcsvファイルにしたら「Segmentation fault」に
もう何がなんだかわからない状態です。ちゃんとfopen,fcloseはしてあるのですが…

補足

ありがとうございます。ご提示いただいたやり方かどうか分かりませんが、自分なりにつくってみました。こちらのプログラムで
test.csvファイルの表示、2列目をキーとしたソート(test1.csv)、キリのいい所の表示(test2.csv)は出来たのですが、
あとはそれぞれのファイルへの書き込み処理をどうしようか迷っています…

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

#define NAME "test.csv"
#define SIZE 1024

struct tb{
char a[SIZE];
char b[SIZE];
char c[SIZE];
};

int comp(const void *, const void *);

int main(main)
{
struct tb tbl[SIZE];
FILE *fp;

char buff[SIZE], string_buff[SIZE];
int j,n;

clock_t start,end;

start = clock();

if((fp=fopen(NAME,"r")) == NULL){
printf("ファイル%sが開けません\n",NAME);
return -1;
}

j=0;

while(fgets(buff,SIZE,fp)!=NULL){
strcpy(tbl[j].a,strtok(buff,",\""));
strcpy(tbl[j].b,strtok(NULL,",\""));
strcpy(tbl[j].c,strtok(NULL,",\""));

j++;
}
fclose(fp);

printf("test.csvの表示");
for(n=0; n<j; n++){
printf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
}
printf("\n");

qsort(tbl, j, sizeof(struct tb), comp);
printf("ソート後\n(test2.csv?)");
for(n=0;n < j;n++) {
printf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
}
printf("\n");

printf("キリのいい所だけ(test3.csv?)\n");

for(n=0; n<j ;n++){
if(n%10==0){
printf("%s %s %s\n",tbl[n].a,tbl[n].b,tbl[n].c);
}
}

end = clock();

printf("%.30f秒かかりました\n",(double)(end-start)/CLOCKS_PER_SEC);
printf("\n");

return 0;
}

int comp(const void *_p0, const void *_p1) {
struct tb *p0 = (struct tb *)_p0;
struct tb *p1 = (struct tb *)_p1;

return strcmp(p0->b, p1->b);
}