@GetMapping("/details") public ResponseInfo<String> Details(@RequestParam String id){ List<eventCamerVo> eventVos = null; boolean bFlag = false; eventCamerVo eventCameraVO = JSONObject.parseObject(id, eventCamerVo.class); Event event = eventService.getById(eventCameraVO.getId()); if (1 == event.getEvent_type()){ eventVos = eventMapper.getAll(id); } if (3 == event.getEvent_type()){ eventVos = slopeWarMapper.getAllSlope(id); } return ResponseInfo.success(eventVos.toString()); }从这段代码可以看出前端传来一个id,我拿到id去查event表,如果查到这条数据的type==1就调用这个方法,List<eventCamerVo> getAll(@Param("id") String id); 如果拿到的type==3就调用这个方法List<eventCamerVo> getAllSlope(@Param("id") String id);,请根据需求指出代码的错误并修改
时间: 2023-06-18 18:04:00 浏览: 83
从代码中看不出具体的错误,但是需要对代码进行优化和修改,如下所示:
1. 对于前端传来的id参数,需要进行非空判断,如果为null或者空字符串,需要返回相应的错误提示。
2. 对于从数据库中查询到的Event对象,需要进行非空判断,如果为null,需要返回相应的错误提示。
3. 对于event.getEvent_type()的返回值,需要进行非空判断,如果为null,需要返回相应的错误提示。
4. 根据需求,如果拿到的type不等于1或者3,需要返回相应的错误提示。
5. 对于getAll和getAllSlope方法的调用,需要进行非空判断,如果返回null或者空列表,需要返回相应的错误提示。
修改后的代码如下所示:
@GetMapping("/details")
public ResponseInfo<String> Details(@RequestParam String id) {
if (id == null || id.isEmpty()) {
return ResponseInfo.fail("参数id不能为空");
}
eventCamerVo eventCameraVO = JSONObject.parseObject(id, eventCamerVo.class);
Event event = eventService.getById(eventCameraVO.getId());
if (event == null) {
return ResponseInfo.fail("查询不到对应的事件信息");
}
Integer eventType = event.getEvent_type();
if (eventType == null) {
return ResponseInfo.fail("查询到的事件类型为空");
}
List<eventCamerVo> eventVos;
if (eventType == 1) {
eventVos = eventMapper.getAll(id);
} else if (eventType == 3) {
eventVos = slopeWarMapper.getAllSlope(id);
} else {
return ResponseInfo.fail("不支持的事件类型");
}
if (eventVos == null || eventVos.isEmpty()) {
return ResponseInfo.fail("未查询到相关数据");
}
return ResponseInfo.success(eventVos.toString());
}
阅读全文