• 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?

VPS Starter Arubacloud
0 głosów
883 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 (344,100 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 (344,100 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 (344,100 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 (344,100 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 (194,920 p.)

Lub jeszcze krócej:

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

 

Podobne pytania

0 głosów
2 odpowiedzi 2,679 wizyt
pytanie zadane 5 stycznia 2017 w C i C++ przez Ala123456 Użytkownik (760 p.)
0 głosów
2 odpowiedzi 1,964 wizyt
pytanie zadane 22 kwietnia 2016 w C i C++ przez niezalogowany
0 głosów
1 odpowiedź 142 wizyt
pytanie zadane 13 stycznia 2021 w C i C++ przez Pelo Użytkownik (520 p.)

92,454 zapytań

141,263 odpowiedzi

319,099 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

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

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...