无法捕获异常

Question

我在产品实现中有以下代码

public void updateProduct(Product product) throws ProductNotFoundException {

    Iterator<Product> it = allProducts.iterator();
    Product p = null;
    if (!allProducts.isEmpty()) {
        while (it.hasNext()) {
            p = it.next();
            if (p.getId() == product.getId()) {
                p.setPrice(product.getPrice());
                System.out.println("Successfully updated the product "
                        + product.getId());
            }
        }
    } else {
        System.out.println("No such product " + product.getId());
        throw new ProductNotFoundException(product.getName());
    }
}

//主要

ipod.setId(Integer.MAX_VALUE);

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

Junit正在抛出以下异常

Exception in thread "main" productdb.util.AssertionFailedError: should've gotten ProductNotFoundException
    at productdb.util.Assert.fail(Assert.java:43)
    **at productdb.ProductDBClient.testProductServer(ProductDBClient.java:85)**
    at productdb.ProductDBClient.main(ProductDBClient.java:20)

不幸的是，我无法使用try和catch。 编译器抛出错误，说它无法被捕获。

你能不能给我一些关于我做错的指示？

非常感谢！！

Answer 1

您忘记了每个可用产品的情况p.getId（）== product.getId（）为false。 遵循通用代码片段逻辑，如果没有id匹配请求，您还应该抛出异常。 像这样改写：

    public void updateProduct(Product product) throws ProductNotFoundException {
    for (Product existing : allProducts) {
        if (existing.getId() == product.getId()) {
            existing.setPrice(product.getPrice());
            return;
        }
    }
    System.out.println("No such product " + product.getId());
    throw new ProductNotFoundException(product.getName());
}

您还可以查看一些功能样式的库，如Guava，以简化集合操作。

Answer 2

我没有看到任何原因会抛出“ProductNotFoundException”

if (!allProducts.isEmpty()) {
        //Code
} else {
        System.out.println("No such product " + product.getId());
        throw new ProductNotFoundException(product.getName());
}

显然，如果你的allProducts不为空，它将永远不会抛出异常

正确版本的代码[假设您无法更新产品，抛出异常]

public void updateProduct(Product product) throws ProductNotFoundException {

    Iterator<Product> it = allProducts.iterator();
    Product p = null;
    if (!allProducts.isEmpty()) {
        while (it.hasNext()) {
            p = it.next();
            if (p.getId() == product.getId()) {
                p.setPrice(product.getPrice());
                System.out.println("Successfully updated the product "
                        + product.getId());
                return;
            }
        }
    } 
    throw new ProductNotFoundException(product.getName());
}

Answer 3

使用您拥有的代码，很难给出明确的答案。 问题是没有达到代码的else路径，你应该使用调试器来找出原因。

但是解决问题的一些提示：

1）改变if (p.getId() == product.getId()) { to if (p.getId().equals(product.getId())) {我认为你的意图是检查是否相等id对象而不是它们的引用。

2）逐步通过调试器查看是否到达了else，或者让初学者验证try-catch是否正常工作只是注释掉代码，如果哪个会让你相信try-catch正在工作并且你的代码保留在if path并因此不抛出异常

Answer 4

原因是，即使在现有产品中找不到产品，您也永远不会抛出异常。 您只在没有任何产品时抛出异常。请尝试以下代码：

public void updateProduct(Product product) throws ProductNotFoundException {

    Iterator<Product> it = allProducts.iterator();
    Product p = null;
    if (!allProducts.isEmpty()) {
        while (it.hasNext()) {
            p = it.next();
            if (p.getId() == product.getId()) {
                p.setPrice(product.getPrice());
                System.out.println("Successfully updated the product "
                        + product.getId());
            }
        }
    }

    if (p == null ) {
        System.out.println("No such product " + product.getId());
        throw new ProductNotFoundException(product.getName());
    }
}