06/12/27 14:25:35
俺は処理単位毎に関数を分割する主義だが、最低限処理単位毎にコメントを入れてくれれば
１つの関数に複数の処理が入って数千行あってもいいと思っている。
コメントなしで数千行あったら、たいてい書いた本人以外は一目で理解できないので、リジェクト
してる。
修正指示は、大きく分けてコメントだけ追加してもらう場合と関数分割してもらう場合の２通りが
あるけど、ネストが深くて中が長い場合はたいてい関数分割だな。
ポイントは、流し読みして処理手順の概略が理解できるかどうか。
別に処理単位に分割することを押し付けるつもりは無いのだけど、数千行もあると本人も実は
処理フローをよく理解できていなくて、流れが汚くなっていたり、対になる処理が抜けていたり、
２つの処理が混じっていたりというのをちらほら見かけるので。