[C++] Utilizzare STL::Map

[beppe66]

Scusate, se ancora una volta rompo le scatole...
ma perchè il codice che scrivo io non è corretto, cerco di farmi capire.. Sono riuscito soprattutto grazie all'aiuto di Shodan a scriver la struttura map come nel codice sopra riportato.. ora non faccio altro che:
Settare una chiave di ricerca...e definire un'iteratore alla struttura...

map< vector<int>,DataMsg,MyArrayCompare <vector<int> >>::iterator elem;
vector <int>key(3);
    
    key[0]=packmsg->getPrevious(0);
    key[1]=packmsg->getPrevious(1);
    key[2]=packmsg->getPrevious(2);

ora cerco un'elemento all'interno della struttura....almeno così ho visto che si fa, dagli esempi riportati suu un manulae c++

elem = mp.find(key);

e incredibile ma vero mi spara fuori 200mila errori, insomma nn compila...
Ma cosa ho, la sfiga addosso oltre ad esser un'imbranato a programmare...
Perodonate questo sfogo ma vado a picco.

Giuseppe

[shodan]

Attento alla dichiarazione:

map< vector<int>,DataMsg,MyArrayCompare <vector<int> >>::iterator elem;

gli attuali compilatori (col prossimo standard la cosa dovrebbe risolversi) interpretano le due angolari rosse come operatore >> non come dichiarazione di template.
Basta separarle da uno spazio e risolvi il problema.

map< vector<int>,DataMsg,MyArrayCompare <vector<int> > >::iterator elem;

Ti consiglio comunque di fare un typedef della dichiarazione:
1) per evitare errori di sintassi
2) per semplificarti la vita.

// Questo va messo prima della dichiarazione della mappa globale.
typedef  map< vector<int>,DataMsg,MyArrayCompare <vector<int> > > MyMap;
MyMap mp; // mappa globale.
// in seguito, se serve
MyMap::iterator elem
// etc.

Ultima cosa.
Nel functor ho messo

std::size_t maxComparingSize = std::min(a.size(),b.size());

in caso serva ciclare tra gli elementi del vector (che può non essere vincolato a soli 3 elementi)
e non oltrepassare i limiti.
Se sai già che la locazione è valida e non ti serve un ciclo for, puoi anche ometterla.

[beppe66]

Ciao Shodan,
il problema che ti ho riportato, non era risolvibile ponendo uno spazio tra i ">>", ho provato infatti a mettere uno spazio tra i due segni e il risultato non cambia...
Per chiarezza voglio riportarti l'errore del compilatore:

eror C3848:espressione con tipo 'const MyArrayCompare<T> 
deve perdere alcuni qualificatori const-volatile per poter chiamare 
'bool MyArrayCompare<T>::operator()(const T&,const T&)'
with
[
   T=std::vector<int>
]
......

Tu ci capisci qualcosa?
Per esser più sicuro ho voluto far una prova, ho tolto nuovamente nella definizione di map il riferimento a MyArrayCompare e le cose sembrano funzionare...

typedef  map< vector<int>,DataMsg> MyMap;

tengo a precisare che l'errore viene sollevato in corrispondenza della riga di "find" ossia:

elem = mp.find(key);

[shodan]

A me compila senza problemi con VC++ Express. Che compilatore usi?
A prima vista sembra che non gradisca const T&, prova semplicemente con T (anche se meno efficiente)

[beppe66]

Dopo aver sbattuto la testa per un bel pò e grazie all'aiuto di gente che ne sa più di me..(il che non è difficile), siamo giunti alla soluzione del problema:
Bisognava modificare la funzione "bool operator"... riporto qui di seguito il codice:

template <typename T>
struct MyArrayCompare {
	 bool operator()(const T& a,const T& b) const{
             
		return (a[2] < b[2]);
	}
};

insomma bastava aggiungere il modificatore "const" alla firma del metodo.
A volte odio la programmazione, basta un'errore se pur banale per andare a picco.

Grazie1000
Beppe.

[oregon]

Originariamente inviato da beppe66
A volte odio la programmazione, basta un'errore se pur banale per andare a picco.

E invece proprio questo è il bello della programmazione ...

[shodan]

Originariamente inviato da beppe66
insomma bastava aggiungere il modificatore "const" alla firma del metodo.
A volte odio la programmazione, basta un'errore se pur banale per andare a picco.

Il problema di fondo sono anche i compilatori che in presenza dello stesso codice hanno differenti comportamenti.
Come già detto, a me quel codice compilava senza problemi. Il tuo compilatore invece si lamentava.

[beppe66]

Cmq caro shodan voglio ringraziarti,
senza di te non ce l'avrei mai fatta....

Sei grande...
Giuseppe

[shodan]

Grazie, ma sono altri i grandi.