埃拉托斯特尼实施的筛子不检查某些数字

Question

我正在编写一个程序，使用厄拉多塞算法的好筛子来寻找质数(虽然有一个扭曲)。为了将所需字节数组的大小减半，我不表示任何偶数，而坚持使用奇数(通过整数除以2来计算它们在数组中的位置)。

然而，我有一个问题。我的程序似乎无法将17或113识别为质数，尽管它们确实是质数。这是我的代码：

public class Eratosthes {

    byte[] checklist;
    int range;

    public EratosthesConcurrent(int range) {
        this.range = range/2;
        this.checklist = new byte[this.range];
        this.initAllChecked();
        this.findPrimesSeq();
        this.printPrimes();
    }

    private void initAllChecked(){
        for(int i = 1; i < checklist.length; i++){
            checklist[i] = 1;
        }
    }

    private void crossOut(int i){
        checklist[i/2] = 0;
    }

    private void findPrimesSeq(){
        int curr = 3;
        outer: while(curr < range*2){
            for(int i = curr*curr; i < range*2; i+=2*curr){
                if(i != curr && i%curr == 0){
                    if(i == 113){
                        System.out.println(curr);
                    }
                    crossOut(i);
                }
            }

            secondLoop: for(int i = curr; i < range*2; i++){
                if(checklist[i/2] == 1 && curr != i){
                    curr = i;
                    break secondLoop;
                } else if(i == range*2-1 || i == range*2-2){ //Should be changed, might miss the last prime
                    break outer;
                }
            }
        }
    }

    public void printPrimes(){
        for(int i = 1; i < range; i++){
            if(checklist[i] == 1){
                System.out.println("Prime found: " + ((i*2)+1));
            }
        }
    }
}

将以下内容添加到 crossOut 方法不会生成任何输出：

if(i == 17 || i == 113){
    System.out.println("Found one of the numbers");
}

这对我来说很奇怪，因为程序的打印输出不包括17和113，因此不检查它们的任何倍数。这给我留下了一个字节数组，其中所有素数都被检查了...以及 17 或 113 的任意倍数。

如果有人能在我的程序中找到错误，我会很高兴的。我自己已经调试了几个小时，没有结果。提前谢谢。

Answer 1

for(int i = curr*curr; i < range*2; i+=2*curr){

第一个值= 9，然后到15

货币= 3 - 3*3 = 9

i = 2 * curr = 9 6

伪：

outerNum = sqrt[range]
array[boolean] size of range
array[0 & 1] = false

for i = 2 to outernum
       for j  = i to range
           if j % i != 0
               array[j] = false

for i = array[boolean] length
     print if array[i] = true

这是从记忆中得出的，所以如果有任何错误，请原谅。

Answer 2

问题似乎是你没有检查2除法。

您在某个时刻调用了crossOut（16）（特别是作为44的倍数），它去掉了 17的相应值。同样， 112导致113被删除。

改变：

if (i != curr && i % curr == 0) {

到：

if (i % 2 != 0 && i != curr && i % curr == 0) {

这两个都被打印为prime。

另外，我有点担心这个循环到底在做什么-我没有太详细地查看输出，虽然它实际上看起来是所有的素数，但我想你应该循环curr的倍数，第一个是2*curr（not >）， 0.05的增量，虽然我并不是说你所做的是不正确的，因为我需要更多的时间来确切理解你的代码是如何解决这个问题的。