質問編集履歴

2

参考URLと内容の追加

2019/10/05 04:35

投稿

namatomato
namatomato

スコア5

test CHANGED
File without changes
test CHANGED
@@ -18,7 +18,7 @@
18
18
 
19
19
 
20
20
 
21
- ※追記
21
+ ※追記1
22
22
 
23
23
  至らぬ点が多々あり、申し訳ございません。
24
24
 
@@ -205,3 +205,39 @@
205
205
 
206
206
 
207
207
  どのようにアプローチすればよいでしょうか。
208
+
209
+
210
+
211
+
212
+
213
+
214
+
215
+
216
+
217
+
218
+
219
+ ※追記2
220
+
221
+
222
+
223
+ [開発者が知っておくべきSOLIDの原則](https://postd.cc/solid-principles-every-developer-should-know/)
224
+
225
+ [ソフトウェア原則[1] - OCP(Open-Close Principle)](http://objectclub.jp/technicaldoc/object-orientation/principle/principle01)
226
+
227
+ [Laravelチップシリーズ 2:SOLIDの世界1](https://www.bravesoft.co.jp/blog/archives/2458)
228
+
229
+
230
+
231
+
232
+
233
+ 以上のリンク先から、変更の理由=キャラクターの追加と設計段階で予想されるなら、キャラクターをそれぞれ共通のインターフェースを持った別のクラスにすることで既存のクラスの修正を行わなくてよいのが開放閉鎖原則のメリットだと考えておりました。
234
+
235
+ リンク先の内容が全て或いは一部誤っているのか、記載の内容はあっていて私の理解だけが誤っているのか自分には判断できません。
236
+
237
+ 申し訳ありませんが、こちらのご指摘からお願いします。
238
+
239
+
240
+
241
+
242
+
243
+ 「階層」とはオブジェクトの所持関係のことです。追記1で記述したコードの場合、キャラクタークラスのフィールドのオブジェクトが更に別のフィールドを持ち、その初期化の内容がキャラクターごとなら追記1の通り初期化が複雑で膨大になるのでは?と考えていました。

1

内容の訂正とコードの追加

2019/10/05 04:35

投稿

namatomato
namatomato

スコア5

test CHANGED
File without changes
test CHANGED
@@ -13,3 +13,195 @@
13
13
 
14
14
 
15
15
  こういった場合に、開放閉鎖原則を守りながらクラス設計をするにはどうしたらよいでしょうか?
16
+
17
+
18
+
19
+
20
+
21
+ ※追記
22
+
23
+ 至らぬ点が多々あり、申し訳ございません。
24
+
25
+
26
+
27
+ 「これは変更の理由ごとにクラスを分割することで、新たな仕様追加には新たなクラスを増やすことで対応するというものだと解釈しました。」
28
+
29
+ これは解釈以前に内容が伝わる文章になっていませんでした。
30
+
31
+ 変更の理由=新たな仕様追加時には新たなクラスを追加するような設計にするという意味です。
32
+
33
+
34
+
35
+
36
+
37
+ 想定しているケースの簡単な例を記載します。拙いコードで申し訳ありませんが、ご確認下さい。
38
+
39
+
40
+
41
+ 前提条件
42
+
43
+ ・キャラクターを採集して点数を競うイベントがある
44
+
45
+ ・点数の計算式はキャラクターごとに異なる
46
+
47
+ ・点数の計算式は時間帯や曜日によっても異なる
48
+
49
+
50
+
51
+ ```C#
52
+
53
+ class Monster
54
+
55
+ {
56
+
57
+ //★モンスターごとに初期化が必要なフィールド
58
+
59
+ //攻撃、防御など複数の戦闘用パラメーター
60
+
61
+ //習得中の技
62
+
63
+ //場所ごとの遭遇率
64
+
65
+ //etc...
66
+
67
+
68
+
69
+
70
+
71
+ //時間帯と曜日により点数を返すメソッド
72
+
73
+ public int GetScore(TimeZone timeZone,Week week)
74
+
75
+ {
76
+
77
+ //省略
78
+
79
+ }
80
+
81
+
82
+
83
+
84
+
85
+ //危惧している点:分岐ごとの処理メソッドが沢山増える
86
+
87
+ private GetScoreByMoning()
88
+
89
+ {
90
+
91
+ //省略
92
+
93
+ }
94
+
95
+
96
+
97
+ private GetScoreByNone()
98
+
99
+ {
100
+
101
+ //省略
102
+
103
+ }
104
+
105
+
106
+
107
+ private GetScoreByNight()
108
+
109
+ {
110
+
111
+ //省略
112
+
113
+ }
114
+
115
+
116
+
117
+ }
118
+
119
+
120
+
121
+ class MonsterList : List<Monster>
122
+
123
+ {
124
+
125
+ //List内のMonsterの点数を合計する
126
+
127
+ public int GetTotalScore()
128
+
129
+ {
130
+
131
+ //省略
132
+
133
+ }
134
+
135
+
136
+
137
+ }
138
+
139
+
140
+
141
+ class Contest
142
+
143
+ {
144
+
145
+ //MonsterListのTotalScoreによりGiftを返す
146
+
147
+ public Gift GetGift(MonsterList monsters)
148
+
149
+ {
150
+
151
+ //省略
152
+
153
+ }
154
+
155
+
156
+
157
+
158
+
159
+ }
160
+
161
+
162
+
163
+ public enum Gift
164
+
165
+ {
166
+
167
+ //省略
168
+
169
+ }
170
+
171
+
172
+
173
+ public enum Week
174
+
175
+ {
176
+
177
+ //省略
178
+
179
+ }
180
+
181
+
182
+
183
+ public enum TimeZone
184
+
185
+ {
186
+
187
+ //省略
188
+
189
+ }
190
+
191
+ ```
192
+
193
+
194
+
195
+ 問題として考えている点は主に2つになります。
196
+
197
+ ・キャラクター固有の特徴を表すフィールドの初期化が複雑で膨大になる
198
+
199
+ ・条件分岐ごとのメソッドにより、メソッド数が膨大になる
200
+
201
+
202
+
203
+ 回避策として、例えばMonsterAに対してMonsterAScoreクラスを作成して処理を委譲することを考えたのですが、そうするとキャラクターが増えるごとにいくつものクラスを作成しなければいけなくなります。
204
+
205
+
206
+
207
+ どのようにアプローチすればよいでしょうか。