Błędy przy wskaźnikach?

Question

Powoli pisze sobie gierke w konsoli. Mam problem przy nakładaniu przedmiotów (na postać). Przedmioty są podzielone na kilka klas typu zbroja, helm itd. Wszystkie dziedziczą z jednej klasy wirtualnej, zalozmy Cloth.
Ekwipunek jest wektorem wskaźników Cloth. Poniżej funkcje klasy postaci odpowiedzialne za nakładanie/zdejmowanie przedmiotów (dla broni funkcja jest przeciążona z widocznych powodów) :

template <class T>
void cloth_on(T* w)
{
   if(this->hlvl>=w->lvl){
     this->def+=w->def;
     if(w->bid!=0){
                     if(w->bid==1) this->maxhp+= w->b_pow;
                     else if(w->bid==2) this->def+= w->b_pow;
                     else if(w->bid==3) this->rzuty+= w->b_pow;
                     else if(w->bid==4) this->dmg+= w->b_pow;
                     else if(w->bid==5) this->crt+= w->b_pow;
                  }
     w->on=true;
   }
}

void cloth_on(Miecz* w)
{
     if(this->hlvl>=w->lvl)
     {
               w->on=true;
               rzuty+=w->rzuty;
               dmg+=w->dmg;
               crt+=w->crt;
        if(w->bid!=0){
                     if(w->bid==1) this->maxhp+= w->b_pow;
                     else if(w->bid==2) this->def+= w->b_pow;
                     else if(w->bid==3) this->rzuty+= w->b_pow;
                     else if(w->bid==4) this->dmg+= w->b_pow;
                     else if(w->bid==5) this->crt+= w->b_pow;
                  }
              }
}

template <class U>
void cloth_off(U* w)
{
     this->def-=w->def;
     if(w->bid!=0){
                     if(w->bid==1) this->maxhp-= w->b_pow;
                     else if(w->bid==2) this->def-= w->b_pow;
                     else if(w->bid==3) this->rzuty-= w->b_pow;
                     else if(w->bid==4) this->dmg-= w->b_pow;
                     else if(w->bid==5) this->crt-= w->b_pow;
                  }
     w->on=false;
}
void cloth_off(Miecz* w)
{
              w->on=false;
              rzuty-=w->rzuty;
              dmg-=w->dmg;
              crt-=w->crt;
        if(w->bid!=0){
                     if(w->bid==1) this->maxhp-= w->b_pow;
                     else if(w->bid==2) this->def-= w->b_pow;
                     else if(w->bid==3) this->rzuty-= w->b_pow;
                     else if(w->bid==4) this->dmg-= w->b_pow;
                     else if(w->bid==5) this->crt-= w->b_pow;
                  }
}

Konkretnie chodzi o to, że postać dostaje kompletnie randomowe wartości obrony, dmg itd. W przypadku obrony są to czesto liczby olbrzymie, ale czasami też zero. Wspominam o tym, bo w przypadku innych statystyk są to liczby znacznie bliżej tej właściwej. Dmg i rzuty są czasami nawet dodawane poprawnie :D . Przy zdejmowaniu jest odejmowana ta sama wartość która została dodana przy założeniu.
Wywołania wyglądają tak:

h.cloth_off(v[choice]);

h.cloth_on(v[choice]);

hero - obiekt klasy postaci, v- wektor ekwipunku

Answer 1

Kolego, a mam takie jedno pytanie - czy konstruktor domyślny postaci zeruje startowo te wartości, bądź inicjalizuje jakąś wartością? Bo jeśli nie, to wartości, które dodaje funkcja są sumowane ze śmieciami, które znajdowały się w pamięci poprzednio. Mógłbyś pokazać jeszcze kod konstruktora klasy postaci oraz listę jej składowych?

 

@Edit:

Poza tym, ja w pierwszej kolejności przetestowałbym jakie wartości zawiera twoja składowa b_pow.

Możesz np. zmodyfikować kod w ten sposób i stwierdzić, czy wartości które przechowuje są poprawne dodając np. operację wyjścia w którymś z if'ów. :)

Comments

Dodawałem je na szybko przed chwilą :P za chwile zobacze.

---

Na 90% to jest problemem - używasz wskaźnika do klasy cloth, więc posługując się wskaźnikiem this używasz wartości składowej z tej klasy, ponieważ przechowujesz adres obiektu w wskaźniku polimorficznym.

A tworząc klasę inicjalizujesz składową klasy pochodnej, a nie bazowej. Proszę, powiedz, że teraz działa, bo to jest pomysł ostateczny i logicznie wyjaśnia dlaczego tak się dzieje.

---

Cóż, wzystkie przedmioty poza klasą Miecz są zakładane i zdejmowane poprawnie. Ale w Mieczu atrybuty zawsze są odejmowane o_O pole on zmienia sie poprawnie, ale reszta jest odejmowana niezależnie od użytej funkcji.

update: jeszcze pokombinuje, ale wieczorem, narazie musze lecieć.

---

Okej, daj znać, czy się udało.

A tutaj przykład czego NIE robić i dlaczego.

#include <iostream>

class A
{
    public:
        int x;
        A(){}
};

class B: public A
{
    public:
        int x;
        B(int n): x(n){}
};

int main( void )
{
    B Obiekt(10);
    A* Wskaznik = &Obiekt;

    std::cout << "Kiedy odczytuje z obiektu pochodnego: " << Obiekt.x << std::endl;
    std::cout << "Kiedy odczytuje z wskaznika polimorficznego na klase bazowa: " << Wskaznik->x << std::endl;

    return 0;
}

Output:

Kiedy odczytuje z obiektu pochodnego: 10
Kiedy odczytuje z wskaznika polimorficznego na klase bazowa: 4272976

 

---

Ok, już wszystko ładnie działa :D Kopiowałem część kodu i zapomniałem zamienić - na +. Dzięki za pomoc :)

Answer 2

Mógłbyś podać kod "h.cloth_on"?

Jak i prostą definicję klasy tego clouthu. (przedmiotu)

Podejrzewam że błąd jest w cloth_on gdyż zły adres pobiera.

Albo w obiekcie przedmiotu gdyż dziwną wartośc posiada.

 
Zrób test -- stwórz przedmiot a następnie sprawdz statystyki czy wszystko ok.

Jeśli tak to problem leży w metodzie.

Comments

Zrób test -- stwórz przedmiot a następnie sprawdz statystyki czy wszystko ok.

W obiekcie przedmiotu jest wszystko ok.
Kod jest w spoilerze.
 

---

Nie żebym wnikał ale nie powinno być?:

h.cloth_on<Tutaj_wpisz_tyb_zmiennej>(v[choice]);

 

 

---

Przy wywołaniu na pewno nie. Przy definicji/deklaracji tak, ale w specjalizacjach szablonów bodajze, tutaj mam zwykłe przeciążenie.

---

Nie, ponieważ to jest funkcja szablonowa, a nie cała klasa.

---

Wiem że to jest funkcyjna.

Ale co ma Miecz do ubrania? (np. zakładasz pierścień za pomocą w której zakładasz miecz)

no chyba że ekwipujesz broń to zwracam honor.