コードサイズ

1行の長さ

昔は80文字に収める事が理想とされていましたが
それは端末の表示幅が80文字だったからです。
現在、ほとんどの環境でこれ以上の表示幅が確保できる現実を考えれば
もう少し伸ばしてもいいでしょう。
家の環境では 100 文字にしています。

CheckStyle : LineLength

Javaでは、メソッドが処理の最小単位です。
よって、長いメソッドは色々な点でデメリットになります。

理想的には、全てのメソッドが1ページに収まるのが良いでしょう。
家の環境では、50ステップ位に収めるように CheckStyle のチェックを入れています。
これならば、ほぼ1画面に収まります。

メソッドの引数が多過ぎる事によるデメリットは
多くの方が実際に経験している事と思います。
とにかく読みずらいし理解が難しくなります。
家の環境では、最大数を6に制限しています。本当は4〜5にしたいところですが…

メソッドより大きい単位となるのが、クラスです。
これも、あまり大き過ぎるとアウトライン表示などで一覧を見るときに苦労します。
1000行程度に収まれば良いでしょう。

if文をネストしたりif-elseを連続で繰り返したりすると、
ロジックの複雑さが増加します。
コードチェッカでは、Cyclomatic Complexity 値という単位で
これらを計測しています。