ファイルに文字列を書く

質問

ファイルを2つ読み込んでファイルを書き直そうとしたのですが
2度目に書き込もうとしたときに改行の書き込みがおかしくなってしまいます
してほしいところで改行を一回だけしてほしいと思っているのですが
実際には多数改行されます
どうしたら改行が一回だけになりますか？

---ソース---
#include <stdio.h>

void main(void){

FILE *fp;
char *tm[1000];
char buf[400];
int i=1,sei;

fp= fopen("now.txt","w+");
fprintf(fp,"もも");
fclose(fp);

//何で最初に書き込んでるんだ？
//という突っ込みがあるでしょうが本当に作りたいプログラムは最初にファイルに書き込まないといけないためです。

fp= fopen("now.txt","r");
while( fgets( buf, 400, fp ) != NULL ){
tm[0]=(char*)malloc(strlen(buf)+1);
strcpy(tm[0], buf);
}

fclose(fp);

fp =fopen("moto.txt","r");
while( fgets( buf, 400, fp ) != NULL ){
if(i<999){
tm[i] = (char*)malloc(strlen(buf+1));
strcpy(tm[i], buf);
i++;
}
else{
tm[999] = (char*)malloc(strlen(buf+1));
strcpy(tm[999], buf);
}

}

fclose(fp);

if(i<1000){
sei=i;
}

else{
sei=1000;
}

fp =fopen("chat_deta.txt","w+");

for(i=0;i<sei;i++){
fprintf(fp,"%s\n",tm[i]);
}

if(i!=(sei-1)){
fprintf(fp,"\n");
}

for(i=0;i<sei;i++){
free(tm[i]);
}

fclose(fp);

}

}

---now.txt---
もも

---moto.txt---
オレンジ
みかん

回答No.3

kmee · ベストアンサー

最初に書いたときに、「余計な改行」が付く
↓
次に読むのは、その「余計な改行」が付いたファイル
それを出力するときに、また「余計な改行」が付く

... を繰り返してるんじゃないですか?
このchat_data.txt を、コピーして次はmoto.txt にしてるとか無いですか?

あと
> tm[999] = (char*)malloc(strlen(buf+1));
これ、1000行以上あると、解放できませんね。

補足

＜＜最初に書いたときに、「余計な改行」が付く
↓
次に読むのは、その「余計な改行」が付いたファイル
それを出力するときに、また「余計な改行」が付く

... を繰り返してるんじゃないですか?

まさしくそんな感じですね
もしかしたら余計な改行を入れていたのが原因かもしれませんね
もう一度ソースを見直してみますね

＜＜このchat_data.txt を、コピーして次はmoto.txt にしてるとか無いですか?
言われてきづいたのですが質問用のプログラムとして書いたときに変更し忘れていました
chat_data.txtではなくmoto.txtと書き換えてください

そのためおかしな質問をしていました
申し訳ありません

＞＞あと
> tm[999] = (char*)malloc(strlen(buf+1));
これ、1000行以上あると、解放できませんね。

1001行目をなしにするつもりで書きました
ちょっとここら辺はこのバグを何とかしたらちゃんと1000行になるか確認したかったので、ちゃんとできてる傘は確認してません

回答No.2

Tacosan

fgets の仕様については既に指摘されているところだったりします.
cf. http://okwave.jp/qa/q7819645.html

そしてこのプログラムにもしっかりバグが入っている.

ところで
if(i!=(sei-1)){
fprintf(fp,"\n");
}
における if の存在意義ってなんだろう.

お礼

ちなみに５個なら

tm[0]
tm[1]
\n
tm[2]
\n
\n
tm[3]
\n
\n
\n
tm[4]

となります

ｔｍを個と数えましたが
正確にはプログラムを数回起動させて処理したときの個数のときになります

tm[0]
\n
tm[1]
\n
tm[2]
\n
tm[3]
\n
tm[4]
\n

上の感じならfgetsのバグだと私も思うんですけどね
以前もあったので

補足

これは全体の一部をくりぬいて質問用のプログラムにしているのですが、元のプログラムは一行だけ他と違うように変える必要があったのです

そのためへんなプログラムに見えたらすみませんね

もう少しバグについて話すならtmが２つだけなら問題がなく
3個なら

tm[0]
tm[1]
\n
tm[2]

4個なら

tm[0]
tm[1]
\n
tm[2]
\n
\n
tm[3]

となってしまいます

回答No.1

m-take0220

fgetsは読み込んだデータの改行をそのまま残すので、そのデータに対して改行を付加して書き込めば、当然改行が二重になります。

補足

えっと読み込んだときには待ったく問題なく表示されたのですが、書き込むときにはおかしくなっちゃうのです