BAEL-2388: moved examples java-collections-maps -> core-java-long. Fixing String comparisons

Author: cror

core-java-lang/pom.xml

@@ -66,6 +66,12 @@
             <artifactId>mail</artifactId>
             <version>${javax.mail.version}</version>
         </dependency>
+        <dependency>
+            <groupId>nl.jqno.equalsverifier</groupId>
+            <artifactId>equalsverifier</artifactId>
+            <version>${equalsverifier.version}</version>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <build>
@@ -424,6 +430,7 @@
         <maven-shade-plugin.version>3.1.1</maven-shade-plugin.version>
         <spring-boot-maven-plugin.version>2.0.3.RELEASE</spring-boot-maven-plugin.version>
         <exec-maven-plugin.version>1.6.0</exec-maven-plugin.version>
+        <equalsverifier.version>3.0.3</equalsverifier.version>
     </properties>
 
 </project>

…ava/com/baeldung/map/hashCode/Money.java …a/com/baeldung/equalshashcode/Money.javajava-collections-maps/src/main/java/com/baeldung/map/hashCode/Money.java renamed to core-java-lang/src/main/java/com/baeldung/equalshashcode/Money.java java-collections-maps/src/main/java/com/baeldung/map/hashCode/Money.java renamed to core-java-lang/src/main/java/com/baeldung/equalshashcode/Money.java

