在Java中找到素数时获取奇怪的输出

Question

我有两种方法可以找出java方法中的质数-2工作正常，但是从方法一中得到错误的输出，可以帮助我在逻辑上做错的地方。 提前致谢

我的整个代码

package prepare;

import java.util.Scanner;

    public class Squar {
        //Method - 1 to find prime number
        boolean isPrime(int num){
            int exp = (int)Math.sqrt(num);
            for(int i=2;i<exp;i++){
                if(exp%2==0){
                    return false;
                }
            }return true;
        }

    public static void main(String[] args) {
        Scanner scan = new Scanner(System.in);
        int num = scan.nextInt();
        Squar s = new Squar();
        System.out.println("From M1 "+s.isPrime(num));
        scan.close();
        System.out.println("From M2 "+s.isPrimeNumber(num));
    }
    //Method - 2 to find prime number
    public  boolean isPrimeNumber(int number) {
        if(number == 1){
            return false;
        }
        if (number == 2 || number == 3) {
            return true;
        }
        if (number % 2 == 0) {
            return false;
        }
        int sqrt = (int) Math.sqrt(number) + 1;
        for (int i = 3; i < sqrt; i += 2) {
            if (number % i == 0) {
                return false;
            }
        }
        return true;
    }

}

输入：63个实际输出的质数为假，但与方法一输出的输出不同

63
From M1 true
From M2 false

Answer 1

在isPrime()方法中，您不应该检查num % i == 0而不是exp % 2 == 0吗？

Answer 2

看一下这段代码

if(exp%2==0){

它应该是num % i

Answer 3

像这样更改isPrime函数。

 boolean isPrime(int num) {
        int exp = (int) Math.sqrt(num);
        for (int i = 2; i < exp; i++) {
            if (num % i == 0) {
                return false;
            }
        }
        return true;
    }

因为在if条件下，您正在检查exp%2 == 0 。 但是，当对i < exp迭代时，该语句不会更改。 因此，此逻辑应以num % i == 0开启

Answer 4

好吧，我认为罪魁祸首是if(exp%2==0){并且它在迭代i<exp时引起问题，因此您可能需要将其调整为num%i==0

我试图给出其他一些方法来解决这个问题。 我希望这会有所帮助。

我认为有一个诱使您使用的原因
（int）Math.sqrt（num）;

我试图在下面详细说明。

考虑以下3种方法。 所有这些都是正确的，但是前两种方法有一些缺点。

方法1

boolean isPrime(int num) {
for(int i=2;i<num;i++) {
    if(num%i==0)
        return false;
}
return true;
}

我们有能力使其更快。

考虑一下， 如果2除以某个整数n，那么（n / 2）也将n除。 这表明我们不必尝试2到n之间的所有整数。

现在我们可以修改算法：

方法2

//checks whether an int is prime or not.
boolean isPrime(int num) {
for(int i=2;2*i<num;i++) {
    if(num%i==0)
        return false;
}
return true;
}

最后，我们知道2是“奇数”素数-它恰好是唯一的偶数素数。 因此，我们只需要单独检查2，然后遍历奇数直到n的平方根即可。 我认为这可能会诱使您使用（int）Math.sqrt（num）;

方法3

//checks whether an int is prime or not.
boolean isPrime(int num) {
//check if num is a multiple of 2
if (num%2==0) return false;
//if not, then just check the odds
for(int i=3;i*i<=num;i+=2) {
    if(num%i==0)
        return false;
}
return true;
}

因此， 我们从检查每个整数（最多检查n个以发现一个数为质数），到仅检查整数的一半直至平方根 。 这不是一种改进，特别是考虑到数量很大时。

Answer 5

好吧，您的第一个算法几乎是正确的（用％i代替％2）。 我不知道第二种算法，但是我一定会将其更改为以下形式：

public boolean isPrime(int n) {
   if (n <= 1) {
       return false;
   }
   for (int i = 2; i < Math.sqrt(n); i++) {
       if (n % i == 0) {
           return false;
       }
   }
   return true;
}