Errore di compilazione tramite iteratori di mappe

Question

Nel mio file di intestazione ho incluso la std :: map e uso lo spazio dei nomi appropriato.
Uno dei miei soci è:

map<unsigned int, double> pT_Spam; 

E nel mio file cpp tento di fare qualcosa che ho fatto spesso da qualche tempo:

for(map<unsigned int, double>::iterator it=pT_Spam.begin() ; it!=pT_Spam.end() ; it++) {/*code*/} 

Quanto sopra è anche menzionata in uno gli esempi di utilizzo di std :: map su cplusplus.com. Anche se ho fatto praticamente lo stesso in altre parti del codice che causano errori di compilazione, in questa particolare linea ottengo il seguente errore da Cygwin:

error: conversion from `std::_Rb_tree_const_iterator<std::pair<const unsigned int, double> >' to non-scalar type `std::_Rb_tree_iterator<std::pair<const unsigned int, double> >' requested 

Che sembra piuttosto strano. Qualche idea di cosa potrebbe essere sbagliato? (La mia intestazione è, ovviamente, inclusa nel mio cpp)

Answer 1

Sembrerebbe che allo scopo questo ciclo esista, la mappa sia const. Ad esempio, il ciclo in un metodo di classe dichiarato const, in questo modo?

void method() const // const method 
{ 
    // Do stuff. 
} 

o passato come argomento const, come questo?

void function(const map<unsigned int, double>& pT_Spam) 
{ 
    // Do stuff. 
} 

Se lo è, è necessario utilizzare gli iteratori const:

for(map<unsigned int, double>::const_iterator it=pT_Spam.begin() ; it!=pT_Spam.end() ; it++) 
{ 
    /*code*/ 
} 

Oppure, se si sta utilizzando C++ 11, quindi si dovrebbe utilizzare la parola chiave auto:

for(auto it=pT_Spam.begin() ; it!=pT_Spam.end(); it++) 
{ 
    /*code*/ 
} 

Dato che nel caso in cui hai mostrato devi usare gli iteratori const, non puoi usarli per modificare la mappa oi dati al suo interno. Questa è la correttezza costante, ed è una buona cosa :).

Answer 2

È necessario utilizzare const_iterators per le mappe, quindi dovrebbe essere:

for(map<unsigned int, double>::const_iterator it = \\and so on 

Edit: Come sottolineato quanto sopra è giusto, ma per completamente errati motivi (le mappe hanno iteratori non costanti, cosa stavo pensando esattamente? non so). Molto probabilmente la tua mappa è definita come const (come indicato in un'altra risposta).

Answer 3

Bene, l'errore indica che stai provando a trasmettere un const_iterator a un iteratore. Tu dici che pT_Spam è un membro. È un membro di un oggetto const? Se lo è, begin() e end() restituiranno const_iterators.

Answer 4

per (mappa :: iterator it = pT_Spam.begin();! It = pT_Spam.end(), ma ++)

chage al

per (mappa :: const_iterator it = pT_Spam. begin(); it! = pT_Spam.end(); it ++)

Poiché punta al valore costante, l'iteratore deve essere anche di tipo const.