@@ -1,4 +1,4 @@
-package com.baeldung.map.hashcode;
+package com.baeldung.equalshashcode;
 
 class Money {
 
@@ -17,15 +17,19 @@ public boolean equals(Object o) {
         if (!(o instanceof Money))
             return false;
         Money other = (Money)o;
+        boolean currencyCodeEquals = (this.currencyCode == null && other.currencyCode == null)
+          || (this.currencyCode != null && this.currencyCode.equals(other.currencyCode));
         return this.amount == other.amount
-          && this.currencyCode == other.currencyCode;
+          && currencyCodeEquals;
     }
 
     @Override
     public int hashCode() {
         int result = 17;
         result = 31 * result + amount;
-        result = 31 * result + currencyCode.hashCode();
+        if (currencyCode != null) {
+            result = 31 * result + currencyCode.hashCode();
+        }
         return result;
     }
 

…java/com/baeldung/map/hashCode/Team.java …va/com/baeldung/equalshashcode/Team.javajava-collections-maps/src/main/java/com/baeldung/map/hashCode/Team.java renamed to core-java-lang/src/main/java/com/baeldung/equalshashcode/Team.java java-collections-maps/src/main/java/com/baeldung/map/hashCode/Team.java renamed to core-java-lang/src/main/java/com/baeldung/equalshashcode/Team.java

@@ -1,4 +1,4 @@
-package com.baeldung.map.hashcode;
+package com.baeldung.equalshashcode;
 
 class Team {
 

core-java-lang/src/main/java/com/baeldung/equalshashcode/Voucher.java

@@ -0,0 +1,38 @@
+package com.baeldung.equalshashcode;
+
+class Voucher {
+
+    private Money value;
+    private String store;
+
+    Voucher(int amount, String currencyCode, String store) {
+        this.value = new Money(amount, currencyCode);
+        this.store = store;
+    }
+
+    @Override
+    public boolean equals(Object o) {
+        if (o == this)
+            return true;
+        if (!(o instanceof Voucher))
+            return false;
+        Voucher other = (Voucher)o;
+        boolean valueEquals = (this.value == null && other.value == null)
+          || (this.value != null && this.value.equals(other.value));
+        boolean storeEquals = (this.store == null && other.store == null)
+          || (this.store != null && this.store.equals(other.store));
+        return valueEquals && storeEquals;
+    }
+
+    @Override
+    public int hashCode() {
+        int result = 17;
+        if (this.value != null) {
+            result = 31 * result + value.hashCode();
+        }
+        if (this.store != null) {
+            result = 31 * result + store.hashCode();
+        }
+        return result;
+    }
+}

…com/baeldung/map/hashCode/WrongTeam.java …m/baeldung/equalshashcode/WrongTeam.javajava-collections-maps/src/main/java/com/baeldung/map/hashCode/WrongTeam.java renamed to core-java-lang/src/main/java/com/baeldung/equalshashcode/WrongTeam.java java-collections-maps/src/main/java/com/baeldung/map/hashCode/WrongTeam.java renamed to core-java-lang/src/main/java/com/baeldung/equalshashcode/WrongTeam.java

@@ -1,5 +1,11 @@
-package com.baeldung.map.hashcode;
+package com.baeldung.equalshashcode;
 
+/* (non-Javadoc)
+* This class overrides equals, but it doesn't override hashCode.
+*
+* To see which problems this leads to:
+* TeamUnitTest.givenMapKeyWithoutHashCode_whenSearched_thenReturnsWrongValue
+*/
 class WrongTeam {
 
     String city;

core-java-lang/src/main/java/com/baeldung/equalshashcode/WrongVoucher.java

@@ -0,0 +1,47 @@
+package com.baeldung.equalshashcode;
+
+/* (non-Javadoc)
+* This class extends the Money class that has overridden the equals method and once again overrides the equals method.
+*
+* To see which problems this leads to:
+* MoneyUnitTest.givenMoneyAndVoucherInstances_whenEquals_thenReturnValuesArentSymmetric
+*/
+class WrongVoucher extends Money {
+
+    private String store;
+
+    WrongVoucher(int amount, String currencyCode, String store) {
+        super(amount, currencyCode);
+
+        this.store = store;
+    }
+
+    @Override
+    public boolean equals(Object o) {
+        if (o == this)
+            return true;
+        if (!(o instanceof WrongVoucher))
+            return false;
+        WrongVoucher other = (WrongVoucher)o;
+        boolean currencyCodeEquals = (this.currencyCode == null && other.currencyCode == null)
+          || (this.currencyCode != null && this.currencyCode.equals(other.currencyCode));
+        boolean storeEquals = (this.store == null && other.store == null)
+          || (this.store != null && this.store.equals(other.store));
+        return this.amount == other.amount
+          && currencyCodeEquals
+          && storeEquals;
+    }
+
+    @Override
+    public int hashCode() {
+        int result = 17;
+        result = 31 * result + amount;
+        if (this.currencyCode != null) {
+            result = 31 * result + currencyCode.hashCode();
+        }
+        if (this.store != null) {
+            result = 31 * result + store.hashCode();
+        }
+        return result;
+    }
+}

…baeldung/map/hashCode/MoneyUnitTest.java …eldung/equalshashcode/MoneyUnitTest.javajava-collections-maps/src/test/java/com/baeldung/map/hashCode/MoneyUnitTest.java renamed to core-java-lang/src/test/java/com/baeldung/equalshashcode/MoneyUnitTest.java java-collections-maps/src/test/java/com/baeldung/map/hashCode/MoneyUnitTest.java renamed to core-java-lang/src/test/java/com/baeldung/equalshashcode/MoneyUnitTest.java

@@ -1,4 +1,4 @@
-package com.baeldung.map.hashcode;
+package com.baeldung.equalshashcode;
 
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.assertFalse;

…/baeldung/map/hashCode/TeamUnitTest.java …aeldung/equalshashcode/TeamUnitTest.javajava-collections-maps/src/test/java/com/baeldung/map/hashCode/TeamUnitTest.java renamed to core-java-lang/src/test/java/com/baeldung/equalshashcode/TeamUnitTest.java java-collections-maps/src/test/java/com/baeldung/map/hashCode/TeamUnitTest.java renamed to core-java-lang/src/test/java/com/baeldung/equalshashcode/TeamUnitTest.java

@@ -1,4 +1,4 @@
-package com.baeldung.map.hashcode;
+package com.baeldung.equalshashcode;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;

java-collections-maps/pom.xml

@@ -41,12 +41,6 @@
             <artifactId>streamex</artifactId>
             <version>0.6.5</version>
         </dependency>
-        <dependency>
-            <groupId>nl.jqno.equalsverifier</groupId>
-            <artifactId>equalsverifier</artifactId>
-            <version>3.0.3</version>
-            <scope>test</scope>
-        </dependency>
     </dependencies>
 
     <properties>