[C++] Problemy z dynamiczną alokacją tablicy obiektów

[Pneumokok]

Witam!
Otóż mam problem z dynamiczną alokacją tablicy obiektów w konstruktorze. Przedstawię kody:

Main.cpp:

Ukryta treść:    

Kod: Zaznacz cały

/* 
 * File:   main.cpp
 * Author: Abc
 *
 * Created on 29 kwiecień 2013, 10:17
 */

#include <cstdlib>
#include "Pociag.h"
using namespace std;

/*
 * 
 */
int main(int argc, char** argv) {
    Pociag pociag;
    pociag.wypiszInfo();
    return 0;
}

Pociag.h:

Ukryta treść:    

Kod: Zaznacz cały

/* 
 * File:   Pociag.h
 * Author: Abc
 *
 * Created on 29 kwiecień 2013, 10:20
 */

#ifndef POCIAG_H
#define	POCIAG_H
#include<iostream>
#include<string>
#include "Wagon.h"
class Pociag {
public:
    Pociag(int liczbaWagonow=5, std::string nazwaPociagu="Wacek", Wagon* wagony=NULL);
    Pociag(const Pociag& orig);
    virtual ~Pociag();
    void wypiszInfo();
private:
    int liczbaWagonow;
    Wagon* wagony;
    std::string nazwaPociagu;
};

#endif	/* POCIAG_H */

Pociag.cpp:

Ukryta treść:    

Kod: Zaznacz cały

/* 
 * File:   Pociag.cpp
 * Author: Abc
 * 
 * Created on 29 kwiecień 2013, 10:20
 */

#include "Pociag.h"

Pociag::Pociag(int liczbaWagonow, std::string nazwaPociagu, Wagon* wagony){
    this->liczbaWagonow=liczbaWagonow;
    this->nazwaPociagu=nazwaPociagu;
    this->wagony=wagony;
    wagony=new Wagon[liczbaWagonow];
}

Pociag::Pociag(const Pociag& orig) {
}

Pociag::~Pociag() {
    delete[]wagony;
    wagony=NULL;
    std::cout<<"Zdealokowano!"<<std::endl;
}

void Pociag::wypiszInfo(){
    std::cout<<"Pociag nazywa sie: "<<nazwaPociagu<<std::endl<<std::endl;
    for (int i=0; i<liczbaWagonow; ++i){
        std::cout<<"W wagonie numer "<<i<<" jest "<<
                wagony[i].getLiczbaMejsc()<<" miejsc"<<std::endl;
    }
}

Wagon.h:

Ukryta treść:    

Kod: Zaznacz cały

/* 
 * File:   Wagon.h
 * Author: Abc
 *
 * Created on 29 kwiecień 2013, 10:25
 */

#ifndef WAGON_H
#define	WAGON_H

class Wagon {
public:
    Wagon(int liczbaMiejsc=50);
    Wagon(const Wagon& orig);
    virtual ~Wagon();
    int getLiczbaMejsc();
private:
    int liczbaMiejsc;
    

};

#endif	/* WAGON_H */

Wagon.cpp:

Ukryta treść:    

Kod: Zaznacz cały

/* 
 * File:   Wagon.cpp
 * Author: Abc
 * 
 * Created on 29 kwiecień 2013, 10:25
 */

#include "Wagon.h"
#include <iostream>

Wagon::Wagon(int liczbaMiejsc) {
    this->liczbaMiejsc=liczbaMiejsc;
}

Wagon::Wagon(const Wagon& orig) {
    liczbaMiejsc=orig.liczbaMiejsc;
}

Wagon::~Wagon() {
    liczbaMiejsc=0;
    std::cout<<"Wyzerowano!"<<std::endl;
}

int Wagon::getLiczbaMejsc(){
    return (this->liczbaMiejsc);
}

Debugger narzędzia Netbeans IDE 7.2.1 (kompilator MinGW) twierdzi, że wyciek pamięci następuje w funkcji getLiczbaMiejsc() w pliku Wagon.cpp, a ta funkcja jest wywoływana w funkcji wypiszInfo() w pliku Pociag.cpp. Przy czym.... No jest alokacja pamięci, jest inicjacja liczby miejsc w wagonach dzięki konstruktorowi domniemanemu, a operator new[] zachowuje się tak jakby nie zwracał adresu zaalokowanej dla tablicy pamięci. W obecnej wersji kodu np wskaźnik ma cały czas wartość zero (NULL). Nie mam pojęcia, zdaje mi się że operatora new[] używam właściciwe....

Z góry dziękuję za okazaną pomoc.

Pozdrawiam,
Pneumokok.

[Afish]

Kod: Zaznacz cały

this->wagony=wagony;
wagony=new Wagon[liczbaWagonow];

