Refactor packet sending to use SendReliable method for improved reliability and remove unnecessary logging
This commit is contained in:
@@ -99,10 +99,7 @@ namespace KCM.Packets.Network
|
|||||||
chunkSize = chunk.Length,
|
chunkSize = chunk.Length,
|
||||||
saveDataIndex = sent,
|
saveDataIndex = sent,
|
||||||
totalChunks = chunks.Count
|
totalChunks = chunks.Count
|
||||||
}.Send(clientId, Riptide.MessageSendMode.Reliable);
|
}.SendReliable(clientId);
|
||||||
|
|
||||||
Main.helper.Log(" ");
|
|
||||||
|
|
||||||
packetsSent++;
|
packetsSent++;
|
||||||
sent += chunk.Length;
|
sent += chunk.Length;
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -20,9 +20,6 @@ namespace KCM.Packets
|
|||||||
|
|
||||||
if (!Main.clientSteamIds.ContainsKey(clientId))
|
if (!Main.clientSteamIds.ContainsKey(clientId))
|
||||||
return null;
|
return null;
|
||||||
|
|
||||||
//Main.helper.Log($"SteamID: {Main.GetPlayerByClientID(clientId).steamId} for {clientId} ({Main.GetPlayerByClientID(clientId).id})");
|
|
||||||
|
|
||||||
if (Main.kCPlayers.TryGetValue(Main.GetPlayerByClientID(clientId).steamId, out p))
|
if (Main.kCPlayers.TryGetValue(Main.GetPlayerByClientID(clientId).steamId, out p))
|
||||||
return p;
|
return p;
|
||||||
else
|
else
|
||||||
@@ -107,6 +104,37 @@ namespace KCM.Packets
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public void SendReliable(ushort toClient)
|
||||||
|
{
|
||||||
|
try
|
||||||
|
{
|
||||||
|
if (KCServer.IsRunning && toClient != 0)
|
||||||
|
{
|
||||||
|
KCServer.server.Send(PacketHandler.SerialisePacket(this), toClient, Riptide.MessageSendMode.Reliable);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
catch (Exception ex)
|
||||||
|
{
|
||||||
|
Main.helper.Log($"Error sending reliable packet {packetId} {this.GetType().Name} from {clientId}");
|
||||||
|
|
||||||
|
Main.helper.Log("----------------------- Main exception -----------------------");
|
||||||
|
Main.helper.Log(ex.ToString());
|
||||||
|
Main.helper.Log("----------------------- Main message -----------------------");
|
||||||
|
Main.helper.Log(ex.Message);
|
||||||
|
Main.helper.Log("----------------------- Main stacktrace -----------------------");
|
||||||
|
Main.helper.Log(ex.StackTrace);
|
||||||
|
if (ex.InnerException != null)
|
||||||
|
{
|
||||||
|
Main.helper.Log("----------------------- Inner exception -----------------------");
|
||||||
|
Main.helper.Log(ex.InnerException.ToString());
|
||||||
|
Main.helper.Log("----------------------- Inner message -----------------------");
|
||||||
|
Main.helper.Log(ex.InnerException.Message);
|
||||||
|
Main.helper.Log("----------------------- Inner stacktrace -----------------------");
|
||||||
|
Main.helper.Log(ex.InnerException.StackTrace);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
public abstract void HandlePacketServer();
|
public abstract void HandlePacketServer();
|
||||||
public abstract void HandlePacketClient();
|
public abstract void HandlePacketClient();
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user