我正在实现一个简单的Java udp套接字程序。详细信息如下:
这是问题所在: 客户端的for循环永无止境!这意味着将永远不会收到2500个数据包!尽管我检查了服务器端,并已将它们全部发送出去。
我尝试使用以下方法将套接字的接收缓冲区大小设置为10 * packetSize
:
socket.setReceiveBufferSize(10 * packetSize)
但是它不起作用。
您认为我如何解决这个问题?我知道UDP不可靠,但是客户端和服务器都在同一台计算机的不同端口上运行!
这是服务器端的代码:
for (int i = 0; i < packets; i++) {
byte[] currentPacket = new byte[size];
byte[] seqnum = intToByteArray(i);
currentPacket[0] = seqnum[0];
currentPacket[1] = seqnum[1];
currentPacket[2] = seqnum[2];
currentPacket[3] = seqnum[3];
for (int j = 0; j < size-4; j++) {
currentPacket[j+4] = finFile[i][j];
}
sendPacket = new DatagramPacket(currentPacket, currentPacket.length, receiverIP, receiverPort);
socket.send(sendPacket);
try {
Thread.sleep(2);
} catch (InterruptedException e) {
e.printStackTrace();
}
}
和客户端:
int k = 0;
while (true) {
receivedBytes = new byte[size];
receivePacket = new DatagramPacket(receivedBytes, size);
socket.receive(receivePacket);
allBytes.add(receivePacket.getData());
k++;
if (k == packets)
break;
}
allBytes
只是一个包含已接收字节的链表。我用它来重组最终文件。
P.S。此代码非常适合100Mb以下的文件。 谢谢
答案 0 :(得分:1)
更新:
tl;博士摘要:packets
未正确初始化或使用TCP,或者在UDP数据包中添加了序列号,以便您的客户端知道它是否丢弃了数据包,并且您可以编写代码来处理该数据包(请求一个转播)。无论如何,这本质上使其成为基本的TCP。
我怀疑您从未初始化packets
,所以您从未击中break
。将while
重写为for
循环可以很容易地检查这是否成立。假设您发送的第一个数据包包含将要接收的数据包,并且您正确初始化了packets
,则如果丢失了数据包,则客户端程序将不会结束,因为receive()是一种阻塞方法。
如果您强烈怀疑丢失了数据包,请调试客户端,查看LinkedList中有多少个接收到的数据包,并将其与服务器端发送的数据包进行比较。
for(int i = 0; i < packets; i++) {
receivedBytes = new byte[size];
receivePacket = new DatagramPacket(receivedBytes, size);
socket.receive(receivePacket);
allBytes.add(receivePacket.getData());
}
System.out.println("All " + packet + " received.");
切换到上面的代码将使您知道,如果您从未使用过print语句,那么您将知道正在丢失数据包,因为receive()是一种阻塞方法,这意味着您的客户端被卡在了for
循环。这是因为for
循环无法满足,因为如果服务器发送了2500个数据包,但客户端仅接收了2300个数据包,则它仍将在receive()行的for循环中等待2301、2302 ,...数据包等。
由于您有一个大于100MB的文件需要组装,因此我认为您不能容忍丢失,因此请使用将满足此要求的TCP或通过在您的代码中创建自己的标头来处理这种可能性每个数据包。该报头可以像客户端将要接收和读取的递增序号一样简单,如果它跳过前一个数据包中的数字,则它将知道一个数据包丢失了。此时,您可以让客户端请求服务器重新广播该特定数据包。但是,此时您仅实现了自己的原始TCP。