使用Rest Controller的MultipartFile上传图片时出现文件损坏问题

看起来你遇到的问题确实和并发场景下的文件写入竞态有关，我帮你分析下可能的原因和解决方案：

核心问题分析 #

你代码里的uploadFile方法存在竞态条件：当多个请求同时上传时，可能出现两个线程都通过Files.exists(path)的检查，然后同时写入同一个文件路径，导致文件内容被混合覆盖，最终出现乱码损坏。另外，如果你的generateRandomString方法不是线程安全的（比如用了new Random()而不是ThreadLocalRandom），还可能生成重复的文件名，加剧这个问题。

具体解决方案 #

1. 使用UUID生成绝对唯一的文件名 #

放弃自定义的随机字符串生成，直接用UUID.randomUUID().toString()来生成唯一文件名，这样可以从根源上避免文件名冲突，不需要再做文件存在性检查：

private String uploadFile(MultipartFile file) {
    if (file.isEmpty()) {
        return "";
    }
    final String UPLOADED_FOLDER = "photos";
    // 用UUID生成唯一文件名，避免冲突
    String uniqueName = UUID.randomUUID().toString();
    // 保留原文件后缀
    String fileExtension = "";
    int dotIndex = file.getOriginalFilename().lastIndexOf(".");
    if (dotIndex != -1) {
        fileExtension = file.getOriginalFilename().substring(dotIndex);
    }
    String filePath = UPLOADED_FOLDER + "/" + uniqueName + fileExtension;
    new File(UPLOADED_FOLDER).mkdirs();
    try (InputStream inputStream = file.getInputStream()) {
        Files.copy(inputStream, Paths.get(filePath), StandardCopyOption.REPLACE_EXISTING);
    } catch (IOException e) {
        e.printStackTrace();
        return "";
    }
    return filePath;
}

2. 消除竞态条件（备选方案） #

如果一定要保留自定义文件名生成，需要用原子操作来避免检查-创建的竞态。可以用Files.createFile，它会在文件已存在时抛出异常，我们可以捕获异常后重新生成文件名：

private String uploadFile(MultipartFile file) {
    if (file.isEmpty()) {
        return "";
    }
    final String UPLOADED_FOLDER = "photos";
    new File(UPLOADED_FOLDER).mkdirs();
    Path folderPath = Paths.get(UPLOADED_FOLDER);
    String originalFilename = file.getOriginalFilename();
    Path filePath;
    do {
        String uniqueName = generateRandomString(); // 确保这个方法线程安全
        filePath = folderPath.resolve(uniqueName + originalFilename);
    } while (Files.exists(filePath)); // 循环直到找到不存在的路径
    try (InputStream inputStream = file.getInputStream()) {
        Files.copy(inputStream, filePath);
    } catch (IOException e) {
        e.printStackTrace();
        return "";
    }
    return filePath.toString();
}

同时，修改generateRandomString为线程安全的实现：

private String generateRandomString() {
    // 用ThreadLocalRandom替代Random，线程安全
    int length = 16;
    StringBuilder sb = new StringBuilder(length);
    String chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789";
    for (int i = 0; i < length; i++) {
        int index = ThreadLocalRandom.current().nextInt(chars.length());
        sb.append(chars.charAt(index));
    }
    return sb.toString();
}

3. 检查Multipart配置（可选） #

虽然不是直接原因，但确保你的Spring Boot Multipart配置足够处理并发上传，在application.properties里添加：

spring.servlet.multipart.max-file-size=10MB
spring.servlet.multipart.max-request-size=100MB
spring.servlet.multipart.enabled=true

为什么之前的InputStream方式没用？ #

因为问题根本不是读取MultipartFile的方式，而是多个线程同时写入同一个文件路径导致的内容重叠。只要文件名冲突的问题解决，不管用getBytes还是InputStream都能正常工作。

内容的提问来源于stack exchange，提问作者Moler