Java用for循环配合JFileChooser批量存文件时重复存储、长度失效问题

核心故障根因 #

• 循环长度取值时机错误
  代码中int writingLength = fc.getSelectedFiles().length的执行时机早于保存对话框弹出操作，此时用户尚未进行任何文件选择，JFileChooser内部的选中文件数组为空，读取到的长度为0，循环根本不会触发，因此完全看不到保存界面。
• 共享成员变量的异步竞态
  代码中使用类级成员变量outputFilename向后台导出任务传值，而ReportDataExtracter属于Swing异步任务（SwingWorker），循环的执行速度远快于后台任务的IO速度：第一次循环刚启动导出任务还未读取路径值，第二次循环就已经将outputFilename覆盖为新的路径，最终两个后台任务读取的都是最后一次赋值的路径，直接导致最后一个文件重复保存两次。
• JFileChooser的状态缓存问题
  循环内每次弹出保存界面前未清空上一次的选中缓存，第一次用户选择完文件后，JFileChooser会自动复用该选中路径作为第二次弹出的默认值，进一步加重重复保存的问题。
• 批量保存的流程逻辑倒置
  当前逻辑是循环N次每次弹出一次保存框，既不符合批量保存的交互预期，也容易因用户操作、状态缓存引发重复路径问题，正常批量保存的流程应当是一次性获取所有待保存的路径列表后，再批量执行导出操作。

可落地的排查&修正步骤 #

• 调整流程顺序：先调用fc.showSaveDialog()等待用户完成选择交互，拿到合法返回值后再读取选中文件的数组长度作为循环次数，不要在弹窗前提前读取选中值。
• 替换变量作用域：将循环内使用的outputFilename从类级成员变量改为方法内的局部变量，创建ExcelOutput实例时直接传入局部变量的引用值，切断异步任务间的变量共享通道，避免值被覆盖。
• 补充状态重置：每次弹出保存界面前，调用fc.setSelectedFile(null)清空上一次的选中缓存，避免复用旧路径。
• 理顺异步执行时序：不要在循环中连续触发多个后台导出任务，否则会反复覆盖全局的progressMonitor实例，导致进度条混乱、并发IO冲突，建议加入任务排队逻辑，等上一个导出任务完成后再触发下一个任务。
• 修正配置不一致问题：当前代码中FileNameExtensionFilter匹配的是.csv后缀，但自动生成的建议文件名是.xlsx后缀，后缀不匹配会导致JFileChooser强制修改文件名、出现格式不识别的问题，需统一文件后缀配置。

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