//统一查看 访问该方法需要两个条件 userId 和 userFlag public List<CommonUse> view(CommonUse commonUse){ String userFlag = commonUse.getUserFlag(); if("1".equals(userFlag)){ return planterDao.view(commonUse); }else if("2".equals(userFlag)){ return processDao.view(commonUse); }else if("3".equals(userFlag)){ return logisticsDao.view(commonUse); }else if ("4".equals(userFlag)){ return saleDao.view(commonUse); } return null; }
时间: 2023-06-27 19:08:15 浏览: 68
这段代码是一个公共方法,需要传入一个CommonUse类型的参数,并且该参数中包含了两个必要的属性userId和userFlag。根据userFlag的不同取值,分别调用planterDao、processDao、logisticsDao和saleDao中的view方法,并返回对应的结果列表。如果userFlag的值不属于这四种情况,则返回null。
相关问题
优化这段代码//统一数据删除 public int delete(CommonUse commonUse){ String userFlag = commonUse.getUserFlag(); if("1".equals(userFlag)){ return planterDao.delete(commonUse); } return -1; } //通过key查询所有的信息 public CommonUse getAllInfo(String key,String userFlag){ CommonUse commonUse = new CommonUse(); commonUse.setPlanterOnlyKey(key); CommonUse commonUse1 = commonUseDao.searchOneByOnlyKey(commonUse); commonUse1.setUserFlag(userFlag); return commonUse1; }
为了优化这段代码,可以考虑以下两点:
1. 增加参数校验
在第二个方法 `getAllInfo` 中,应该增加对传入参数的校验,比如判断 `key` 是否为空,如果为空则直接返回 null。这样可以避免出现空指针异常等问题。
2. 将返回值改为 Optional
在第二个方法 `getAllInfo` 中,可以将返回值改为 `Optional<CommonUse>` 类型,这样可以避免返回 null 值带来的风险,同时也能更好地处理查询结果为空的情况。
下面是优化后的代码:
```
// 统一数据删除
public int delete(CommonUse commonUse) {
String userFlag = commonUse.getUserFlag();
if ("1".equals(userFlag)) {
return planterDao.delete(commonUse);
}
return -1;
}
// 通过key查询所有的信息
public Optional<CommonUse> getAllInfo(String key,String userFlag) {
if (StringUtils.isBlank(key)) {
return Optional.empty();
}
CommonUse commonUse = new CommonUse();
commonUse.setPlanterOnlyKey(key);
CommonUse commonUse1 = commonUseDao.searchOneByOnlyKey(commonUse);
if (commonUse1 != null) {
commonUse1.setUserFlag(userFlag);
return Optional.of(commonUse1);
}
return Optional.empty();
}
```
优化后的代码增加了参数校验和返回值改为 `Optional`,可以更加健壮和安全地执行相关操作。
@RequestMapping("/view") public ReturnCodeUtil view(@RequestBody CommonUse commonUse) { ReturnCodeUtil returnCodeUtil = new ReturnCodeUtil(); if(String.valueOf(commonUse.getUserId()).equals("0") || "".equals(commonUse.getUserFlag()) || commonUse.getUserFlag() == null ){ returnCodeUtil.setResult(false); return returnCodeUtil; } List<CommonUse> view = commonUseService.view(commonUse); returnCodeUtil.setResult(true); returnCodeUtil.setData(view); return returnCodeUtil; }
这段代码看起来像是一个Spring Boot框架中的Controller类中的一个方法。
@RequestMapping("/view") 表示这个方法处理以/view结尾的HTTP请求。
@RequestBody CommonUse commonUse 表示从请求体中获取一个CommonUse类型的对象,并将其赋值给方法中的commonUse参数。
ReturnCodeUtil是一个自定义的返回类,用于封装处理结果和返回数据。
方法中的逻辑是先判断commonUse中的userId和userFlag属性是否为空,如果其中有一个为空,则返回结果为false。
如果都不为空,则调用commonUseService的view方法获取数据,并将数据封装到ReturnCodeUtil对象中,最后返回该对象。
需要注意的是,这段代码中返回结果的格式是JSON格式。
阅读全文