Operator + pozwalający dodać dwie klasy do siebie.

Question

Witajcie, borykam się z problemem poprawnego wykonania linijki ---- 9 ---- oraz wyświetlenia jej w linijce ---- 10 -----.
Nie mam pojęcia jak inaczej mogę wykonać dodawanie dwóch obiektów tego samego typu (BazaDanych).
Proszę o pomoc, podpowiedzi, cokolwiek :)

 

#include <iostream>
#include <string>
#include <fstream>

using namespace std;
class Osoba
{
private:
    string imie_;
    string nazwisko_;
    long long int pesel_;
public:
    //konstruktor domyslny
    Osoba()
    {
        imie_ = "Czlowiek";
        nazwisko_ = "Domyslny";
        pesel_ = 0;
    }


    //konstruktor parametryczny
    Osoba(string imie_, string nazwisko_, long int pesel_)
    {
        this->imie_ = imie_;
        this->nazwisko_ = nazwisko_;
        this->pesel_ = pesel_;
    }
    //metody dostepowe
    string &imie() {return imie_;}
    string &nazwisko() {return nazwisko_;}
    long long int &pesel() {return pesel_;}

    //operator <<
    friend ostream& operator<<(ostream& os, Osoba& o);
    //operator >>
    friend istream& operator>>(istream& os, Osoba& o);
    //operator ==
    friend bool operator==(const Osoba& a, const Osoba& b);

};
    ostream& operator<<(ostream& os, Osoba& o)
    {
        os << o.imie_ << " " << o.nazwisko_ << " " << o.pesel_ << endl;
        return os;
    }

    istream& operator>>(istream& is, Osoba& o)
    {
        is >> o.imie_ >> o.nazwisko_ >> o.pesel_;
        return is;
    }

    bool operator==(const Osoba& a, const Osoba& b)
    {
        return a.pesel_== b.pesel_;
    }
class BazaDanych
{
private:
    //tablica obiektow typu Osoba w pamieci dynamicznej
    int liczbaOsob = 0;
    Osoba* osoby_ = new Osoba[liczbaOsob];
public:
    //konstruktor domyslny
    BazaDanych()
    {
        if(!osoby_)
        osoby_ = nullptr;
        liczbaOsob = 0;
    }

    //konstruktor kopiujacy
    BazaDanych(BazaDanych& b)
    {
        liczbaOsob = b.liczbaOsob;
        osoby_ = new Osoba [liczbaOsob];
            for(int i = 0; i < liczbaOsob; ++i)
            {
                osoby_[i] = b.osoby_[i];
            }
    }

    //destruktor
    ~BazaDanych(){};

    //operator indeksowania []

    //operator <<
    friend ostream& operator<<(ostream& os, BazaDanych& b);

    //operator dodajacy osobe do bazy danych
    BazaDanych& operator+(Osoba & o)
    {
        Osoba* temp = new Osoba[liczbaOsob + 1];
        for (int i = 0; i < liczbaOsob; ++i)
        {
            temp[i] = osoby_[i];
        }

    temp[liczbaOsob] = o;
    delete [] osoby_;
    osoby_ = new Osoba[liczbaOsob + 1];

        for (int i = 0; i < liczbaOsob+ 1; ++i)
        {
            osoby_[i] = temp[i];
        }
    delete [] temp;
    liczbaOsob++;
    return *this;
    }

    //operator przypisania kupujacego
    BazaDanych& operator=(BazaDanych& bb)
    {
        if(this != &bb)
        {
        delete [] osoby_;
        liczbaOsob = bb.liczbaOsob;
        osoby_ = new Osoba[liczbaOsob];
            for (int i = 0; i < liczbaOsob; ++i)
            {
                osoby_[i] = osoby_[i];
            }
        }
        return *this;
    }
    //operator + dodajacy dwie bazy
    friend BazaDanych& operator+(BazaDanych& b1, BazaDanych& b2);
};
    ostream& operator<<(ostream& os, BazaDanych& b)
    {
        for(int i = 0; i < b.liczbaOsob; ++i)
        {
            os << b.osoby_[i];
        }
        return os;
    }

    BazaDanych& operator+(BazaDanych& b1,BazaDanych& b2)
    {
        BazaDanych temp(b1);
            for(int i = 0; i < b2.liczbaOsob; ++i)
            {
                temp = temp + b2.osoby_[i];
            }
            return temp;
    }
