如何简化按8行拆分文件并存入ArrayList<String>的Java代码？

嘿，你的代码确实能完成需求，但存在冗余（比如先把所有行存到另一个ArrayList再处理）和计数逻辑复杂的问题。咱们可以从减少内存占用、简化逻辑、遵循最佳实践这几个方向来优化，让代码更轻量易读。

核心优化思路 #

• 移除全量行存储：原代码先把所有行读入fileSplitted再处理，大文件会浪费内存，改成边读边分组
• 简化计数逻辑：只用一个计数器追踪当前组的行数，去掉容易混淆的countTotal
• 自动资源管理：用try-with-resources确保Scanner自动关闭，避免资源泄漏
• 明确处理换行符：和你的预期输出对齐，拼接时保留换行

优化后的代码 #

import java.io.File;
import java.io.FileNotFoundException;
import java.util.ArrayList;
import java.util.Scanner;

public class FileSplitter {
    // 可选：把每组行数做成常量，后续修改只需改一处
    private static final int LINES_PER_GROUP = 8;

    public static ArrayList<String> splitFile(File file) throws FileNotFoundException {
        ArrayList<String> splittedGroups = new ArrayList<>();
        StringBuilder currentGroup = new StringBuilder();
        int currentLineCount = 0;

        // try-with-resources自动关闭Scanner，无需手动调用close()
        try (Scanner scanner = new Scanner(file)) {
            while (scanner.hasNextLine()) {
                String line = scanner.nextLine();
                // 追加当前行并加上换行符，匹配预期输出格式
                currentGroup.append(line).append("\n");
                currentLineCount++;

                // 满8行就存入结果，重置当前组
                if (currentLineCount == LINES_PER_GROUP) {
                    splittedGroups.add(currentGroup.toString());
                    currentGroup.setLength(0);
                    currentLineCount = 0;
                }
            }

            // 处理最后一组不足8行的剩余内容
            if (currentGroup.length() > 0) {
                splittedGroups.add(currentGroup.toString());
            }
        }

        return splittedGroups;
    }
}

优化细节说明 #

• 内存优化：边读边分组，不需要把整个文件的内容都存在内存里，处理大文件时优势明显
• 逻辑简化：只用currentLineCount追踪当前组的行数，条件判断清晰，没有嵌套复杂的else-if
• 资源安全：try-with-resources是Java 7+的特性，会自动关闭实现了AutoCloseable的资源（比如Scanner），避免忘记关闭导致的资源泄漏
• 格式对齐：手动添加\n，确保分组后的字符串和你预期的格式一致（原代码拼接时没有换行，会导致所有行连在一起）
• 可扩展性：把LINES_PER_GROUP做成常量或参数，以后要改每组行数只需要改一处

可选进阶优化 #

• 用List<String>代替ArrayList<String>作为返回类型，遵循面向接口编程的原则，更灵活：

  public static List<String> splitFile(File file) throws FileNotFoundException {
      List<String> splittedGroups = new ArrayList<>();
      // ... 剩余逻辑不变
  }
  

• 如果不需要最后一行的换行符，可以用currentGroup.toString().trim()来移除末尾的换行

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