バッファリングする fgetc () を作成する問題

Question

###前提・実現したいこと

C言語で open(2), close(2), read(2) 等を用いて（標準入出力ライブラリ関数を使わず）ファイルから１文字ずつ読み出すプログラムを作ろうとしています。

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

abcという中身のファイルを読み出すと65、66、67、aと出てしまいます。
myfgetcかmyfopenが怪しいと思うのですが、どこをどう直して良いかわかりません。

###該当のソースコード

#include <stdio.h>
#include <unistd.h>
#include <fcntl.h>
 
// バッファーサイズ
//  動作確認しやすくするため小さめの値とする
enum {
  BUFFERSIZE = 5
};
 
typedef struct myfile {
  int fd;
  int count;
  int pointer;
  unsigned char buffer[BUFFERSIZE];
} MYFILE;
 
// 簡単化のため同時にオープンできるファイルは1個だけ
static MYFILE myfile;
 
void print_myfile (MYFILE *fp) {
  int i;
  if (fp) {
    fprintf (stderr, "[MYFILE] fd:%d c:%d p:%d ",
         fp->fd, fp->count, fp->pointer);
    for (i = 0; i < BUFFERSIZE; i ++) {
      fprintf (stderr, " %02x", fp->buffer[i]);
    }
    fputc ('\n', stderr);
  }
}
 
MYFILE *myfopen (char *filename) {
  //
  // この部分を作成する
  //
  // 与えられたファイル名のファイルを開く
  int fp = open(filename, O_RDWR);
  myfile.fd = fp;
  myfile.count = 0;
  myfile.pointer = fp;
  // http://www5c.biglobe.ne.jp/~ecb/c/13_09.html
  // fdopen(fp, r+)
  return &myfile;
}
 
int myfclose (MYFILE *fp) {
  //
  // この部分を作成する
  //
  close (fp->fd);
  return 0;
}
 
int myfgetc (MYFILE *fp) {
  //
  // この部分を作成する
  // 1文字読み込み、その文字を返します
   
  int readCount = fp->count;
  fp->count += 1;
  if (fp->count < BUFFERSIZE) {
    read(fp->fd, fp->buffer, 1);
    return  (int)*(fp->buffer + readCount);
  } else {
    return EOF;
  } 
}
 
int main (int argc, char *argv[]) {
  char *filename = "abc.txt";
  MYFILE *fp;
  int c;
  if (argc == 2) filename = argv[1];
  fp = myfopen (filename);
  if (fp) {
    print_myfile (fp);
    while ((c = myfgetc (fp)) != EOF) {
      printf ("#[%02x]%c#\n", c, c);
      print_myfile (fp);
    }
    print_myfile (fp);
    myfclose (fp);
    return 0;
  } else {
    return 1;
  }
}
 

###試したこと
課題に対してアプローチしたことを記載してください
// この部分を作成する、と書かれた関数は自作して何とか動作までは漕ぎ着けました。
ただ、openやread、バッファリングなどの仕組みについて理解が足りないようで、正常に開けていない或いは正常に読み込めていません。

Answer 1

abc.txtというファイルを作って中身にabcと入力、
質問分に記載されているソースをそのままコピペしてmacのgccでコンパイル＆実行

結果

[MYFILE] fd:3 c:0 p:3  00 00 00 00 00
#[61]a#
[MYFILE] fd:3 c:1 p:3  61 00 00 00 00
#[00]#
[MYFILE] fd:3 c:2 p:3  62 00 00 00 00
#[00]#
[MYFILE] fd:3 c:3 p:3  63 00 00 00 00
#[00]#
[MYFILE] fd:3 c:4 p:3  0a 00 00 00 00
[MYFILE] fd:3 c:5 p:3  0a 00 00 00 00

結果の右側の値をみると、61,62,63,0aとfp->buffer[0]にはabc[LF]と読み込まれているようですが？
また本来myfgetcで返すはずの文字が表記ではおかしいので
myfgetc関数のファイル読み込みのバッファの位置を少し変更

read(fp->fd, &fp->buffer[fp->count], 1);

で実行結果は

[MYFILE] fd:3 c:0 p:3  00 00 00 00 00
#[00]#
[MYFILE] fd:3 c:1 p:3  00 61 00 00 00
#[61]a#
[MYFILE] fd:3 c:2 p:3  00 61 62 00 00
#[62]b#
[MYFILE] fd:3 c:3 p:3  00 61 62 63 00
#[63]c#
[MYFILE] fd:3 c:4 p:3  00 61 62 63 0a
[MYFILE] fd:3 c:5 p:3  00 61 62 63 0a

なりました。
質問分の65、66、67は再現しないのですが。。。

Comments

[nazonoP]

すみません、値が間違っていたようです
これで合っていると思います