共通化すれば良いとは限らない - Object.create(null)

ここのところ偶然なのか「共通化」という言葉を多く聞いているのですが, その言葉を聞くたびに身構えていることに気がついたので, この気持ちの出どころを共有しておきます. なぜ身構えているかというと, 共通化が必ずしもコードを良い状態にするとは限らないにも関わらず, それ自体が目的になってしまっている (ように見える) ことが多いからです. この手のリファクタリングの目的はあくまでコードの改善のはずで, そのことを忘れて共通化するだけで満足してしまうと, 良くてリファクタリングの効果が半減, 悪ければ逆効果になってしまいます. 個人的にコードを共通化する上で注意してほしいと思っているのは以下の二つです. コードを共通化すべきでない場合もある 共通化されたコードは一般的な原則にしたがって設計されなければならない 似たようなことは歴史の中で何度も繰り返し言われていることだろうと思いますが, 改めて

[NOV1975]

シンプルな話、FWが何故か用意していない共通機能を開発するときにも考えるべきこと。言語組込みのAPIで2行くらいで書けることを共通メソッド化してした上に呼出し側でエラーハンドリングに数行掛けているのを見かける

[ledsun]

インライン展開するより、あとから共通化する方がコストが高いので共通化しとけばよい。違うものだと気がついたらインライン展開すればよい。一旦共通化したものを崩すのを恐れない方がよい。

[nharuki]

ただただコードが類似しているから、というだけで共通化すると逆に要らぬ依存関係を生む羽目になる。知識が重複している＝役割が近しいとも言い換えられる。

[pascal256]

この動画を思い出したhttps://twitter.com/MinoDriven/status/1380773721032433674?t=ScmIIk2GTRfC0HdP569JTA&s=19

[jay-es]

自分もよく表面だけ見て共通化してしまうことがあるので気をつけたい

[motchang]

良い記事だな

[ydah]

おだいじ

[iga_k]

短く書かれていてよさ

[tofu-kun]

“コードを共通化すべきでない場合もある”

[t2wave]

“コードを共通化するときには, なぜ共通化するのか, なぜその方法で共通化するのかについて説明できるように”

[sgo2]

変更が発生した時に地獄。ただ似たようなコードを機械的に洗い出せる点は利点。(コピペだと漏れが発生し易い)

[Pasta-K]

良いまとめ

[atsushifx]

ミノ駆動氏の動画 https://twitter.com/minodriven/status/1127539251761909760 だね

[houyhnhm]

低レベル共通ライブラリとかになるのが負け、と思いますね。

[ngsw]

すき

[at_yasu]

過度な抽象化の話。同じコードが３つありゃ抽象化のサインだと思ってる。微妙に違う時は、それは違うものだと認識してる。

[toro-chan]

同じなものは同じにする。違うものは違うように扱う。言うと簡単だけどなかなか難しいな。。

[hitode909]

いい話