Dziedziczenie wielokrotne + wielopokoleniowe

Question

Hej mam kod odnośnie klas:

podatnik,hpp
 

class Podatnik
{
    std::string nip_;
public:
    Podatnik(std::string nip_)
    :nip_(nip_)
    {
       //sprawdzamy czy to nip, jak nie mo¿emy poprosiæ o nowe dane podanie albo coœ....
    }
    virtual ~Podatnik(void) = default;
    auto Nip(void)const ->std::string {return nip_;}
    auto pokaz(void) const -> std::string { return "nip: " + nip_;}
    virtual auto wizytowka(void) const ->std::string {return nip_;}

};

Osoba_fizczyna.hpp

class Osoba_fizyczna: public Podatnik
{
   std::string imie_;
   std::string nazwisko_;
   std::string data_;
public:
    Osoba_fizyczna(std::string imie, std::string nazwisko, std::string nip, std::string data)
    :Podatnik(nip) , imie_(imie), nazwisko_(nazwisko),data_(data){}


    auto Imie(void)      const -> std::string {return imie_;}
    auto Nazwisko(void)  const -> std::string {return nazwisko_;}
    auto Data(void)      const -> std::string {return data_;}
    auto pokaz(void)     const -> std::string{ return  Imie()+ " " + Nazwisko()+ Data() +" nip: " + Nip();}
    auto wizytowka(void) const ->std::string override {return  Imie()+ " " + Nazwisko()+" "+Data() + "\n "+" nip: " + Nip()+ "\n "+ "\n ";}
    
};

Firma.hpp

class Firma: public Podatnik
{

    std::string nazwa_firmy_;
public:
    Firma(std::string nazwa_firmy, std::string nip)
    :Podatnik(nip), nazwa_firmy_(nazwa_firmy){}

    auto firma(void) const -> std::string {return nazwa_firmy_;}
    auto pokaz(void) const -> std::string{ return firma()+ "nip: "+ Nip();}
    auto wizytowka()const ->std::string override {return firma()+ "\n " + " nip: "+ Nip()+ "\n "+ "\n ";}

};

dzialalnosc.hpp

class Dzialalnosc: public Osoba_fizyczna, public Firma
{
  std::string regon_;
  public:
      auto REG(void) const -> std::string {return regon_;}

      Dzialalnosc(std::string imie, std::string nazwisko, std::string nip, std::string data, std::string nazwa_firmy, std::string regon)
      :Firma(nazwa_firmy, nip),Osoba_fizyczna(imie, nazwisko, nip, data) ,regon_(regon){}

      auto wizytowka(void) const ->std::string override {return  Imie()+ " " + Nazwisko()+" "+Data() + "\n "+" nip: " + Nip()+ "\n "+ firma()+ "Regon: " +REG()+"\n ";}

};

Próbując skompilować kod dostaję błąd w klasie działalnośc,hpp (8 linia kodu)

class "Dzialalnosc" does not have any field named "Firma"
Nie mogę znaleźć tego co robię źle, wie ktoś co jest nie tak?

 

Answer 1

To jest jedyny błąd? Jeśli nie - podaj wszystkie.

Kilka uwag:

Firma i Osoba_fizyczna powinny dziedziczyć po Podatnik wirtualnie. A to z powodu, że Dzialalnosc dziedziczy po obu tych klasach, więc ostatecznie dziedziczy po klasie Podatnik dwukrotnie. To prowadzi do dwuznaczności przy odwoływaniu się do pól dziedziczonych z Podatnik-a. Tutaj poczytaj o problemie i samym dziedziczeniu wirtualnym:
https://en.wikipedia.org/wiki/Virtual_inheritance
https://isocpp.org/wiki/faq/multiple-inheritance#mi-diamond

I raczej nie używaj takiej deklaracji typu zwracanego. Czasami się przydaje, ale tutaj jest to zbędne. Po prostu niepotrzebnie komplikuje kod.

Comments

Czytałem o tym i wiem na czym polega problem jak i  w jaki sposób 'vitrual' go rozwiązuje jednak to nie powinno stać na przeszkodzie stworzenia dwóch podklas(?) Podatnik, prawda?

Błąd jest też w drugiej linii i  patrząc teraz chyba na tym polega cały problem
"expected class-name before '{' token " ale część 

class Dzialalnosc: public Osoba_fizyczna, public Firma

wydaje mi się być napisana ok.
 

---

Widocznie Dzialalnosc nie wie czym jest Firma. Zaincludowales wlasciwe pliki?

---

Ok, znalazłem błąd.

Było :

#include "Podatnik.hpp"
#include "Osoba_fizyczna.hpp"
#include "dzialalnosc.hpp"
#include "Firma.hpp"

zamiast 

#include "Podatnik.hpp"
#include "Osoba_fizyczna.hpp"
#include "Firma.hpp"
#include "dzialalnosc.hpp"

Dzięki