以下代码是大型项目的一部分。
概述是我正在尝试使用Spring MVC访问数据库。我想根据请求更新字段,并发送有关数据库发回的值的响应。
代码:
@Override
@Transactional
public EmployeeResponse update(EmployeeRequest employeeRequest) {
Employee emp = new Employee();
UUID empId = UUID.fromString(employeeRequest.getId());
Employee foundEmployee = employeeRepository.findOne(empId);
if (foundEmployee != null) {
foundEmployee.setAddress(employeeRequest.getAddress());
// similarly set 4 fields of foundEmployee
emp = employeeRepository.save(foundEmployee);
}
EmployeeResponse response = new EmployeeResponse();
response.setAddress(emp.getAddress());
// similarly set 4 fields of response
return response;
}
我发现new Employee()
没有foundEmployee
,因为有emp
。
我不确定,但我认为这会导致异常。
我是对的吗?
另外,请告诉我foundEmployee
为null
时应该抛出的异常。
其他信息 - 这是帮助显示的内容:
org.springframework.data.repository.CrudRepository
public T findOne(ID id)
Retrieves an entity by its id.
Parameters:
id - must not be null.
Returns:
the entity with the given id or null if none found
Throws:
IllegalArgumentException - if id is null
答案 0 :(得分:8)
在第
行Employee foundEmployee = employeeRepository.findOne(empId);
我们可以假设EmployeeRepository.findOne()
将返回Employee
的实例。这不会导致编译器错误,如果在运行时发生异常,它将在findOne()
内。
关于在null
foundEmployee
的情况下您应该做些什么,这实际上是您必须做出的设计决定。一种选择是让方法返回null
让消费者知道传入的EmployeeRequest
存在严重问题。
另一种选择是创建自己的Exception
,然后将其投放到null
foundEmployee
的情况下。
<强>更新强>
鉴于您需要将某些内容传递回UI,另一种选择是创建一个空的EmployeeReponse
对象并返回该对象:
EmployeeResponse response = new EmployeeResponse();
response.setAddress(null);
response.setName(null);
确保您的框架可以将null
值编组为用户友好的内容,例如所有字段的空字符串。