fwrite fread した時値が異なる

Question

fwriteした後の値とfreadした時の値を一致させたい

ある配列をbainry保存した後に、binaryファイルを読み込み、値を確認すると正しくないことが判明しました。
c言語のbinary保存について誰か教えて下さい！

発生している問題・エラーメッセージ

fwrite時のnum>0の個数：928568
fread時のnum>0の個数：0

該当のソースコード

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

//#define debug
//#define aaa
using namespace std;

int depred_interframe(const char *inname, const char *outname)
{
  short int buf;
  FILE *infp, *outfp;
  infp=fopen(inname,"rb");
  if(infp==NULL){
    perror("input file open error");
    exit(-1);
  }
  outfp=fopen(outname,"wb");
  if(outfp==NULL){
    perror("output file open error");
    exit(-1);
  }

  short int **matrix, *base_matrix;
  int frame=622, channel=512, winsize=7;
  int tmp=channel*winsize*winsize;

  matrix=(short int**)calloc(sizeof(short int *),frame);
  base_matrix=(short int*)calloc(sizeof(short int),frame*tmp);
  for(int i=0;i<frame;i++){
    matrix[i] = base_matrix+i*tmp;
  }

  int num=0;
  for(int i=0;i<frame;i++){
    for(int j=0;j<tmp;j++){
      fread(&buf,2,1,infp); 
      matrix[i][j]=buf;
      if (buf>0){
        num++;
      }
#ifdef aaa
      if((j+1)%49==0){
        printf("%3d\n\n", matrix[i][j]);
      }else if((j+1)%7==0){
        printf("%3d\n", matrix[i][j]);
      }else{
        printf("%3d ", matrix[i][j]);
      }
#endif
    }
  }
  printf("%d\n", num);

  fclose(infp);
  fclose(outfp);
}


int pred_interframe(const char *inname, const char *outname)
{
  unsigned char buf;
  FILE *infp, *outfp;
  infp=fopen(inname,"rb");
  if(infp==NULL){
    perror("input file open error");
    exit(-1);
  }
  outfp=fopen(outname,"wb");
  if(outfp==NULL){
    perror("output file open error");
    exit(-1);
  }

  unsigned char **matrix, *base_matrix;
  int frame=622, channel=512, winsize=7;
  int tmp=channel*winsize*winsize;

  matrix=(unsigned char**)calloc(sizeof(unsigned char *),frame);
  base_matrix=(unsigned char*)calloc(sizeof(unsigned char),frame*tmp);
  for(int i=0;i<frame;i++){
    matrix[i] = base_matrix+i*tmp;
  }

  for(int i=0;i<frame;i++){
    for(int j=0;j<tmp;j++){
      fread(&buf,1,1,infp); 
      matrix[i][j]=buf;
#ifdef debug
      if((j+1)%49==0){
        printf("%3d\n\n", matrix[i][j]);
      }else if((j+1)%7==0){
        printf("%3d\n", matrix[i][j]);
      }else{
        printf("%3d ", matrix[i][j]);
      }
#endif
    }
  }

  unsigned char *pre_frame,*lat_frame;
  int num=0;

  for(int i=0;i<frame;i++){
    pre_frame=matrix[i-1];
    lat_frame=matrix[i];
    for(int j=0;j<tmp;j++){
      short int y;
      if(i==0){
        y=matrix[i][j];
        if(y>0){num++;}
        fwrite(&y,2,1,outfp);
      }else{
        //printf("pre : %d, lat : %d\n", pre_frame[j], lat_frame[j]);
        //printf("diff : %d\n", pre_frame[j]-lat_frame[j]);
        y=pre_frame[j]-lat_frame[j];
        if(y>0){num++;}
        fwrite(&y,2,1,outfp);
      }
    }
  }
  printf("%d\n", num);

  fclose(infp);
  fclose(outfp);
}

int main(void){
  //char inname[100] = "./input.txt";
  pred_interframe("input.txt","output.bin");
  depred_interframe("output.bin","output.bin");

  return 0;
}

試したこと

matrixのまま保存してみましたが、値は0値ばかりになり正しく保存できませんでした。

補足情報（FW/ツールのバージョンなど）

g++-5 (Ubuntu 5.4.0-6ubuntu1~16.04.11) 5.4.0 20160609

Answer 1

depred_interframeのoutfpは何のために存在しているのですか？
読み取り対象と同一のファイルであるoutput.binを書き込みモードで開いてしまっているのがまずいのかも？

Answer 2

ちょっと見ですが・・・

short int buf;

unsigned char buf;

この違いはなぜでしょう？

shortとcharではバイト並びが違いますよｄ＾＾

Comments

[cateye]

参考：バイトオーダ - ビッグエンディアン/リトルエディアン
http://www.ertl.jp/~takayuki/readings/info/no05.html

[Lizard_knight]

pred_interframe()関数では、unsigned char型をfreadで読み込んで、計算結果で負の値が現れるのでshort intで宣言したyをoutput.binで保存しました。
そのoutput.binを読み込むに当たり、short int buf;を使ってfreadしているためです。

Answer 3

depred_interframe()の方はnumが0初期化されていないため不定値が入っているのでは？

(追記)
すみません、VCではfopenは排他オープンになるため
動作確認の際
depred_interframe()の

 outfp=fopen(outname,"wb");
  if(outfp==NULL){
    perror("output file open error");
    exit(-1);
  }

を消して確認していました。
”wb”を指定してfopen()しているので上書きオープンされ、ファイルが0バイトになっていますよ。
上記の不要なfopen()を消せば一致すると思います。

Comments

[Lizard_knight]

num=0;で初期化してみたところ、
fwrite時のnum>0の個数：928568
fread時のnum>0の個数：0
となりました。結局の所、fwriteが正しくできてない感じです。

[SHOMI]

こちらでは一致しましたが…