質問編集履歴
2
参考URLと内容の追加
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チップシリーズ ２：SOLIDの世界１](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
内容の訂正とコードの追加
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
|
+
問題として考えている点は主に２つになります。
|
|
196
|
+
|
|
197
|
+
・キャラクター固有の特徴を表すフィールドの初期化が複雑で膨大になる
|
|
198
|
+
|
|
199
|
+
・条件分岐ごとのメソッドにより、メソッド数が膨大になる
|
|
200
|
+
|
|
201
|
+
|
|
202
|
+
|
|
203
|
+
回避策として、例えばMonsterAに対してMonsterAScoreクラスを作成して処理を委譲することを考えたのですが、そうするとキャラクターが増えるごとにいくつものクラスを作成しなければいけなくなります。
|
|
204
|
+
|
|
205
|
+
|
|
206
|
+
|
|
207
|
+
どのようにアプローチすればよいでしょうか。
|