□メソッドのやることは基本的には１つ。このくらい細かくていい def price nil end def end? period_to.nil? ? false : period_to < Date.today end def for_only_pc? useragents.size == 1 && useragents.first.name == 'PC' end def self.order 0 end

最初はスケーラビィリティとか考えないで、とかく綺麗に作るべき 継承とかMix-inとか使えるなら使う。

まとめまくると、エラーメッセージが出せなくなる。 やっていることは1つでも、それを組み合わせているメソッドがあって、 それを呼んだとき、読んだ側ではどの種類のエラーが起こったか 分からないときがある。まぁ、メッセージを返して、if **.class == Us…

userがnew_recのときは、nilになってくれる。nil.idエラーにならないから便利