いがいが日記

_ アジャイルプラクティス読書会

会社での読書会、(あまにょさんGJ!と言わざるを得ない)

6章「アジャイルなコーディング」は私が進行担当。

この章はプラクティスが多かったので、

最初に人気投票して人気順にやることにした。

25.意図を明確にするコードを書く 5票

26.コードで伝える 7票

27.トレードオフを積極的に考慮する 3票

28.インクリメンタルにコードを書く 3票

29.シンプルにすること 1票

30.凝集度の高いコードを書くこと 8票

31."Tell, Don't Ask" - 求めるな、命じよ 3票

32.取り決めを守ってコードを置き換える 0票

1人何票でも投票可、参加人数は10人くらい。

「凝集度の高いコードを書くこと」は

インパクト大な挿絵による貢献にぐっときた方が多かった模様。

（「おまえピアノかよ！」の絵）

凝集度が高いって読んだだけだとよく分からなかったのだけど、

みんなで話をしていたら理解が深まった気がする。

一言でいうと、

「責務が同じものは集めて、違うものは関連しないようにする」

のが凝集度の高いコードなのだろう。

「そのクラスのキャラを立てる」と言い換えてもいい。

あとはどこまで集めるかは見方次第、バランス重要、だから難しいのだろな。

もう1話題、"Tell, Don't Ask"の節で

「コマンドと問い合わせの分離」の話題を話したかったので独断で持ち出す。

状態を変える場合がコマンドで、

「いまどうよ？」って聞くだけの場合は問い合わせ。

問い合わせのときは状態を変えるべきではない。

変えられると困る。困ってる。まさにいま。(^^;)

ポイントは「コードで伝える」の節にある「悪い名前は間違った情報を伝える」ということ。

Getって名前だと問い合わせだろうと思って使いまくったら、

状態が変わってしおしおしお、、、とか。

そこで出た秀逸な議論。

「ファイルのリードみたいにポインタの指す場所が変わるものはどうする？」

確かに問い合わせなんだけど、読み出すポインタの場所が進んでいく。

結論としては、そういう場合はreadという名前を使えば、

皆ファイルのread(fread()とか)をイメージするので良いのでは、と。

なるほど納得。

そしてそーいう肝になる名付けはチームのコーディングルールで規定して意識あわせするのが吉ではと。

そういうときに使える言葉サンプル集ってどっかに書いてないですかね？

例えばget,set,read,write,update,,,とかとか。

CodeCraftには具体的な例はなかった。

Imprementation Patterns にあるかもとの話もでましたが、

どこかで見かけた方は教えてください。

ところで票数に私の数は入ってないのですが、

私が投票するならトレードオフ、シンプル、"Tell, Don't Ask"かな。