抛出异常 - 并进一步执行代码

Question

遇到这种情况——在main方法中，调用了一个子方法，它检查对象，在这个子方法中抛出异常（列表中的对象之一为NULL）。但是main方法的代码还是继续执行！ 示例代码：

@Transactional
public boolean addCompany(List<Company> companies, List<Address> addresses) throws Exception{
    checkAddress(addresses);
    try{
        for(int i = 0; i < companies.size(); i++){
            if(findCompany(companies.get(i).getId()) == null && !isExistsCompany(companies.get(i))){
                companies.get(i).setAddress(addresses.get(i));
                this.em.persist(companies.get(i));
            }
        }
    }catch(Exception e){
        return false;
    }
    return true;
}

public void checkAddress(List<Address> addresses) throws Exception{
    try{
        if(addresses == null)
            throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid parameter: list is null");
        for(Address a : addresses)
            if(a == null)
                throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid list item: object is null");
    }catch(Exception e){
        e.printStackTrace();
    }
}

在这方面，出现了几个问题： - 为什么代码不会停止？ - 作为一种选择，现在是否有必要通过将 checkAddress 方法的类型从 void 更改为 boolean 并在 main 方法中处理 true/false 来摆脱这种情况？ - 如何在前端正确处理此类错误 - 文本是否向前端发送异常或仅处理代码 500，如果是，那么为什么在后端生成异常 - 以帮助开发过程？如何胜任处理？ 请指教。 提前致谢。

Answer 1

您正在捕捉Exception(s)，当您不重新抛出Exception Java 运行时认为它已处理时。如果您希望程序执行停止，那么您需要将 Exception(s) 传播给调用者。比如在checkAddress改

} catch(Exception e) {
    e.printStackTrace();
}

类似

} catch(Exception e) {
    e.printStackTrace();
    throw e; // <-- re-throw the Exception
}

或只需将try 和catch 完全删除，然后Exception 就会自动抛出给调用者。此外，在 Java 8+ 中，您可以使用 Stream。喜欢，

public void checkAddress(List<Address> addresses) throws Exception {
    if (addresses == null) {
        StackTraceElement[] ste = Thread.currentThread().getStackTrace();
        throw new Exception(ste[2].getClassName() + "."
                + ste[2].getMethodName() + "." + ste[1].getMethodName()
                + ": Invalid parameter: list is null");
    }
    if (addresses.stream().anyMatch(a -> a == null)) {
        StackTraceElement[] ste = Thread.currentThread().getStackTrace();
        throw new Exception(ste[2].getClassName() + "."
                + ste[2].getMethodName() + "." + ste[1].getMethodName()
                + ": Invalid list item: object is null");
    }
}

Answer 2

您需要从 checkAddress() 方法中删除 try.. catch 块。这样，任何从checkAddress() 内部抛出的异常都会传播给它的调用者。

在addCompany() 方法中，将对checkAddress() 方法的调用放入try .. catch 并在那里处理异常。

当checkAddress()抛出异常时，代码执行会跳转到catch块。

Answer 3

e.printStackTrace();

这一行抑制异常，导致您的代码继续而不是失败。它打印堆栈跟踪，这可能使它看起来像是正在抛出异常，但它并没有比那行更进一步。

您几乎从不想使用printStackTrace()，而是应该正确处理您打算使用的异常，或者只是让异常传播给您的方法的调用者。

Answer 4

试试这个：

public void checkAddress(List<Address> addresses) throws Exception{
        if(addresses == null)
            throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid parameter: list is null");
        for(Address a : addresses)
            if(a == null)
                throw new Exception(Thread.currentThread().getStackTrace()[2].getClassName() + "." + Thread.currentThread().getStackTrace()[2].getMethodName() + "." + Thread.currentThread().getStackTrace()[1].getMethodName() + ": Invalid list item: object is null");
    }

}

关于线程的那些东西是什么？疯狂的代码。

首先不允许任何人将空实例添加到列表中。

我可以这样写：

public void checkAddresses(List<Address> addresses) {
    if (addresses == null) throw new IllegalArgumentException("Address List cannot be null");
    for (Address a : addresses) {
        if (a == null) throw new IllegalArgumentException("Address cannot be null");
    }
}