优化这段代码List<MultipartFile> files = Lists.newArrayList(file); if(fileService.checkForbidExtension(forbidExtension,files)){ return ResultBody.failed().msg(FORBID_MSG); } String status = ExportCenterConstant.EXPORT_STATUS_1; String[] resultArray; try { resultArray = fdfsService.upload(file); } catch (Exception e){ log.error("导出中心上传文件到服务器失败:{}", e.getMessage(), e); resultArray = new String[2]; status = ExportCenterConstant.EXPORT_STATUS_3; }
时间: 2024-02-14 22:09:08 浏览: 77
使用springmvc 实现文件上传
这段代码可以进行如下优化:
```java
List<MultipartFile> files = Lists.newArrayList(file);
if (fileService.checkForbidExtension(forbidExtension, files)) {
return ResultBody.failed().msg(FORBID_MSG);
}
String[] resultArray;
String status = ExportCenterConstant.EXPORT_STATUS_1;
try {
resultArray = fdfsService.upload(file);
} catch (Exception e) {
log.error("导出中心上传文件到服务器失败:{}", e.getMessage(), e);
resultArray = new String[2];
status = ExportCenterConstant.EXPORT_STATUS_3;
}
// 可以将逻辑处理和返回结果分离出来,提高代码可读性
if (status.equals(ExportCenterConstant.EXPORT_STATUS_1)) {
return ResultBody.success().data(resultArray);
} else {
return ResultBody.failed().msg(ERROR_MSG);
}
```
优化点:
1. 将 `List<MultipartFile> files = Lists.newArrayList(file);` 改为直接创建集合对象 `List<MultipartFile> files = new ArrayList<>();`,这样可以避免创建不必要的集合对象。
2. 将逻辑处理和返回结果分离出来,可以提高代码的可读性,使代码更加清晰。
阅读全文