Konstruktory - czy to prawidłowy przykład ich zastosowania?

Question

Witam. Uczę się konstruktorów. Czy programik który napisałem prawidłowo wykorzystuje ich możliwości? 

plik.cpp

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

using namespace std;

void Konstruktor::wypisz()
{
    int aktualna_liczba = 1;
    int pomocnicza;

    for (int i=1; i<=ile; i++)
    {
        pomocnicza = i*i;
        if (i<10)
        {
            cout<<" "<<i<<" | "<<pomocnicza<<endl;
        }
        else
        {
            cout<<i<<" | "<<pomocnicza<<endl;
        }
    }
}
void Konstruktor::wpisz()
{
    cout<<"Ile kolejnych liczb? ";
    cin>>ile;
}

Konstruktor::Konstruktor(int a)
{
    ile = a;

    wpisz();
    wypisz();
}

Konstruktor::~Konstruktor()
{
    cout<<"Pamiec zostala zwolniona";
}

 

plik.h

#include <iostream>

using namespace std;

class Konstruktor
{
    int ile;

    public:

    Konstruktor(int ile = 1);
    ~Konstruktor();

    void wypisz();
    void wpisz();
};

 

plik main.cpp

#include <iostream>
#include "konstruktory/konstruktory.h"

using namespace std;

int main()
{
    Konstruktor();

    return 0;
}

 

Comments

To jest programik? To nic nie robi,wiec nie ma co sprawdzać. Nie rozumiem jedynie tego:

int main()
{
    Konstruktor();
 
    return 0;
}

 

---

Jak to nic nie robi? Wywołuje w sobie dwie metody zawarte w klasie o nazwie Konstruktor, które pobierają dane które są przeliczane i wypisywane na ekranie..

Answer 1

Hej :)

Trochę myląca nazwa klasy,ale postaram się wyjaśnić jakie tutaj popełniono błędy.

Pierwszą rzecz to najlepiej jest napisać private: int ile; dla innego programisty ułatwienie(wiem,że to z automatu idzie)

W pliku cpp postaraj się rozpisywać funkcje od Konstruktora do innych funkcji , dzięki temu już można będzie się zorientować wstępnie co dana klasa będzie robiła co wykorzystuje.

Po co wrzucać funkcje które stworzyłeś w obiekcie do konstruktora? Zostawić tylko przypisanie zmiennych. I później:

Konstruktor mojKonstruktorek; // domyślnie ile będzie 1;

mojKonstruktorek.wpisz();

mojKonstruktorek.wypisz();