Trochę nielogiczny fragment kodu - wagony to argument konstruktora, domyślnie jest nullem, ale może być wskaźnikiem na tablicę wagonów lub wagon bądź jego potomka. Przypisujesz ten argument do pola klasy o tej samej nazwie, a linijkę niżej tworzysz nową tablicę. Co powinieneś zrobić, jeżeli ten argument nie jest nullem? Ponadto zauważ, że tworzysz tablicę i przypisujesz jej adres argumentowi funkcji, więc po zakończeniu konstruktora tej tablicy już nie zwolnisz (bo nie masz wskaźnika). Poprawniej byłoby
this->wagony=new Wagon[liczbaWagonow];
ale ciągle jest to odrobinkę nielogiczne.

[Pneumokok]

Ja pierrniczę, wiem już co źle zrobiłem Faktycznie - jeśli argument funkcji nazywa się tak samo jak pole klasy istotny jest własnie ten ,,this".
Z przekazaniem wskaźnika, domyślnie NULLa, chodziło mi o to by konstruktor domniemany inicjalizował wszystkie pola klasy z otrzymanej listy argumentów. Czy to konieczne? Szczerze mówiąc nie wiem. Natomiast gdy zmieniłem konstruktor domniemany na:

Kod: Zaznacz cały

Pociag(int liczbaWagonow=5, std::string nazwaPociagu="Wacek");

Kod: Zaznacz cały

Pociag::Pociag(int liczbaWagonow, std::string nazwaPociagu){
    (...)
}

W pliku main.cpp dostaję błąd: main.cpp:16: undefined reference to `Pociag::Pociag(int, std::string, Wagon*)'.
Czyli tak jakby faktycznie standard wymagał, bym w konstruktorze domniemanym inicjalizował każde z pól klasy ,,z zewnątrz", czyli z argumentów które dostaje konstruktor. Natomiast w Symfonii Grębosz pisze:

Kostruktorem, który można wywołać bez żadnych argumentów jest na przykład konstruktor ze wszystkimi argumentami domniemanymi

Z czego wnioskuję, że chodzi o argumenty domniemane konstruktora, a nie pola klasy, gdyż konstruktor może 1 argument przyjmować a pól klasy może być wiele.
Dość dziwne to, wychodzi na to że autor tutaj chyba się myli, o ile dobrze odczytuję jego słowa**.

Co powinieneś zrobić, jeżeli ten argument nie jest nullem?

Fakt, zgubię wtedy to co przekazałem Dobrym rozwiązaniem jest alokacja tylko wtedy, gdy (!wagony) ?


**
No i chyba jednak nie, dopisuje sobie zmienne w klasie Pociąg i kompilator nie zgłasza błędu o niezdefiniowanej referencji. Tak jakby przy tamtym błędzie zapisał gdzies sobie, że miał konstruktor z 3 argumentami i nie chciał z takowego zrezygnować.

[Afish]

Z przekazaniem wskaźnika, domyślnie NULLa, chodziło mi o to by konstruktor domniemany inicjalizował wszystkie pola klasy z otrzymanej listy argumentów. Czy to konieczne?

Nie, nie trzeba tego robić jawnie (niejawnie pola i tak zostaną zainicjalizowane).

Natomiast gdy zmieniłem konstruktor domniemany na:

Kod: Zaznacz cały

Pociag(int liczbaWagonow=5, std::string nazwaPociagu="Wacek");

Kod: Zaznacz cały

Pociag::Pociag(int liczbaWagonow, std::string nazwaPociagu){
    (...)
}

W pliku main.cpp dostaję błąd: main.cpp:16: undefined reference to `Pociag::Pociag(int, std::string, Wagon*)'.

Błąd najprawdopodobniej wynika z braku rekompilacji pliku Main.cpp. Dla pewności przebuduj cały projekt, gdyż usunięcie jednego argumentu (o ile usunąłeś go zarówno z pliku cpp jak i z pliku h, a wygląda na to, że tak właśnie zrobiłeś) błędu nie powinno spowodować.

Co powinieneś zrobić, jeżeli ten argument nie jest nullem?

Fakt, zgubię wtedy to co przekazałem Dobrym rozwiązaniem jest alokacja tylko wtedy, gdy (!wagony) ?

Jest to rozwiązanie, ale czy dobre? Kto wtedy powinien być odpowiedzialny za zwolnienie pamięci? No i co wtedy oznaczałby argument liczbaWagonow? Najlepiej nie kombinować, tylko przyjąć jedno rozwiązanie i się go trzymać - albo wywołujący konstruktor tworzy wagony i później je zwalnia, albo pociąg sam tworzy wagony i sam zajmuje się sprzątaniem.