- 897 名前:名無しさん@お腹いっぱい。 mailto:sage [2009/08/24(月) 12:16:53 ID:eaD1KRag]
- >>896
Cプログラミングの基本がなってない。 ○str1, str2, str3, str4 はほとんど似たようなものを格納している。 このような変数は配列にすること。 ○case内のインデントがswitch部から数えると2段になっている。 2段インデントはきわめて見にくくなるのでやめるべき。 (caseをインデントしないのが良いと思う) ○ScreenUp(), ScreenDown()内では、ほとんど同じ処理を3回書いている。 見てすぐにほとんど同じとわかるようなコードは、1個のコードにすること。 ○関数back_xmb(), ScreenUp(), ScreenDown()と、変数str1-str4, textは このソース中でしか使用されない。このような関数、変数にはstaticを つけること。
|
|