回答編集履歴
3
不変オブジェクトらしい
test
CHANGED
@@ -32,6 +32,10 @@
|
|
32
32
|
|
33
33
|
|
34
34
|
|
35
|
-
ちなみに、本筋と関係ないので、書いていないのだと想像しますが、
|
35
|
+
<削除>ちなみに、本筋と関係ないので、書いていないのだと想像しますが、
|
36
36
|
|
37
|
-
コンストラクタで、dateを直接保持するのでなくcloneとか別のオブジェクトとして保持させるはず。
|
37
|
+
コンストラクタで、dateを直接保持するのでなくcloneとか別のオブジェクトとして保持させるはず。</削除>
|
38
|
+
|
39
|
+
|
40
|
+
|
41
|
+
クラスは不変でスレッドセーフなら問題ないです。
|
2
DateとLocalDateが、ごっちゃでした。
test
CHANGED
@@ -4,9 +4,9 @@
|
|
4
4
|
|
5
5
|
はい。そのとおりです。
|
6
6
|
|
7
|
-
ただ、そもそも前提としてあるのが、ServiceDateクラスはDateクラスじゃないということです。
|
7
|
+
ただ、そもそも前提としてあるのが、ServiceDateクラスはLocalDateクラスじゃないということです。
|
8
8
|
|
9
|
-
日付の演算なんて本来不要なのです。
|
9
|
+
日付の演算なんて本来不要(として設計されている)なのです。
|
10
10
|
|
11
11
|
|
12
12
|
|
@@ -16,7 +16,7 @@
|
|
16
16
|
|
17
17
|
という発想自体が、設計として誤っているというか、必要なメソッド(夏季の判断)以外は公開する必要がないです。
|
18
18
|
|
19
|
-
極論いってしまえば、内部の日付はDateでなく文字列でもいいです。
|
19
|
+
極論いってしまえば、内部の日付はLocalDateでなく文字列でもいいです。
|
20
20
|
|
21
21
|
|
22
22
|
|
1
コンストラクタについて追記
test
CHANGED
@@ -29,3 +29,9 @@
|
|
29
29
|
|
30
30
|
|
31
31
|
というところです。
|
32
|
+
|
33
|
+
|
34
|
+
|
35
|
+
ちなみに、本筋と関係ないので、書いていないのだと想像しますが、
|
36
|
+
|
37
|
+
コンストラクタで、dateを直接保持するのでなくcloneとか別のオブジェクトとして保持させるはず。
|