続ｊファイルに文字列を書く

Question

あれから後一歩と言うところにきました

たぶんmallocのバグだと思いますがチャットのシステムを作ろうとしているのですが
読み込み時にエラーになります

ソースの注目部分を見てほしいのですが、
注目部分はbuf+1 にすると　文字列が1文字ずつ消えていくバグになり
bufにすると　3行目を書き込んだ時点で3行目がの頭の部分の文字列がおかしくなり4行目を書き込もうとするとエラーになります

ちゃんと動作するにはどのように書けば良いですか？

---ソース---
#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 ){
   tm[i] = (char*)malloc(strlen(buf+1));
   strcpy(tm[i], buf+1);                  //ここを注目
   
   if(i<999){
    i++;
   }
  }
  
  fclose(fp);
  
  if(i<=1000){
   sei=i;
  }
  
  else{
   sei=1000;
  }

fp =fopen("moto.txt","w");
  
  for(i=0;i<sei;i++){
   if(i==0){   
    fprintf(fp,"%s
",tm[0]);
   }
   
   else{
    fprintf(fp,"%s",tm[i]);
           }
  }

}

---now.txt---
もも

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

Wr5 · Accepted Answer

>+1を入れると\0がはいるんじゃないんですか？

strlen()は「引数で渡されたアドレスから」文字列終端までの長さを返します。
'\0'が入る領域を用意するのであれば、「strlen()の返した値」に+1する必要があります。
ですので、"123456789"と9文字入っているアドレスの"2"が入っているところから数えたら8文字しかありませんよね？
掲示されたコードは"2"から数え始めてね。と指定しているのです。
# そして、"2"からコピーしてね。としてコピーしています。

>fp= fopen("now.txt","r");
>while( fgets( buf, 400, fp ) != NULL ){
>tm[0]=(char*)malloc(strlen(buf)+1);
>strcpy(tm[0], buf);
>}
では、正しく「文字数+1」となっているのに、問題の箇所では「先頭2文字目からの文字数」となっているのはなぜですか？

>つまりstrcpy(tm[i], buf+3);
>にすれば何とかなるのですね

そりゃエラーにはならないかも知れませんが、それが想定している動作なんですか？
掲示されたファイルの例(Shift-JIS想定)だと……
1回目で(3行目がヘンになっているかも知れない。半角カナになっている為)
---moto.txt---
激塔W
(空行)
ゥん

2回目の実行で
---moto.txt---
ジ
(空行)
(空行)
(空行)
になりますかね。
# 3行目が空行かゴミが出るかは状況次第のギャンブル。

>想定外と申されましても試行錯誤して書き換えているので想定そのものができません

では期待動作はなんなんでしょうか？

Wr5 · Answer

>再三指摘されているfgets()での改行コードに

失礼しました。
再三指摘されていたのは別人でした。
たぶんお友達だと思いますけど。

Wr5 · Answer

>ちゃんと動作するにはどのように書けば良いですか？

どういう動作が期待している動作なのか？
が不明です。

とりあえず、問題にしている箇所でバッファオーバーランを発生させないようにするのであれば、

tm[i] = (char*)malloc(strlen(buf+1));
strcpy(tm[i], buf+1);
は
tm[i] = (char*)malloc(strlen(buf));  // buf文字分しかないけど次のコピーで1文字カットするから'\0'の分はある。
strcpy(tm[i], buf+1);
とすべきでしょう。

まあ、これで読み込んだ行の先頭1文字を削除できるか？は、ファイル次第ですけど。
Shift-JISなテキストファイルで1文字目に日本語とか書かれていたら…ヘンになるかも知れませんし、
ASCIIで記述されたテキストファイルならたぶん正しく1文字目が削除されるでしょう。
# 現状のmoto.txtだとたぶん壊れます。

再三指摘されているfgets()での改行コードに関しては…まぁ、問題ないのでしょう。
私的には気持ち悪い処理内容ですが……。
# 部下や教えている生徒がこういうコード書いてきたら、なぜそうなるのかキチンと説明を求めるレベル。
「1行目の次に空行を挿入して、それ以降の行は空行を挿入しない処理です。」以外の回答があるのかは謎。

Wr5 · Answer

malloc()のバグではなく、「使い方」の問題でしょう。
今時malloc()にバグが潜在するようなライブラリなんてないでしょうし。

>tm[i] = (char*)malloc(strlen(buf+1));
の意図ってなんでしょう？
なぜ「bufのアドレスに+1」する必要があるんですか？
文字列終端の'\0'の分が確保されませんから、次の
>strcpy(tm[i], buf+1); //ここを注目
で、『予定通り』バッファオーバーランを実行します。
# tm[i] = (char*)malloc(strlen(buf)+1);
# だったら、「bufに入っている文字列をコピーするのに必要なサイズ」が確保されますけど…
# 掲示されたコードでは「bufに入っている文字列を2文字少なくコピーできるサイズ」しかありません。

バッファオーバーランで想定外の箇所を書き換えているのですから…
>文字列が1文字ずつ消えていく
>3行目を書き込んだ時点で3行目がの頭の部分の文字列がおかしくなり4行目を書き込もうとするとエラー
という動作になっても想定内のハズです。

Tacosan · Answer

他のところと見比べればわかっていいはずなんだが....

「malloc して strcpy」する関数を, なぜ作らない?

続ｊファイルに文字列を書く

>+1を入れると\0がはいるんじゃないんですか？

この回答への補足

>再三指摘されているfgets()での改行コードに

この回答への補足

>ちゃんと動作するにはどのように書けば良いですか？

この回答への補足

malloc()のバグではなく、「使い方」の問題でしょう。

この回答への補足

他のところと見比べればわかっていいはずなんだが....

この回答への補足

関連するカテゴリからQ&Aを探す

デイリーランキングこのカテゴリの人気デイリーQ&Aランキング

マンスリーランキングこのカテゴリの人気マンスリーQ&Aランキング