style: include RV_ABSOLUTE_VALUE_OF_HASHCODE (#5273)

vil02 · web-flow · commit 5bc96cf78955 · 2024-07-02T22:47:35.000+03:00
diff --git a/spotbugs-exclude.xml b/spotbugs-exclude.xml
@@ -89,9 +89,6 @@
     <Match>
         <Bug pattern="RCN_REDUNDANT_NULLCHECK_OF_NONNULL_VALUE" />
     </Match>
-    <Match>
-        <Bug pattern="RV_ABSOLUTE_VALUE_OF_HASHCODE" />
-    </Match>
     <!-- fb-contrib -->
     <Match>
         <Bug pattern="OCP_OVERLY_CONCRETE_PARAMETER" />
diff --git a/src/main/java/com/thealgorithms/datastructures/hashmap/hashing/GenericHashMapUsingArray.java b/src/main/java/com/thealgorithms/datastructures/hashmap/hashing/GenericHashMapUsingArray.java
@@ -47,8 +47,7 @@ public void put(K key, V value) {
 
     // tells which bucket to go to
     private int hashFunction(K key) {
-        int hc = key.hashCode();
-        return Math.abs(hc) % buckets.length;
+        return Math.floorMod(key.hashCode(), buckets.length);
     }
 
     private void reHash() {

Original file line number	Diff line number	Diff line change
`@@ -47,8 +47,7 @@ public void put(K key, V value) {`
`47`	`47`
`48`	`48`	`// tells which bucket to go to`
`49`	`49`	`private int hashFunction(K key) {`
`50`		`- int hc = key.hashCode();`
`51`		`- return Math.abs(hc) % buckets.length;`
	`50`	`+ return Math.floorMod(key.hashCode(), buckets.length);`
`52`	`51`	`}`
`53`	`52`
`54`	`53`	`private void reHash() {`