1

下面的代码旨在获取一个 byte[] 和其他一些东西（参见它给出的代码）。然后它构建一个具有一定长度的数据包并返回它。我的问题是当我将文件读入主程序时，我需要数据报包的缓冲区为剩余要发送的字节数（如果小于 512）+ 4。这是为了处理读取请求的最后一个数据包TFTP 服务器。我在“buffer[bufferOffset] = data[dataOffset];”这一行得到一个索引越界异常。谁能帮我理解为什么会发生这种情况以及最好的解决方法是什么？

public DatagramPacket doRRQ(byte[] data, int block, InetAddress address, int port, long fileSize, long sentBytes){

//takes file data and creates a buffer for a DATA packet to be used in a RRQ. This packet 
//is passed to MyFirstTFTPServer.java to be modified and sent to client. This is done 
int bufferOffset = 3;
int dataOffset = -1;
byte[] buffer = null;
long difference = 0;
int i = 512;
byte byteBlock = (byte)block++;//possible issue with block no increment. test.

if (sentBytes + 512 > fileSize){

difference = fileSize - sentBytes;
    difference = difference + 4;
    i = (int)difference;
    buffer = new byte[i];

} else {

    buffer = new byte[516];

}

DatagramPacket packet = new DatagramPacket(buffer, buffer.length, address, port);
buffer[0] = 0;
buffer[1] = 3;
buffer[2] = 0;
buffer[3] = byteBlock;
for(byte item:data){

    bufferOffset++;
    dataOffset++;
    buffer[bufferOffset] = data[dataOffset];

    //System.out.println(bufferOffset);

    //System.out.println(dataOffset);

}

return packet;

}
4

2 回答 2

2

我看不到您在哪里看过 data.length. Data.length 是否可能大于 sendBytes？如果是这样，您的循环应该超过 sentBytes，而不是超过数据。

基本上，您决定缓冲区的缓冲区长度的逻辑似乎与 sentBytes 的大小不一致。

于 2012-05-22T20:34:42.373 回答
1

这是重构为紧凑形式的代码：

public DatagramPacket doRRQ(byte[] data, int block, InetAddress address,
    int port, long fileSize, long sentBytes)
{
  final byte[] buffer =
    new byte[sentBytes + 512 > fileSize? (int)(fileSize - sentBytes) + 4 : 516];
  final DatagramPacket packet = 
    new DatagramPacket(buffer, buffer.length, address, port);
  buffer[1] = 3;
  buffer[3] = (byte)block;
  for (int i = 0; i < data.length; i++) buffer[i+3] = data[i];
  return packet;
}

现在很明显发生了什么。您的缓冲区大小表达式不直接取决于data.length，因此问题可能在于方法的参数值，就像它可能在于代码本身一样。你来做法官。

于 2012-05-22T20:52:28.733 回答