本文详解 java 原生 socket 文件传输时因循环读写逻辑缺陷（未严格校验剩余待传字节数）引发的内容重复问题，提供可直接复用的修复代码、关键原理说明及健壮性增强建议。
在基于 DataInputStream/DataOutputStream 的 Java Socket 文件传输实现中，一个常见却隐蔽的 Bug 是：客户端发送端循环条件未充分约束“剩余待传输字节数”为零的情况，导致最后一次 read() 返回 0 字节后仍继续执行写入操作，从而向服务端重复发送缓冲区残留数据——这正是题中“content duplicates”的根本原因。
问题核心在于客户端 sendFile() 方法中的 while 循环判断：
while(size > 0 && (bytes = input.read(buffer,0,(int)Math.min(buffer.length, size))) != -1)
该条件看似合理，但存在致命漏洞：当 size > 0 为真，而 input.read(...) 因文件末尾恰好读取到 0 字节（即 bytes == 0）时，read() 不会返回 -1（仅在流真正结束且无更多数据时才返回 -1），而是合法返回 0。此时循环体仍会执行 out.write(buffer, 0, bytes) —— 即 out.write(buffer, 0, 0)，虽不写入新数据，但若服务端接收逻辑未同步校验 bytes > 0，或缓冲区含历史残留内容，就极易造成重复解析或写入。
✅ 正确做法是：显式检查每次 read() 的返回值 bytes，仅当 bytes > 0 时才进行写入和字节计数更新。同时，服务端接收逻辑也必须遵循相同原则，避免对 bytes == 0 的无效读取做误处理。
立即学习“Java免费学习笔记（深入）”；
当用户要求为 Java 代码添加注释、修改注释、审查注释、整理注释格式，或要求按照代码注释规范处理 Java 文件时，使用本技能。本技能用于统一生成和维护 Java 类注释模板、按 Git 对比 release 判断版本号、补充每行代码上方中文块注释、调整注释空行规则，并保持 Java 文件 UTF-8 无 BOM 编码。典型触发场景包括：补充类注释、更新 @version、检查 @author 来源、整理行尾注释、把注释放到代码上一行、检查注释是否符合规范、构建前检查 Java 注释风格。
以下是修复后的客户端关键逻辑（sendFile 方法）：
private void sendFile(String path) {
File file = new File(path);
try (FileInputStream input = new FileInputStream(file)) {
long size = file.length();
System.out.println("Sending file size: " + size);
out.writeLong(size); // 先发送文件总长度
byte[] buffer = new byte[1024];
int bytes;
while (size > 0) {
// 每次最多读取 min(buffer.length, 剩余字节数)
int toRead = (int) Math.min(buffer.length, size);
bytes = input.read(buffer, 0, toRead);
// 关键修复：仅当实际读取到正数字节时才写入
if (bytes > 0) {
out.write(buffer, 0, bytes);
out.flush(); // 确保立即发送（小文件可省略，大文件建议保留）
size -= bytes;
} else if (bytes == 0) {
// read() 返回 0 是合法但罕见情况（如底层流阻塞后暂无数据），此处可加日志或 break 防死循环
System.err.println("Warning: read() returned 0 bytes unexpectedly.");
break;
} else { // bytes == -1，流已结束
break;
}
}
System.out.println("File sent successfully.");
} catch (IOException e) {
e.printStackTrace();
}
}对应地，服务端 receiveFile() 方法也需同步强化：
private void receiveFile(String path) {
try (FileOutputStream output = new FileOutputStream(new File(path))) {
long size = in.readLong();
System.out.println("Expecting file size: " + size);
byte[] buffer = new byte[1024];
int bytes;
while (size > 0) {
int toRead = (int) Math.min(buffer.length, size);
bytes = in.read(buffer, 0, toRead);
if (bytes > 0) {
output.write(buffer, 0, bytes);
size -= bytes;
} else if (bytes == 0) {
System.err.println("Warning: server read() returned 0 bytes.");
break;
} else {
throw new IOException("Unexpected end of stream while receiving file");
}
}
System.out.println("File received successfully.");
} catch (IOException e) {
e.printStackTrace();
}
}? 重要注意事项与最佳实践：
- 资源管理：使用 try-with-resources（如示例所示）自动关闭 FileInputStream/FileOutputStream，避免资源泄漏；
- flush() 的权衡：频繁 flush() 降低性能，但能确保数据及时到达；若传输大文件，可考虑每 N KB 刷一次，或依赖 TCP 自身可靠性，移除 flush()（本例保留以保证教学清晰性）；
- 字符打印调试慎用：原代码中遍历 buffer 强制转 (char) 并打印，易因编码/乱码导致控制台异常，且干扰二进制文件（如图片、jar）传输——生产环境应移除所有非必要字符输出逻辑；
- 异常处理完整性：捕获 IOException 后不应静默吞掉，至少记录日志；NullPointerException 的空 catch 块是严重反模式，必须删除；
- 扩展性提示：若需传输多个文件或带元数据，建议在协议层增加文件名、MIME 类型等字段，而非依赖单一长度前缀。
通过以上修正，即可彻底解决因 read() 返回 0 引发的重复写入问题，构建出稳定、可预测的 Socket 文件传输通道。
