试试这个：

public <T> ResponseEntity<T> validateRequest(Request request) {
  if(agentRepository.findAgentByRequestPoint(request.getPoint()).isEmpty())
    return ResponseEntity.ok((T)ErrorDTO.from(bundle.getString("point.not.set")));
    
  if(!request.getAdvanced().getFunction().equals("CheckAcc"))
    return ResponseEntity.ok((T) response.wrongCheck(1, 1));
  if(ServiceRepository.findServiceByServiceId(Long.parseLong(request.getAdvanced().getService())).isEmpty())
    return ResponseEntity.ok((T)  response.wrongCheck(1, 4));
}

我建议对变量/方法进行更多的验证或空检查，因为像request.getAdvanced().getFunction().equals("CheckAcc")或request.getAdvanced().getService()这样的字段似乎容易受到NPE的攻击。
或者至少尝试捕获该方法并记录堆栈跟踪以供将来调试之用。
希望能有所帮助。

提高代码可读性的最佳方法是将代码拆分为不同的方法：

public <T> ResponseEntity<T> validateRequest(Request request) {
    if (!isAgentExists(request)) {
        return ResponseEntity.ok((T) ErrorDTO.from(bundle.getString("point.not.set")));
    }
    if (!isRequestFunctionCorrect(request, "CheckAcc")) {
        return ResponseEntity.ok((T) response.wrongCheck(1, 1));
    }
    if (!isServiceExists(request)) {
        return ResponseEntity.ok((T)  response.wrongCheck(1, 4);
    }
    // do the logic if everything is correct
}
private boolean isAgentExists(Request request) {
    return agentRepository.findAgentByRequestPoint(request.getPoint()).isPresent();
}
private boolean isRequestFunctionCorrect(Request request, String func) {
    if (request.getAdvanced() != null) {
        return func.equals(request.getAdvanced().getFunction());
    }
    return false;
}
private boolean isServiceExists(Request request) {
    Long serviceId = Long.parseLong(request.getAdvanced().getService());
    return ServiceRepository.findServiceByServiceId(serviceName).isPresent()
}

在某些情况下，我添加了额外的空值检查以避免NPE。
希望对你有帮助：）