int main()
{
  Osoba o1;
  cout << "---- 1 ----" << endl;
  cout << o1 << endl;

  Osoba o2("Jan", "Kowalski", 11111111111);
  cout << "---- 2 ----" << endl;
  cout << o2 << endl;

  cout << "---- 3 ----" << endl;
  cout << boolalpha << (o1 == o2) << endl;

  BazaDanych db1;
  cout << "---- 4 ----" << endl;
  cout << db1;

  db1 = db1 + o1;
  cout << "---- 5 ----" << endl;
  cout << db1;

  ifstream plik_we("zadanie7.txt");
  if(!plik_we)
  {
    cout << "---- 6 ----" << endl;
    return 1;
  }

  while(plik_we >> o2)
  {
      db1 = db1 + o2;
  }
  cout << "---- 7 ----" << endl;
  cout << db1;

  {
      BazaDanych db2(db1);
      cout << "---- 8 ----" << endl;
      cout << db2;
  }
  cout << "---- 9 ----" << endl;
  cout << db1;
  BazaDanych db3;
  db3 = db1 + db1;

  cout << "---- 10 ----" << endl;
  cout << db3;
    /*
  for(int i=0; i<db3.liczbaOsob(); ++i)
  {
      cout << db3[i] << endl;
  }
  /*
  cout << "---- 11 ----" << endl;

  db1 = db1 - o1;
  cout << "---- 12 ----" << endl;
  cout << db1;

  cout << "---- 13 ----" << endl;
  cout << db3;
  cout << "---- 14 ----" << endl;
    */;
  return 0;
}

 

Comments

Dodając 2 bazy ni rób tego po 1 elemencie - n razy allokujesz tablicę o 1 większa i dodajesz element. Od razu zaalokuj sumę dlugości baz i skopiuj dane.

---

@haitoke, czyżby kolejny uczestnik kursu iście przestarzałego i słabego dydaktyczno-merytorycznie kursu c++ pana Zelenta?

naprawdę odradzam.

---

Zadanie z laboratoriów na jednej z politechnik 

 

---

a to spoczko

Answer 1

Operator + powinien zwracać nowy obiekt, a u Ciebie operator dodający osobę zwraca obiekt this. Zachowuje się tak, jakby był operatorem +=.

Operator dodający dwie bazy zwraca referencję na obiekt lokalny - to błąd. Obiekt lokalny poza funkcją nie istnieje. Zwracaj przez wartość.

Powinieneś zadbać o const correctness. Czyli metody, które nie zmieniają stanu obiektu, powinny być const, parametry przekazywane przez referencję/wskaźnik także powinny być const, jeśli nie zamierzasz zmieniać ich stanu.

 

BTW, skoro masz zdefiniowany ctor kopiujący, to operator kopiujący możesz zredukować do:

BazaDanych& operator= (BazaDanych bb)
{
    std::swap(osoby_, bb.osoby_);
    std::swap(liczbaOsob, bb.liczbaOsob);
    return *this;
}

Zresztą masz w nim błąd.

Destruktor BazaDanych powinien usunąć osoby_.

Użyj std::vector zamiast bawić się w tablice dynamiczne.

Answer 2

Online jakoś dział 9 i 10 działa ale nie wiem czy poprawnie. Jak coś poprawie to z edytuję. Ale rzeczywiście odpowiedz @J23 powinna być najlepsza.

edit :: Dodam troszeczkę poprawiony kod