Dotaz: C++ tridy -- nevim, kde mam chybu (easy)

[Bundas]

Zdravim. Je mi 14 let, ucim se programovat. Ted se snazim pochopit tridy v C++ a napsal sem jednoduchou aplikaci.. Bohuzel visual studio tam furt nekde vidi chybu..

Zdrojak:

hlavicka.h -- https://drive.google.com/file/d/0B_1o37MpTnYUTG5USjZiLWdQbTg/edit?usp=sharing

main.cpp -- https://drive.google.com/file/d/0B_1o37MpTnYUN3BkZFlncTc3NUU/edit?usp=sharing

Pomozte mi to opravit prosim

Odpovědi

[:::]

1. chybi ti tam implementace konstruktoru Circle()

2. Co maji delat tyhle radky?:

cin >> c.getX
cin >> c.getY;
cin >> c.getRadius;
cin >> c.getColor;

prepisovat adresu funkce ?

3. Co ma delat tenhle radek?:

c.printValues;

4. Guess: predtim jsi programoval v pascalu ?

[Logik]

Ahoj,

no máš tam těch problémů dost , ale v poho. Snad jen že na učení objektovýho programování jsou lepší jazyky než C++ (např. i ta ukecaná Java), ale naučit se to dá i na C++ (můj případ).

1) V C++ objektu by měla bejt oddělená deklarace objektu (tj. první část hlavičky.h od definice jejich metod. Takhle když ten objekt budeš chtít použít na víc místech, tak tam budou ty metody dvakrát a linker bude řvát. Ale na todle jednoduchý vyzkoušení to zatim tak nechat můžeš.

2) Když vytvoříš kruh bez parametrů, tak je v nějakym nedefinovanym stavu. To neni sice syntaktická chyba, ale není to dobrý zvyk (objekt by měl bejt vždy v pořádku, tj. měl by mít konstruktor se všemi potřebnými parametry).

2) getX a getY jsou funkce. Lze je tedy buďto volat, popř. vzít si jejich adresu, nic jinýho. Ty do nich zapisuješ, to nejde. Nadefinuj si funkce void Circle::setX(int x), načti x do lokální proměnný a pak volej c.setX(promenna).

Ale uplne spravne (když už chceš používat iostream, mě to k srdci nepřirostlo a i když píšu striktně objektově, tak používám getc/scanf) by bylo, kdybys v metodě Circle přetížil operátor >>(&istream, &Circle)

[Bundas]

1) V C++ objektu by měla bejt oddělená deklarace objektu (tj. první část hlavičky.h od definice jejich metod. Takhle když ten objekt budeš chtít použít na víc místech, tak tam budou ty metody dvakrát a linker bude řvát. Ale na todle jednoduchý vyzkoušení to zatim tak nechat můžeš.

Diky za radu. Tohle by me nenapadlo.

2) Když vytvoříš kruh bez parametrů, tak je v nějakym nedefinovanym stavu. To neni sice syntaktická chyba, ale není to dobrý zvyk (objekt by měl bejt vždy v pořádku, tj. měl by mít konstruktor se všemi potřebnými parametry).

Takze k tomu konstruktoru Circle() mam pridat parametry? Jaky? (jsem vazne zacatecnik. Ucim se C++ teprve tyden, takze odpust prosim moji neschopnost )

3) getX a getY jsou funkce. Lze je tedy buďto volat, popř. vzít si jejich adresu, nic jinýho. Ty do nich zapisuješ, to nejde. Nadefinuj si funkce void Circle::setX(int x), načti x do lokální proměnný a pak volej c.setX(promenna).

A mam tam ty gettery nechavat? Nebo staci jenom vytvorit ty funkce? Definice fuknce void Circle::setX(int x) by byla: Circle::setX(int x) {x = x;} ?

kdyz mam volat c.setX(promenna) tak by to v praxi vypadalo nejak takhle?: c.setX(21); nebo c.setX(getX())? Jak by to slo udelat, aby tam byl umoznen vstup uzivatele? (ze by inicializaci promenny x vykonaval uzivatel)

Ale uplne spravne (když už chceš používat iostream, mě to k srdci nepřirostlo a i když píšu striktně objektově, tak používám getc/scanf) by bylo, kdybys v metodě Circle přetížil operátor >>(&istream, &Circle)

Tak daleko sem se jeste nedostal

[Logik]

2) No takový parametry, aby po zavolání konstruktoru cicrle byl kruh definovanej. Tj. můžeš ho inicializovat pomocí x,y,radius, nebo pomocí istreamu, z kterýho načteš hodnoty, dělává se i tzn. copy contruktor, tj. z jiné instance třídy Circle.

3a) No jde o to, jestli má circle nabízet ke čtení z vnějšku své vlastnosti. Jestli ano, nechat, jestli ne, tak smazat.

3b) Circle::setX(int x) {x = x;} ? Nejde, protože lokální definice x ze SetX Ti zastíní tu z třídy. Musíš buď lokální pojmenovat jinak, nebo asi lepší je si zvyknout psát: Circle::setX(int x) {this->x = x;} ?

3c) c.setX(getX())? Je přeci blbina Jednak nemáš definovanou funkci getX, pouze metodu getX třídy Circle, takže by to muselo bejt c.setX(c.getX()), jednak si to zkus přeložit do "lidského jazyka": "Kružnici c . změň X (na hodnotu vlastnosti x kružnice c)". A to asi nechceš.

3d) Inicializaci proměnný x udělá uživatel. Jestli má jít opravdu o inicializaci kružnice, mělo by to bejt v konstruktoru (v tomto případě s parametrem istream). Jestli má jít o změnu existující kružnice, tak buď dopsat další metodu setX s parametrem istream (v jejímž těle pak může bejt stream >> this->x), nebo tady asi rozumější je: int tmp_x; cin >> tmp_x ; c.setX(tmp_x)

Anebo se vykašlat na setter a psát cin >> c.x, ale to je proti principům objektovýho programování. U takhle jednoduchýho objektu a programu se ale neprojeví žádná nevýhoda plynoucí z porušení principů, jen si navykneš "bad habits"

[Jardík]

void Circle::setCentre(int xn, int yn){
        xn = x;
        yn = y;
};

Tohle máš asi přehozený, smyslem bylo asi nastavit x a y. Takto akorát nastavíš novou hodnotu těm argumentům a pak s tim už nic neděláš. Z pohledu volajícího metoda nemá žádný efekt a i kompilátor pravděpodobně nic nevygeneruje.

Zbytek jsem už línej číst, protože mi to hází nějakou kriploidní hlášku, že mám zastaralý prohlížeč a že mám přejít na Google Chrome. Fk you špehující Google, nepřejdu na tvůj prohlížeč.