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

Dynamiczna alokacja pamięci programowanie obiektowe

VPS Starter Arubacloud
0 głosów
375 wizyt
pytanie zadane 23 kwietnia 2020 w C i C++ przez monterek4444 Początkujący (350 p.)
edycja 23 kwietnia 2020 przez monterek4444

Witam, 

program losuje współczynniki wielomianu po podaniu do konstruktora stopnia wielomianu, następnie ma dodawać wielomiany za pomocą przeciążania operatora "+", operator ">" ma podmieniać współczynniki wielomianu, opeartor "<" ma wyświetlać wielomian,

Problem polega na tym, że program działa "poprawnie" tylko i wyłącznie, gdy nie zwalniam zaalokowanej pamięci destruktorem, gdy zwolnię pamięc program się wysypuje, a konkretnie wartości zapisane w tablicach pokazują przykładowe wartości z pamięci. Nie mam pojęcia jak rozwiązać ten problem, prosiłbym o jakiekolwiek wskazówki.

Z góry dziękuje.

 

 

 

#include <iostream>
#include <ctime>
#include <cstdlib>

using namespace std;
class Wielomian
{
public:
    int* t; int st; int *tab;


    Wielomian(int st);
    Wielomian(int st, int* t);
    Wielomian ();
    Wielomian (const Wielomian &f); // konstruktor kopiujacy
    ~Wielomian();



    //operator =
    Wielomian& operator=(const Wielomian& c)
    {
        st = c.st;
        t = c.t;
        return *this;
    }

    // operator +
    Wielomian operator+(const Wielomian Drugi);


};


//konstruktor bezparametrowy
Wielomian::Wielomian() {
    int st=5;
    this->st = st;
    t = new int[st+1]{};

}




//konstruktor parametrowy losuje wspolczynniki
Wielomian::Wielomian(int st)
{
   this->st=st;
     t = new int[st+1];

    tab[st+1];
    tab = t;

    for (int i=0; i<=st; i++)
    {
        int x = (rand() % 50) + 0;
        t[i]=x;
        tab[i]=x;
    }
}




//konstruktor kopiujacy
Wielomian::Wielomian (const Wielomian &f)
{
    st=f.st;
    t=f.t;
}


//operator "<<"
ostream& operator<<(ostream& pokaz, Wielomian c)
{

    for (int i=(c.st);  i>=0; i--) {
        if (c.t[i] == 0)
            {
                if(i==0)
                    pokaz << "a";
                else
                    pokaz << "a" << i << "x^" << i << " + ";
            }
        else
            {
                if (i == 0)
                    {
                        pokaz << c.t[i];
                    }
                else
                    pokaz << c.t[i] << "x^" << i << " + ";
            }
    }

    return pokaz;


}



//operator ">>"
istream& operator>>(istream& wpisz, Wielomian &c)
{

      for (int i=c.st; i>=0; i--)
      {
          cout <<endl<<"prosze podac wspolczynnik przy potedze "<<i <<"w celu podmiany: ";
          wpisz >>c.t[i];
      }
      return wpisz;
}


//operator "+"
Wielomian Wielomian::operator+(const Wielomian Drugi )
{
    Wielomian wynik = *this;

for (int i=0; i<=wynik.st ; i++)
wynik.t[i] = Drugi.t[i] + wynik.t[i];

return wynik;

}



//destruktor
Wielomian::~Wielomian()
{
    //delete[] t; t=NULL;
    //cout <<endl<<"destruktor" <<endl;
}


int main()
{
srand (time(NULL));

 Wielomian c1(3);
 Wielomian c2(3);

 cout <<c1 <<endl;
 cout <<c2 <<endl;
 cout <<c1+c2 <<endl;
 cin >> c1;
 cout <<c1 <<endl;

    return 0;
}

 

komentarz 23 kwietnia 2020 przez Whistleroosh Maniak (56,900 p.)
No i jaki jest problem?
komentarz 23 kwietnia 2020 przez monterek4444 Początkujący (350 p.)

@monterek4444,

Usunąłem przypadkowo komentarz, już poprawiłem post.

komentarz 23 kwietnia 2020 przez tkz Nałogowiec (42,000 p.)
Na pierwszy rzut oka, to Twój operator dodawania jest bezsensu. Zauważ, że właśnie w nim tworzysz obiekt, który później "znika".

1 odpowiedź

+1 głos
odpowiedź 23 kwietnia 2020 przez j23 Mędrzec (194,920 p.)
wybrane 27 kwietnia 2020 przez monterek4444
 
Najlepsza

Masz źle napisany operator przypisania (równie dobrze mogłeś go w ogóle nie pisać):

Wielomian& operator=(const Wielomian& c)
{
    delete[] t;
    t = new int[c.st + 1];
    st = c.st;
    std::copy(c.t, c.t + c.st + 1, t);
    return *this;
}

Konstruktor kopiujący też popraw, bo jest źle.

komentarz 23 kwietnia 2020 przez monterek4444 Początkujący (350 p.)

@j23, Jednak kompiluje poprawnie, tylko zamiast operatora "<=" dałem "<".

Dzięki bardzo za pomoc!

komentarz 23 kwietnia 2020 przez monterek4444 Początkujący (350 p.)

@j23, "tab" było w celach testowych, zapomniałem usunąć.

 

komentarz 23 kwietnia 2020 przez j23 Mędrzec (194,920 p.)
Jeśli zastosowałeś wszystkie poprawki, to wszystko powinno działać z destruktorem bez problemu.
komentarz 24 kwietnia 2020 przez monterek4444 Początkujący (350 p.)
Działa prawidłowo, dzięki za pomoc.
komentarz 24 kwietnia 2020 przez j23 Mędrzec (194,920 p.)
Zatem zaakceptuj odpowiedź, by dopełnić całości.

Podobne pytania

0 głosów
1 odpowiedź 261 wizyt
pytanie zadane 7 kwietnia 2023 w C i C++ przez Zuzan Początkujący (390 p.)
0 głosów
1 odpowiedź 415 wizyt
pytanie zadane 7 maja 2020 w C i C++ przez Hubertius Bywalec (2,970 p.)
0 głosów
1 odpowiedź 245 wizyt

92,453 zapytań

141,262 odpowiedzi

319,088 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!

...