Manipulace se zapouzdřenými třídami – C / C++ – Fórum – Programujte.com

Dobrý den,

momentálně řeším problém týkající se práce s třídami. Pravděpodobně jsem při studiu OOP něco důležitého přehlédl, protože dlouhodobě bojuji s tím, jak mám z třídy vracet instanci privátního člena.

class Obal {
	private:
		Prvek prvek;

	public:
		Prvek getPrvek1() const { // 1
			return this->prvek;
		}

		Prvek &getPrvek2() const {
			return this->prvek;
		}

		Prvek *getPrvek3() const {
			return &this->prvek;
		}
}

Pokud bych chtěl z instance třídy Obal dostat vnitřní třídu Prvek, kterou z metod bych měl použít? V prvním případě se Prvek "překopíruje", takže bych ve výsledku neměnil vnitřní instanci.

Ve druhém případě pravděpodobně bude kompilátor skřehotat kvůli převodu z const Prvek na &Prvek. 

Třetí případ dopadne stejně jako druhý. Problém s převodem const Prvek na *Prvek.

Ve třetím případě by se chyba odstranila smazáním const za jménem metody. To mi ale znemožní takovou metodu volat například v kopírovacím konstruktoru:

Obal::Obal( const Obal &original ) {
	this->prvek = *original.getPrvek3();
}

Takže na jednu stranu potřebuju, aby metoda byla const - pro použití v kopírovacím konstruktoru, ale také, aby nebyla const - a mohl jsem volat nekonstantní metody třídy Prvek.

Jde o špatný návrh tříd? Dědění se mi v tomto případě zdá jako špatný nápad. Obal není speciálním případem Prvku, jen využívá jeho metod. 

Jak se řeší podobné případy? 

Děkuji.

4 moznost:   :

const Prvek &getPrvek2() const {
	return this->prvek;
}

A je možné na takto vrácený Prvek volat nekonstantní metody?

#4 vitamin

Nie, ak chces volat aj nekonstantne metody tak musis urobyt kopiu premennej

To neumožní editovat vnitřní instanci "zapouzdřené" třídy.

vytvorit nekonstantny geter

Jde vytvořit přetíženou metodu na základě toho, zda je const? Jednu pro kopírovací konstruktor a druhou pro získání nekonstantní reference pro použití "zvenčí".

pripadne dat premennu verejnu a vykaslat sa na getre

To vypadá jako možné řešení. Nicméně moje otázka stále trvá, není to, že řeším podobné problémy způsobeno špatným návrhem tříd a jejich vzájemné spolupráce?

#5 Lorin

Metoda moze byt pretazene na zaklade const:

#include <iostream>
#include <vector>

struct C{
	void fnc(){std::cout << "mutable\n";}
	void fnc()const{std::cout << "const\n";}
};

int main(){
	C c;

	c.fnc();
	reinterpret_cast<const C&>(c).fnc();
	return 0;
}

//vystup je:
mutable
const

Ak toto všetko riešiš iba kvôli kopírovaciemu konštruktoru tak to môžeš jednoducho napísať takto:

Obal::Obal( const Obal &original ) {
	this->prvek = original.prvek;
}

a nepotrebuješ žiadne get metódy.

Ano v případě tříd, které mají být manipulovány i "zvenčí" bude nejjednodušší možností jejich přesun do "public" sekce.

viz. akceptovaná odpověď od #4 vitamin

pripadne dat premennu verejnu a vykaslat sa na getre