[HowTo]Pickup Bug - willkürliches Aufheben über Tastatur

Hallo,

Wie macht sich der Bug den bemerbar?

Wenn einzelnde Gegenstände um einen herum liegen mag alles noch relativ gut funktionieren, werden es aber mehr Items und Yang nähe des Charakters so merkt man das man manchmal nichts mehr aufheben kann. Wenn man um den gesamten Drop herumläuft und öfters ^ oder Y drückt kann es sein das er manchmal Items aufhebt und manchmal nicht. Selten passiert es auch das man trotz mehrerer Drops aufheben kann, aber dann nimmt er nicht das was am nähesten ist zuerst sondern sehr willkürlich.

Ein großen Dank an Lead der sich um dieses Problem gekümmert hat und nun wie es scheint einen vollständigen Fix dafür geschrieben hat!

Zitat von Lead0b110010100

So, hier der Fix:

In PythonItem.h über der Klasse einfügen:

C: PythonItem.h

1. inline int32_t DISTANCE_APPROX(int32_t dx, int32_t dy)
2. {
3. int32_t min, max;
4. if (dx < 0)
5. dx = -dx;
6. if (dy < 0)
7. dy = -dy;
8. if (dx < dy)
9. {
10. min = dx;
11. max = dy;
12. }
13. else
14. {
15. min = dy;
16. max = dx;
17. }
18. return max - min;
19. }

Alles anzeigen

Hier meine Funktionen für die PythonItem.cpp, einfach bei euch anpassen wie ihr es braucht. Ihr könnts eig. ersetzen denke ich.

C: PythonItem.cpp

1. bool CPythonItem::GetCloseMoney(const TPixelPosition &c_rPixelPosition, uint32_t*pdwItemID, uint32_t dwDistance)
2. {
3. uint32_t dwCloseItemID = 0;
4. uint32_t dwCloseItemDistance = dwDistance * dwDistance;
5. TGroundItemInstanceMap::iterator i;
6. for (i = m_GroundItemInstanceMap.begin(); i != m_GroundItemInstanceMap.end(); ++i)
7. {
8. TGroundItemInstance *pInstance = i->second;
9. if (pInstance->dwVirtualNumber != VNUM_MONEY)
10. continue;
11. uint32_t dwxDistance = DISTANCE_APPROX(c_rPixelPosition.x, pInstance->v3EndPosition.x);
12. uint32_t dwyDistance = DISTANCE_APPROX(c_rPixelPosition.y, pInstance->v3EndPosition.y);
13. uint32_t dwDistance = uint32_t(dwxDistance * dwxDistance + dwyDistance * dwyDistance);
14. if (dwDistance < dwCloseItemDistance)
15. {
16. dwCloseItemID = i->first;
17. dwCloseItemDistance = dwDistance;
18. }
19. }
20. if (dwCloseItemDistance > dwDistance * dwDistance)
21. return false;
22. *pdwItemID = dwCloseItemID;
23. return false;
24. }
25. bool CPythonItem::GetCloseItem(const TPixelPosition &c_rPixelPosition, uint32_t*pdwItemID, uint32_t dwDistance)
26. {
27. uint32_t dwCloseItemID = 0;
28. uint32_t dwCloseItemDistance = dwDistance * dwDistance;
29. TGroundItemInstanceMap::iterator i;
30. for (i = m_GroundItemInstanceMap.begin(); i != m_GroundItemInstanceMap.end(); ++i)
31. {
32. TGroundItemInstance *pInstance = i->second;
33. uint32_t dwxDistance = DISTANCE_APPROX(c_rPixelPosition.x, pInstance->v3EndPosition.x);
34. uint32_t dwyDistance = DISTANCE_APPROX(c_rPixelPosition.y, pInstance->v3EndPosition.y);
35. uint32_t dwDistance = uint32_t(dwxDistance * dwxDistance + dwyDistance * dwyDistance);
36. if (dwDistance < dwCloseItemDistance)
37. {
38. dwCloseItemID = i->first;
39. dwCloseItemDistance = dwDistance;
40. }
41. }
42. if (dwCloseItemDistance > dwDistance * dwDistance)
43. return false;
44. *pdwItemID = dwCloseItemID;
45. return true;
46. }

Alles anzeigen

Erklärung: Schuld waren zwei Dinge.

1) DWORD dwCloseItemDistance = 1000 * 1000; - Die Variable nutzt hardcode 1000 als maximale Distance, es sollte aber dwDistance genutzt werden was entweder 500 (auf dem Pferd) oder 300 (normaler Spieler) ist.

2) c_rPixelPosition.y-(-pInstance->v3EndPosition.y) - Kann negativ sein. DWORD(...) ihr versteht schon, Wertebereich geht bei unsigned Datentypen nicht ins Negative, deshalb können sie ja doppelt so groß sein wie signed Zahlen.

___________________________________________________________________________________________________________

Bonus:

Hier die CItem::DistanceValid - Funktion auf Serverseite. Falls der User läuft und dabei Dinge aufhebt, ändert er seine Position. Bis das Packet beim Server ankommt können das einige viele Koordis sein und der Spieler muss zurücklaufen zum Aufheben, zum Fixxen hab ich eine Tolleranzvariable eingeführt + das mit dem Pferd beachtet.

C

1. bool CItem::DistanceValid(LPCHARACTER ch)
2. {
3. if (!GetSectree())
4. return false;
5. int32_t iDist = DISTANCE_APPROX(GetX() - ch->GetX(), GetY() - ch->GetY());
6. uint16_t max_distance = ch->IsRiding() ? 500 : 300;
7. uint16_t tolerance = 150;
8. if (iDist > max_distance + tolerance)
9. {
10. if (test_server)
11. ch->ChatPacket(CHAT_TYPE_INFO, "%d: Distance is not valid: %d > %d", GetID(), iDist, max_distance + tolerance);
12. return false;
13. }
14. return true;
15. }

Alles anzeigen

Alles anzeigen

Mit freundlichen Grüßen!