Constructors BigInteger and BigDecimal used to wrap primitives should never be used. (#3627)
Constructors BigInteger and BigDecimal used to wrap primitives should never be used. Co-authored-by: Debasish Biswas <debasishbsws.abc@gmail.com>
This commit is contained in:
parent
cc17d60d5c
commit
c8ecd23183
@ -2438,16 +2438,16 @@ public class AES {
|
||||
public static BigInteger[] keyExpansion(BigInteger initialKey) {
|
||||
BigInteger[] roundKeys = {
|
||||
initialKey,
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
new BigInteger("0"),
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
BigInteger.ZERO,
|
||||
};
|
||||
|
||||
// initialize rcon iteration
|
||||
|
@ -63,7 +63,7 @@ public class RSA {
|
||||
publicKey = BigInteger.valueOf(3L);
|
||||
|
||||
while (m.gcd(publicKey).intValue() > 1) {
|
||||
publicKey = publicKey.add(BigInteger.valueOf(2L));
|
||||
publicKey = publicKey.add(BigInteger.TWO);
|
||||
}
|
||||
|
||||
privateKey = publicKey.modInverse(m);
|
||||
|
@ -10,7 +10,7 @@ public class KaprekarNumbers {
|
||||
digits and sum of these parts is equal to the original number. */
|
||||
|
||||
// Provides a list of kaprekarNumber in a range
|
||||
public static ArrayList<Long> kaprekarNumberInRange(long start, long end)
|
||||
public static List<Long> kaprekarNumberInRange(long start, long end)
|
||||
throws Exception {
|
||||
long n = end - start;
|
||||
if (n < 0) throw new Exception("Invalid range");
|
||||
@ -26,12 +26,12 @@ public class KaprekarNumbers {
|
||||
// Checks whether a given number is Kaprekar Number or not
|
||||
public static boolean isKaprekarNumber(long num) {
|
||||
String number = Long.toString(num);
|
||||
BigInteger originalNumber = new BigInteger(number);
|
||||
BigInteger originalNumber = BigInteger.valueOf(num);
|
||||
BigInteger numberSquared = originalNumber.multiply(originalNumber);
|
||||
if (number.length() == numberSquared.toString().length()) {
|
||||
return number.equals(numberSquared.toString());
|
||||
} else {
|
||||
BigInteger leftDigits1 = new BigInteger("0");
|
||||
BigInteger leftDigits1 = BigInteger.ZERO;
|
||||
BigInteger leftDigits2;
|
||||
if (numberSquared.toString().contains("0")) {
|
||||
leftDigits1 =
|
||||
|
@ -2,7 +2,8 @@ package com.thealgorithms.maths;
|
||||
|
||||
import static org.junit.jupiter.api.Assertions.*;
|
||||
|
||||
import java.util.ArrayList;
|
||||
import java.util.List;
|
||||
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
public class KaprekarNumbersTest {
|
||||
@ -50,7 +51,7 @@ public class KaprekarNumbersTest {
|
||||
@Test
|
||||
void testForRangeOfNumber() {
|
||||
try {
|
||||
ArrayList<Long> rangedNumbers = KaprekarNumbers.kaprekarNumberInRange(
|
||||
List<Long> rangedNumbers = KaprekarNumbers.kaprekarNumberInRange(
|
||||
1,
|
||||
100000
|
||||
);
|
||||
|
Loading…
Reference in New Issue
Block a user