Question

我对eratosthenes的筛子有些麻烦。所以我从一本名为“Schaum的轮廓”的书中得到了筛子的数学计算，但我认为这本书编写了错误的代码...... 这是本书中的代码：

public class Sieve
{
    final static int P = 800;
    static boolean[] isPrime = new boolean[count];

Sieve();
{
    for (int i = 2; i<P; i++)
    {
        isPrime[i] = true;
    }
    for (int i = 2; i<P/2; i++)
    {
        if (isPrime[i])
        {
            for (int j = 2*i; j<P; j += i)
            {
                isPrime[j] = false;
            }
        }   
    }
}
public static void main(String[] args)
{
    new Sieve();
    print();
}

static void print()  {
    for (int i=0; i<count; i++)
        if (isPrime[i]) System.out.println(i + " ");
        else if (i%90==0) System.out.println();
        System.out.println();
}}

所以是的，我使用了代码并进行了一些细微的更改，因为“Sieve（）”无法识别。 以下是我的代码：

public class Primenumbers
{
    final static int count = 1000;
    static boolean[] isPrime = new boolean[count];

    public static sieve(int count, boolean isPrime);
    {
        for (int i = 2; i<count; i++)
        {
            isPrime[i] = true;
        }
        for (int i = 2; i<count/2; i++)
        {
            if (isPrime[i])
            {
                for (int j = 2*i; j<count; j += i)
                {
                    isPrime[j] = false;
                }
            }   
        }
    }
    public static void main(String[] args)
    {
        for (int i=0; i<count; i++)
        {
            if (isPrime[i])
            {
                System.out.println(i + " ");
            }
        }
    }
}

那么......我做错了什么？谢谢你的帮助！

Answer 1

我做错了什么？

您已定义sieve()，但您从未对其进行过调用。您需要在打印前调用它：

public static void main(String[] args)
{
    sieve(); // <<== Here
    for (int i=0; i<count; i++)
    {
        if (isPrime[i])
        {
            System.out.println(i + " ");
        }
    }
}

它在本书中起作用的原因是初始化已在构造函数中完成。您修改后的代码将初始化移动到静态函数中，但无法调用该初始化。

您还需要从sieve声明中删除参数，因为它已经可以访问isPrime和count。

另一种方法是完全删除sieve方法，将其替换为静态初始化程序。而不是

public static sieve() 
{
    // Code to initialize isPrime
}

写

static
{
    // Code to initialize isPrime
}

此更改使sieve()方法成为静态初始值设定项，在您的类中的任何其他内容执行之前，它始终会被调用。

关于Eratosthenes的筛子

1 个答案: