Release slot beendet Upload zu früh

[Cobain86]

hi
ich habe folgendes problem ich will einen Release slot reinhaben, das klappt auch ganz gut, zumindest mit der score verteilung.

jetz hab ich aber das problem bei meinem up sys,
ich habe es so reingemacht

bool CUploadQueue::CheckForTimeOver(CUpDownClient* client){
   uchar      requpfileid[16];
   CKnownFile* currequpfile = theApp.sharedfiles->GetFileByID(requpfileid);
   //Cuba Mod Up Kicker
   //TOTAL 9.5
   if(thePrefs.IsTotal95Enabled())
   {
      if(!client->IsFriend()
         || !client->IsDownloading()
         || !thePrefs.IsRSlotEnabled() && currequpfile->GetUpPriority()== !PR_VERYHIGH )  <---!!
      {
      if(client->Credits()->GetUploadedTotal() > 9.5*1024*1024)
      {
         Log(LOG_STATUSBAR, _T("User: %s hat bei QR: %u ca. %u KB bekommen"),client->GetUserName(),client->GetRemoteQueueRank(),client->GetSessionUp()/1024);
         return true;
       }
      }

jedocht haut er den client mit dem Release file immer wieder raus(also auch bei den andren up sys die nach 1,5mb kicken usw.

weiß jemand wie ich die upFile Prio in der UploadQueue.cpp abfrage?

[edit by Pathfinder: Überschrift umformuliert um das Wort 'Problem' zu vermeiden! Siehe Board Rules und Checkliste vor dem Posten]

[Xman]

der Fehler liegt hier:

if(client->Credits()->GetUploadedTotal() > 9.5*1024*1024)

nu überleg mal

[Stulle]

ich steig mal ein beim raten...
if(client->Credits() && client->Credits()->GetUploadedTotal() > 9.5*1024*1027)

würdest du n geraden wert wie 9 oder 10 nehmen könntest du den code schöner schreiben mit "10<<20", aber ich denke nicht das das bei nem float auch ohne weiteres geht... könnte mich aber irren...

[Xman]

Stulle.. bring ihn nicht auf flsche Fährten... lies mal was da steht:
client->Credits() && client->Credits()->GetUploadedTotal()

[Stulle]

siehste, wer lesen kann ist klar im vorteil!
im übrigen hatte ich sonst aber auch schon am anfang probleme weil ich nicht geschaut habe ob der Credits() pointer überhaupt existierte und nicht NULL war...
in anbetracht dessen, dass dies ja nu aber code für den UL ist sollte man wohl keine credits funktion nutzen, denn der eigentliche upload sollte nicht vom gesamt traffic der je geleistet worden ist beeinflusst werden...

trotzdessen stimmt mein vorschlag zur crash/ exception sicherheit wenn man wirklich unsinnigerweise dieses ziel haben sollte...

mfg stulle