メタプログラミングをして割に合うかの判断基準：処理を1箇所に局所化できるか - 2014-01-16 - ククログ

毎日他の人のコミットをながめる文化で生活していると、理由は浮かばないけど「ん？このコミットはなんか気になる」と感じるようになります。それは、新しいことを知ることができたコミットだったり、真似したくなるようなコードが入っているコミットだったり、なんかまずそうな気がするコミットだったり、様々です。 「ん？」と感じてコミットを見直してみても、何が気になったか自分でもすぐにわからない場合があります。そんなとき、気になったことをコミットした人に伝えるために、コミットへのコメントをまとめ始めます。「コミットした人に伝えるため」というように、他の人に伝えようとすることがポイントです。他の人に伝えるためにまとめようとすると、思いの外なにが気になったかまとまるものです。 今回は、メタプログラミングを使ってコードを整理したコミットで「ん？」と感じたときのことについて紹介します。このおかげで「メタプログラミング

[thesecret3]

これ、、caseにするよりelsifぐらいでいいのでは。最初のが読みやすい。

[theatrical]

「なんか変とか、気に入らない」って言われると納得できなくて、「ふざけんなお前の価値観じゃねーか」とか思うけど、こうやって言語化されると納得できていいね。自分でも言語化して伝えるように心がけたい。

[hush_puppy]

最初のが一番頭を使わずに一瞬で読める気もする

[joker1007]

ちゃんと言語化できることが凄い。

[kmizushima]

これは、メタプログラミングをして割に合うかとは直交した話をしているのではないかという気がする。

[Rinta]

楽になってる抽象化ならいいんだけどね。楽=保守コストの低減なので、そこに資するかどうか、で判断するかな、私は。そこには３年生ぐらいのヤツでも保守できる程度かどうかも含まれる。

[nilab]

メタプログラミングをして割に合うかの判断基準：処理を1箇所に局所化できるか - ククログ(2014-01-16)

[t2y-1979]

なるほど、詳細を知らなくても使えるくらいの抽象化、これは分かりやすい

[tarchan]

＞変更箇所を局所化できていないと、このあたりのコードをいじるたびに、「このメタプログラミングをしているところはどういう仕組みで動いているか」を理解しなければいけません。これは通常のコードを理解するより

[northlight]

こういうのをメタプログラミングっていうの？

[VoQn]

個人プロダクトはともかく、他者がメンテする前提だったら短さより「分かりやすいか」で考えた方がいいですよね

[hayato240]

メタプログラミングをして割に合うかの判断基準：処理を1箇所に局所化できるか - ククログ(2014-01-16) @_clear_codeさんから

[willnet]

よい記事

[kasumani]

メタプログラミングをして割に合うかの判断基準：処理を1箇所に局所化できるか 毎日他の人のコミットをながめる文化で生活していると、理由は浮かばないけど「ん？このコミットはなんか気になる」と感じるようになり

[isseium]

メタプロは詳しくないけど，メンバーがメンテできるかは大事な指針だと思う．メンバー全員がこういうのを気に掛けていればいいけど．デザインやビジネスじゃなくてどちらかというとアートの世界な気が

[kazuyadesse]

( ´ ▽ ` )ﾉ

[aki03]

なんとなーく、わかる

[uzuki-first]

なるほど感。

[manaten]

最後のが読みにくい言ってる人はRubyに慣れてないだけなのでは感ある。正直この程度の短い処理なら、読む人がどのコミュニティに属してるかによってどれが読みやすいか変わりそう

[kazuya030]

最初の類似名関数群の存在から気になる。最初から引数でとりたい / 関数名を文字列で使う部分も同意できない。IDE は関数の参照認識できないんじゃね？ / 想定される更新の時にどこをいじるかの想定は常に気にしたい

[houyhnhm]

JavaScript読めないけど、Hashでeachで中にif文書くのって普通なの？Hashに関数ごとぶっ込んだら駄目なん？わからんなあ・・・・・・

[atsushifx]

メタプログラミングというよりはリファクタリングしすぎとかデザインパターン適用しすぎとかの問題に見える。こういうのはバランスが大事で、文章などにアウトプットすることが自分の考えを明確にする

[suginoy]

"変更箇所を局所化できるほど抽象化できていないメタプログラミングのコードはオーバースペック"

[oktnzm]

この人の話だとSUB_FORMATTERSの値がクロージャだったら妥当って話になるの？つかパラメータ名にあわせて関数作っちまえばいいのに･･･。あとneed~をなくすのとif/caseは別問題

[y_guriko]

なるほど。話それるけど、ここでcaseは正しい選択なのに読みにくさときたら...

[nebokegao]

「変更箇所を局所化できるほど抽象化できていないメタプログラミングのコードはオーバースペック」

[kaipu1224]

ケースバイケースっすね。

[iakio]

"抽象化のレイヤーにヒビが入っていて、違うレイヤーがのぞいてしまっているイメージです。"

[nodens161]

TKGHatebuReaderより

[mather314]

“メタプログラミングでやっていることの詳細を知らなくても使えるくらい抽象化しているか” なるほど。それは抽象化レイヤには必要な考え方だよな。

[lizy]

データ駆動に書き換えたところが、たまたまメタプログラミング？をやっていた場合

[labocho]

“メタプログラミングでやっていることの詳細を知らなくても使えるくらい抽象化しているか” なるほど

[stealthinu]

どこまでやるのが良いかという話から、なぜ？というのを言語化しようとすることで考えが整理されるという話。なんかもう一度リーダブルコードを読み直さなくちゃという気になる。