ArrayList 的 Java 问题

Question

我有一个包含两个数组列表的类，我试图将对象存储到其中，每个对象类型一个。在我的主类中，我像这样插入对象：

 for (int i =0; i < 3; i++)
 {
     Cat cat = new Cat("meow",i);
     Dog dog = new Dog("woof",i);
     objList.addCat(cat);
     objList.addDog(dog);
 }

我的 ObjectList (objList) 类的设置如下：

import java.util.ArrayList;

public class ObjectList {
    public ArrayList  cats;
    public ArrayList  dogs;

    public ObjectList()
    {
        this.cats   = new ArrayList();
        this.dogs   = new ArrayList();
    }

    public void addCat(Cat c)
    {
        this.cats.add(c);
    }

    public void addDog(Dog d)
    {
        this.dogs.add(d);
    }
}

但是，我从objList.addCat(cat); 行开始得到java.lang.NullPointerException。我在这一行之前打印出了 cat 对象的属性，这两个值似乎都已设置。我试图查看是否可以将一个 int 传递给我的 addCat arrayList，但我得到了同样的错误，所以我假设我在我的类中错误地使用了 arraylist。我的代码中是否存在显而易见的错误？

Answer 1

我建议的几件事：

1. 不要公开您的字段；
2. 尽可能使用接口（List 而不是 ArrayList）；
3. 支持immutability，或者如果您不打算更改数据成员，至少将它们设为最终成员；和
4. （与此无关）采用 Java 代码约定。

所以试试这个：

public class ObjectList {
  private final List cats;
  private final List dogs; 

  public ObjectList() {   
    cats = new ArrayList();
    dogs = new ArrayList();
  }

  public void addCat(Cat c) {
    cats.add(c);
  }

  public void addDog(Dog d) {
    dogs.add(d);
  }
}

这应该避免任何可能发生的外部变化。或者，更好的是，使用泛型：

public class ObjectList {
  private final List<Cat> cats;
  private final List<Dog> dogs; 

  public ObjectList() {   
    cats = new ArrayList<Cat>();
    dogs = new ArrayList<Dog>();
  }

  public void addCat(Cat c) {
    cats.add(c);
  }

  public void addDog(Dog d) {
    dogs.add(d);
  }
}

你正在初始化你的对象列表实例吗？

ObjectList objList = new ObjectList();
for (int i=0; i<3; i++)  {
  Cat cat = new Cat("meow",i);
  Dog dog = new Dog("woof",i);
  objList.addCat(cat);
  objList.addDog(dog);
}

Answer 2

你在使用 objList 之前是否对其进行了初始化？

ObjectList objList = new ObjectList();

（在循环之前）