[C++] Problem z grą - Exception thrown: read access violation

Dobra, trochę się pospieszyłem... Rozumiem, że błąd leży tu:

Kod:

---

for(int u = 0; u < this->iloscGraczy; u++)
        {
              if (status[u] == 2 && status[u+1] != 2) continue;
              else if (najlepszyGracz == -1) najlepszyGracz = u;
              else if (tablicaGraczy[u]->policzPunkty() > tablicaGraczy[najlepszyGracz]->policzPunkty())
              najlepszyGracz = u;
        }

---

I wiem, że muszę to przerobić tak, aby w przypadku kiedy dwie lub więcej osób przegrało, program nie próbował odwołać się do elementu tablicy -1. Dla dwóch graczy umiem to jakoś koślawo zrobić ale jak przerobić tą funkcję dla różnej ilości graczy? (maksymalnie 8). Znowu zastygłem przy tym więc proszę o jakiegoś tipa... ;d

Cytuj:

---

Pumpkin napisał

typie ale on to chce na wskaźnikach zrobić a nie kopiować obiekty,

---

no po tym co mu polecilem to owszem kopiowany bylby obiekt ;p
nie zaglebialem sie w algorytm

a zeby sobie skopiowac adres moze sobie uzyc &

Cytuj:

---

Dio Paladin napisał

Dobra, trochę się pospieszyłem... Rozumiem, że błąd leży tu:

Kod:

---

for(int u = 0; u < this->iloscGraczy; u++)
        {
              if (status[u] == 2 && status[u+1] != 2) continue;
              else if (najlepszyGracz == -1) najlepszyGracz = u;
              else if (tablicaGraczy[u]->policzPunkty() > tablicaGraczy[najlepszyGracz]->policzPunkty())
              najlepszyGracz = u;
        }

---

I wiem, że muszę to przerobić tak, aby w przypadku kiedy dwie lub więcej osób przegrało, program nie próbował odwołać się do elementu tablicy -1. Dla dwóch graczy umiem to jakoś koślawo zrobić ale jak przerobić tą funkcję dla różnej ilości graczy? (maksymalnie 8). Znowu zastygłem przy tym więc proszę o jakiegoś tipa... ;d

---

A coś takiego nie byłoby lepsze?

Kod:

---

najlepszyGracz = 0;
for(int u = 0; u < this->iloscGraczy; u++)
      if (tablicaGraczy[u]->policzPunkty() > tablicaGraczy[najlepszyGracz]->policzPunkty() && status[u] != 2)  najlepszyGracz = u;

---

Cytuj:

---

kuba_k31 napisał

A coś takiego nie byłoby lepsze?

Kod:

---

najlepszyGracz = 0;
for(int u = 0; u < this->iloscGraczy; u++)
      if (tablicaGraczy[u]->policzPunkty() > tablicaGraczy[najlepszyGracz]->policzPunkty() && status[u] != 2)  najlepszyGracz = u;

---

---

Królu złoty, zielony już wyleciał. Program działa pięknie, problem został rozwiązany.

A tutaj:

Kod:

---

for(int u = 0; u < this->iloscGraczy; u++)
        {
              if (status[u] == 2 && status[u+1] != 2) continue;
              else if (najlepszyGracz == -1) najlepszyGracz = u;
              else if (tablicaGraczy[u]->policzPunkty() > tablicaGraczy[najlepszyGracz]->policzPunkty())
              najlepszyGracz = u;
        }

---

mając np. 2 graczy (zmienna iloscGraczy = 2) pętla wykonała by się 2 razy, ale za drugim razem sprawdzałbyś wartości dla status[1] i status[2], czyli drugiego gracza z trzecim (który w grze nie uczestniczy).
Nie wiem czy przez to Ci sie sypalo, ale na przyszłość warto na to uważać :D