単一責任原則で無責任な多目的クラスを爆殺する - Qiita

この記事は クラウドワークスアドベントカレンダー2020 8日目の記事です。 概要 こんにちは、クソコードを爆殺リファクタリングするのが大好きなミノ駆動です。 今回は単一責任原則の話です。 単一責任原則はSOLID原則のひとつとして有名で、2020年のオブジェクト指向カンファレンスのアンケートでも、SOLID原則の中で最も人気がありました。 皆さんは単一責任原則を遵守した設計をしていますか。 どんな構造が単一責任設計で、一方どんな構造が単一責任でない設計か、明確に意識していますか。説明できますでしょうか。 ところで「単一責任原則とはなんぞや」について、少なくとも私の観測範囲では、概念的な話にとどまっているものが多く、コードレベルで具体的に説明しているものは少ないように感じます。 そうした状況からか、単一責任原則の解釈が人によって違っていたりしているように感じます。 本記事は、今一度単一責任

[nakag0711]

このケースはむしろビジネス上の方針変更があったということなので、当初の設計はその時点の判断としては間違ってない気がするな

[okzk]

ダメな例がSRPには違反していないように見える。夏季割引仕様が「通常割引に加えて〇〇する」だったら実装としては正しいかもだし、そうじゃなければ流用すべきでないロジックを流用してバグったというだけで。

[buhoho]

こういう型指定で外堀埋めるようガチガチに作れるのがオブジェクト指向のいいところだったなって、改めて勉強になった

[honma200]

変化を予め予測した設計、拡張可能なようにするためにこう作るんだろうけど、自分みたいなのが使うとどこまでValueObjectにしていいか分かんなくなる奴だよね

[kazuau]

要件定義で業務担当と夏季は条件こそ違うが割引自体の仕様は通常と同じって合意が(間違って)されてしまっているからこそこういうコードになのでは。どちらに倒すにせよ開発者が勝手なことしちゃ駄目だよねと。

[ssids]

仕様を帰納的に抽象化するならば上位概念を作るべきで、似ている横に雑に寄せてはいけない的な

[iwasiman]

爆殺のタイトルでミノ駆動さんの記事と思ったらそうだった。解説も丁寧です。

[yarumato]

“まず定価を考えます。単なるint型変数と定義すると容易に不正値を代入できて正常動作に責任を持てない。オブジェクト指向におけるクラスは、インスタンス変数を正常に制御するメソッド (←ココ重要)を持つのが基本”

[otchy210]

この割引の例だと、唯一の DiscountManager クラスに割引に関する責任を全て与え、外部から割引ルールを指定するように作りたくなるな。新たなキャンペーンの度にクラスが増えるのはちょっと。

[youko03]

“単一責任原則を遵守するには、ビジネス理解が必須。 ビジネス理解が浅いと細かな概念の違いを区別できなくなる。 概念の違いを区別できないとロジックレベルで密結合になりがち。 ビジネス理解と概念分析には、ドメ

[tettekete37564]

この概念が理解できていない新人とかに読ませるのに良さげ。

[blueeyedpenguin]

クソコード書くの楽しそう

[kotetsu306]

クソコードでは「getDiscountPrice」と英語的に変な名前だったのが、改善コードでは「DiscountedPrice」にさりげなく直してあるの好き

[katariya0116]

あ、Javaってコンストラクタで例外投げて良いんだ……と思ったC++er

[itotto]

[雑記[開発]

[su_zu_ki_1010]

こういう視点は保守運用をやってないと出てこないよなぁ。開発だけして終わり、だと動けばええんねん！なスタイルになるので。

[Wafer]

読みづらくて目が滑った

[aukusoe]

“名前は点ではなく範囲 であることです”

[volx]

“ビジネスへの関心が薄く理解が浅いと単一責任設計が困難“、"単に仕様通りに動くだけのコードを書くのは論外。仕様書には現れないビジネス概念は、意外なほど沢山あります。"　ここが刺さりまくって泣いちゃった…

[yo_waka]

"ビジネスに関心がないと単一責任設計が困難" 分かる

[kazkun]

ちょっと面白そう。でもこれでもまだビジネスの実装の泥臭さが足りないな。

[hiroaki256]

それは文脈、意味合いが違っていたからです。 「300円割り引く」仕様が最初たまたま同じであっただけで、あれらは 通常割引金額 夏季限定割引金額

[tkrd]

http://blog.cleancoder.com/uncle-bob/2014/11/24/FPvsOO.html

[NOV1975]

まあ、最初からこうなることは少ない気もする。

[peketamin]

"概念が違えばDRYにすべきではないのです" それそれ。

[password1234]

重箱の隅をつつくようだけどMIN_AMOUNT = 0 とする無駄な定数化は好きではないな

[iekusup]

ほー。

[rgfx]

お、クソコード動画の人か