Skip to content

Commit 13f5144

Browse files
committed
feat: 通用上传文件接口返回值改为保存的文件名
1 parent 9a514ca commit 13f5144

2 files changed

Lines changed: 22 additions & 15 deletions

File tree

framework/framework_svc/src/main/java/com/github/cadecode/uniboot/framework/svc/controller/CommonController.java

Lines changed: 14 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,9 @@
2121
import javax.servlet.http.HttpServletResponse;
2222
import java.io.File;
2323
import java.io.IOException;
24+
import java.util.Arrays;
25+
import java.util.List;
26+
import java.util.stream.Collectors;
2427

2528
/**
2629
* 通用API
@@ -37,51 +40,47 @@ public class CommonController {
3740

3841
@ApiOperation("上传文件")
3942
@PostMapping("common/upload")
40-
public boolean upload(@RequestPart("file") MultipartFile file) {
43+
public String upload(@RequestPart("file") MultipartFile file) {
4144
String originalFilename = file.getOriginalFilename();
4245
boolean checked = FileUploadUtil.checkAllowedExtension(originalFilename);
4346
AssertUtil.isFalse(checked, FrameErrorEnum.EXTENSION_NOT_ALLOWED, originalFilename + "不被允许");
4447
String renamedFileName = FileUploadUtil.renameByUUID(originalFilename);
45-
String targetFilePath = FileUploadUtil.uploadPath() + renamedFileName;
48+
String targetFilePath = FileUploadUtil.uploadPath(renamedFileName);
4649
try {
4750
File targetFile = FileUtil.file(targetFilePath);
4851
FileUtil.mkParentDirs(targetFile);
4952
file.transferTo(targetFile);
5053
} catch (IOException e) {
5154
throw ApiException.of(FrameErrorEnum.UPLOAD_FILE_FAIL, e, originalFilename + "上传失败");
5255
}
53-
return true;
56+
return renamedFileName;
5457
}
5558

5659
@ApiOperation("上传多文件")
5760
@PostMapping("common/upload_files")
58-
public int uploadFiles(@RequestPart("files") MultipartFile[] files) {
59-
int res = 0;
60-
for (MultipartFile file : files) {
61-
upload(file);
62-
res++;
63-
}
64-
return res;
61+
public List<String> uploadFiles(@RequestPart("files") MultipartFile[] files) {
62+
return Arrays.stream(files)
63+
.map(this::upload)
64+
.collect(Collectors.toList());
6565
}
6666

6767
@ApiOperation("下载文件")
6868
@GetMapping(FileUploadUtil.DEFAULT_DOWNLOAD_API)
69-
public void download(HttpServletRequest request, HttpServletResponse response, String fileName) throws IOException {
69+
public void download(HttpServletRequest request, HttpServletResponse response, String fileName) {
7070
boolean checked = FileUploadUtil.checkAllowedExtension(fileName);
7171
AssertUtil.isFalse(checked, FrameErrorEnum.EXTENSION_NOT_ALLOWED, fileName + "不被允许");
7272
// 写文件流
73-
String targetFilePath = FileUploadUtil.downloadPath() + fileName;
73+
String targetFilePath = FileUploadUtil.downloadPath(fileName);
7474
boolean exist = FileUtil.exist(targetFilePath);
7575
AssertUtil.isFalse(exist, FrameErrorEnum.FILE_NOT_FOUND, fileName + "不存在");
7676
ServletUtil.write(response, new File(targetFilePath));
7777
}
7878

7979
@ApiOperation("下载临时文件")
8080
@GetMapping(FileUploadUtil.DEFAULT_DOWNLOAD_TEMP_API)
81-
public void downloadTemp(HttpServletRequest request, HttpServletResponse response, String fileName) throws IOException {
81+
public void downloadTemp(HttpServletRequest request, HttpServletResponse response, String fileName) {
8282
download(request, response, fileName);
83-
String targetFilePath = FileUploadUtil.downloadPath() + fileName;
83+
String targetFilePath = FileUploadUtil.downloadPath(fileName);
8484
FileUtil.del(targetFilePath);
8585
}
86-
8786
}

framework/framework_svc/src/main/java/com/github/cadecode/uniboot/framework/svc/util/FileUploadUtil.java

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,14 @@ public static String downloadPath() {
5151
return DOWNLOAD_PATH;
5252
}
5353

54+
public static String uploadPath(String fileName) {
55+
return UPLOAD_PATH + fileName;
56+
}
57+
58+
public static String downloadPath(String fileName) {
59+
return DOWNLOAD_PATH + fileName;
60+
}
61+
5462
public static String downloadUrl(String fileName) {
5563
return DEFAULT_DOWNLOAD_API + "?fileName=" + fileName;
5664
}

0 commit comments

Comments
 (0)