java - 筛网故障筛

我试图用Java创建一个Eratosthenes筛，但是我编写的代码似乎有缺陷。
我想知道是否有人可以发现我的错误，因为我不能。我得到的输出只是[2]，这意味着我的主循环不起作用。我才刚刚开始使用Java，所以如果您能给出详细的答案，我将不胜感激。

我的代码：

public static int[] primes(int n)
{
    //Variable assignement
    double sqrt;
    List<Integer> primes= new ArrayList<Integer>();
    //Adds 2 to primes so i don't need to include the even numbers in my for loop.
    if(n>1)
    {
        primes.add(2);
    }
    //For loop that goes through the oneven numbers up to n
    for(int counter1=3;counter1<=n;counter1+=2)
    {
        sqrt=Math.floor(Math.sqrt(0.0+counter1));
        //for loop that tests if the first for loops number is prime
        for(int counter2=0;sqrt<=primes.get(counter2);counter2++)
        {
            if(counter1 % primes.get(counter2) != 0 && counter2 ==sqrt)
            {
                primes.add(counter1);
            }

            if(counter1 % primes.get(counter2)==0)
            {
                break;
            }
        }
    }
    return convertIntegers(primes);
}
//Converts the list to an array
public static int[] convertIntegers(List<Integer> integers)
{
    int[] ret = new int[integers.size()];
    for (int i=0; i < ret.length; i++)
    {
        ret[i] = integers.get(i).intValue();
    }
    return ret;
}

最佳答案

我没有彻底检查您的代码，但可能涉及精度问题。
请注意，double可能无法精确地表示整数值，因此即使counter2 ==sqrt似乎是整数，sqrt也可能为假。

为防止这种情况，请尝试以下操作：

//this is not necessarily a square root anymore, btw :)
int sqrt = (int) Math.floor( Math.sqrt( (double)counter1 ) );
...
for( ... ) {
  ...
  if(... && counter2 ==sqrt ) {
    ...
  }
  ...
}

编辑：

通过更多的分析得出以下结果：

假设n = 5，那么将在您的代码中执行以下步骤：


counter1 = 3
sqrt = 1 //楼板（sqrt（2））
counter2 = 0
counter1 % primes.get(counter2) != 0 && counter2 ==sqrt为false，因为3 % 2 = 1但0 != 1
counter1 % primes.get(counter2)==0为false，因为3 % 2 = 1
counter2 = 1（下一次迭代）
primes.get(counter2)引发异常，因为primes.get(1)需要primes至少具有2个元素。


您可以尝试使用counter2 <=sqrt代替。

关于java - 筛网故障筛，我们在Stack Overflow上找到一个类似的问题：https://stackoverflow.com/questions/10088871/