JAVA更新ArrayList对象方法不更新

Question

我有一个实现另一个超类的接口测试类。在测试类中，我有一个方法应该从数组列表中更新一个对象；首先它应该检查对象是否在列表中，如果是，它将删除并添加一个新对象（替换）。如果找不到对象，它将抛出异常消息。这是我实现的代码：

public class ProductDBImpl implements ProductDB {

// field declarations
ArrayList<Product> products = new ArrayList<Product>();
@Override
public void updateProduct(Product product) throws ProductNotFoundException 
{
    // TODO Auto-generated method stub
    Iterator<Product> pritr = products.iterator();
    while(pritr.hasNext())
    {
        Product pr = pritr.next();

        if (!pr.getId().equals(product.getId()))
        {

            throw new ProductNotFoundException("Product does no exist");

        }
        pritr.remove();

    }
    products.add(product);

}

首先我不知道这是否是正确的方法。而且，当我使用测试客户端脚本对其进行测试时，我收到一条错误消息：

Exception in thread "main" productdb.util.AssertionFailedError: should've gotten ProductNotFoundException

测试客户端代码如下：

    ipod.setId(Integer.MAX_VALUE);
    try {
        productDB.updateProduct(ipod);
        Assert.fail("should've gotten ProductNotFoundException");
    } catch (ProductNotFoundException pnfe) {
        // expecting this
    }

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

最新编辑 我根据收到的反馈更新了我的代码，我收到了第一个引发错误的项目：

public void updateProduct(Product product) throws ProductNotFoundException 
    {
        // TODO Auto-generated method stub
        Iterator<Product> pritr = products.iterator();
        while(pritr.hasNext())
        {
            Product pr = pritr.next();
            System.out.println(pr.getId());
            System.out.println(product.getId());
            if (pr.getId().equals(product.getId()))
            {

                pritr.remove();

            }
            else
            {
                throw new ProductNotFoundException("Product Not Found");
            }

        }
        products.add(product);

    }

另一个更新

public void updateProduct(Product product) throws ProductNotFoundException 
{
    // TODO Auto-generated method stub
    Iterator<Product> pritr = products.iterator();
    boolean match = true;
    while(pritr.hasNext())
    {
        Product pr = pritr.next();
        if (pr.getId().equals(product.getId()))
        {

            pritr.remove();

        }
        else
        {
            match = false;
        }

    }
    if (match == false)
    {
        new ProductNotFoundException("Product not found");
    }
    else
    {
        products.add(product);
    }


}

Answer 1

再次，遍历您的逻辑。你有这个：

public void updateProduct(Product product) throws ProductNotFoundException 
{
    // TODO Auto-generated method stub
    Iterator<Product> pritr = products.iterator();
    boolean match = true;
    while(pritr.hasNext())
    {
        Product pr = pritr.next();
        if (pr.getId().equals(product.getId()))
        {

            pritr.remove();

        }
        else
        {
            match = false;
        }

    }
    if (match == false)
    {
        new ProductNotFoundException("Product not found");
    }
    else
    {
        products.add(product);
    }


}

翻译为：

start method
    set match to true
    while loop
        if product found remove original product
        else match is set to false  // this will always happen one or more times!
    end while loop

    // match is almost guaranteed to be false!
    check match. if false, throw exception
    else if true, add new product
end method

现在假设数据是：

No match  
No match  
No match  
match  
No match  
No match

然后看看会发生什么。根据您的逻辑，布尔值在不应该的时候最终会是假的。

将此与应该引发异常的数据进行比较：

No match  
No match  
No match  
No match  
No match  
No match

再看看会发生什么。

你想要什么：

start method
    set match to false
    while loop
        if product found 
             remove original product
             replace with new product
             set match to true
        // no else block needed
    end while loop

    check match. if false, throw exception
    else if true, add new product
end method

或

start method
    // no need for match variable
    while loop
        if product found 
             remove original product
             replace with new product
             set match to true
             return from method // *****
    end while loop

    // if we reach this line, a match was never found
    throw exception

end method

Answer 2

Assert.fail 在没有抛出异常时执行，如果出现这种情况：

• 列表 (products) 的 first（这似乎是一个不必要的错误！）元素中的任何一个都具有相同的 id
• 或列表 (products) 为空

列表的内容是什么？