Java

时间:2015-12-18 10:37:21

标签: java algorithm primes prime-factoring

我正在尝试在解决此问题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.

5 个答案:

答案 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的最大素数因子为2549的最大素因子为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);
  }
}