JAVA ArrayList 从 ArrayList 方法中检索所有对象

Question

我有一个实现另一个超类的接口测试类。在测试类中，我有一个方法应该检索 ArrayList 中的所有对象。首先它检查它是否为空。如果不是，它应该返回整个列表。这是我尝试过的。

public class ProductDBImpl implements ProductDB {

// field declarations
ArrayList<Product> products = new ArrayList<Product>();
@Override
public List<Product> getAllProducts() 
{
    // TODO Auto-generated method stub
    // TODO Auto-generated method stub
    // create an iterator for the Arraylist

    if (products.isEmpty())
    {
        return new ArrayList<Product>() ;
    }
    return products;
}

不幸的是，当我运行我的测试脚本（测试接口测试类和超类）时，getAllProducts() 方法无法正常工作，它返回了productdb.util.AssertionFailedError: expected:<1> but was:<0> 消息。下面是测试脚本测试 getAllProducts() 方法的方式：

    // testing getAllProducts()
    List<Product> productList = productDB.getAllProducts();     
    int size = productList.size();      
    productList.remove(0);      
    productList = productDB.getAllProducts();       
    Assert.assertEquals(size, productList.size());

请帮我找出我的错误。谢谢。

Answer 1

您返回对 ArrayList 对象的引用。然后修改对象。然后你得到另一个对它的引用，发现对象被修改了。

如果你想保护数据结构不被外部修改，那么返回一个新的List或者一个UnmodifiableList：

@Override
public List<Product> getAllProducts() 
{
    return new ArrayList<>(products);
}

或者，为了避免额外的内存分配和复制（请注意，这将导致在您的测试代码中调用 .remove() 失败；重要的部分是您是否必须允许调用者修改他们收到的列表） ：

@Override
public List<Product> getAllProducts() 
{
    return Collections.unmodifiableList(products);
}