[C++] Warning che non capisco ...

[trallallero]

Perchè questa linea mi da un warning ?

	LLUnit* pUnit = m_UnitHandler->GetEntity("AUDIO");

il warning:

src/LLEngine.cpp:345: warning: invalid conversion from ‘LLBaseEntity*’ to ‘LLUnit*’

La funzione è così:

	LLBaseEntity* GetEntity (string sName) { return SingleTone->GetEntityBase(SingleTone->GetEntityIdBase(sName)); }

e LLUnit deriva da LLBaseEntity:

class LLUnit : public LLBaseEntity

Vabbè che è venerdì e son fuso ma mi sfugge qualcosa

[wingman87]

Se LLUnit deriva da LLBaseEntity devi fare un cast per fare quell'assegnamento (sempre che sia lecito).

[trallallero]

Originariamente inviato da wingman87

Se LLUnit deriva da LLBaseEntity devi fare un cast per fare quell'assegnamento (sempre che sia lecito).

Ma perchè, scusa ? io vorrei proprio evitare un reinterpret_cast che bello non è

[wingman87]

Purtroppo di C++ non so niente, quindi non so dirti quale tipo di cast è necessario, però così mi sembra normale il warning, ti sta dicendo solo che se metti dei frutti in un cesto di mele devi assicurarti che i frutti siano proprio mele.

Se mi sto sbagliando scusami, non voglio sembrare saccente, provo solo a dare una mano.

[trallallero]

Originariamente inviato da wingman87

Purtroppo di C++ non so niente, quindi non so dirti quale tipo di cast è necessario, però così mi sembra normale il warning, ti sta dicendo solo che se metti dei frutti in un cesto di mele devi assicurarti che i frutti siano proprio mele.

Se mi sto sbagliando scusami, non voglio sembrare saccente, provo solo a dare una mano.

Figurati, a quest'ora di venerdì anche i consigli di un panettiere potrebbero servire

grazie

[cionci]

Il cast implicito da LLBaseEntity * a LLUnit * non è lecito, devi farlo esplicito. E' illecito perché non hai alcuna garanzia che il puntatore ritornato sia un puntatore a LLUnit.
Al contrario da LLUnit * a LLBaseEntity * è lecito.

[trallallero]

Originariamente inviato da cionci

Il cast implicito da LLBaseEntity * a LLUnit * non è lecito, devi farlo esplicito. E' illecito perché non hai alcuna garanzia che il puntatore ritornato sia un puntatore a LLUnit.
Al contrario da LLUnit * a LLBaseEntity * è lecito.

Mi hanno rotto le palle (amichevolmente) perchè ho usato qualche reinterpret_cast nel progetto.

Secondo te è "sbagliata" (diciamo criticabile) un'istruzione del genere ?

LLUnit* pUnit = reinterpret_cast<LLUnit*>(m_UnitHandler->GetEntity("AUDIO"));

Sto usando m_UnitHandler che è di tipo LLUnitHandler quindi so per certo che contiene LLUnit e basta.
LLUnitHandler deriva da LLBaseHandler che contiene praticamente tutte le funzioni di handling delle entità tranne un paio di funzioni astratte che vengono implementate nei vari handlers derivati.

Boh, secondo me va bene ... ok, reinterpret_cast è una forzatura ma so quello che faccio, non forzo un puntatatore ad una classe a caso.

Che poi LLUnit è astratta e chiamando il suo metodo "pure virtual" ExecuteLogic, sfrutto la potenza del polimorfismo.
Per me è elegante.

[cionci]

Usa un dynamic_cast

[trallallero]

Originariamente inviato da cionci

Usa un dynamic_cast

Mai usato ... si vede che vengo dal C, vè ?

ok grazie, leggerò qualcosa sui cast.

[cionci]

Originariamente inviato da trallallero

Mai usato ... si vede che vengo dal C, vè ?

ok grazie, leggerò qualcosa sui cast.

dynamic_cast si usa proprio quando devi fare un cast da una classe base ad una derivata da questa.
http://www.cplusplus.com/doc/tutorial/typecasting.html

Attenzione comunque alla nota scritta lì. E' da verificare nel tuo caso.

[trallallero]

Originariamente inviato da cionci

dynamic_cast si usa proprio quando devi fare un cast da una classe base ad una derivata da questa.
http://www.cplusplus.com/doc/tutorial/typecasting.html

Attenzione comunque alla nota scritta lì. E' da verificare nel tuo caso.

