meetup #5 report

話題

「ポコタンいないと開始早いねー」
「最近ずっとモーニング娘。聞いてます」「すごいんですよ。誰が誰だか分からないでしょう？」
「RubyとかどうでもいいのでHeartbeatとしてmeetupを使いましょう」「やりてえこと持ってこい」「宣言して、やる」「ビールが (ﾟДﾟ)ｳﾏｰ」
「コードをエディタで読むと読みやすくていいよ」「rtagsがたぶんRuby読むにはいちばんいいよ」「スワップがオギャー」
「ぼくの装備はゆるふわなんです」「最後尾から攻撃魔法使います」「TDDをATDDにしたら『価値』の共有やりやすくなったりしませんかね」「ユーザーストーリー勉強したらいいのかなと思ってます」「モーニング娘。の52枚目のシングルがですね」

コードレビューとRubyっぽいコードの話をグダグダと1時間以上
変数が多い
ブロックが長い
意味、意図を名前にしてメソッド作ろう
Arrayの順番に意味のある戻り値にしちゃうと使う人が必ずその意味を分かってなきゃいけないので変更があったら壊れやすいよ
attr_reader でそのままインスタンス変数取れるようにしたら？オブジェクトそのまま渡したうえで getter から取れるようにすればごちゃごちゃ考えなくていいよ
getter には get_ も () も付けないよ派 vs プロパティとメソッド区別したいよ派
return あるとキモいよ派 vs return ないとキモいよ派
? がおしりにあるメソッドは Boolean を返す約束
! がおしりにあるメソッドは 破壊的
後置 if キモい！
このカウンタと比較してる if って先に slice したら要らないよね。if はないに越したことない。
この each 使ってるやつは map 使ったうえでこのメソッド自身も副作用のないメソッドにするのが好きですねドドドヤぁ