• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

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

Fiszki IT
Fiszki IT
0 głosów
391 wizyt
pytanie zadane 23 stycznia 2019 w C i C++ przez Alan Kruszyński Obywatel (1,410 p.)

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..."

1 odpowiedź

0 głosów
odpowiedź 23 stycznia 2019 przez adrian17 Ekspert (297,680 p.)
edycja 23 stycznia 2019 przez adrian17
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
komentarz 23 stycznia 2019 przez Alan Kruszyński Obywatel (1,410 p.)
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ć?
komentarz 23 stycznia 2019 przez adrian17 Ekspert (297,680 p.)

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?

komentarz 23 stycznia 2019 przez Alan Kruszyński Obywatel (1,410 p.)

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();
}

 

komentarz 23 stycznia 2019 przez adrian17 Ekspert (297,680 p.)

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)

komentarz 23 stycznia 2019 przez Alan Kruszyński Obywatel (1,410 p.)
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
komentarz 23 stycznia 2019 przez adrian17 Ekspert (297,680 p.)

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));

 

1
komentarz 24 stycznia 2019 przez j23 Mędrzec (164,200 p.)

Lub jeszcze krócej:

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

 

Podobne pytania

0 głosów
2 odpowiedzi 1,794 wizyt
pytanie zadane 5 stycznia 2017 w C i C++ przez Ala123456 Użytkownik (770 p.)
0 głosów
2 odpowiedzi 1,162 wizyt
pytanie zadane 22 kwietnia 2016 w C i C++ przez niezalogowany
0 głosów
1 odpowiedź 52 wizyt
pytanie zadane 13 stycznia w C i C++ przez Pelo Użytkownik (520 p.)
Porady nie od parady
Wynikowy wygląd pytania, odpowiedzi czy komentarza, różni się od tego zaprezentowanego w edytorze postów. Stosuj więc funkcję Podgląd posta znajdującą się pod edytorem, aby upewnić się, czy na pewno ostateczny rezultat ci odpowiada.Podgląd posta

84,758 zapytań

133,559 odpowiedzi

295,986 komentarzy

56,012 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...