if のネストが深い。さっさとReturnせず、いちいち全部を判定してからReturnしようとする。

Boolean型の変数をいちいちTrue/Falseと等しいかどうか判定する。

Switch(Select case )を使わず、ElseIfを多用する。

コメントの場所が不適切。どれに対応したコメントなのかが分からない。

スコープを意識した命名規則になっていない。特にローカル変数。

 

 

コメントを残す

メールアドレスが公開されることはありません。 * が付いている欄は必須項目です