add sasa.c #15

sasai3 · 2018-12-27T11:14:05Z

No description provided.

akkyie · 2019-01-17T10:16:31Z

cat/sasa.c

+  int fdescript;
+	ssize_t readbytes;
+	ssize_t writebytes;
+  char buf[1024];


1024 みたいなマジックナンバーはstatic constか#defineにしたいね

akkyie · 2019-01-17T10:16:57Z

cat/sasa.c

+
+int main(int argc, char * argv[]){
+  int fdescript;
+	ssize_t readbytes;


どうでもいいけどタブとスペースが混ざっている気がする

akkyie · 2019-01-17T10:32:53Z

cat/sasa.c

+			fdescript = open(argv[i], O_RDONLY);
+			if (fdescript == -1)
+				continue;
+		}


インデントが揃っていなくてわかりにくいけど、"引数を複数指定したら最初に開けたファイルを出力する"という仕様になっているっぽい？
オリジナルのcatの仕様（となぜcatという名前がついているか）を確認してみるといいかも

akkyie · 2019-01-17T10:34:45Z

cat/sasa.c

+				writebytes = write(1,buf,readbytes);
+			} else if (readbytes == 0) {
+				close(fdescript);
+				writebytes = write(1,"\n",1);


writebytesは代入しているけど使っていないのかな？
書き込み失敗した時のエラーハンドリングもしたいところですね

笹井　雄貴 and others added 2 commits December 27, 2018 20:11

add sasa.c

b9f62e8

git configの確認

a700423

akkyie reviewed Jan 17, 2019

View reviewed changes

Provide feedback