- 119 名前：デフォルトの名無しさん mailto:sage [2009/10/09(金) 16:22:38 ]
- ・scanf()の使い方が危険。
せめて、scanf("%63s", fname)としよう。
・if文中での代入は避けた方が読み易い(但し異論もあり)。
変数の寿命の点からも、file = fopen(); if (file == NULL) {}とした方がいいと思うが。
・エラーメッセージは標準エラー出力を使おう。
・Cのループは0オリジンがお約束。
間違いじゃないけれど、for (cnt = 0; cnt < no; ++cnt) {}とした方が多くの人が見慣れている。
・変数の局在化のためにも、変数宣言を関数の先頭に集中させるのはやめよう。
例えば、int chはforの内側で宣言すれば充分。
