![](https://cdn-ak-scissors.b.st-hatena.com/image/square/843983b98d4c86c116c78db38dbaaac032859230/height=288;version=1;width=512/https%3A%2F%2Fqiita-user-contents.imgix.net%2Fhttps%253A%252F%252Fcdn.qiita.com%252Fassets%252Fpublic%252Fadvent-calendar-ogp-background-7940cd1c8db80a7ec40711d90f43539e.jpg%3Fixlib%3Drb-4.0.0%26w%3D1200%26mark64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZ3PTk3MiZoPTM3OCZ0eHQ9JUU1JThEJTk4JUU0JUI4JTgwJUU4JUIyJUFDJUU0JUJCJUJCJUU1JThFJTlGJUU1JTg5JTg3JUUzJTgxJUE3JUU3JTg0JUExJUU4JUIyJUFDJUU0JUJCJUJCJUUzJTgxJUFBJUU1JUE0JTlBJUU3JTlCJUFFJUU3JTlBJTg0JUUzJTgyJUFGJUUzJTgzJUE5JUUzJTgyJUI5JUUzJTgyJTkyJUU3JTg4JTg2JUU2JUFFJUJBJUUzJTgxJTk5JUUzJTgyJThCJnR4dC1jb2xvcj0lMjMzQTNDM0MmdHh0LWZvbnQ9SGlyYWdpbm8lMjBTYW5zJTIwVzYmdHh0LXNpemU9NTYmdHh0LWFsaWduPWxlZnQlMkN0b3Amcz03YjgzNGViNWUxY2Q3NGM1NmMzMjMyNDc1MTk3Y2NmMQ%26mark-x%3D142%26mark-y%3D100%26blend64%3DaHR0cHM6Ly9xaWl0YS11c2VyLWNvbnRlbnRzLmltZ2l4Lm5ldC9-dGV4dD9peGxpYj1yYi00LjAuMCZoPTc2Jnc9OTcyJnR4dD0lNDBNaW5vRHJpdmVuJnR4dC1jb2xvcj0lMjMzQTNDM0MmdHh0LWZvbnQ9SGlyYWdpbm8lMjBTYW5zJTIwVzYmdHh0LXNpemU9MzYmdHh0LWFsaWduPWxlZnQlMkN0b3Amcz1hMWVmZTFhZGU4ZjBiODkzOTRiMGE0OTg3MzY0YmI2Nw%26blend-x%3D142%26blend-y%3D455%26blend-mode%3Dnormal%26txt64%3DaW4gUkVBRFlGT1LmoKrlvI_kvJrnpL4%26txt-width%3D972%26txt-clip%3Dend%252Cellipsis%26txt-color%3D%25233A3C3C%26txt-font%3DHiragino%2520Sans%2520W6%26txt-size%3D36%26txt-x%3D156%26txt-y%3D550%26s%3Dcde34a544b6e2d87ba474a4d130ba575)
エントリーの編集
![loading...](https://b.st-hatena.com/bdefb8944296a0957e54cebcfefc25c4dcff9f5f/images/v4/public/common/loading@2x.gif)
エントリーの編集は全ユーザーに共通の機能です。
必ずガイドラインを一読の上ご利用ください。
記事へのコメント28件
- 注目コメント
- 新着コメント
![kazuau kazuau](https://cdn.profile-image.st-hatena.com/users/kazuau/profile.png)
kazuau
要件定義で業務担当と夏季は条件こそ違うが割引自体の仕様は通常と同じって合意が(間違って)されてしまっているからこそこういうコードになのでは。どちらに倒すにせよ開発者が勝手なことしちゃ駄目だよねと。
![yarumato yarumato](https://cdn.profile-image.st-hatena.com/users/yarumato/profile.png)
yarumato
“まず定価を考えます。単なるint型変数と定義すると容易に不正値を代入できて正常動作に責任を持てない。オブジェクト指向におけるクラスは、インスタンス変数を正常に制御するメソッド (←ココ重要)を持つのが基本”
![okzk okzk](https://cdn.profile-image.st-hatena.com/users/okzk/profile.png)
okzk
ダメな例がSRPには違反していないように見える。夏季割引仕様が「通常割引に加えて〇〇する」だったら実装としては正しいかもだし、そうじゃなければ流用すべきでないロジックを流用してバグったというだけで。
![otchy210 otchy210](https://cdn.profile-image.st-hatena.com/users/otchy210/profile.png)
otchy210
この割引の例だと、唯一の DiscountManager クラスに割引に関する責任を全て与え、外部から割引ルールを指定するように作りたくなるな。新たなキャンペーンの度にクラスが増えるのはちょっと。
![youko03 youko03](https://cdn.profile-image.st-hatena.com/users/youko03/profile.png)
youko03
“単一責任原則を遵守するには、ビジネス理解が必須。 ビジネス理解が浅いと細かな概念の違いを区別できなくなる。 概念の違いを区別できないとロジックレベルで密結合になりがち。 ビジネス理解と概念分析には、ドメ
![volx volx](https://cdn.profile-image.st-hatena.com/users/volx/profile.png)
volx
“ビジネスへの関心が薄く理解が浅いと単一責任設計が困難“、"単に仕様通りに動くだけのコードを書くのは論外。仕様書には現れないビジネス概念は、意外なほど沢山あります。" ここが刺さりまくって泣いちゃった…
注目コメント算出アルゴリズムの一部にLINEヤフー株式会社の「建設的コメント順位付けモデルAPI」を使用しています
![アプリのスクリーンショット](https://b.st-hatena.com/bdefb8944296a0957e54cebcfefc25c4dcff9f5f/images/v4/public/entry/app-screenshot.png)
いまの話題をアプリでチェック!
- バナー広告なし
- ミュート機能あり
- ダークモード搭載
関連記事
単一責任原則で無責任な多目的クラスを爆殺する - Qiita
この記事は クラウドワークスアドベントカレンダー2020 8日目の記事です。 概要 こんにちは、クソコード...
この記事は クラウドワークスアドベントカレンダー2020 8日目の記事です。 概要 こんにちは、クソコードを爆殺リファクタリングするのが大好きなミノ駆動です。 今回は単一責任原則の話です。 単一責任原則はSOLID原則のひとつとして有名で、2020年のオブジェクト指向カンファレンスのアンケートでも、SOLID原則の中で最も人気がありました。 皆さんは単一責任原則を遵守した設計をしていますか。 どんな構造が単一責任設計で、一方どんな構造が単一責任でない設計か、明確に意識していますか。説明できますでしょうか。 ところで﹁単一責任原則とはなんぞや﹂について、少なくとも私の観測範囲では、概念的な話にとどまっているものが多く、コードレベルで具体的に説明しているものは少ないように感じます。 そうした状況からか、単一責任原則の解釈が人によって違っていたりしているように感じます。 本記事は、今一度単一責任
2020/12/09 リンク