- C言語なら俺に聞け（入門編）Part 131 [転載禁止]©2ch.net
35 ：デフォルトの名無しさん[sage]：2015/11/23(月) 00:31:50.54 ID:fxSBPtHV - >>32
さっと見た感じ読みやすいコードだと思います。
気になった点としては、
1. せっかくtypedef unsigned char _UBYTEとかで型を隠してるのに使われてない
2. #define hoge 0とか1とかは可能な限りenumを使いたい
3. そのファイル内でしか使われない関数はstaticを付けてスコープを制限したい
4. 3の結果グローバルな関数のプロトタイプ宣言は.hに移す
5. main()とか若干長いのでもうちょっと短くしたい(冒頭の処理をhoge_init()にまとめるとか)
あまり細かく見てないけどこんな感じです。
|
- C言語なら俺に聞け（入門編）Part 131 [転載禁止]©2ch.net
36 ：デフォルトの名無しさん[sage]：2015/11/23(月) 00:33:56.81 ID:fxSBPtHV - >>35の補足
アンダースコアで始まるシンボル名はコンパイラ側でリザーブされているので使わないように。
インクルードガードとかもね。どうしても付けたいなら末尾に。
|
- C言語なら俺に聞け（入門編）Part 131 [転載禁止]©2ch.net
45 ：デフォルトの名無しさん[sage]：2015/11/23(月) 20:46:54.29 ID:fxSBPtHV - >>40
エラーメッセージそのままだよ。定義(実装)の前に参照しようとしている。
関数ならプロトタイプ宣言を先にしておく、もしくは関数ブロック自体を使うより前に配置する。
ファイルの上から下へ流れることを思い出すべし。
|