BufferedOutputStream не записывает все в файл

Question

Я пытаюсь создать 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

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;

}

}

Answer 1

Этот код мне кажется очень подозрительным:

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

Ваш выходной массив (данные) имеет длину length - 4, но вы только копируете length - 8 байты к нему.Если байты, которые следует игнорировать в packageData, — это первые 4 байта, это должно быть

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

потому что последний аргумент - это не длина, а to индекс (эксклюзивный).Видеть Javadoc для получения подробной информации.