Skip to content

style: include NAB_NEEDLESS_BOOLEAN_CONSTANT_CONVERSION #5149

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 8, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 0 additions & 3 deletions spotbugs-exclude.xml
Original file line number Diff line number Diff line change
Expand Up @@ -126,9 +126,6 @@
<Match>
<Bug pattern="OCP_OVERLY_CONCRETE_PARAMETER" />
</Match>
<Match>
<Bug pattern="NAB_NEEDLESS_BOOLEAN_CONSTANT_CONVERSION" />
</Match>
<Match>
<Bug pattern="LSC_LITERAL_STRING_COMPARISON" />
</Match>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,10 @@ public boolean clock() {

private boolean getMajorityBit() {
Map<Boolean, Integer> bitCount = new TreeMap<>();
bitCount.put(false, 0);
bitCount.put(true, 0);
bitCount.put(Boolean.FALSE, 0);
bitCount.put(Boolean.TRUE, 0);

registers.forEach(lfsr -> bitCount.put(lfsr.getClockBit(), bitCount.get(lfsr.getClockBit()) + 1));
return bitCount.get(false) <= bitCount.get(true);
return bitCount.get(Boolean.FALSE) <= bitCount.get(Boolean.TRUE);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -34,15 +34,15 @@ void dijkstra(int[][] graph, int src) {

for (int i = 0; i < k; i++) {
dist[i] = Integer.MAX_VALUE;
Set[i] = false;
Set[i] = Boolean.FALSE;
}

dist[src] = 0;

for (int c = 0; c < k - 1; c++) {
int u = minDist(dist, Set);

Set[u] = true;
Set[u] = Boolean.TRUE;

for (int v = 0; v < k; v++) {
if (!Set[v] && graph[u][v] != 0 && dist[u] != Integer.MAX_VALUE && dist[u] + graph[u][v] < dist[v]) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ void primMST(int[][] graph) {
// Initialize all keys as INFINITE
for (int i = 0; i < V; i++) {
key[i] = Integer.MAX_VALUE;
mstSet[i] = false;
mstSet[i] = Boolean.FALSE;
}

// Always include first 1st vertex in MST.
Expand All @@ -65,7 +65,7 @@ void primMST(int[][] graph) {
int u = minKey(key, mstSet);

// Add the picked vertex to the MST Set
mstSet[u] = true;
mstSet[u] = Boolean.TRUE;

// Update key value and parent index of the adjacent
// vertices of the picked vertex. Consider only those
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ public int compareTo(Job otherJob) {
// Function to print the job sequence
public static String findJobSequence(ArrayList<Job> jobs, int size) {
Boolean[] slots = new Boolean[size];
Arrays.fill(slots, false);
Arrays.fill(slots, Boolean.FALSE);

int[] result = new int[size];

Expand All @@ -40,7 +40,7 @@ public static String findJobSequence(ArrayList<Job> jobs, int size) {
for (int j = jobs.get(i).deadline - 1; j >= 0; j--) {
if (!slots[j]) {
result[j] = i;
slots[j] = true;
slots[j] = Boolean.TRUE;
break;
}
}
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/com/thealgorithms/sorts/CircleSort.java
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ private <T extends Comparable<T>> Boolean doSort(T[] array, int left, int right)
boolean swapped = false;

if (left == right) {
return false;
return Boolean.FALSE;
}

int low = left;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,14 +1,17 @@
package com.thealgorithms.bitmanipulation;

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;

import org.junit.jupiter.api.Test;

class IsEvenTest {
@Test
void testIsEven() {
assertEquals(true, IsEven.isEven(2));
assertEquals(true, IsEven.isEven(-12));
assertEquals(false, IsEven.isEven(21));
assertTrue(IsEven.isEven(0));
assertTrue(IsEven.isEven(2));
assertTrue(IsEven.isEven(-12));
assertFalse(IsEven.isEven(21));
assertFalse(IsEven.isEven(-1));
}
}
Original file line number Diff line number Diff line change
@@ -1,17 +1,18 @@
package com.thealgorithms.dynamicprogramming;

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;

import org.junit.jupiter.api.Test;

class SumOfSubsetTest {

@Test
void basicCheck() {
assertEquals(false, SumOfSubset.subsetSum(new int[] {1, 2, 7, 10, 9}, 4, 14));
assertEquals(false, SumOfSubset.subsetSum(new int[] {2, 15, 1, 6, 7}, 4, 4));
assertEquals(true, SumOfSubset.subsetSum(new int[] {7, 3, 2, 5, 8}, 4, 14));
assertEquals(true, SumOfSubset.subsetSum(new int[] {4, 3, 2, 1}, 3, 5));
assertEquals(true, SumOfSubset.subsetSum(new int[] {1, 7, 2, 9, 10}, 4, 13));
assertFalse(SumOfSubset.subsetSum(new int[] {1, 2, 7, 10, 9}, 4, 14));
assertFalse(SumOfSubset.subsetSum(new int[] {2, 15, 1, 6, 7}, 4, 4));
assertTrue(SumOfSubset.subsetSum(new int[] {7, 3, 2, 5, 8}, 4, 14));
assertTrue(SumOfSubset.subsetSum(new int[] {4, 3, 2, 1}, 3, 5));
assertTrue(SumOfSubset.subsetSum(new int[] {1, 7, 2, 9, 10}, 4, 13));
}
}
21 changes: 11 additions & 10 deletions src/test/java/com/thealgorithms/maths/PythagoreanTripleTest.java
Original file line number Diff line number Diff line change
@@ -1,21 +1,22 @@
package com.thealgorithms.maths;

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;

import org.junit.jupiter.api.Test;

public class PythagoreanTripleTest {

@Test
public void Testpythagoreantriple() {
assertEquals(true, PythagoreanTriple.isPythagTriple(3, 4, 5));
assertEquals(true, PythagoreanTriple.isPythagTriple(6, 8, 10));
assertEquals(true, PythagoreanTriple.isPythagTriple(9, 12, 15));
assertEquals(true, PythagoreanTriple.isPythagTriple(12, 16, 20));
assertEquals(true, PythagoreanTriple.isPythagTriple(15, 20, 25));
assertEquals(true, PythagoreanTriple.isPythagTriple(18, 24, 30));
assertEquals(false, PythagoreanTriple.isPythagTriple(5, 20, 30));
assertEquals(false, PythagoreanTriple.isPythagTriple(6, 8, 100));
assertEquals(false, PythagoreanTriple.isPythagTriple(-2, -2, 2));
assertTrue(PythagoreanTriple.isPythagTriple(3, 4, 5));
assertTrue(PythagoreanTriple.isPythagTriple(6, 8, 10));
assertTrue(PythagoreanTriple.isPythagTriple(9, 12, 15));
assertTrue(PythagoreanTriple.isPythagTriple(12, 16, 20));
assertTrue(PythagoreanTriple.isPythagTriple(15, 20, 25));
assertTrue(PythagoreanTriple.isPythagTriple(18, 24, 30));
assertFalse(PythagoreanTriple.isPythagTriple(5, 20, 30));
assertFalse(PythagoreanTriple.isPythagTriple(6, 8, 100));
assertFalse(PythagoreanTriple.isPythagTriple(-2, -2, 2));
}
}
13 changes: 7 additions & 6 deletions src/test/java/com/thealgorithms/others/TwoPointersTest.java
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
package com.thealgorithms.others;

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;

import org.junit.jupiter.api.Test;

Expand All @@ -10,34 +11,34 @@ public class TwoPointersTest {
void twoPointersFirstTestCase() {
int[] arr = {2, 6, 9, 22, 121};
int key = 28;
assertEquals(true, TwoPointers.isPairedSum(arr, key));
assertTrue(TwoPointers.isPairedSum(arr, key));
}

@Test
void twoPointersSecondTestCase() {
int[] arr = {-1, -12, 12, 0, 8};
int key = 0;
assertEquals(true, TwoPointers.isPairedSum(arr, key));
assertTrue(TwoPointers.isPairedSum(arr, key));
}

@Test
void twoPointersThirdTestCase() {
int[] arr = {12, 35, 12, 152, 0};
int key = 13;
assertEquals(false, TwoPointers.isPairedSum(arr, key));
assertFalse(TwoPointers.isPairedSum(arr, key));
}

@Test
void twoPointersFourthTestCase() {
int[] arr = {-2, 5, -1, 52, 31};
int key = -3;
assertEquals(true, TwoPointers.isPairedSum(arr, key));
assertTrue(TwoPointers.isPairedSum(arr, key));
}

@Test
void twoPointersFiftiethTestCase() {
int[] arr = {25, 1, 0, 61, 21};
int key = 12;
assertEquals(false, TwoPointers.isPairedSum(arr, key));
assertFalse(TwoPointers.isPairedSum(arr, key));
}
}
Original file line number Diff line number Diff line change
@@ -1,23 +1,24 @@
package com.thealgorithms.strings;

import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertTrue;

import org.junit.jupiter.api.Test;

public class ValidParenthesesTest {

@Test
void testOne() {
assertEquals(true, ValidParentheses.isValid("()"));
assertTrue(ValidParentheses.isValid("()"));
}

@Test
void testTwo() {
assertEquals(true, ValidParentheses.isValid("()[]{}"));
assertTrue(ValidParentheses.isValid("()[]{}"));
}

@Test
void testThree() {
assertEquals(false, ValidParentheses.isValid("(]"));
assertFalse(ValidParentheses.isValid("(]"));
}
}