java - 编写 checkOrElseThrow 泛型函数的更好方法

Question

我有两个针对 Employee 和 Address DAO 类的函数调用，我在其中检查员工姓名或地址是否已被使用

为了使检查和抛出异常变得通用，我创建了以下通用函数

CommonUtil.java中的

checkOrElseThrow

public static <R, C, T extends Throwable> R checkOrElseThrow(R rtn, C chk, Supplier<? extends T> ex) throws T
{
    if (chk != null)
    {
        throw ex.get();
    }
    return rtn;
}

上面的通用函数在 EmployeeDAO.java 和 AddressDAO.java 中被调用，如下所示

EmployeeDAO.java中的

checkAndReturnEmployee

public Employee checkAndReturnEmployee(Employee employee) {
    return checkOrElseThrow(
        employee,
        employee.getAddressName(),
        () -> new EntityNotFoundException("Employee already in use for another address"));
}

AddressDAO.java中的

checkAndReturnAddress

public Address checkAndReturnAddress(Address address) {
    return checkOrElseThrow(
        address,
        address.getEmployeeName(),
        () -> new EntityNotFoundException("Address already in use for another address"));
}

问题

我的解决方案工作正常，但我想知道是否有其他更好的方法来重写我编写的通用函数(checkOrElseThrow)

Answer 1

最好的写法就是不写。

public Employee checkAndReturnEmployee(Employee employee) {
    if (employee.getAddressName() == null) {
      throw new EntityNotFoundException("Employee already in use for another address"));
    }
    return employee;
}

上面的代码同样简短，但更具可读性。条件是什么以及不满足时会发生什么更加清楚。

您的自定义函数仅用于尝试为 Java 创建一种新语法，其他人不会理解这种语法，而且您可能很快也会忘记。