Dynamiczne tworzenie zmiennej a wektor.

pytanie zadane 4 grudnia 2015 w C i C++ przez Rafajah Bywalec (2,090 p.)

Witajcie.

Od jakiegoś czasu uczę się programować w c++. Mam pewien nurtujący mnie problem, którego za nic nie mogę rozwiązać i potrzebuję wytłumaczenia. Jak to jest, że to działa:

  vector <int> vec;

    int *x = new int;
    *x=0;
    vec.push_back(*x);
    delete x;

Natomiast to już nie:

vector <Postac> postacie;
    Postac *postac = new Postac;
    postacie.push_back(*postac);
    delete postac;

Niby się kompiluje ale program przestaje działać w momencie użycia delete postac;

Wrzuczam jeszcze caly kod gdyby to w klasach leżał problem:

http://wklej.org/hash/c19501402c7/

Z góry dzięki za wyrozumiałość i ewentualne odpowiedzi ;)

@Edit:

Skoro już zdecydowałem się na absorbowanie Waszego czasu to od razu zapytam.

Jaki jest najlepszy sposób aby bez wycieku pamięci tworzyć dynamicznie obiekty, dodawać je do wektora, a następnie usuwać wraz ze zwolnieniem pamięci?

3 odpowiedzi

odpowiedź 4 grudnia 2015 przez adrian17 Ekspert (344,860 p.)

Skoro już zdecydowałem się na absorbowanie Waszego czasu to od razu zapytam.

Jaki jest najlepszy sposób aby bez wycieku pamięci tworzyć dynamicznie obiekty, dodawać je do wektora, a następnie usuwać wraz ze zwolnieniem pamięci?

1. Nie alokować ich dynamicznie.

2. Po co potrzebujesz je alokować dynamicznie? Polimorfizm? Jak nie, to opisz o co chodzi.

3. Jeśli bardzo potrzebujesz je alokować dynamicznie, owiń je w std::shared_ptr.

komentarz 4 grudnia 2015 przez Rafajah Bywalec (2,090 p.)

Po prostu brzydko mówiąc "ujebało" mi się, że trzeba je alokować dynamicznie żeby mieć kontrolę nad pamięcią. Nie wiedziałem, że wektor robi to za mnie.

Dzięki za odpowiedź, poczytam również o shared_ptr gdybym faktycznie musiał kiedyś jednocześnie użyć polimorfizmu i wektorów.

komentarz 4 grudnia 2015 przez adrian17 Ekspert (344,860 p.)

(Są powody na dynamiczną alokację pamięci inne niż polimorfizm, ale zdarzają się raczej rzadko.)

odpowiedź 4 grudnia 2015 przez Patrycjerz Mędrzec (192,320 p.)

Korzystając z std::vector używa się metod zawartych w tej klasie - użycie operatora delete może powodować niezdefiniowane skutki. Przeczytaj sobie: http://cpp0x.pl/kursy/Kurs-STL-C++/Kontener-tablicy-std-vector/119, http://www.cplusplus.com/reference/vector/vector/.

komentarz 4 grudnia 2015 przez Rafajah Bywalec (2,090 p.)

Na cpp0x.pl szukałem info o wektorach zanim napisałem ale nie znalazłem tego linka, dzięki za niego ;)

odpowiedź 5 grudnia 2015 przez criss Mędrzec (172,590 p.)

Nie musisz alokować dynamicznie, żebyś dodane do vectora elementy mógł usuwać z pamięci :P Tym zajmuje się vector (zainteresuj się czym wlaściwie jest i jak działa). Z resztą metoda push_back kopiuje obiekt. Także jeśli stworzysz obiekt, a potem go dodasz do vectora - będziesz miał 2 różne obiekty. Chyba, że stworzysz vector wskaźników, ale powtarzam - to bez sensu dopóki w gre nie wchodzi polimorfia.

Co do drugiego pytania - jeśli już musisz. Najlepszym sposobem jest std::shared_ptr. http://kacperkolodziej.pl/artykuly/programowanie/263-cpp11-wskazniki-unikalne-i-wspoldzielone-unique-ptr-shared-ptr.html

edit: właściwie przepisałem kod Adriana, nie przeczytalem jego odpowiedzi zanim zacząłem pisać swoją.