回答編集履歴
2
ハッシュコードはhashCodeの規約
test
CHANGED
|
@@ -47,3 +47,19 @@
|
|
|
47
47
|
|
|
48
48
|
|
|
49
49
|
という問題があります。
|
|
50
|
+
|
|
51
|
+
|
|
52
|
+
|
|
53
|
+
[Object#equals](https://docs.oracle.com/javase/jp/8/docs/api/java/lang/Object.html#equals-java.lang.Object-)
|
|
54
|
+
|
|
55
|
+
|
|
56
|
+
|
|
57
|
+
|
|
58
|
+
|
|
59
|
+
> 通常、このメソッドをオーバーライドする場合は、hashCodeメソッドを常にオーバーライドして、「等価なオブジェクトは等価なハッシュ・コードを保持する必要がある」という**hashCodeメソッドの汎用規約**に従う必要があることに留意してください。
|
|
60
|
+
|
|
61
|
+
|
|
62
|
+
|
|
63
|
+
ハッシュコードの話はequalsではなくhashCodeの汎用規約でした。
|
|
64
|
+
|
|
65
|
+
|
1
完全に規約違反だった
test
CHANGED
|
@@ -7,3 +7,43 @@
|
|
|
7
7
|
|
|
8
8
|
|
|
9
9
|
という点で問題があります。
|
|
10
|
+
|
|
11
|
+
|
|
12
|
+
|
|
13
|
+
---
|
|
14
|
+
|
|
15
|
+
|
|
16
|
+
|
|
17
|
+
ごめんなさい。それ以前に大きな問題がありました。
|
|
18
|
+
|
|
19
|
+
このequalsメソッドは引数をMyStuffにキャストしたときに、引数に名前があるか否かだけで判定しています。
|
|
20
|
+
|
|
21
|
+
そのため、
|
|
22
|
+
|
|
23
|
+
- 名前がnullのMyStuffを作ると、自分自身にfalse判定を出してしまう(反射性違反)
|
|
24
|
+
|
|
25
|
+
```java
|
|
26
|
+
|
|
27
|
+
MyStuff x = new MyStuff(null);
|
|
28
|
+
|
|
29
|
+
System.out.println(x.equals(x)); //反射性によりtrueを返すべきだが、falseになる
|
|
30
|
+
|
|
31
|
+
```
|
|
32
|
+
|
|
33
|
+
- 名前がnullと非nullのインスタンスを作って比較すると、x.equals(y)とy.equals(x)の結果が一致しない(対称性違反)
|
|
34
|
+
|
|
35
|
+
```java
|
|
36
|
+
|
|
37
|
+
MyStuff x = new MyStuff("name");
|
|
38
|
+
|
|
39
|
+
MyStuff y = new MyStuff(null);
|
|
40
|
+
|
|
41
|
+
System.out.println(x.equals(y)); //yの名前がnullによりfalse
|
|
42
|
+
|
|
43
|
+
System.out.println(y.equals(x)); //xの名前がnullでないためtrue
|
|
44
|
+
|
|
45
|
+
```
|
|
46
|
+
|
|
47
|
+
|
|
48
|
+
|
|
49
|
+
という問題があります。
|