Question

我一直在注意一些项目Euler问题的主题。他们需要大量的时间来完成。（像Largest Palindrome和10,001st Prime这样的问题）我希望有一些方法可以优化代码以使其更好。我让这一个跑了超过24分钟而且没有完成。

public class Seven
{
    public static void main(String[] args)
    {
        //Declare Variables
        int primeCount = 0;
        int numCount = 1;
        int latestPrime = 0;

        while(primeCount <= 10001)
        {
            if(isPrime(numCount))
            {
                primeCount++;
                latestPrime = numCount;
            }
            numCount++;
        }

        System.out.println("The 10,001st prime is: " + latestPrime);
    }

    //This method will determine if a number is prime
    public static boolean isPrime(long num)
    {
        //Check for even number
        if(num % 2 == 0)
            return false;

        //Check for non-prime odd numbers
        for(long i = 3; i <= num; i += 2)
        {
            if(num % i == 0)
                return false;
        }

    //Return that the number is prime
        return true;
}

}

Answer 1

你做了很多重复，你不需要在for循环中遍历整个数字，只需要直到平方根。

这会在一秒钟内运行。

 // Check for non-prime odd numbers
      for (long i = 3; i <= Math.sqrt(num) +1; i += 2) {
         if (num % i == 0)
            return false;
      }

如果您想进一步优化，可以将素数存储在一个数组中，只检查该数字是否可被前一个素数整除。

Answer 2

该程序打印10001个第一个素数。正如@Tejash Desai建议的那样，可以进行2次优化：

（1）将找到的素数保存在一个列表中，以便只对该列表中的项目进行新素数的测试，而不是所有奇数;和

（2）测试反对因子只能达到数字的平方根。

void printPrimes() {
    // Find first 10001 primes
    int p = 0;
    for (var i = 2; i <= 10001; i++) {
        p = nextPrime();
        Console.WriteLine("Prime #{0}: {1}", i, p);
    }
}

// Set of known primes
List<int> primes = new List<int>() {2};

// Find the prime next to the last one that was found
int nextPrime() {
    int k = primes[primes.Count - 1] + 1;
    while (!testPrimeUsingKnownSetOfPrimes(k)) {
        k = k + 1;
    }

    primes.Add(k);
    return k;
}

// Check if a number is prime, using the set of known primes
bool testPrimeUsingKnownSetOfPrimes(int n) {

    foreach (var p in primes) {
        // Largest prime factor of a number can't be greater than square root of the number
        if (p > Math.Sqrt(n)) {
            return true;
        }

        if (n % p == 0) {
            return false;
        }
    }

    return true;
}

PS 用C＃编写。

Answer 3

仅循环直到小于或等于n的平方根

//Check for non-prime odd numbers for(long i = 3; i <= Math.sqrt(num); i += 2) { if(num % i == 0) return false; }

另外，另一个优化方法是在ArrayList中保存你到目前为止所遇到的所有素数，然后只遍历该列表中的循环。

//Check for non-prime odd numbers for(int i = 0; i <= arrayList.size(); i ++) { if(num % arrayList.get(i) == 0) return false; }             arrayList.add(num);

项目Euler＃7优化

3 个答案: