C++ vector a referencja

Question

Witajcie,

dopiero zaczynam naukę programowania obiektowego i natknąłem się następujący problem.

Mam klase Update, w której jest: virtual void update() = 0;

W mainie utworzyłem obiekt klasy Potion heart1 oraz Update *wsk;

Czemu zapis " wsk = &heart1 " nie jest poprawny? Bo dostaje komunikat o błędzie "Cannot convert Potion* to Update* in assignment".

I jeszcze dodatkowe pytanie: mam utworzone: vector<Potion> potionArray;

Jak mam przekazać wartość potionArray do wsk? Dodam, że jest to używane w pętli:

licznik = 0;
        for(iter6 = pickUpArrayHeart.begin(); iter6 != pickUpArrayHeart.end(); iter6++)
        {
            pickUpArrayHeart[licznik].update(); // to chciałbym zastąpic wsk -> update();
            window.draw(pickUpArrayHeart[licznik].rect);
            window.draw(pickUpArrayHeart[licznik].sprite);

            licznik++;
        }

pickUpArrayHeart[licznik].update(); // to chciałbym zastąpic wsk -> update();

 

Z góry dziękuję za pomoc

Comments

Na pierwsze pytanie dostajesz odpowiedź w opisie błędu. Tworzysz wskaźnik na obiekt klasy Update, a następnie próbujesz przypisać do niego wskaźnik na obiekt klasy Potion. Nie można tak

Answer 1

Jeśli Potion dziedziczy z Update, to nie powinno być problemów.

Z komentarzy pod odpowiedzią Ehlerta:

expected class name before '{' token. a po usunięciu drugiego dziedziczenia program się uruchamia i wszystko działa poprawnie

 Widocznie nie masz zaincludowanego Update.

pickUpArrayHeart[licznik].update(); // to chciałbym zastąpic wsk -> update();

Ale.. po co?

Btw. pętla wrzucona przez ciebie. Jeśli korzystasz z iteratorów, to... korzystaj z iteratorów:

        for(iter6 = pickUpArrayHeart.begin(); iter6 != pickUpArrayHeart.end(); iter6++)
        {
            iter6->update();
            window.draw(iter6->rect);
            window.draw(iter6->sprite);
         }

Kolejna uwaga: od tego jest sf::Drawable, żeby z niej dziedziczyć, a nie pisać coś takiego:

window.draw(pickUpArrayHeart[licznik].rect);
window.draw(pickUpArrayHeart[licznik].sprite);

Tzn. masz jakąś tam klase Potion i tak:

class Potion : public sf::Drawable
{
public:
   void draw(sf::RenderTarget & target, sf::RenderStates states) const override
   {
      target.draw(rect, states);
      target.draw(sprite, states);
   }

private:
   sf::RectangleShape rect;
   sf::Sprite sprite;
};

//...

Potion p;
window.draw(p);

 

Answer 2

Czemu wskaźnik ustawiasz na obiekt, który nie ma nic wspólnego z nim? Żeby wsk = &heart1 powiodło się to klasa Potion musi dziedziczyć po Update i przeładować jej wirtualne metody. A tego 2 pytania to w ogóle nie rozumiem. Co chcesz osiągnąć?

Answer 3

Jeżeli klasa Potion dziedziczy po klasie Update to możesz przypisać adres do tego wskaźnika i błędu nie będzie.

Answer 4

// Example program
#include <iostream>
#include <string>
#include <vector>

class Update{
public:
    virtual void update() = 0;
};

class Potion : public Update{
    public:
  virtual void update() {
    std::cout << "Update()";   
  }
};

int main()
{
    Potion a;
    Update* wsk = (Update*)&a;
    //poczytaj o dynamic_cast
    wsk->update();
    
    std::vector<Potion> container;
    for (int i = 0; i < 5; ++i)
        container.push_back(Potion());
        
    for(auto iter = container.begin(); iter != container.end(); ++iter)
        iter->update();
}

Mi tam wszystko działa. Oczywiście mam nadzieję, że Potion dziedziczy z update bo inaczej nie mamy o czym rozmawiać,

Comments

A czemu nie wsk = &a; ?

---

Bez również zadziała, ale to zależy pod jakim kompilatorem. Na cpp.sh działa.

---

Dzięki wielkie, dynamic cast to było to! 

Oczywiście Potion dziedziczy z Update, ale gdy zrobiłem Update* wsk = (Update*)&a to wyskoczył błąd w klasie Potion 

#ifndef POTION_H_INCLUDED
#define POTION_H_INCLUDED

#include "collect.h"

class Potion :public Collect, public Update
{
public:

    int valueHeart = 10;

    Potion();
    ~Potion();

    virtual void update();
};

#endif // POTION_H_INCLUDED

expected class name before '{' token. a po usunięciu drugiego dziedziczenia program się uruchamia i wszystko działa poprawnie. O co chodzi?

---

Nie mam pojęcia. Piszesz pod Visual Studio?

---

Nie, w Code Blocksie

---

Pisz w czymś bardziej ambitnym. 

• Windows: Visual Studio
• Linux: Clion

---

@Ehlert:
Ale... jaki dynamic cast? 

dynamic_cast - jest taki operator rzutowania, którego w twoim kodzie nie ma.
Użyłeś rzutowania z C. Tzn. kompilator sam sobie zrobi co mu się podoba.

Poza tym - dynamic_cast służy do rzutowania w dół - ze wskaźnika/referencji na klase bazową do wskaźnika/referencji na klase pochodna. U ciebie mamy tylko rzutowanie w góre do którego już żadnej magii nie potrzebujemy i wystarczy najprostszy static_cast. Z resztą niepotrzebne - każdy kompilator C++ bez problemu wykonuje niejawnie konwersje ze wskaźnika klasy pochodnej na wskaźnik klasy bazowej.

---

Chylę czoła, zapamiętam.