[C++] passaggio per riferimento

[trallallero]

Ho questo prototipo:

void LLDeviceSwitch::SetState(int State, mapIdState& IdStateMap)

e adesso mi accorgo che non sempre ho bisogno di una mappa IdStateMap ma non posso non passarne una valida quando chiamo la funzione.
Mi piacerebbe poter chiamare la funzione in questo modo:
SetState( LLDevice::LL_DEVICE_STATE_OFF, NULL );

non mi era mai successo di provare a passare NULL ad un parametro per riferimento ed ho visto che al compilatore non va proprio giù.
C'è qualche diavoleria e/o trucchetto che possa risolvere il problema.

Potrei cambiare il prototipo, lo so, ma ciò comporterebbe la modifica di n^n funzioni :muro:

grazie

[^TiGeRShArK^]

Ho questo prototipo:

void LLDeviceSwitch::SetState(int State, mapIdState& IdStateMap)

e adesso mi accorgo che non sempre ho bisogno di una mappa IdStateMap ma non posso non passarne una valida quando chiamo la funzione.
Mi piacerebbe poter chiamare la funzione in questo modo:
SetState( LLDevice::LL_DEVICE_STATE_OFF, NULL );

non mi era mai successo di provare a passare NULL ad un parametro per riferimento ed ho visto che al compilatore non va proprio giù.
C'è qualche diavoleria e/o trucchetto che possa risolvere il problema.

Potrei cambiare il prototipo, lo so, ma ciò comporterebbe la modifica di n^n funzioni :muro:

grazie
..ah quant'è bello eclipse...
ALT+SHIFT+C, modifichi il tipo del parametro, INVIO e fa tutto lui :D
Sinceramente non ho idea come fare in C++.. :p

[trallallero]

ma io lavoro con Eclipse ...
ALT+SHIFT+C ??? :wtf:

[banryu79]

ma io lavoro con Eclipse ...
ALT+SHIFT+C ??? :wtf:

Lascia stare, credo sia una feature dell'IDE per fare Refactoring, ci sono anche in NeatBeans.

(selezioni un parametro nella firma del metodo che vuoi cambiare, rinomini, e in ogni punto del codice del tuo progetto dove quel metodo viene chiamato viene sostituito il vecchio parametro con il nuovo, comodo :) )

Cmq NULL dovrebbe essere una define, giusto? Invece di passare NULL direttamente, potresti istanziare un puntatore a "mapIdState" e inizializzarlo a NULL, e poi passare quello?

[trallallero]

Lascia stare, credo sia una feature dell'IDE per fare Refactoring, ci sono anche in NeatBeans.

(selezioni un parametro nella firma del metodo che vuoi cambiare, rinomini, e in ogni punto del codice del tuo progetto dove quel metodo viene chiamato viene sostituito il vecchio parametro con il nuovo, comodo :) )

Cmq NULL dovrebbe essere una define, giusto? Invece di passare NULL direttamente, potresti istanziare un puntatore a "mapIdState" e inizializzarlo a NULL, e poi passare quello?

No lasciate stare, son pirla io :doh:

Prima faccio le cose fatte bene e poi mi dimentico di averle fatte :D

È tutto così ben incastrato (oggetto contiene oggetti che contiengono oggetti ...)
che devo solo cambiare i prototipi e le funzioni.
In pratica la "insert( make_pair( " della mappa la uso solo (e giustamente) in un punto, quindi cambio solo quello.

Comunque grazie.

Però mi piacerebbe approfondire la cosa perchè è sempre meglio avere qualche trucco in tasca :D

Il tuo consiglio non l'ho capito

[banryu79]

Il tuo consiglio non l'ho capito


Per forza, non ha senso: ho riletto con calma e ho sparato una vaccata, chiedo venia :stordita:

[trallallero]

Per forza, non ha senso: ho riletto con calma e ho sparato una vaccata, chiedo venia :stordita:

succede dai :D

[trallallero]

ho provato anche a reinterpret_castare l'impossibile ma non riesco a fregarlo :mad: !

[cionci]

mapIdState che roba è ?

La prima cosa che mi viene in mente, se mapIdSate è una classe, è quello di creare un NullObject...e mettere un factory method.

Dovrebbe risultare qualcosa di questo tipo:

SetState( LLDevice::LL_DEVICE_STATE_OFF, mapIdState::NullObject());

[trallallero]

mapIdState che roba è ?

La prima cosa che mi viene in mente, se mapIdSate è una classe, è quello di creare un NullObject...e mettere un factory method.

Dovrebbe risultare qualcosa di questo tipo:

SetState( LLDevice::LL_DEVICE_STATE_OFF, mapIdState::NullObject());

typedef map<int, int > mapIdState;
mi serve per mappare ogni Id elemento con il suo stato

però è interessante 'sto NullObject() ... che roba è ?
(ti stavo aspettando, lo sapevo che avresti risposto con qualcosa di figo :D)

[cionci]

Mi sa che in questo caso ti conviene passare un riferimento ad un mapIdState vuoto, molto più semplice.

In ogni caso il NullObject è questo (AbstractBase fa da interfaccia):


class AbstractBase
{
private:
AbstractBase() {};
public:
void add(int i) = 0;
int get(int index) = 0;
};

class ADerived: public AbstractBase
{
vector <int> v;
public:
ADerived() {};
void add(int i) { v.push_back(i); };
int get(int index) { return v.at(i); };
};

class MyNullObject: public AbstractBase
{
public:
MyNullObject() {};
void add(int i) { };
int get(int index) { return 0; };
};

In pratica si implementa l'interfaccia della classe che normalmente usiamo, ma di fatto il NullObject non fa niente ;) Serve per evitare controlli espliciti sui NULL (if(a != NULL) fai questo; )
http://en.wikipedia.org/wiki/Null_Object_pattern

[trallallero]

wow, figata ... mi dovrò salvare anche questo 3d :D

in effetti non è il mio caso perchè non passo una classe ma una semplice map.
E comunque la map è sempre vuota, la passo perchè qualche elemento potrebbe riempirla con qualche state change o set value.
Infatti adesso il prototipo ovunque ha questi parametri
funz(mapIdState* IdStateMap, mapIdVoidval* IdVoidvalMap)

Quindi le mappe sono vuote all'andata e (forse) piene al ritorno.

Certe volte so che l'evento non può creare state changes quindi volevo passare NULL ma per riferiemento pare sia impossibile.

[cionci]

Certe volte so che l'evento non può creare state changes quindi volevo passare NULL ma per riferiemento pare sia impossibile.
Che ti importa...lo passi comunque il map.

[trallallero]

Che ti importa...lo passi comunque il map.

no, perchè certe azioni partono localmente e servono solo a settare lo state senza un output, senza una modifica di leds vari o stringhe.

quindi prima dovevo fare una cosa del tipo

mapIdState m;

SetState( LLDevice::LL_DEVICE_STATE_OFF, m );
e non mi piaceva proprio :stordita:

[cionci]

Allora semplicemente fai un'altra SetState con prototipo:

void LLDeviceSwitch::SetState(int State)

e in

void LLDeviceSwitch::SetState(int State, mapIdState& IdStateMap)

suddividi le azioni in modo da richiamare SetState(int State) così da eliminare la duplicazione.

[trallallero]

Allora semplicemente fai un'altra SetState con prototipo:

void LLDeviceSwitch::SetState(int State)

e in

void LLDeviceSwitch::SetState(int State, mapIdState& IdStateMap)

suddividi le azioni in modo da richiamare SetState(int State) così da eliminare la duplicazione.

:doh: non c'ho pensato

vedi che lavorare troppo fa male !