バイナリーファイルのコピー

Question

バイナリーファイルをコピーするプログラムです。
以下は正常に動作したプログラムです。

#include<stdio.h>

#define BSIZE 1024

int main(int argc, char* argv[]){
    int n;
    FILE *src, *dst;
    unsigned char buf[BSIZE];

    if(argc != 3){
        fprintf(stderr, "パラメータが不正です。\n");
    } else {
        if((src = fopen(*++argv, "rb")) == NULL){
            fprintf(stderr, "ファイル%sをオープンできませんでした。\n", *argv);
            return 1;
        } else if((dst = fopen(*++argv, "wb")) == NULL){
            fprintf(stderr, "ファイル%sをオープンできませんでした。\n", *argv);
            fclose(src);
            return 1;
        } else {
            while((n = fread(buf, sizeof(unsigned char), BSIZE, src)) > 0){
                fprintf(stderr, "n = %d\n", n);
                fwrite(buf, sizeof(unsigned char), n, dst);
            }
            fclose(src);
            fclose(dst);
        }
    }

    return 0;
}

ところが、私が持っている本の中では、コピーを行っている部分が以下のようになっていました。

 while((n = fread(buf, BSIZE, 1, src)) > 0){
                fprintf(stderr, "n = %d\n", n);
                fwrite(buf, n, 1, dst);
            }

最初この通りに作ったのですが、うまく行かなかったので自分で作り変えました。

しかしよく考えると、疑問です。
下の方でも良いような気がします。
なぜ下のやり方ではうまく行かないのでしょうか？？
fprintf はテスト用に入れてあります。

下のプログラムを実行すると n = 1と表示されて終わります。
つまりループが一回しか実行されていないことになります。

なんで一回で終わってしまうんでしょうか？
下のやり方ではできない理由を教えてください。

実行環境は Linux 64bit gcc です。
皆さんの環境では、どうなりますかね？
正常に動作しますか？？

Answer 1

freadの扱いが間違ってますね
要素数を返すため、読み取ったバイト数は 第2引数 * 戻り値 です

1が戻った場合はBSIZE１個分読んだよってことです

---

ちなみに、動かす方法

• BSIZE を 1にする事で（偶然）動く
• 多分本来コレを書くつもりだった

 while((n = fread(buf, 1, BSIZE, src)) > 0){
                fprintf(stderr, "n = %d\n", n);
                fwrite(buf, n, 1, dst);
            }

Comments

[strike1217]

１６進数は２桁で１Bですよね。
この関数、第二パラメータを１以外にする場面ってあるんでしょうか？

[strike1217]

バイナリは数字が連続しているので　第２パラメータに1024Bを入れると 1024B分取ってくるという意味ではないのですか？

それだと、
fread(buf, 1, BSIZE, src);
fread(buf, BSIZE, 1, src);

何が違うんですかね？
上は１Bを 1024個
下は1024Bを１個ですよね？

結局同じような気するんですが・・・

[asm]

>１６進数は２桁で１Bですよね
1バイトは8bitですから16進数2桁で表わせますが逆が常に正しいかというと
…どれだけ相手が意地悪な人かで変わってきそうです
ちなみに私なんぞは生粋の天邪鬼なので、"3F"(文字列)や、(int)0x3Fなどを考えてしまいます。

> この関数、第二パラメータを１以外にする場面ってあるんでしょうか？
構造体の配列等を扱うときにはちょっと楽できます
歴史的な経緯があるのかもしれませんが

> 何が違うんですかね？
その後のfwrite(buf, n, 1, dst);でfwrite(buf,1,1,dst);になって1byteしか書き出していません

[strike1217]

fwriteで1Bしか書き出していないのはわかるのですが・・・

ん？
ちょっとまってください。
なんでループは一回で終わっているんですかね？

Answer 2

fwrite(buf, n, 1, dst);が間違ってます。
n*BSIZE*1バイト書かないといけないので、下記のいずれかにする。

fwrite(buf, n, BSIZE, dst);
fwrite(buf, BSIZE, n, dst);
fwrite(buf, n*BSIZE, 1, dst);

Comments

[strike1217]

freadの方は問題がないということでしょうか？

[strike1217]

ああ・・・できました。
freadの方は２つとも問題ないようですね。

fwrite()の方に問題があったということですね。

while文の継続条件は、n > 0 ですが、なんでfwrite()修正するとちゃんとループするんでしょうか？？

fwrite()は変数nに影響を与えていませんよね？？
なんでループが１回で終了してしまうんでしょうか？

[otn]

ファイルサイズがBSIZE以下だと、ほとんどの場合ループが1回で終わるでしょう。

[strike1217]

確かに、その通りです。
しかし、試したのはELF実行ファイルで1024Bよりも大きいです。

なんででしょうかね・・・？

[otn]

2048B以下だと、2回目のfreadでは0が返りますね。
実際には、1024B未満が読み込まれていますが。

[strike1217]

fread(buf, BSIZE, 1, src)
fread(buf, sizeof(unsigned char),BSIZE,src);

この２つは、結局は同じ。
fwrite() についても理解できました。

ループが謎すぎます。><

[strike1217]

コピーしたファイルの大きさは、ls -lhで調べてみました。
8.7KBのようですね。

[otn]

返り値が違いますね。返り値は、第二引数サイズの要素の個数なので、0以上第三引数以下です。

[strike1217]

返り値とはfread()の返り値のことですよね？
あ、確かに、そうですね。

というより、再度異常のあるコードをコンパイルしたら、ループしました。

.....??? 再度コンパイルするとできるようになるという現象がよく起きるんですよね。
なんですかね・・・これ・・・
（この問題については、再度質問するかもしれません。gccのバグでしょうかね？）

fwrite() はどの道間違っていましたね。
とりあえず、fread() と fwrite() が分かったので、ベストアンサーにさせてもらいます。

[otn]

> 再度コンパイルするとできるようになるという現象がよく起きるんですよね。
> なんですかね

それは気のせいで、コンパイルがミスっているでしょう。
1つ前のソースが動いていたとか。

[strike1217]

おそらく、私のミスです。
ケアレスミスが多いので、どこかで勘違いを起こしているかもしれません。