考えることを減らすという指針 - Konifar's WIP

最近チームメンバーが増えてコードレビューの数も増えてきました。 レビューでお互いが納得できるような説明を何度もするのも嫌だし、『よいコードとは何か』ということについて自分の思考を整理すべきかなぁと考えていました。 よいコードについてはすでに議論されつくされていて、最後は好みの問題という人もいるしケースバイケースだという人もいると思います。たしかにそうだなぁとも思うんですが、レビューで指摘する際にそのように伝えてしまうと相手は次から何をどう考えてコードを書けばいいかわからなくなってしまうし、最低限自分なりの指針を明確にしておく必要があります。 で、自分の指針を整理した結果、 考えることを減らす に落ち着きました。 この指針に沿って、考えたことをまとめておこうと思います。項目とか綺麗にまとまってないですが、このブログはタイトルの通りwork-in-progressなのでまぁいいかなぁと言い訳し

[aceraceae]

わりと共感できる。小さな処理なのにいろいろなところを参照しないとならないのは鬱陶しい。ただ、一回しか使わない処理の括りだしは、その複雑さしだいじゃないかと思う。

[hush_puppy]

コード設計もUI設計も、ほんとこれ。

[koogawa]

レビューから学ぶことも多いので「考えること」自体が悪いことだとは思わないけど、確かに無駄に考える時間は減らすべきだと思った

[zyzy]

自分も一回限りの一行isHoge関数は作っちゃうけど、なんというか、括弧の入れ子を複雑にしたくない、増やしたくない、って気持ちがあるんだよなぁ。(A != (B.pyo(C.ro)) == D.fuga(E)) != F みたいなのをIF内に置きたくないというか