类对象的ArrayList上的java.lang.IndexOutOfBoundException

Question

今天好，我有一个设计问题，我不能真正弄明白我的头，我希望我能解释清楚，让所有人都能理解。我有一个ClassA，它是一个包含ClassB的Arraylist的单例，所以ClassA是：

   private classA(){
    //private constructor
}

public static synchronized classA getClassA(){
    if(ClassA == null){         
        ClassA = new ClassA();
    }
    return ClassA;
}

private ArrayList<ClassB> classb = new ArrayList<ClassB>(5);

public ArrayList<ClassB> getClassB(){
    return this.classb;
}

public ClassB getLastClassB() {
    return this.classb.get(this.classb.size() - 1);
}

    public ClassB() {

}

private ArrayList<ClassD> classd = new ArrayList<ClassD>();


    public ArrayList<ClassD> getClassD(){
        return this.ClassD;
    }

    public ClassD getLastClassD() {
        return this.ClassD.get(this.ClassD.size() - 1);
    }

 ClassA  A = ClassA.getClassA();


        Class B B = new ClassB();

         ClassD D = new ClassD();
           //do some setters here for D

           B.getClassD.add(D)

           // do some things in B

           A.getClassB.add(B)

但是当我试图在另一个类中访问它时，我得到一个java.lang.IndexOutOfBoundsException：index：0,size:0，我猜这是来自ClassB保存ClassD的ArrayList的，因为其他函数也在工作。

  for(int i = 0; i < classA.getClassB.size(); i++)
   if(classA.getClassB().get(i).getClassD().size() > 0){ //java.lang.indexOutOfBoundsException

有什么想法，什么可能是错误的，或者是我的设计实现的方式吗？…我不能使ClassB成为另一个单例，因为这样我就会在ArrayList ClassA中拥有相同的ClassB对象。有什么更好的实现方法吗？..事先多谢。

Answer 1

在this.classd.get(This.classd.size()-1)和另一个类似的代码行中，size()调用是否总是保证返回大于零的值？换句话说，你能保证列表总是至少有一个元素吗？显然不是，所以是的，这是一个糟糕的设计。

您的getLastClassX()方法应准备好返回null并因此在尝试获取大小为（）-1的项之前调用列表中的！isEmpty()。