• 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

Object Storage Arubacloud
0 głosów
430 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,980 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ź 287 wizyt
pytanie zadane 7 kwietnia 2023 w C i C++ przez Zuzan Początkujący (390 p.)
0 głosów
1 odpowiedź 449 wizyt
pytanie zadane 7 maja 2020 w C i C++ przez Hubertius Bywalec (2,970 p.)
0 głosów
1 odpowiedź 258 wizyt

92,655 zapytań

141,543 odpowiedzi

319,959 komentarzy

62,024 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

Kolejna edycja największej imprezy hakerskiej w Polsce, czyli Mega Sekurak Hacking Party odbędzie się już 20 maja 2024r. Z tej okazji mamy dla Was kod: pasjamshp - jeżeli wpiszecie go w koszyku, to wówczas otrzymacie 40% zniżki na bilet w wersji standard!

Więcej informacji na temat imprezy 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!

...