二つのファイル内容判定の仕方

Question

いつもお世話になっています。プログC初心者でただいま勉強中です。自分の作成した(mydata.out) と(data.out) のファイルをプログラム中で開き、先頭から１文字ずつ比較して、二つのファイルの内容が一致するかどうかを調べて結果を表示するプログラムを作りたいのですが、文字を比較するのは読み込みでするということでいいですよね？流れを教えてください。以下のコードの流れであっていますか?

コード
```#include<stdio.h>
#include<stdlib.h>
#define MAX 1000
int main()
{
  char buf[MAX];
  int i,n,c,j;
  FILE *fpin1,*fpin2,*fpout;
  fpin1=fopen("mydata.out","r"); 
  fpin2=fopen("data.out","r");
  if(fpin1==NULL || fpin2==NULL){
    printf("read file open error\n");
    exit(1);
  }
  /*ここで、文字比較する*/
    fclose(fpin1);/*ファイルオープン終了*/
 fclose(fpin2);/*ファイルオープン終了*/
    exit(2);/*強制終了*/
  fclose(fpin);/*読込みファイルクローズ*/
  return 0;/*戻り値0を返す*/
}
/*ここでファイル一致か不一致かの判定結果を表示する*/

return 0;
}

Answer 1

参考になるかわかりませんが、流れをプログラム中に書いてみました。
流れのご説明ですので、各所提示プログラム内容と異なっております。
*fpout;ではなく標準出力としております。
char buf[MAX];バッファ読込ではなくストリーム読込としています。
その他等々...
プログラム自身は、例題ですので動きません（あしからず）

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

int main()
{
    int i,n,c1,c2;
    FILE *fpin1,*fpin2;

    /* FILE をオープンします */
    fpin1=fopen("mydata.out","r");
    fpin2=fopen("data.out","r");

    /* どちらがのファイルオープンでエラーが有ればエラーを出力して終了 */
    if(fpin1==NULL || fpin2==NULL){
        printf("read file open error\n");
        exit(1);
    }

    int ln = 1;
    int cl = 0;

    /* 双方のファイルを一文字づつ読込比較を行う */
    while ((c1=fgetc(fpin1))!=EOF && (c2=fgetc(fpin2))!=EOF){

        /* ライン数をカウントする LFを改行としています MACならば\rとしてください*/
        ln += ( c1 == '\n') ? 1 : 0;
        /* カラム数をカウント */
        cl  = ( c1 == '\n') ? 0 : ++cl;

        /* 文字を比較します */
        if (c1 != c2 ){
            /* 文字が異なる場合は、エラーを出力して終了 全角の場合文字化けするので注意 */
            printf ("Difference found [%c][%c] at line %d col %d\n",c1,c2,ln,cl);
            fclose(fpin1);/*ファイルオープン終了*/
            fclose(fpin2);/*ファイルオープン終了*/
            /* 違いを見つけたら終了 ここで継続しても良いが大量のエラー出力になる可能性有り*/
            exit(2);/*強制終了*/
        }
    }
    /*ここでファイル一致か不一致かの判定結果を表示する*/
    printf ("No difference found\n");

    fclose(fpin1);/*ファイルオープン終了*/
    fclose(fpin2);/*ファイルオープン終了*/
    return 0;/*戻り値0を返す*/
}

Answer 2

こんにちは。

文字を比較するのは読み込みでするということでいいですよね？

その通りです。

流れを教えてください。以下のコードの流れであっていますか?

細かい問題はありますが、流れはあっています。

細かい問題は下記２点です。
fpin2側をオープンできなかった時、fpin1を開いたまま終了していることが気になります。
実害はないですが、開いたものは閉じるという原則を守れていないので。
意図的にこのようにすることはありますが、その必要がありデメリットと比較してでもやるべきであると判断した時にのみ行います。理由がない限り、開いたら閉じる、借りたら返すの原則を守るべです。

exit(2);/強制終了/

fclose(fpin);/読込みファイルクローズ/

の処理が意味不明です。消し忘れでしょうか？