Mam stworzoną klasę ekwipunku, i podczas zamiany przedmiotów miejscami wykonywana jest taka operacja:
Item *tempItem = inventoryBar[j].item;
inventoryBar[j].item = inventoryBar[i].item;;
inventoryBar[i].item = tempItem;
Czy jest to poprawne? Jeśli nie to w jaki sposób poprawnie zamienić te przedmioty miejscami?
Na oko jest ok.
Poczytaj sobie o tym tutaj:
https://en.wikipedia.org/wiki/Swap_(computer_science)#Using_a_temporary_variable
A czy ten tempItem można po prostu zostawić, czy on nie będzie powodował wycieków pamięci? Bo jak się go usuwa potem to item znika.
Przecież to jest tylko wskaźnik. Nic z nim nie rób. Wyciek pamieci jest kiedy nie zwolnisz zaalokowanej pamięci i zgubisz do niej wskaźnik. A ty przecież masz 2 obiekty i caly czas trzymasz do nich wskaźniki i wszystko jest ok.