public void Run2()
{
DateTime nbc = DateTime.Now;
while (true)
{
try
{
if (Kernel.Clients.Count > 0)
{
foreach (Player role in Kernel.Clients.Values)
{
//if (role.NobilityRecord != null)
role.Send(role.GetNobilityIconPacket());
}
}
}
catch { }
Thread.Sleep(1000);
}
}
no ...Quote:
.... few issues there...
#1: Before doing anything, switch Kernel.Clients to a threadsafe collection. I thought I had but according to error someone else got, i didn't. Because of that you'll get lots of issues.
#2: Why are you re-sending nobility information to all users every 1 second? There's no need! It updates AS PEOPLE DONATE. If it doesn't then it might have never been fully finished but that doesn't mean you write a wasteful thread to run it constantly... you just fix the system as it should work (async vs sync really here... Only change when it needs to be rather then constantly checking if it does need to be updated... or worse, updating it regardless of changes)
check out you source ...Quote:
What do you mean 'error'? If they both have the same exact amount of silver donated then whoever had the amount first should be ranked higher then them iirc.
Ok ...i was fixed the bugQuote:
That's a very minor issue really... Just change the sorting of ranks and you're good to go.