数组隐私 Java - 基本

Question

对于（另一个）公认的非常基本的问题，我很抱歉，但我在一些 Java 代码中遇到了更多问题，我必须在课堂上解决这些问题。任务是筛选 Eratosthenes 程序，虽然我的构造函数似乎可以工作（从 main 方法调用 myArray[50] 给出了它被初始化的值），但它没有反映其他类的任何更改旨在使myArray。此外，print() 不打印任何内容（可能是因为该函数看到了 myArray 的统一版本。）。

目标是 SieveEm() 和 deleteStuff() 将所有非素数标记为假，print() 来并打印所有素数（标记为真）。

我似乎在 Internet/StackOverflow 上找不到任何解决问题的方法。我做错了什么？

    public class Sieve {
    private int count;
    private boolean[] myArray;
    public Sieve(int length){
        count = length+1;
        boolean[] newArray = new boolean[length+1];
        if(length>=1){
            newArray[0]=false;
            newArray[1]=false;
            if(length>=2){
                for(int i=0;i<=length;i++){
                    newArray[i]=true;
                }
            }
        }
        this.myArray = newArray;
    }
    public void SieveEm(){
        System.out.println("here now");
        System.out.println(this.myArray[50]==true);
        for(int i=0; i<count;i++){
            System.out.println("We got this far");
            if(this.myArray[i]){
                deleteStuff(i);
            }
        }
    }
    public void deleteStuff(int current){
        int increment= current;
        current+=increment;
        while(current<count){
            this.myArray[current]=false;
            current+=increment;
        }
    }
    public void print(){
        this.SieveEm();
        for(int i=2;i<count;i++){
            if(this.myArray[i]){
                System.out.println(i);
            }
        }
    }
    public static void main(String[] args){
        Sieve mySieve = new Sieve(100);
        mySieve.print();
        System.out.println(mySieve.myArray[50]);
    }
}

Answer 1

我会逐个解决任何有问题的地方，因为似乎有一点需要纠正。

对于public void deleteStuff(int current),，这个int increment= current; current+=increment; 似乎相当多余和有害，因为increment 没有做任何特别的事情。更重要的是，您的while - loop 将永远不会终止，因为在第一次通过时current = 0; 和increment = 0;。如果您执行current++;，那么您的while-loop 将终止。此外，如果您的 while 循环确实进行并遍历数组，则所有 true 值都将设置为 false。

转到print()。此方法不打印任何内容，因为您将整个 myArray 设置为 false。当然，由于我之前提到的while-loop 问题，永远无法到达要打印的for-loop，因为你被困在一个永无止境的while-loop. 所以，如果你解决了while - loop 问题，你将每个元素都设置在数组为假，所以下面的语句永远不会执行。

if(this.myArray[i]) { System.out.println(i); }

如果我注意到其他任何事情，我会告诉你。