C++ “erreur: le passage de 'const std::map<int, std::basic_string<char> >' que 'ce' argument de ...”

Avec le code suivant (extrait par souci de concision):

couleur.h:

class color {
public:
    color();

    enum colorType {
        black, blue, green, cyan, red,
        magenta, brown, lightgray, nocolor
    };

    colorType getColorType();
    void setColorType(colorType cColortype);

    string getColorText() const;

private:
    colorType cColortype = nocolor;
    map<int, string> colors = {
        {black, "black"},
        {blue, "blue"},
        {green, "green"},
        {cyan, "cyan"},
        {red, "red"},
        {magenta, "magenta"},
        {brown, "brown"},
        {lightgray, "lightgray"},
        {nocolor, "nocolor"}};
};

color.cpp:

color::color() {
}

color::colorType color::getColorType() {
    return cColortype;
}

void color::setColorType(colorType cColortype) {
    this->cColortype = cColortype;
}

string color::getColorText() const {
    return colors[cColortype];
}

J'obtiens l'erreur suivante:

couleur.rpc:16:29: erreur: passage de 'const std::map >' que 'ce' argument de 'std::map<_Key, _Tp, _Compare, _Alloc>::mapped_type& std::map<_Key, _Tp, _Compare, _Alloc>::operator[](std::map<_Key, _Tp, _Compare, _Alloc>::key_type&&) [avec _Key = int; _Tp = std::basic_string; _Compare = std::less; _Alloc = std::allocator > >; std::map<_Key, _Tp, _Compare, _Alloc>::mapped_type = std::basic_string; std::map<_Key, _Tp, _Compare, _Alloc>::key_type = int] " rejets qualificatifs [-fpermissive]

L'erreur fait référence à "restituer les couleurs[cColortype];" dans getColorText.

J'écris ceci pour un projet de classe et je peux l'obtenir pour travailler pour le bien de la cession, par la suppression de la const déclaration dans le getColorText signature, mais je suis en train d'apprendre ou d'adopter de bonnes pratiques, et d'adhérer à la recommandation de l'utilisation de const pour les fonctions de membres qui ne modifiez pas les données donc je veux savoir comment faire face à cette aller de l'avant.

D'habitude je suis vraiment bon à débogage/dépannage mais le message d'erreur est tellement alambiqué qu'il n'est pas beaucoup d'aide.

Toute aide est appréciée.

Ma première suggestion serait de supprimer l'indice de l'opérateur de couleur::getColorText() const avec un std::map<>::find de façon appropriée.

InformationsquelleAutor WXB13 | 2013-11-29

c++const map

33
```
string color::getColorText() const {
    return colors[cColortype];
}
```
Le problème est que vous avez marqué à la fonction de const. Le operator[] sur std::map est marquée comme non-const, et ne peut pas être utilisé dans un const fonction comme ceci. Vous devez manuellement utilisation de std::map::find (ou autre mécanisme) à la recherche pour le type d'entrée et de gérer le cas où il ne l'est pas.

Si vous êtes à l'aide de C++11, vous pouvez à la place utiliser std::map::at, qui EST autorisé à être utilisé sur une constante de la carte, et lève une exception si l'élément n'est pas présent.
- std::map::au s'inscrit le projet de loi parfaitement. Je vous remercie beaucoup.
- Oui, std::map::est ce dont j'ai réellement besoin.
InformationsquelleAutor Dave S
5

La clé est près de la fin: "les rejets des qualificatifs". getColorText est un const fonction membre, donc colors est const. Mais map::operator[]() n'est pas const.

InformationsquelleAutor Pete Becker
2

Première : la carte map<int, string> colors doit être une carte à partir de cColorType à la chaîne, au lieu de int :
```
map<cColorType, string> colors
```
Deuxième : Comme certaines personnes déjà répondu : map::operator[]() n'est pas const. La raison en est que cet opérateur renvoie une référence, qui permet de modifier sa valeur.

Permettez-moi de suggérer la solution suivante : Vous pouvez créer un deuxième attribut privé : la couleur dans la chaîne de format. Par conséquent, vous aurez 2 Obtenir des fonctions (un pour chaque type de couleur), et une fonction de Jeu (qui va modifier les 2 attributs de couleur).
- En fait cColorType est un nom de variable et non un type. Peut-être que vous avez voulu dire colorType à la place. Ce n'est effectivement pas nécessaire que colorType est un enum qui correspond à l ' int de sorte que le "map<int, string> les couleurs" signature fonctionne très bien. La partie du code que j'ai quitté de mon post est un accesseur qui prend une chaîne de caractères comme argument et annule la carte de recherche sur les couleurs pour obtenir le correspondant colorType. La carte avec des recherche inversée élimine la nécessité de stocker les paramètres de couleur dans 2 formats différents.
- Oui, vous avez raison à propos de la cColorType, mes excuses. En gardant la première entrée de la carte comme "int" de travail, mais la mise à colorType est plus souple : imaginez que vous (ou quelqu'un d'autre) va réécrire ce code dans l'avenir, et aimeraient quelque chose d'autre qu'un enum pour colorType. Ce codeur n'aurez pas à modifier la déclaration de la carte. Comment procédez-vous pour "inverser la carte de recherche" ? Je suis impatient de voir 🙂
- Inverser la carte de recherche de: stackoverflow.com/questions/5749073/reverse-map-lookup/...
- Votre carte de recherche inversée utilise find_if, qui est linéaire complexité. Si vous stockez des 2 attributs différents, le en 2, les fonctions s'exécutent en temps constant chaque. Et depuis la mise en correspondance entre les 2 types de couleur est parfait, vous n'avez pas besoin de recherche inversée. Vous pouvez de code au lieu de deux cartes qui représentent des fonctions inverses les uns des autres.
InformationsquelleAutor Brainless

Vous devez vous connecter pour publier un commentaire.