我正在尝试在解决此问题here时找到数字的最大素数因子。我认为我做的一切都是正确的,但是其中一个测试用例(#2)失败了,我想不出任何可能失败的极端情况。这是我的代码,请看看并尝试发现一些东西。
public class ProblemThree
{
public static void main(String[] args)
{
Scanner scanner = new Scanner(System.in);
int T = scanner.nextInt();
for (int i = 0; i < T; i++)
{
System.out.println(largestPrime(scanner.nextLong()));
}
}
private static long largestPrime(long n)
{
while (n % 2 == 0)
{
n = n / 2; // remove all the multiples of 2
}
while (n % 3 == 0)
{
n = n / 3; // remove all the multiples of 2
}
// remove multiples of prime numbers other than 2 and 3
while (n >= 5)
{
boolean isDivisionComplete = true;
for (long i = 5; i < Math.ceil(Math.sqrt(n)); i++)
{
if (n % i == 0)
{
n = n / i;
isDivisionComplete = false;
break;
}
}
if (isDivisionComplete)
{
break;
}
}
return n;
}
}
基本上,我所做的是:
Largest_Prime(n):
1. Repeatedly divide the no by any small number, say x where 0 < x < sqrt(n).
2. Then set n = n/x and repeat steps 1 and 2 until there is no such x that divides n.
3 Return n.
答案 0 :(得分:3)
您的代码中似乎有一些错误,就像您输入 16 largestPrime 函数返回1时一样。当输入为3的幂时,这是正确的。
答案 1 :(得分:2)
详细的算法描述: 你可以通过保留三个变量来做到这一点: 您想要考虑的数字(A) 现在的除数店(B) 最大的除数店(C) 最初,让(A)成为您感兴趣的数字 - 在这种情况下,它是600851475143.然后让(B)为2.有条件检查(A)是否可以被(B)整除。如果它是可分的,则将(A)除以(B),将(B)重置为2,然后返回检查(A)是否可被(B)整除。否则,如果(A)不能被(B)整除,则将(B)增加+1,然后检查(A)是否可被(B)整除。运行循环直到(A)为1.返回的(3)将是600851475143的最大质数。
public static void main(String[] args) {
Scanner in = new Scanner(System.in);
int t = in.nextInt();
for(int a0 = 0; a0 < t; a0++){
long n = in.nextLong();
long A=n;
long B=2;
long C=0;
while(Math.pow(B,2)<=A)
{
if(A%B==0)
{
C=B;
A=A/B;
B=2;
}
else
B++;
}
if(A>=C)
C=A;
if(A==1)
{ C=2;
break;
}
System.out.println(C);
}
}
答案 2 :(得分:1)
为什么要删除2的倍数和3的倍数?这样,如果你的数字是2和3的任意组合,你的答案就是1,这显然是错误的。
对于这个问题,你可以采用天真的方式从2循环到sqrt(n)并存储除以n的最大数,当你完成循环时,只返回你找到的最高除数。
答案 3 :(得分:1)
1你的循环为2和3.如果没有,你得不到2,2x2,3,2x3,...所有2和3的倍数
2将你的循环改为停在2(而不是5):
$main
如果2
则停止3次while (n >= 2)
{
从2开始的4循环
和
5循环直到sqrt(n),其中&lt; =并且不仅&lt; (如果不是,你就不会获得素数X Prime)
if (n==2) return 2;
答案 4 :(得分:-1)
提取素数因子的一种简单方法是:
/**
* Prime factors of the number - not the most efficient but it works.
*
* @param n - The number to factorise.
* @param unique - Want only unique factors.
* @return - List of all prime factors of n.
*/
public static List<Long> primeFactors(long n, boolean unique) {
Collection<Long> factors;
if (unique) {
factors = new HashSet<>();
} else {
factors = new ArrayList<>();
}
for (long i = 2; i <= n / i; i++) {
while (n % i == 0) {
factors.add(i);
n /= i;
}
}
if (n > 1) {
factors.add(n);
}
return new ArrayList<>(factors);
}
那些第一个循环是一个问题。他们会将所有偶数减少到1
- 因此缺少2
因子。更改要使用的代码:
while (n > 2 && n % 2 == 0) {
n = n / 2; // remove all the multiples of 2
}
while (n > 3 && n % 3 == 0) {
n = n / 3; // remove all the multiples of 2
}
您还有其他问题 - 例如您报告25
的最大素数因子为25
,49
的最大素因子为49
。
只需使用您的代码运行此代码即可查看您的代码失败:
for (long i = 1; i < 1000; i++) {
long largestPrime = largestPrime(i);
List<Long> primeFactors = primeFactors(i, true);
if (primeFactors.size() > 0) {
Collections.sort(primeFactors, Collections.reverseOrder());
long highestFactor = primeFactors.get(0);
if (largestPrime != highestFactor) {
System.out.println("Wrong! " + i + " " + largestPrime + " != " + primeFactors);
}
} else {
System.out.println("No factors for " + i);
}
}