getlineをfgetsで書き換えたい

###前提・実現したいこと
C言語を用いて、ファイル入力で読み込んだテキストに行番号を付けるプログラムをgetline関数を用いて作り、同じプログラムをfgetsで入力しても実行できるようにしたいのですが、上手くいきません

###発生している問題・エラーメッセージ
正しい実行結果は、

 1 apple
 2 banana
 3 character
 4 doctor
 5 elephant
 6 firefighter
 7 giant
 8 My_name_is_Tarou_Satou
 9 Python

上記のような形なのですが、

 1  apple
 2  banana
 3  charact     4  er
 5  doctor
 6  elephan     7  t
 8  firefig     9  hter
10  giant
11  My_name    12  _is_Tar    13  ou_Sato    14  u
15  Python

このようになってしまいます。

###該当のソースコード
#【getlineで作ったソース】

#include<stdio.h>
#include<stdlib.h>
int main(){

  FILE* fr = fopen("abc.txt","r"); //読み込みファイル
  if(fr==NULL){perror("ReadFileOpenError"); return 0;}
  FILE* fw = fopen("xyz.txt","w"); //書き出しファイル
  if(fw==NULL){perror("WriteFileOpenError"); return 0;}
  size_t sz = 16;
  char *str = (char*)malloc(sz);
  if (str == NULL) sz = 0;
  int i=1;
  while (getline(&str, &sz, fr) != -1){
    fprintf(fw,"%6d %s",i,str);
    i++;
  }
}

#【fgetsで作っているソース】

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

int main() {
  FILE* fr = fopen("abc.txt","r"); //読み込みファイル
  if(fr==NULL){ perror("ReadFileOpen");return 0;}
  FILE* fw = fopen("def.txt","w"); //書き込みファイル
  if(fw==NULL){ perror("WriteFileOpen");return 0;}
  size_t sz = 16; //文字列のサイズ
  char* str = (char*)malloc(sizeof(char)*(sz+1)); //読み込み文字列格納用
  if(str==NULL){ sz=0; }
  int line=1; //行番号
  while(fgets(str,sizeof(str),fr) != NULL){
    fprintf(stdout,"%6d  ",line);
    fputs(str,stdout);
    line++;
  }
  free(str);
  fclose(fw);
  fclose(fr);
}

《abc.txt》
apple
banana
character
doctor
elephant
firefighter
giant
My_name_is_Tarou_Satou
Python

よろしくお願いします

LouiS0616

2017/11/02 13:08

見やすいように工夫しているのは素晴らしいことですが、実は『コードブロック』を用いることが出来ます。編集画面を開き、コードを選択した状態で<code>ボタンを押してください。インデントや#なども正しく表示することが出来ます。

kmmtprgm

2017/11/02 13:12

ご指摘ありがとうございます。修正いたしました。

行動規範の内容に同意します

回答1件

ベストアンサー

次のように書き換えることで、私の環境では正常に動作しました。
追記：また、szの値を32に変更していました。simiteiさん、ありがとうございます。

C
1while(fgets(str, sz, fr) != NULL){
2    fprintf(stdout, "%6d  ", line);
3    fputs(str, stdout);
4    line++;
5}

fgetsは次のような引数を取ります。引用元

引数
s: ストリームから読み取った文字列を格納するための配列
n: 1 行の最大文字数
stream: ストリーム

strはこのときchar*型の変数ですから、sizeof(str)は同一環境では一定です。

kmmtprgmさんの環境では、sizeof(str)が8であるようです。
隠れているヌル文字を含め、8文字取得・表示できているようですから。

ついでなので、ちょっと気になった点を追記します。
プログラム内のwhile文は、for文に置き換えることで読みやすくなるかと思います。

C
1for(int line = 1; fgets(str, sz, fr) != NULL; line++){
2    fprintf(stdout,"%6d  ",line);
3    fputs(str, stdout);
4}

あと、ファイルを複数開く場合、ファイルポインタの解放はもうちょっと複雑になります。

C
1FILE* fr = fopen("abc.txt", "r");
2if(fr == NULL){ 
3    perror("ReadFileOpen");
4    return 0;
5}
6
7FILE* fw = fopen("def.txt", "w");
8if(fw == NULL){
9    perror("WriteFileOpen");
10    fclose(fr);
11    return 0;
12}