C言語でのCSVへの出力がうまくいきません…

質問

風力発電の発電データを1秒ごとに1日分を記録したCSVファイルを読み込み、集計して、集計用の新しいCSVファイルに書き込むプログラムを組もうとしています。以下にソースコードを記載します。

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

static char filename[100];
static char wfname[100];
char fname2[256];

void str_copy(char[],char[]); //文字列のコピー
void str_cat(char[],char[]); //文字列の連結

struct kaze {
_char jikoku[16];
_double fuusoku;
_double fuuko;
_double denatu;
_double denryoku1;
_double denryoku2;
_double co2;
};

struct kaze data[86400];

int main()
{
_char date2[50];
_char date3[50];
_char month2[10];
_char year2[10];
_char path1[50]="F:\\風力発電\\風車";
_char path2[10]="年";
_char path3[10]="月\\WP";
_char buff[256];
_int month1,month,year,date;
_int kensuu,i,j,uruu;
_FILE *fpin,*fpout;

_printf("集計する年を入力してください。\n");
_scanf("%d",&year);

_printf("集計する月を入力してください。\n");
_scanf("%d",&month);
_if(month==2){
__printf("閏年ですか？(はい -> 1　いいえ -> 2)");
__scanf("%d",&uruu);
_}

_if(month==2){
__if(uruu==1){
___month1=29;
__}
__else{
___month1=28;
__}
_}
_else if(month==4 || month==6 || month==9 || month==11){
__month1=30;
_}
_else{
__month1=31;
_}

_date=year*100+month;
_sprintf(date3,"%d",date);
_str_copy(wfname,date3);
_str_cat(wfname,".csv");
_sprintf(year2,"%d",year);
_sprintf(month2,"%d",month);
_sprintf(fname2,"%s%s%s%s%s%s",path1,year2,path2,month2,path3,wfname);

_if((fpout=fopen(fname2,"w"))==NULL){
__printf("ファイルが作成できません。 --- %s\n",fname2);
__exit(EXIT_FAILURE);
_}
_fprintf(fpout,"日付,平均風速,平均風向,平均電圧,平均発電電力、発電電力量,CO2削減量\n");
_fprintf(fpout,"年/月/日,m/s,°,V,KW,KWh,kg-c\n");
_printf("集計用ファイルを作成しました。 --- %s\n",fname2);
_fclose(fpout);

_date=year*10000+month*100;
_for(j=1;j<=month1;j++){
__double fuusoku=0,fuuko=0,denatu=0,denryoku1=0,denryoku2=0,co2=0;
__date++;
__sprintf(date2,"%d",date);
__str_copy(filename,date2);
__str_cat(filename,".csv");

__sprintf(buff,"%s%s%s%s%s%s",path1,year2,path2,month2,path3,filename);
__if((fpin=fopen(buff,"r"))==NULL){
___printf("ファイルが見つかりません。--- %s\n",buff);
___exit(EXIT_FAILURE);
__}

__for(kensuu = 0; fscanf(fpin, "%[^,],%lf,%lf,%lf,%lf,%lf,%lf\n", data[kensuu].jikoku, &data[kensuu].fuusoku,
&data[kensuu].fuuko, &data[kensuu].denatu, &data[kensuu].denryoku1,
&data[kensuu].denryoku2, &data[kensuu].co2) != EOF; kensuu++);

__for(i = 0; i < kensuu; i++){
___fuusoku+=data[i].fuusoku;
___fuuko+=data[i].fuuko;
___denatu+=data[i].denatu;
___denryoku1+=data[i].denryoku1;
}
__fuusoku/=86400;
__fuuko/=86400;
__denatu/=86400;
__denryoku1/=86400;
__denryoku2=data[kensuu-1].denryoku2;
__co2=data[kensuu-1].co2;

__fclose(fpin);

__printf("\n%d年%d月%d日の集計結果\n平均風速：%lf(m/s)\n平均風向：%lf(°)\n
__________平均電圧：%lf(V)\n",year,month,j,fuusoku,fuuko,denatu);
__printf("平均発電電力：%lf(kW)\n発電電力量：%lf(kWh)\nCO2削減量：%lf(kg-c)\n"
,denryoku1,denryoku2,co2);

//ここまでは正常に動作する。

●if((fpout=fopen(fname2,"a"))==NULL){
___printf("ファイルに追記できません。--- %s\n",fname2);
___exit(EXIT_FAILURE);
__}
__fprintf(fpout,"%d/%d/%d,%s,%s,%s,%s,%s,%s\n",year,month,j,fuusoku,fuuko,denatu,denryoku1
,denryoku2,co2);

__printf("以上の結果を記録しました。 --- %s",fname2);
__fclose(fpout);
}
_return EXIT_SUCCESS;
}

void str_copy(char dst[],char src[])
{
_int i;

_for(i=0;src[i]!='\0';i++){
__dst[i]=src[i];
_}
_dst[i]='\0';
}
void str_cat(char dst[],char src[])
{
_int i,j;

_for(i=0;dst[i]!='\0';i++)
;
_for(j=0;src[j]!='\0';j++,i++){
_dst[i]=src[j];
_}
_dst[i]='\0';
}

集計用ファイルの作成と項目の書き込み、データファイルの読み込み、集計、ディスプレイへの表示までは問題なく動作するのですが、●印のところで変数fname2がおかしなことになっており、

　　　ファイルに追記できません。--- 23:59:48

と表示されます。どこかでfname2が書き換えられているのだとは思いますが、実際にどこでどうなっているのか、全くわかりません。
どなたか原因がわかる方、もしくは「ここじゃないかな？」など心当たり程度でも構いません。
ご意見をお寄せください！お願いします！

回答No.4

麻野 なぎ（@AsanoNagi） · ベストアンサー

まずは、fname2 を出力してみて、どうなっているか確認すれば良いとは思います。

__sprintf() って、標準の sprintf() と機能的に同等だとして……。

__sprintf(month2,"%d",month);
で、「月」の部分を生成しているので、9月までだと、ファイル名の中にスペースが入り、悪さをしそうなきはします。

で、わざわざ、__sprintf でファイル名を生成しているのですから、
__sprintf(fname2,"%s%s%s%s%s%s",path1,year2,path2,month2,path3,wfname);
は
__sprintf(fname2,"F:\\風力発電\\風車%d年%02d月\\WP%s", year, month, wfname);
とすれば、一発です。（month に対応する書式指定子は %02d にしました）
中間の,
year2 や month2 も不要になります。
それでも、paht1 や path2 をソースに埋め込みたくないのであれば、

#define path1 "F:\\風力発電\\風車"
#define path2 "年"
#define path3 "月\\WP"
として、
__sprintf(fname2, path1 "%d" path2 "%02d" path3 "%s", year, month, wfname);
でも、OKです。（path1 とかの前後がスペースで区切ってあるのは、実は、正解です）

あるいは、

#define fullPath "F:\\風力発電\\風車%d年%02d月\\WP%s"
で、
__sprintf(fname2, fullPath, year, month, wfname);
でもOK。

さらに、
__sprintf(date2,"%d",date);
__str_copy(filename,date2);
__str_cat(filename,".csv");
は、

__sprintf(filename, "%d.csv", date);
と一行ですみます。

お礼

回答ありがとうございました！
fname2を出力してみたところ、途中から入力元のCSVファイルの時刻が格納されていました。
それでよくよく見直してみると、データを格納する構造体配列data[]の数が足りていなかったようです。CSVファイルのデータは86400秒分でも、その上にも行があるのだから、data[86400]では足りないのは当然ですね(笑)

初歩的なミスでお恥ずかしい限りです…///
改めて、ありがとうございました！

回答No.8

麻野 なぎ（@AsanoNagi）

No.4 です。
いろいろと見落としがありました。>#5
特に、

> >_sprintf(month2,"%d",month);
> >で、「月」の部分を生成しているので、9月までだと、ファイル名の中にスペースが入り、悪さをしそうなきはします。
> スペースは入りません。

は、最初見たときには、頭の中で、"%2d" になっていました。
きっと、sprintf(month,"%2d",month); に脳内で変換していたのでしょう。

回答No.7

Tacosan

fscanf で読み込めたかどうかを「EOF との比較」で調べているので, 「ファイルの内容が期待した書式にあっていない」場合にオーバーランする可能性があります＞#5.

単純に行数が多すぎてもアウトだけど.

回答No.6

Wr5

sizeof(int) == 2でintがsignedの環境だった場合…なら、

>for(kensuu = 0; fscanf(fpin, "%[^,],%lf,%lf,%lf,%lf,%lf,%lf\n", data[kensuu].jikoku, &data[kensuu].fuusoku,&data[kensuu].fuuko, &data[kensuu].denatu, &data[kensuu].denryoku1,&data[kensuu].denryoku2, &data[kensuu].co2) != EOF; kensuu++);
で、行数によってはfname2に時刻な情報が入る…かも知れません。
# 先頭から65532行目のデータになる…かな。

もっとも…そんな環境なら、その前に吹っ飛びそうなものですが……。
# 具体的には32768行目で。data[-32768]なんて場所にアクセスしようとしますから……。

回答No.5

Wr5

>ファイルに追記できません。--- 23:59:48

と表示されるのですから、fname2の内容は"23:59:48"になっている…ということですよね？
読み込んでいるデータに"23:59:48"とか含まれていたりしないでしょうか？
どこから"23:59:48"が出てきたのか…が謎ですかねぇ……。

コード見る限りでは…バッファオーバーランやバッファアンダーランなどで書き換えられていると思われるところは判りませんでしたが……。

入力ファイルからのfscanf()が失敗した場合に
>denryoku2=data[kensuu-1].denryoku2;
>co2=data[kensuu-1].co2;
がどうなるやら…というのはありますが……。

ループの中でfname2を表示する。
というprintfデバッグで追いかけることになりますかね……。

>__sprintf() って、標準の sprintf() と機能的に同等だとして……。

頭の_はインデントの代わりのようです。
ということで、「標準の sprintf()」でしょう。
なので……
>_sprintf(month2,"%d",month);
>で、「月」の部分を生成しているので、9月までだと、ファイル名の中にスペースが入り、悪さをしそうなきはします。
スペースは入りません。
# "%2d"なら入るでしょうけど。

回答No.3

asuncion

＞void str_copy(char[],char[]);//文字列のコピー
＞void str_cat(char[],char[]);//文字列の連結

自作する必要が本当にあるのでしょうか。

回答No.2

春原 なの（@ymda）

こんにちわ。

こういうソースの書き方があるんですね。。 ＜ _

とりあえず、これを動かせないので、一部気になる所より・・

＃まとあたってないかもしれませんが・・

■strcat その実装合ってます？
理論上、あってるとはおもうのですが・・念のため strcpyもつけておきます。
（こちらのがシンプルなので、FreeBSDよりソースを拾ってます）

char *strcpy(char *to, char *from)
{
char *save = to;

for (; (*to = *from); ++from, ++to);
return(save);
}

char *strcat(char *s, char *append)
{
char *save = s;

for (; *s; ++s);
while ((*s++ = *append++));
return(save);
}

■正常に動作するの次で、fname2 を printfして
すぐexitすると、どうなりますか？

■これを・・
　fpout=fopen(fname2,"a"))==NULL){
　から
　fpout=fopen(fname2,"w"))==NULL){

に変更すると、どうなりますか？

一応、仕様上、"a" は
追加 (ファイルの最後に書き込む) のために開く。 ファイルが存在していない場合には新たに作成する。 ストリームはファイルの最後に位置される。

ではありますが・・

その他、
最初の、staticを外すとどうか？
単純に文字数等がオーバーフローしている

可能性もあります。

char abc[100]
と記載されているところは、出来る限り
文字数を調べたりしながら
mallocで確保して、不要になったらfreeすることを
おすすめします。

回答No.1

Tacosan

とりあえず printf してみる.