如果我有一个类，并且在这个类中有一个创建 ArrayList 的构造函数...将此数组列表传递给类方法的最佳方法是什么？

Question

抱歉这个愚蠢的问题，但我似乎无法在谷歌上找到答案。我编写了一个类，在该类中有一个创建数组列表的构造函数，在同一个类中有一个通过创建迭代器对象来迭代数组列表的方法。然而，就我的代码而言，它无法识别 arraylists 名称，我应该将数组列表存储在类变量中，还是将其作为参数传递到方法中？

通常这里的最佳实践是什么，因为这总是让我困惑？

如果您无法理解我有些复杂的解释，我的代码如下！抱歉！非常感谢您的阅读:)

import java.util.*;
public class Primes {
  public Primes( int initialCapacity) {
    ArrayList<Integer> listOfPrimeNumbers = new ArrayList<Integer>(initialCapacity);    
    //how do I get the above...
    int index = 2;
    while (index != listOfPrimeNumbers.size())
    {
      if (isPrime(index))
      {
        listOfPrimeNumbers.add(index);  
      }
      index++;
    }
  }
  public static boolean isPrime(int candidateNo) {
    Iterator<Integer> iter = listOfPrimeNumbers.iterator( );
    //in here ! ?
    i=2;
    while ( iter.hasNext( ) ) {
      if (candidateNo%i==0 && i!=1) {
        return false;
      }
      else
        return true;
    }
  }

（另外，如果您发现我的代码有任何可怕的错误，请不要害怕批评我，建设性的批评越多越好！） }

Answer 1

您为 ArrayList 定义一个私有变量，并在构造函数中初始化该变量。
现在您可以访问班级内的列表了:)
希望有帮助。

import java.util.*;
public class Primes {

  private ArrayList<Integer> listOfPrimeNumbers;

  public Primes( int initialCapacity) {
    listOfPrimeNumbers = new ArrayList<Integer>(initialCapacity);    
    //how do I get the above...
    int index = 2;
    while (index != listOfPrimeNumbers.size())
    {
      if (isPrime(index))
      {
        listOfPrimeNumbers.add(index);  
      }
      index++;
    }
  }
  public static boolean isPrime(int candidateNo) {
    Iterator<Integer> iter = listOfPrimeNumbers.iterator( );
    //in here ! ?
    i=2;
    while ( iter.hasNext( ) ) {
      if (candidateNo%i==0 && i!=1) {
        return false;
      }
      else
        return true;
    }
  }

Answer 2

listOfPrimeNumbers 是一个局部变量。您可以将其更改为类的静态成员，使其成为实例变量并使 isPrime 非静态，或者将其作为参数传递给 isPrime。

考虑到此类的明显意图，我会将 isPrime 设置为非静态，并将 listOfPrimeNumbers 设置为实例变量。

Answer 3

您需要重新考虑如何将类组合在一起：

• 在构造函数中完成所有工作通常不是一个好主意。 Java 中的构造函数很棘手，通常我会尝试使它们尽可能简单，并将真正的工作放在其他地方。

• 静态方法被高估了，我尽量避免使用它们。如果您希望方法访问某些数据结构，请将这些方法设为实例方法并将数据结构作为实例变量。 Java 应该是一种面向对象的语言，使用静态方法会阻碍面向对象并限制你的选择。

• 考虑一下您希望如何使用该类，以及您希望如何测试它，并更改该类实现的 API 以与之匹配。

Answer 4

看来您想将 arraylist 设置为类成员变量。但是，您将 isPrime 声明为静态。这两件事放在一起没有意义。如果 isPrime 不需要是静态的，那么只需将 arraylist 存储为类成员即可。如果它确实需要是静态的（无论出于何种原因），则不需要构造函数（因为在这种情况下 isPrime 是静态的没有意义）并在其他地方创建数组列表并将其作为参数传递到 isPrime。

编辑：再想一想，你不需要 isPrime 是静态的，它应该看起来像这样：

public class Primes {
  ArrayList<Integer> listOfPrimeNumbers;
  public Primes( int initialCapacity) {
    listOfPrimeNumbers = new ArrayList<Integer>(initialCapacity);    
    //how do I get the above...
    int index = 2;
    while (index != listOfPrimeNumbers.size())
    {
      if (isPrime(index))
      {
        listOfPrimeNumbers.add(index);  
      }
      index++;
    }
  }
  public static boolean isPrime(int candidateNo) {
    Iterator<Integer> iter = listOfPrimeNumbers.iterator( );
    //in here ! ?
    i=2;
    while ( iter.hasNext( ) ) {
      if (candidateNo%i==0 && i!=1) {
        return false;
      }
      else
        return true;
    }
  }

但是，你似乎还有其他问题。构造函数中的 while 循环将如何结束？你每次都会增加索引，但只有当它是素数时，你才会将一些东西添加到你的列表中。

在你的 isPrime 方法中，你的 while 循环有什么意义？您调用 iter.hasNext()，但不使用其迭代的内容。您只检查 candidateNo 和 i。