Programiranje u C++-u - pitanja i odgovori

Ja volim koristit std c++ strukture, npr std::set bi to automatski posortirao unesene po velicini, a ja bi kod inserta samo gledao jel neparni. A bas jos i boost ukljucit za ovo je mozda mali overkill :) ...iako u stvarnom zivotu bi ga koristio.

 Cisto ako nekom treba, rjesio sam problem, otisao sam pod Project -> Options -> C++ Compiler i stavio "Use Classic compiler" na True.

@Hazar

Samo kroz praksu uči. Kao i matematiku. 

Proučavaj tuđe kodove.

 Kad ljude uče principe objektno orijentiranog programiranja, vrlo često im se zaboravi reći da postoje tkz. oblikovni obrasci. To su jednostavno "načini" kako se koristi OOP za rješavanje nekih problema, pa kroz njihovo učenje, i implementiranje u kodu, barem stekneš neki osjećaj kako koristiti OOP.

Ovo je klasična knjiga:
https://en.wikipedia.org/wiki/Design_Patterns

Naravno, postoje jasne kritike na njih (možeš ih overuseati, neke stvari se rješavati drugačije, itd.), ali zasada se ne zamaraj tim, već kreni čisto od njih, konkrentnije od klasničnog primjera uređivača teksta (koji se analizira već u prvom poglavlju te knjige). 

Pogledaj zadatak 2. ovdje http://www.zemris.fer.hr/~ssegvic/ooup/ooup3lab.shtml (zadatak 1. zanemari ako te ne zanima C dio), i ovo http://www.zemris.fer.hr/~ssegvic/ooup/ooup4lab.shtml

Ovo je tema o C++-u, pa pretpostavljam da to radiš u C++-u, onda se ne bi bilo loše upoznati kako se oop u praksi implementira (da razumijes sto su virtualni pozivi i sl), pa onda pogledaj i ostale lab vjezbe i gradivo: http://www.zemris.fer.hr/~ssegvic/ooup/

Hvala ljudi! Puno ste pomogli.

Koliko vidim iz koda, nije klasa A nadklasa klase B, već obrnuto. Dole si deklarirao klasu B, a gore deklariraš klasu A koja nasljeđuje B. U A si dodao član x, a B nema toga člana.

U primjeru bi ti trebalo biti obrnuto:

class A {
    protected:
    int x;
};

class B : public A {
    void foo() {
        A::x = 5;
    }
};

Dakle, kad kažeš da A može imati N objekata klase B preko templatea, jel misliš da je cijela klasa parametrizirana, tj. da imaštemplate ispred deklaracije klase:

template <int N>
class A {

// ... implementacija

};

U tom slučaju, ne možeš imati da B nasljeđuje A jer A je parametriziran (dakle konkretna instanca ovisi o vrijednosti N), tipa mogao bi imati
class B : public A<5>, ili bi ti i B trebala biti parametrizirana

template <int N>

class B : public A<N> {

///

}

Kod templatea, svaka instanca templatea je zasebna klasa, A<5> bi bilo razlicito od A<4>, isto kao sto je std::vector<std::string> razlicit tip od std::vector<int>.

E sad, meni nije jasno što želiš postići. Dakle želiš da A sadrži N objekata od B, i da B nasljeđuje A. To nije moguće jer onda bi imao B koji je A pa ima N B objekata, koji je svaki A pa svaki ima još po N objekata, i tako beskonačno.

To je u osnovi isti razlog zašto klasa ne može sadržavati član čiji je tip upravo ta klasa jer nema logički smisla. No ono što klasa može sadržavati je referencu/pokazivač na tu klasu. Tipa možeš imati:

class A {
A* drugiClan;
};

Npr. to vidiš kad implementiraš povezanu listu (onda Nodeovi imaju pokazivače na sljedeći Node, itd.). Kod tebe bi mogo imati tipa (samo bi prije toga morao imati deklaraciju: samo class B; ):

class A {

protected:
std::vector<B> drugi;

//..
}

class B : public A {
// implementacija
};

Sad bi trebali vidjeti zašto ti želiš imati gornju stvar, tj. da ti objekti "sadrže sami sebe" (samo ćeš to "sadržavanje" morati implementirati preko pokazivača). Ne znam jel pokušavaš iskoristiti oblikovni obrazac kompozit. U tom slučaju, tipa želio bi imati jednu klasu koja bi mogla predstavljati i jednu ledicu, i red ledica i sl, a da onaj tko primi takav objekt ne mora znati detalje toga.

Dakle, LED predstavlja jedan LED čip? Koliko vidim i ta klasa je parametrizirana preko templatea i to preko tipa. Ne vidim nigdje u kodu da taj tip koristiš.

I onda kad ProgLED nasljeđuje, onda treba nasljeđivati konkrentnu instancu toga templatea, dakle ne može biti class ProgLED : public LED, već mora biti ProgLED : public LED<nekiTip>.  Možda najbolje da to izbaciš jer ti ne treba (jedino ako misliš nešto kasnije s tim raditi), dakle da LED samo ostane klasa, a ne template.

E onda može ProgLED nasljeđivati LED. Sad je samo pitanje zašto želiš da ti ProgLED nasljeđuje LED.  Koliko sam skužio, iz ovog odsječka ti želiš polimorfizam, tipa da imaš LED led, i onda taj led može biti jedan led čip, ili linija ledica, i kad pozovše on, da se upali ili ta jedna ledica, ili da se upali jedan red. U tom slučaju, on će ti imati različitu implementaciju u ProgLED istanci u odnosu na LED instancu, i onda ta funkcija mora biti virtualna.

Dakle u LED moraš staviti virtual void on(), isto i u ProgLED (ili void on() override;). Slično i za ostale metode.

 Mislim da hoće, možda može kompajler nešto optimizirati pri tome, možda se možeš igrati s nekim parametrima pri kompilaciji. Sad je pitanje hoće li ti to biti problem.

Ako bude problem, onda stavi da ProgLED ne bude parametriziran po broju ledica, već da u sebi ima dinamičko polje LED instanci (tipa preko std::vector, ili preko običnog polja, ali moraš paziti da ga u destroktoru oslobodiš), a da broj ledica prima preko konstruktora.

 Mislim da si pomiješao parent i child klasu. Dakle tebi je parent klara LED, a ProgLED nju nasljeđuje. ProgLED ima pristup svemu od LED, ali ne vrijedi obrnuto. Ti sad iz nadklase želiš pristupiti članu svoje podklase, dakle svoga djeteta.

Nije mi uopće jasno kako se ovo što si napisao može kompajlati: color[this->getRIdx()] = r; Dakle ovo je kontekstu metode iz objekta LED, a this je onad pointer na konkrentu instancu LED-a, a LED u sebi nema metodu getRIdx(). Ne znam jesi probao to komapjlati, ili ti samo vs code ne baca errore, ali moguće da ti ne baca errore jer si prije toga stavio ovu template deklaraciju, pa onda ne zna što će s tim.

 Trebalo bi malo promijeniti dizajn, zato je bitno što želiš s tim nasljeđivanjem i klasama konkretno postići. Ako je su indexi stalni, i jednaki za svaki led, onda ih ima smisla staviti u klasu LED, i to kao statičke članova (da nema svaki led objekt kopiju njih, već da je na razini klase, tako štediš i nešto prostora).

Novi kod: https://github.com/silvio3105/ProgLED/tree/dev

Stavio sam class LED: public ProgLED<T> no sad opet moram stavit template(u .cpp) za sve metode iz LED klase.