Jak odwołać się do wektora wewnątrz klasy?

Question

Klasa Magazyn wygląda tak:

class Magazyn {

    vector < Towar > magazyn;

    public:
    void wczytaj_magazyn(Magazyn & obiekt);
    void wyswietl_magazyn();
    void zapisz_magazyn();
    void dostawa();
    void odtowarowanie();
    bool czy_jest_w_magazynie(string nazwa);
    Magazyn();
    ~Magazyn();
};

Obiekt klasy ma nazwę o_magazyn. Jak widać klasa zawiera wektor. Wewnątrz metody wczytaj_magazyn chcę wywołać pushback() na tym wektorze. gdy napiszę magazyn.pushback(jakiś_obiekt); dostaję komunikat "no matching function for call..."

Answer 1

vector `magazyn` jest polem prywatnym. .push_back() musi znaleźć się gdzieś w metodzie klasy.

edit: pełny kod i bliższa prawdzie odpowiedź w komentarzach

Comments

Możesz rozwinąć odpowiedź? Czyli pushback miałoby być metodą klasy Magazyn? Chciałęm skorzystać z metod wbudowanych vectora ale widzę że tak to nei zadziała. Czyli mam napisać funkcję pushback()? Jak miałby taki kod wyglądać?

---

nie, to znaczy że tylko metody Magazyn mogą operować na zmiennej prywatnej. Na przykład:

void Magazyn::wczytaj_magazyn(){
    Towar rzecz; // ...
    magazyn.push_back(rzecz);
}

edit: możliwe że źle zrozumiałem Twój problem. Czy aby po prostu nie napisałeś pushback zamiast push_back?

---

Przecież to właśnie chcę zrobić. Napisałem że wewnątrz metody wczytaj_magazyn chcę użyć magazyn.pushback(). I wtedy dostaję komunikat "no matching function for call to.."

Tak wygląda ta metoda. Pod koniec wywołuję pushback() na wektorze i otrzymuję wspomniany błąd. Co robię źle?

void Magazyn::wczytaj_magazyn(Magazyn & obiekt){ //odczyt z pliku

    fstream plik;
    plik.open("magazyn.txt", ios::in);

    if(plik.good()==false) {
        cout<<"Nie mozna wczytac magazynu";
        exit(0);
    }

    string linia, naz;
    int nr_linii = 1, ile, ind;
    float cen;
    while(getline(plik, linia)){
        switch(nr_linii){
        case 1:
            {
                naz = linia;
                nr_linii++;
                break;
            }
        case 2:
            {
                ile = atoi(linia.c_str());
                nr_linii++;
                break;
            }
        case 3:
            {
                ind = atoi(linia.c_str());
                nr_linii++;
                break;
            }
        case 4:
            {
                cen = atof(linia.c_str());
                Towar * nowy = new Towar(naz, ile, ind, cen);
                o_magazynmagazyn.push_back(nowy);
                nr_linii = 1;
                break;
            }
        }
    }
    plik.close();
}

 

---

OK, widzę.

Pod koniec wywołuję pushback() na wektorze

Nie, na czymś o nazwie o_magazynmagazyn. Twój vector nazywa się `magazyn`.

Po drugie, vector przechowuje Towary. Próbujesz dodać wskaźnik na towar. Zamiast wskaźnika, po prostu dodaj obiekt?

(boczna uwaga: nie trzeba wywoływać plik.close(), destruktor zrobi to za Ciebie)

---

omagazynmagazyn zostało po wykasowaniu dopisanej linijki, nie zauważyłem. Dodaję wskaźniki, ponieważ wykładowca wymaga przechowywania wskaźników na obiekty, nie samych obiektów. Ale moim okiem nadal nie odpowiedziałeś na moje pytanie

---

Odpowiedziałem.

no matching function for call...

Próbujesz wywołać .push_back() przekazując wskaźnik. Kompilator krzyczy, bo .push_back bierze jako argument Towar, nie Towar*. Poprawne wywołanie wygląda na przykład tak:

                Towar nowy(naz, ile, ind, cen);
                magazyn.push_back(nowy);

Lub krócej:

                magazyn.push_back(Towar(naz, ile, ind, cen));

 

---

Lub jeszcze krócej:

magazyn.emplace_back(naz, ile, ind, cen);