C++ wywołanie metody z klasy bazowej

pytanie zadane 31 maja 2018 w C i C++ przez Prime_Bull Obywatel (1,820 p.)

Hej. Chciałem stworzyć swoją klasę stosu, która oferowałaby więcej funkcjonalności. Pomyślałem, żeby dziedziczyć z klasy std::stack. Mam jednak problem z wywołaniem metod dziedziczonych np. push().

class Stos : public std::stack<std::string> { 
      // cos tu bedzie 
};

I chciałem wywołać w taki sposób:

Stos s;
s.push(1);

Error:

||=== Build: Debug in Alg_Lab12v2 (compiler: GNU GCC Compiler) ===|
C:\Users\przem\Documents\Programming\Alg_Lab12v2\main.cpp||In function 'int main()':|
C:\Users\przem\Documents\Programming\Alg_Lab12v2\main.cpp|75|error: no matching function for call to 'Stos::push(int)'|
c:\mingw\lib\gcc\mingw32\6.3.0\include\c++\bits\stl_stack.h|218|note: candidate: void std::stack<_Tp, _Sequence>::push(const value_type&) [with _Tp = std::__cxx11::basic_string<char>; _Sequence = std::deque<std::__cxx11::basic_string<char>, std::allocator<std::__cxx11::basic_string<char> > >; std::stack<_Tp, _Sequence>::value_type = std::__cxx11::basic_string<char>]|
c:\mingw\lib\gcc\mingw32\6.3.0\include\c++\bits\stl_stack.h|218|note:   no known conversion for argument 1 from 'int' to 'const value_type& {aka const std::__cxx11::basic_string<char>&}'|
c:\mingw\lib\gcc\mingw32\6.3.0\include\c++\bits\stl_stack.h|223|note: candidate: void std::stack<_Tp, _Sequence>::push(std::stack<_Tp, _Sequence>::value_type&&) [with _Tp = std::__cxx11::basic_string<char>; _Sequence = std::deque<std::__cxx11::basic_string<char>, std::allocator<std::__cxx11::basic_string<char> > >; std::stack<_Tp, _Sequence>::value_type = std::__cxx11::basic_string<char>]|
c:\mingw\lib\gcc\mingw32\6.3.0\include\c++\bits\stl_stack.h|223|note:   no known conversion for argument 1 from 'int' to 'std::stack<std::__cxx11::basic_string<char> >::value_type&& {aka std::__cxx11::basic_string<char>&&}'|
||=== Build failed: 1 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

Pomyślałem, żeby zrobić coś takiego, ale nie działa:

Stos s;
s.std::stack<std::string>::push(1);

komentarz 31 maja 2018 przez mokrowski Mędrzec (155,460 p.)

Oprócz błędu który już został pokazany, istnieje niebezpieczeństwo że wpakujesz się w problemy. Dziedziczenie z kontenera nie posiadającego wirtualnego destruktora (a ten w bibliotece standardowej go nie ma), może skazać program na niezdefiniowane zachowanie przy mało uważnym programowaniu.

komentarz 31 maja 2018 przez Prime_Bull Obywatel (1,820 p.)

Masz na myśli wycieki pamięci, tak jak jest to omówione w tym filmie?
https://www.youtube.com/watch?v=jELbKhGkEi0

Skoro mogą być wycieki pamięci to dlaczego twórcy biblioteki standardowej nie zastosowali wirtualnych destruktorów? Nie zakładali, że klasy będą dziedziczone?

1	komentarz 31 maja 2018 przez mokrowski Mędrzec (155,460 p.) Tak. Nie zakładali że klasy będą dziedziczone. W przeciwnym wypadku, nie wykorzystując dziedziczenia, płacił byś cenę za wirtualny destruktor. Poza tym lepsze jest zawieranie od dziedziczenia.

komentarz 1 czerwca 2018 przez Prime_Bull Obywatel (1,820 p.)

Szczerze mówiąc to od niedawna programuję obiektowo i miałem właśnie problem z dobrym połączeniem klas jeśli chodzi o dziedziczenie, dopiero później wpadłem na pomysł, że zacznę tworzyć obiekty klas w innych klasach. Czyli można by rzec, że nieświadomie używałem zawierania czy tam kompozycji, prawda?

komentarz 1 czerwca 2018 przez mokrowski Mędrzec (155,460 p.)

Zawieranie to inaczej Agregacja. Jeśli obiekt klasy zawierającej jest niszczony, obiekty klas zawieranych (agregowanych), nie są niszczone i przetrwają.

Kompozycja to zawieranie ścisłe. Jeśli obiekt klasy komponującej jest niszczony, obiekty klas zawieranych (wkomponowanych), są niszczone i nie przetrwają.

Używałeś zapewne jednego z nich.

Po prostu nie podniecaj się dziedziczeniem bo oprócz wielu zalet (np. łatwości wytłumaczenia jak to działa), tworzy często zbyt twarde "spawy w kodzie". Następnie nie użyjesz ponownie kodu jakiejś klasy jeśli nie wyciągniesz także jej rodziców. Powodem będzie "zespawanie" poprzez dziedziczenie. To taka relacja która nie powinna być wybierana "jak nie wiem co to dziedziczenie".... Raczej z dużą rozwagą...