ミノ駆動 on Twitter: "クソコード動画「共通化の罠」 https://t.co/MM750CNXc2"

クソコード動画「共通化の罠」 https://t.co/MM750CNXc2

[mizoguche]

共通化はシステムにおける共通部分の意味をよく考えないと保守で死ぬのでむしろコピペのがマシなこともあるみたいな話よくある

[fashi]

まるごとコピーして余計なコードついたまま使うしか

[akfmi]

Aの特別処理を共通モジュールに入れた時点で破綻しているのでは・・？

[mohno]

これ、共通モジュールを各クラスが“使う”だけなら「コピーしてオシマイ」だよね。条件分岐が各クラスに依存してる、とかでないと現実味が出ないのでは？w（←そこか？ww）

[bearcub]

Aの特別処理を最初の時に共通モジュールに入れずに、Aから関数渡しとかポリモーフィズムとかで注入するのが正解？

[slkby]

しれっとEFが増えてる辺りがリアル

[rti7743]

コピーしてバージョンをつけようとすると、MSが陥ったDLL地獄の再来。

[shogo_okamoto]

A-Fが別の人という設定だけど、大体の場合、過去の自分じゃない?

[el-condor]

処理が似てるから安易に共通化するとこうなるという話。それが本当に抽象として同じなのか考えてみないとダメだよね。あとその分岐本当に必要か、というのも。

[ghostbass]

動かないコードは悪(というか評価に値しないが)だが、動くコードにも良し悪しはある

[htnmiki]

ワロタ

[dot]

コンテキストを無視して同じ処理だからという理由だけで共通化すると失敗するのよく見る。

[Kuichi]

CSSで割とこんな状態になっていたりで辛い…。

[ardarim]

くそわろｗ 共通化自体はそんなに悪いことじゃなくて、共通化部分を密結合にしてしまうと後でハマるってパターンなんだよな。手足はいつでももげるようにしておかないと（そうじゃない）

[knok]

RPGツクールで動画作るのか…

[reachout]

なぜそれぞれのクラスで条件分岐を実装出来なかったんや（呼び出しクラス名の上4桁で分岐するCase文を眺めながら）

[otihateten3510]

DRY!(Don't Repeat Yourself!) ←これ消えて欲しいんだが　／最もヤバいのは、共通モジュールが誰に使われてるか分からなくなるケース。そして共通モジュールを壊したら他の共通モジュールも壊れてジェンガ崩壊。

[KoshianX]

DRYはあくまで原則であって必ずしもいいとは限らないってやつよなあ……。

[akanehara]

似ているだけの別のものかもしれないと立ち止まらず拙速に共通化してDRI！DRI！と喜んでるの、カモノハシを鳥に分類するみたいなアホさを感じる。

[zyzy]

共通化の罠と言うより、継承で共通化する罠のお話。継承は共通化ではなくフレームワークの柔軟性のためにのみ使うべきだよね。

[neogratche]

すごくわかりやすいし面白い

[otakumesi]

依存関係逆転原則の「抽象に依存せよ」という話ですよね

[stp7]

最初の「練炭」で草

[z1h4784]

こういうのは共通モジュールを触らず個別処理をやるためのクラスを別途作るのがベター。コールバック関数とかハンドラーとか使って

[quick_past]

そもそもフレームワーク化してる部分に直接手を入れてる時点で、PMは止めないの？(;´Д`)せめて共通モジュールから継承したクラス内でやればいいのに。

[FutureIsWhatWeAre]

焼肉おごってもらってもあかんものはあかんと言っとかんと

[dorokei]

かっちりと共通化されたプロジェクトに修正入れるとき毎回悩むやつ

[D_first]

コンテキストの異なるものを安易に共通化してはいけない

[tmurakam]

だからといってコピペしまくっていいことにはならない

[gabill]

初めて個別処理が入る時点で、『共通処理を更に細かくバラバラにする→共通処理を、バラバラにしたミニ処理を呼びだす形にする→個別処理入りも同じ要領で別々に作る』とやれれば良いんだけど、時間がないとね...。

[gabari]

共通モジュールといいながら同じlibrary内にあるから起きる問題。別ライブラリにしておけば、特別処理を入れるときに循環参照になるからそもそもそこでストップかかる

[kniphofia]

最後はリファではなくコピーでは。いじらない。