Skip to content

Commit 3b54c7a

Browse files
authored
Merge branch 'master' into dependabot/maven/org.apache.maven.plugins-maven-pmd-plugin-3.24.0
2 parents 290903a + 5840579 commit 3b54c7a

File tree

4 files changed

+11
-12
lines changed

4 files changed

+11
-12
lines changed

pmd-exclude.properties

+1-2
Original file line numberDiff line numberDiff line change
@@ -39,15 +39,14 @@ com.thealgorithms.dynamicprogramming.WineProblem=UselessParentheses
3939
com.thealgorithms.maths.BinomialCoefficient=UselessParentheses
4040
com.thealgorithms.maths.Complex=UselessParentheses
4141
com.thealgorithms.maths.DistanceFormulaTest=UnnecessaryFullyQualifiedName
42-
com.thealgorithms.maths.FibonacciJavaStreamsTest=BigIntegerInstantiation
4342
com.thealgorithms.maths.Gaussian=UselessParentheses
4443
com.thealgorithms.maths.GcdSolutionWrapper=UselessParentheses
4544
com.thealgorithms.maths.HeronsFormula=UselessParentheses
4645
com.thealgorithms.maths.KaprekarNumbers=UselessParentheses
4746
com.thealgorithms.maths.KeithNumber=UselessParentheses
4847
com.thealgorithms.maths.LeonardoNumber=UselessParentheses
4948
com.thealgorithms.maths.LinearDiophantineEquationsSolver=UselessParentheses
50-
com.thealgorithms.maths.MatrixUtil=BigIntegerInstantiation,UselessParentheses
49+
com.thealgorithms.maths.MatrixUtil=UselessParentheses
5150
com.thealgorithms.maths.RomanNumeralUtil=UselessParentheses
5251
com.thealgorithms.maths.SecondMinMax=UselessParentheses
5352
com.thealgorithms.maths.SecondMinMaxTest=UnnecessaryFullyQualifiedName

src/main/java/com/thealgorithms/maths/FibonacciJavaStreams.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public static Optional<BigDecimal> calculate(final BigDecimal index) {
2222
return Optional.of(BigDecimal.ZERO);
2323
}
2424

25-
if (index.compareTo(new BigDecimal(2)) < 0) {
25+
if (index.compareTo(BigDecimal.TWO) < 0) {
2626
return Optional.of(BigDecimal.ONE);
2727
}
2828

src/test/java/com/thealgorithms/maths/FibonacciJavaStreamsTest.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,13 @@ public void testWithNegativeIndexShouldThrowException() {
2121
public void testCheckTheFirst4SequenceElements() {
2222
checkElement(BigDecimal.ZERO, BigDecimal.ZERO);
2323
checkElement(BigDecimal.ONE, BigDecimal.ONE);
24-
checkElement(new BigDecimal(2), BigDecimal.ONE);
25-
checkElement(new BigDecimal(3), new BigDecimal(2));
24+
checkElement(BigDecimal.TWO, BigDecimal.ONE);
25+
checkElement(new BigDecimal(3), BigDecimal.TWO);
2626
}
2727

2828
@Test
2929
public void testCheck10thSequenceElement() {
30-
checkElement(new BigDecimal(10), new BigDecimal(55));
30+
checkElement(BigDecimal.TEN, new BigDecimal(55));
3131
}
3232

3333
@Test

src/test/java/com/thealgorithms/maths/MatrixUtilTest.java

+6-6
Original file line numberDiff line numberDiff line change
@@ -11,20 +11,20 @@ class MatrixUtilTest {
1111
@Test
1212
void add() {
1313
final BigDecimal[][] matrix1 = {
14-
{new BigDecimal(3), new BigDecimal(2)},
14+
{new BigDecimal(3), BigDecimal.TWO},
1515
{BigDecimal.ZERO, BigDecimal.ONE},
1616
};
1717

1818
final BigDecimal[][] matrix2 = {
1919
{BigDecimal.ONE, new BigDecimal(3)},
20-
{new BigDecimal(2), BigDecimal.ZERO},
20+
{BigDecimal.TWO, BigDecimal.ZERO},
2121
};
2222

2323
final BigDecimal[][] actual = MatrixUtil.add(matrix1, matrix2).orElseThrow(() -> new AssertionError("Could not compute matrix!"));
2424

2525
final BigDecimal[][] expected = {
2626
{new BigDecimal(4), new BigDecimal(5)},
27-
{new BigDecimal(2), BigDecimal.ONE},
27+
{BigDecimal.TWO, BigDecimal.ONE},
2828
};
2929

3030
assertTrue(Objects.deepEquals(actual, expected));
@@ -37,7 +37,7 @@ void subtract() {
3737
};
3838

3939
final BigDecimal[][] matrix2 = {
40-
{new BigDecimal(2), BigDecimal.ZERO},
40+
{BigDecimal.TWO, BigDecimal.ZERO},
4141
{new BigDecimal(-2), new BigDecimal(-3)},
4242
};
4343

@@ -55,13 +55,13 @@ void subtract() {
5555
void multiply() {
5656

5757
final BigDecimal[][] matrix1 = {
58-
{BigDecimal.ONE, new BigDecimal(2), new BigDecimal(3)},
58+
{BigDecimal.ONE, BigDecimal.TWO, new BigDecimal(3)},
5959
{new BigDecimal(4), new BigDecimal(5), new BigDecimal(6)},
6060
{new BigDecimal(7), new BigDecimal(8), new BigDecimal(9)},
6161
};
6262

6363
final BigDecimal[][] matrix2 = {
64-
{BigDecimal.ONE, new BigDecimal(2)},
64+
{BigDecimal.ONE, BigDecimal.TWO},
6565
{new BigDecimal(3), new BigDecimal(4)},
6666
{new BigDecimal(5), new BigDecimal(6)},
6767
};

0 commit comments

Comments
 (0)