From ef7bbc849203ec02ef0f2b882fb3036ddc63a0c4 Mon Sep 17 00:00:00 2001 From: Persson-dev Date: Thu, 13 Mar 2025 10:08:36 +0100 Subject: [PATCH] packetpool: remove simulation --- .../app/src/main/java/network/PacketPool.java | 37 ++++++++++++++----- 1 file changed, 27 insertions(+), 10 deletions(-) diff --git a/ChatApp/app/src/main/java/network/PacketPool.java b/ChatApp/app/src/main/java/network/PacketPool.java index 374c93c..1d3a4a4 100644 --- a/ChatApp/app/src/main/java/network/PacketPool.java +++ b/ChatApp/app/src/main/java/network/PacketPool.java @@ -24,7 +24,7 @@ public class PacketPool { private static int MAX_SEND_TRY = 50; private static long SEND_DELAY = 10; - private static long RETRY_INTERVAL = SEND_DELAY * 2; + private static long RETRY_INTERVAL = 100; private static float PACKET_LOSS_PROBABILITY = 0.1f; private static record ReliablePacketAddress(ReliablePacket packet, InetSocketAddress address) { @@ -71,6 +71,7 @@ public class PacketPool { /** * Set the recieve sequence number + * * @param address * @param newValue */ @@ -80,6 +81,7 @@ public class PacketPool { /** * Try to add the address into memory + * * @param address */ private void tryAddContext(InetSocketAddress address) { @@ -88,6 +90,7 @@ public class PacketPool { /** * Construct a PacketPool + * * @param socket */ public PacketPool(Socket socket) { @@ -97,7 +100,7 @@ public class PacketPool { } private void debugPrint(String msg) { -// System.out.println(msg); + // System.out.println(msg); } private void debugSend(ReliablePacket packet, InetSocketAddress address) { @@ -114,6 +117,7 @@ public class PacketPool { /** * Send a packet to the socket + * * @param packet * @param address * @throws IOException @@ -122,13 +126,18 @@ public class PacketPool { var packetsSentTries = this.addressContexts.get(address).packetsSentTries; new Thread(() -> { + // try { + // Thread.sleep(SEND_DELAY); + // if (Math.random() > PACKET_LOSS_PROBABILITY) try { - Thread.sleep(SEND_DELAY); - if (Math.random() > PACKET_LOSS_PROBABILITY) - this.socket.sendPacket(packet, address); - } catch (InterruptedException | IOException e) { - // e.printStackTrace(); + this.socket.sendPacket(packet, address); + } catch (IOException e) { + // TODO Auto-generated catch block + e.printStackTrace(); } + // } catch (InterruptedException | IOException e) { + // // e.printStackTrace(); + // } }).start(); if (packet.getPacket() instanceof AcknowlegdementPacket) @@ -143,7 +152,9 @@ public class PacketPool { } /** - * Send a packet to socket and try to resend if an acknowlegment was not recieved + * Send a packet to socket and try to resend if an acknowlegment was not + * recieved + * * @param packet * @param address * @throws IOException @@ -163,6 +174,7 @@ public class PacketPool { /** * Send a packet (and encapsulate it) + * * @param packet * @param address * @throws IOException @@ -175,6 +187,7 @@ public class PacketPool { /** * Try to resend a packet + * * @param reliablePacketAddress */ private void tryResend(ReliablePacketAddress reliablePacketAddress) { @@ -212,7 +225,6 @@ public class PacketPool { ctx.sendThreads.remove(Thread.currentThread()); } - /** * @param address * @return The smallest sequence number recieved @@ -227,7 +239,9 @@ public class PacketPool { } /** - * Move packets in buffer to the packet recieved queue to be processed by the app + * Move packets in buffer to the packet recieved queue to be processed by the + * app + * * @param address * @return the sequence number of the last packet that was added to the queue */ @@ -251,6 +265,7 @@ public class PacketPool { /** * Process packet when recieved + * * @param packet * @param address * @throws IOException @@ -311,6 +326,7 @@ public class PacketPool { /** * Closes the connexion + * * @param adress */ private void close(InetSocketAddress adress) { @@ -321,6 +337,7 @@ public class PacketPool { /** * Stop the threads of the connexion + * * @param adressContext */ private void close(AdressContext adressContext) {