我有以下方法,如果数据为非空或非空,则会更新用户数据
public Users updateUser(Users requestBody) throws AppServiceException {
Users user = new Users();
try {
user = userDAO.getUserByUserId(requestBody.getUserId());
if (requestBody.getRole() != null && !requestBody.getRole().isEmpty()) {
user.setRole(requestBody.getRole());
}
if (requestBody.getUserName() != null && !requestBody.getUserName().isEmpty()) {
user.setUserName(requestBody.getUserName());
}
if (requestBody.getChannelType() != null && !requestBody.getChannelType().isEmpty()) {
user.setChannelType(requestBody.getChannelType());
}
if (requestBody.getStatus() != null && !requestBody.getStatus().isEmpty()) {
user.setStatus(requestBody.getStatus());
}
if (requestBody.getDevice() != null) {
user.setDevice(requestBody.getDevice());
}
user.setUpdatedDate(new Date());
user = userDAO.updateUser(user);
} catch (Exception e) {
e.printStackTrace();
throw new AppServiceException(AppServiceException._FAIL_TO_UPDATE);
}
return user;
}
我每次都检查nonNull和isEmpty的值。
如何避免这种情况?
答案 0 :(得分:1)
您可以使用Apache Commons lang的StringUtils.isEmpty(java.lang.String)
检查字符串是否为空(“”)或null。
如果您的代码是
if (StringUtils.isEmpty(requestBody.getStatus())) {
答案 1 :(得分:0)
假设我们有两个类:
@Data
class X {
private String field1;
}
@Data
class Y {
private String field2;
}
定义静态方法
static <F, T>void copy(F f, T t, Function<F, String> get, BiConsumer<T, String> set){
String value = get.apply(f);
if(value != null && value.isEmpty()){
set.accept(t, value);
}
}
并在您的代码中使用它:
X x = new X();
Y y = new Y();
copy(x, y, X::getField1, Y::setField2);
答案 2 :(得分:0)
您可以在getter方法内实现此检查逻辑,即
public Optional<String> getChannelType() {
if (channelType.isEmpty() || channelType == null)
return Optional.empty();
else
return Optional.of(channelType);
}
答案 3 :(得分:0)
如果只是在两个具有相同变量名的bean之间进行复制,则可以使用BeanUtils.copyProperties
进行复制。
在这里我们可以说BeanUtil复制非空值。因此代码将减少为1行。
BeanUtils.copyProperties(source, destination, (.. optional parameter to copy non null value)