teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

3

追記

2019/02/17 06:39

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -1,2 +1,5 @@
1
1
  条件部で一回読み取っているんですから、printf("%s", text); で充分では。
2
+
3
+ ---
2
- あと、mallocしたときは必ずfreeしてください。軽微ではありますがメモリリークが起きています。
4
+ あと、mallocしたときは必ずfreeしてください。軽微ではありますがメモリリークが起きています。
5
+ ファイルが開けなかったとき、またメモリ取得失敗時のフローが無いのも気になります。

2

成形

2019/02/17 06:39

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -1,2 +1,2 @@
1
1
  条件部で一回読み取っているんですから、printf("%s", text); で充分では。
2
- あと、mallocしたときは忘れfreeしてください。軽微ではありますがメモリリークこしています。
2
+ あと、mallocしたときはずfreeしてください。軽微ではありますがメモリリークています。

1

追記

2019/02/17 06:38

投稿

LouiS0616
LouiS0616

スコア35678

answer CHANGED
@@ -1,1 +1,2 @@
1
- 条件部で一回読み取っているんですから、printf("%s", text); で充分では。
1
+ 条件部で一回読み取っているんですから、printf("%s", text); で充分では。
2
+ あと、mallocしたときは忘れずにfreeしてください。軽微ではありますがメモリリークを起こしています。