Di nuovo vedo solo ora che hai scritto in questo 3d (perchè ci stavo per scrivere io).
Ma non dovrebbe essere in grassetto il link al 3d quando qualcuno scrive qualcosa di nuovo ?

Vabbè, grazie del link.

Mi da errore in questo caso, cerco di castare da primo a secondo:

‘void (class LLPanelSEM80_90::*)(class LLMsg_t&, class LLMessage*, struct LL_ACTION_SIGNAL&)’
‘void (class LLPanel::*)        (class LLMsg_t&, class LLMessage*, struct LL_ACTION_SIGNAL&)’

Ma non capisco perchè, LLPanelSEM80_90 è una derivata di LLPanel

Errore:

src/Panels/LLPanelSEM80_90.cpp:40: error: cannot dynamic_cast ‘&LLPanelSEM80_90::Button_Power’ (of type ‘void (class LLPanelSEM80_90::*)(class LLMsg_t&, class LLMessage*, struct LL_ACTION_SIGNAL&)’) to type ‘void (class LLPanel::*)(class LLMsg_t&, class LLMessage*, struct LL_ACTION_SIGNAL&)’ (target is not pointer or reference)

Ma non è vero perchè il target è un array di puntatori a metodi della classe LLPanelSEM80_90:

typedef void (LLPanel::*PanelLogicFunctions)(LOGIC_FUNCTIONS_PARAMS);
...
PanelLogicFunctions  m_Functions[SEM80_90_DEV_FUNCTIONS]; //!< Panels' devices' functions
...
m_Functions[0] = dynamic_cast<PanelLogicFunctions>(&LLPanelSEM80_90::Button_Power);

[cionci]

Scusa, ma che c'entra il cast di un puntatore a funzione ? Questo non è un cast da un tipo derivato alla classe base. Quei due tipi non sono derivati l'uno dall'altro.

Sinceramente con la programmazione ad oggetti non mi sono trovato molto spesso ad operare con puntatori a funzioni. Probabilmente li puoi evitare.

[trallallero]

Originariamente inviato da cionci

Scusa, ma che c'entra il cast di un puntatore a funzione ? Questo non è un cast da un tipo derivato alla classe base. Quei due tipi non sono derivati l'uno dall'altro.

Son fuso ... fortuna che dopo il progetto mi faccio 2 settimane di vacanza.

Originariamente inviato da cionci

Sinceramente con la programmazione ad oggetti non mi sono trovato molto spesso ad operare con puntatori a funzioni. Probabilmente li puoi evitare.

Lo so, ma io vengo dal C e questo è stato il mio primo progetto interamente in C++ dove ho dovuto sia implemantare che fare tutto il design.
E mi ricordo anche che mi hai suggerito di usare i pattern invece dei puntatori a metodo (ho ancora il 3d per il futuro ) ma avevo poco tempo ed ho preferito usare qualcosa che conosco bene.

Comunque non è male come design: ho matrici di pulsanti, lo stesso prototipo di funzione (tipo OnClick) per ogni pulsante e quindi una "matrice" (in realtà array) di puntatori a metodi, uno per ogni pulsante.

edit: scusa ma è da molto che volevo chiedertelo: chi è il chitarrista nel tuo avatar ?

[cionci]

Originariamente inviato da trallallero

edit: scusa ma è da molto che volevo chiedertelo: chi è il chitarrista nel tuo avatar ?

Chuck Schuldiner

[trallallero]

Originariamente inviato da cionci

Chuck Schuldiner

Sei un metallaro

[cionci]

Originariamente inviato da trallallero

Sei un metallaro

[trallallero]

Originariamente inviato da cionci

Io ogni sera suono Joe Satriani, conosci ?
ho appena finfito di imparare "satch boogie".

Ok, non è metallo, ma non è certo musica pop

[cionci]

Non lo conosco bene, ma le poche cose che ho ascoltato mi sono piaciute. Se sai suonare Satriani allora sai suonare molto bene

[trallallero]

Originariamente inviato da cionci

Non lo conosco bene, ma le poche cose che ho ascoltato mi sono piaciute. Se sai suonare Satriani allora sai suonare molto bene

Beh, è stato il mio lavoro fino a 28 anni prima di diventare informatico quindi diciamo che me la cavo (certo che prima suonavo 8 ore al giorno, adesso solo 1-2).

Quì suono The Extremist http://www.youtube.com/watch?v=tyqQJi1iWa8
Anche se è di un pò di tempo fa, adesso l'ho migliorata.

Vorrei arrivare ad almeno 12-15 pezzi per fare qualche serata ma sono solo a 8

Vabbè, un pelino OT

[cionci]

E stica