基于分数重排文件的Java代码报错排查与优化咨询

错误原因分析 #

• 数组索引越界：CSV中存在格式错误的行（比如空行、只有用户名没有分数、分隔符缺失），分割后数组长度不足2，代码直接访问索引1导致报错。
• 无法解析null字符串：读取到空字段或null行，尝试将其转为数字类型时抛出异常。

针对性解决步骤 #

1. 过滤无效行：读取每行时先判断是否为空或空白，直接跳过；分割后检查数组长度是否≥2，否则跳过该行。
2. 处理空字段：对分割后的字段做非空校验，若分数字段为空或无法转为数字，跳过该行。
3. 安全解析数字：解析分数时捕获NumberFormatException，避免因格式错误导致程序崩溃。

整体代码优化方案 #

用自定义对象存储玩家信息，结合try-with-resources自动管理IO资源，使用Comparator实现排序，同时加入健壮性校验：

import java.io.*;
import java.util.*;

// 自定义玩家分数对象，提升代码可读性
class PlayerScore {
    private String username;
    private int score;

    public PlayerScore(String username, int score) {
        this.username = username;
        this.score = score;
    }

    public String getUsername() {
        return username;
    }

    public int getScore() {
        return score;
    }

    // 转为CSV行格式
    @Override
    public String toString() {
        return username + "," + score;
    }
}

public class HighScoreManager {
    // 提取常量，方便后续维护
    private static final String CSV_PATH = "highscores.csv";
    private static final int MAX_TOP_SCORES = 5;
    private static final String SEPARATOR = ",";

    public static void main(String[] args) {
        List<PlayerScore> scores = loadScores();
        // 添加新获胜者（可替换为动态输入逻辑）
        scores.add(new PlayerScore("NewWinner", 950));
        // 按分数降序排序，分数相同则按用户名升序
        scores.sort((s1, s2) -> {
            int scoreCompare = Integer.compare(s2.getScore(), s1.getScore());
            return scoreCompare != 0 ? scoreCompare : s1.getUsername().compareTo(s2.getUsername());
        });
        // 保留前5名高分
        if (scores.size() > MAX_TOP_SCORES) {
            scores = scores.subList(0, MAX_TOP_SCORES);
        }
        saveScores(scores);
    }

    private static List<PlayerScore> loadScores() {
        List<PlayerScore> scores = new ArrayList<>();
        // try-with-resources自动关闭IO流，避免资源泄漏
        try (BufferedReader reader = new BufferedReader(new FileReader(CSV_PATH))) {
            String line;
            while ((line = reader.readLine()) != null) {
                line = line.trim();
                // 跳过空行
                if (line.isEmpty()) {
                    continue;
                }
                String[] parts = line.split(SEPARATOR);
                // 校验行格式合法性
                if (parts.length < 2) {
                    System.err.println("跳过无效行：" + line);
                    continue;
                }
                String username = parts[0].trim();
                String scoreStr = parts[1].trim();
                // 校验字段非空
                if (username.isEmpty() || scoreStr.isEmpty()) {
                    System.err.println("跳过字段为空的行：" + line);
                    continue;
                }
                try {
                    int score = Integer.parseInt(scoreStr);
                    scores.add(new PlayerScore(username, score));
                } catch (NumberFormatException e) {
                    System.err.println("跳过分数格式错误的行：" + line);
                }
            }
        } catch (FileNotFoundException e) {
            // 文件不存在时创建空列表，后续写入会自动生成文件
            System.out.println("高分文件不存在，将创建新文件");
        } catch (IOException e) {
            e.printStackTrace();
        }
        return scores;
    }

    private static void saveScores(List<PlayerScore> scores) {
        try (BufferedWriter writer = new BufferedWriter(new FileWriter(CSV_PATH))) {
            for (PlayerScore score : scores) {
                writer.write(score.toString());
                writer.newLine();
            }
            System.out.println("高分记录已更新，保留前" + MAX_TOP_SCORES + "名");
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

优化点说明 #

• 自定义对象：用PlayerScore替代字符串数组，代码可读性和可维护性大幅提升。
• 自动资源管理：try-with-resources确保IO流自动关闭，避免资源泄漏。
• 健壮性校验：多层过滤无效行、空字段、格式错误，彻底解决空指针和数组越界问题。
• 清晰的排序逻辑：使用Lambda表达式实现Comparator，排序规则一目了然。
• 常量提取：将文件路径、分隔符、最大保留数设为常量，便于后续修改。
• 错误提示：对无效行输出明确的错误信息，方便排查问题。

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