Eratosthenes的筛子认为127之后的所有数字都是素数

Question

我的Eratosthenes筛网遇到问题。 我想编写一个Sieve，它不需要由所有数字组成的数组，直到您想要的最大质数，而只是在Sieve到达时跟踪每个质数倍数。 这意味着您不必先进行所有工作，而可以在需要时确定下一个质数。 添加诸如“从N开始查找K素数”之类的接口功能也很容易。 这是伪代码：

Begin with current number set to 2
Loop:
    If prime queue is not empty:
        Peek at the top prime in the queue
        If current > top, we can move top to the next multiple
            Remove the top prime from the prime queue
            Increment top to its next multiple
            Re-add it to the queue
        If current == top, current is not a prime
            Increment current number to next integer
        If current < top, we've found a prime
            Break
Push current number onto prime queue
Increment current number to next integer
Return the new prime

所以这是问题所在：我正确计算了前31个素数（最大为127），但是之后它认为每个数字都是素数。 我已经将代码放在Ideone上了-我希望它是某些Java集合行为或一个小错误，而不是算法本身。 我想不出算法应在一定数量的素数后中断的原因。 我已经手动确认，在127之后，如果正确地对堆进行了排序，我的算法应该将128识别为不是素数，但这不是代码向我显示的内容。

有什么建议么？

http://ideone.com/E07Te

（当然，一旦基本算法生效，我将增加2（以跳过所有非素数偶数）。我可能还会使Sieve成为可迭代的。）

Answer 1

你的问题是

top.multiple == current

与...有关

Integer current = 2;
Integer multiple;

还有就是缓存Integer s的绝对值小， -128到127 ，如果我没有记错，所以使用比较==相同的情况下比较了超过128较小的值但从128，你会得到一个新的盒装Integer为current ，并且与top.multiple引用的对象不同。

使用equals比较或声明int current; 解决它。

并改进您的算法，仅从素数的平方中记下每个素数的倍数。

Answer 2

您没有检查整个列表：

31之后的筛子堆：[[127：127]，[11：132]，[2：128]

您到达132，即> 128，从而break; 在您检查2 * 64之前。