Pourquoi mes valeurs std :: map ne sont-elles pas modifiées?

Question

J'essaie de découper les espaces blancs du début et de la fin des chaînes de ma carte.

Le code échoue 3 des essais:

#include <map> 
#include <string> 

void blanksOut(std::map<int, std::string> & data) 
{ 
    for (auto p : data) { 
     std::string p1 = p.second; 

     size_t first = p1.find_first_not_of(' '); 
     p1 = p1.erase(0, first); 
     size_t last = p1.find_last_not_of(' '); 
     p1.resize(last + 1); 
     p.second = p1; 
    } 
} 

int main() 
{ 
    std::map<int, std::string> m; 
    m[2] = "foo"; 
    m[3] = " bar"; 
    m[5] = " baz "; 
    m[8] = "quux "; 

    blanksOut(m); 

    return(m[2] != "foo") 
     + (m[3] != "bar") 
     + (m[5] != "baz") 
     + (m[8] != "quux"); 
} 

Le seul qui passe est le premier, où aucun changement est nécessaire. Qu'est-ce que je fais mal?

Answer 1

Votre problème est que vous utilisez sur un std::pair temporaire:

for (auto p : data) 

donc quand vous attribuez à p.second, vous ne changez rien dans le std::map. Au lieu de cela, vous devez itérer sur les références:

for (auto& p: data) 

Cette modification entraîne le passage de vos tests.