我尝试创建一个TFTP服务器,但是当它收到文件时,它似乎并非所有的所有都保存到服务器上(一些字节丢失)。创建文件罚款,并且大多数数据被写入,但由于文件未完成,因此被归类为损坏和不可手持。有谁知道如何解决这个问题?

主类

            WRQ WRQ = new WRQ();
            ACK ACK = new ACK();
            DatagramPacket outPacket;
            BufferedOutputStream bufferedOutput = new BufferedOutputStream(new FileOutputStream(filename));
            byte[] bytes;
            byte[] fileOut;
            outPacket = WRQ.firstPacket(packet);
            socket.send(outPacket);

            socket.receive(packet);

            while (packet.getLength() == 516){

            bytes = WRQ.doWRQ(packet);
            bufferedOutput.write(bytes);

            outPacket = ACK.doACK(packet);
            socket.send(outPacket);

            socket.receive(packet); 

            }

            bytes = WRQ.doWRQ(packet);
            bufferedOutput.write(bytes);

            bufferedOutput.close();

            outPacket = ACK.doACK(packet);
            socket.send(outPacket);
.

wrq class

public class WRQ {

public DatagramPacket firstPacket(DatagramPacket packet) throws IOException{

    ACK ACK = new ACK();
    DatagramPacket ACKpacket = ACK.doACK(packet);

    //takes ACK packet and sets block # as 0 to signal that this is the first packet in a WRQ
    byte[] ACKcontents = new byte[3];
    ACKcontents = ACKpacket.getData();
    ACKcontents[2] = 0;
    ACKcontents[3] = 0;
    ACKpacket.setData(ACKcontents);

    return ACKpacket;

}

public byte[] doWRQ(DatagramPacket packet){

    int length = packet.getLength();
    byte[] packetData = packet.getData();
    byte[] data = new byte[length - 4];
    data = Arrays.copyOfRange(packetData, 4, length - 4);

    return data;

}

}
.

有帮助吗?

解决方案

此代码对我来说非常怀疑:

byte[] packetData = packet.getData();
byte[] data = new byte[length - 4];
data = Arrays.copyOfRange(packetData, 4, length - 4);
.

您的输出阵列(数据)是长度生成的,但是您只能将length - 4字节复制到它。如果在packetdata中忽略的字节是前4个字节,则应是

data = Arrays.copyOfRange(packetData, 4, length);
.

因为最后一个参数不是一个长度,而是length - 8索引(独占)。查看 javadoc 详细信息。

许可以下: CC-BY-SA归因
不隶属于 StackOverflow
scroll top