
本文详解为何使用二维字符串数组会导致只打印最后一组数据,并提供更合理、简洁的解决方案:改用 song 对象数组或直接逐对解析文件内容,确保每首作品标题与艺术家正确配对并完整输出。
本文详解为何使用二维字符串数组会导致只打印最后一组数据,并提供更合理、简洁的解决方案:改用 song 对象数组或直接逐对解析文件内容,确保每首作品标题与艺术家正确配对并完整输出。
问题根源在于原始代码中对二维数组 cd[6][6] 的错误索引逻辑与变量复用。具体来看:
column 被反复重置为 0,且在每次循环中先赋值 title 到 cd[i][0],再自增 column++ 后赋值 artist 到 cd[i][1],但随后又将 column = 0 —— 这看似合理,实则掩盖了更严重的问题:title 和 artist 变量在循环末尾被持续覆盖,最终所有 Song 构造均使用最后一次读取的值。
更关键的是,for (int i = 0; i 第二个 for 循环中统一创建——此时 title 和 artist 已是文件末尾的最后两行(即 "The Blue Danube Waltz" 和 "Strauss"),导致 6 次输出完全相同。
✅ 正确做法是:避免复用外部变量,立即构建并存储 Song 实例。推荐以下两种现代、健壮的实现方式(基于 Java 7+):
方案一:使用 Files.readAllLines() + Song[] 数组(清晰可维护)
import java.io.*;
import java.nio.charset.Charset;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.util.List;
public class CompactDisc {
public static void main(String[] args) throws IOException {
// 一次性读取全部行(自动关闭资源,无需显式 close)
List<String> lines = Files.readAllLines(Paths.get("Classics.txt"), Charset.forName("UTF-8"));
// 每两行为一首歌,计算容量
Song[] cd = new Song[lines.size() / 2];
int songIndex = 0;
// 成对解析:第0/1行、2/3行……
for (int i = 0; i < lines.size(); i += 2) {
String title = lines.get(i);
String artist = lines.get(i + 1);
cd[songIndex++] = new Song(title, artist);
}
System.out.println("Contents of Classics:");
for (Song track : cd) {
System.out.print(track); // 自动调用 toString()
}
}
}
class Song {
private final String title;
private final String artist;
public Song(String title, String artist) {
this.title = title;
this.artist = artist;
}
@Override
public String toString() {
return title + " by " + artist + "\n";
}
}方案二:零内存分配,流式输出(最简高效)
// 替换 main 方法内核部分即可:
List<String> lines = Files.readAllLines(Paths.get("Classics.txt"), Charset.forName("UTF-8"));
System.out.println("Contents of Classics:");
for (int i = 0; i < lines.size(); i += 2) {
System.out.println(new Song(lines.get(i), lines.get(i + 1)));
}⚠️ 注意事项:
- 原始代码中 Song 继承自 CompactDisc 是设计错误(Song 不是 CompactDisc 的子类型),应改为独立类(如上所示),避免不必要的继承和 static 上下文冲突;
- FileReader/BufferedReader 易受编码影响,Files.readAllLines() 默认 UTF-8 更可靠;
- 确保 Classics.txt 行数为偶数,否则 i + 1 会抛出 IndexOutOfBoundsException;生产环境建议添加校验;
- 若必须保留二维数组结构(如教学限制),需严格按 cd[i][0] = title; cd[i][1] = artist; 并在同一循环内立即使用,而非延迟到后续循环。
通过以上重构,程序将准确输出全部六首作品,彻底解决“仅显示最后一项”的典型状态覆盖陷阱。


















