makotoshimazu / amiq11-sezemi-2014-readable-code-2 Goto Github PK

with_id の話

わざわざメンターに聞きたいと言ってたので、まじめに言語化してみます。

• id をパラメタで渡すなら with_id は基本つけなくていいと思う
  • show_recipe(id) となっていたら id はなにかしらの使われ方をするのが期待されるし、
    それが出力として印字されるというのは期待のうちの一つに入ると思う
  • 一方、recipe のうち、id で指定された recipe を show するという受け取り方もできるので
    それは関数の仕様による
  • show の実装に名前が縛られすぎるのはよくない (場合が多い)
• 仮につける場合があるとするなら、id を渡しても id を印字しないバージョンの関数があり、
  これとその印字の仕様によって区別することを明示したい場合とか。たとえばデフォルトの id を印字しない (が id は内部で使うので引数として受け取る) show_recipe(id) があり、それではないバージョンの関数として show_recipe_with_id(id) とか。
• ポイントは show_recipe(id) というシグネチャでなにを抽象化したいのか、なので
  それに従って関数名は決まってきそうで、関数名はその仕様を表現して実装を表現しないのがうれしいことが多い。

と思います。