|
|
|
#1 |
|
Messages: n/a
Hébergeur: |
Bonjour,
J'ai cherché des justifications au fait que cette surcharge n'existe pas. Et je n'en ai pas trouvé, hormis le fait que l'inexistance de la clé entraîne l'ajout d'une paire dans la map. Pourquoi ? Existe-t-il une méthode élégante pour contourner ce manque ? Peut-être que je me trompe dans mon interprétation de ce qu'est une map, et que donc je n'utilises pas le bon outil ? Merci ![]() -- Mickaël Wolff aka Lupus Michaelis http://lupusmic.org |
|
|
|
#2 |
|
Messages: n/a
Hébergeur: |
Mickaël Wolff <mickael.wolff@laposte.net> writes:
> Bonjour, > > J'ai cherché des justifications au fait que cette surcharge n'existe > pas. Et je n'en ai pas trouvé, hormis le fait que l'inexistance de la > clé entraîne l'ajout d'une paire dans la map. > > Pourquoi ? Parce que l'inexistance de la clé entraîne l'ajout d'une paire dans la map. > Existe-t-il une méthode élégante pour contourner ce manque ? Tu peux utiliser map::find. > Peut-être que je me trompe dans mon interprétation de ce qu'est une > map, et que donc je n'utilises pas le bon outil ? Je ne sais pas, c'est quoi pour toi une map? -- __Pascal Bourguignon__ http://www.informatimago.com/ Pour moi, la grande question n'a jamais été: «Qui suis-je? Où vais-je?» comme l'a formulé si adroitement notre ami Pascal, mais plutôt: «Comment vais-je m'en tirer?» -- Jean Yanne |
|
|
|
#3 |
|
Messages: n/a
Hébergeur: |
Mickaël Wolff <mickael.wolff@laposte.net> writes:
| Bonjour, | | J'ai cherché des justifications au fait que cette surcharge n'existe | pas. Et je n'en ai pas trouvé, hormis le fait que l'inexistance de la | clé entraîne l'ajout d'une paire dans la map. | | Pourquoi ? operator[] est défini en terme de recherche+insertion. -- Gaby |
|
|
|
#4 |
|
Messages: n/a
Hébergeur: |
Si on ne veut pas que l'inexistence entraine l'ajout de la clé, qu'est
ce qui sera retourné si on fait un accès et que la clé demandée n'est pas présente? Ce n'est pas possible. Du coup, l'ajout est obligatoire, et on ne peut pas fournir d'opérateur en const > Bonjour, > > J'ai cherché des justifications au fait que cette surcharge n'existe > pas. Et je n'en ai pas trouvé, hormis le fait que l'inexistance de la > clé entraîne l'ajout d'une paire dans la map. > > Pourquoi ? > Existe-t-il une méthode élégante pour contourner ce manque ? > Peut-être que je me trompe dans mon interprétation de ce qu'est une > map, et que donc je n'utilises pas le bon outil ? > > Merci ![]() > |
|
|
|
#5 |
|
Messages: n/a
Hébergeur: |
Pascal Bourguignon a écrit :
> Parce que l'inexistance de la clé entraîne l'ajout d'une paire dans > la map. Certes, mais pourquoi ajouter une paire ? Pourquoi ne pas lancer une exception ? >> Existe-t-il une méthode élégante pour contourner ce manque ? > > Tu peux utiliser map::find. Non, j'ai essayé. il n'y a pas de map::find const > Je ne sais pas, c'est quoi pour toi une map? Je le prenais comme un tableau associatif, qui renvoie une valeur pour une clé associée à une valeur -- Mickaël Wolff aka Lupus Michaelis http://lupusmic.org |
|
|
|
#6 |
|
Messages: n/a
Hébergeur: |
On Mon, 21 Apr 2008 02:53:09 +0200, Mickaël Wolff :
> Certes, mais pourquoi ajouter une paire ? Pourquoi ne pas lancer une >exception ? L'opérateur non-const [] ajoute une paire si nécessaire, pour qu'on puisse écrire : map<int,int> m; m[3]= 5; Une conséquence immédiate (et peut-être malheureuse) est que le code ci-dessous ajoute également une paire dans le map<> : map<int,int> m; cout << m[4]; En C++, le mot-clé "const" sert avant tout à avoir un message d'erreur (à la compilation) si on modifie un élément qui, logiquement, n'a pas à être modifié. En particulier, l'ajout ou le retrait d'un "const" (tout comme de "public"/"private") ne devrait pas changer sensiblement le comportement du programme. Ainsi, si j'écris map<int,int> m const; cout << m[4]; j'ai une erreur à la compilation. C'est logique, map: perator[] n'apas été prévu pour être utilisé comme ça. Si, par l'ajout du "const", le comportement du programme avait été silencieusement modifié, avec à la clé une exception lancée de temps en temps, ce serait un cauchemar pour le programmeur. |
|
|
|
#7 |
|
Messages: n/a
Hébergeur: |
On Mon, 21 Apr 2008 02:53:09 +0200, Mickaël Wolff
<mickael.wolff@laposte.net>: > Non, j'ai essayé. il n'y a pas de map::find const Bien sûr que si, heureusement ! Tu dois même pouvoir écrire quelque chose comme ça : template <class Cle, class Valeur, class Less> Valeur GetDansMap (std::map <Cle, Valeur, Less> const& map_, Cle const& cle, Valeur const& valeur_par_defaut= Valeur()) { typename std::map <Cle, Valeur, Less>::const_iterator it= map_.find (cle); if (it == map_.end()) { return valeur_par_defaut; } else { return (*it).second; } } ou bien : template <class Cle, class Valeur, class Less> Valeur GetDansMap (std::map <Cle, Valeur, Less> const& map_, Cle const& cle) { typename std::map <Cle, Valeur, Less>::const_iterator it= map_.find (cle); if (it == map_.end()) { throw quelque_chose; } else { return (*it).second; } } Tu peux aussi écrire ton propre map : template <class Cle, class Valeur> class MonMap { public: void Set (Cle const& cle, Valeur const& valeur) { data[cle]= valeur; } Valeur const& Get (Cle const& cle) const { Data::const_iterator it= data.find (cle); if (it == data.end()) { throw quelque_chose; } return (*it).second; } Valeur const& operator[] (Cle const& cle) const { return Get(cle); } private: typedef typename std::map<Cle,Valeur> Data; Data data; }; |
|
|
|
#8 |
|
Messages: n/a
Hébergeur: |
Mickaël Wolff <mickael.wolff@laposte.net> writes:
| Pascal Bourguignon a écrit : | | > Parce que l'inexistance de la clé entraîne l'ajout d'une paire dans | > la map. | | Certes, mais pourquoi ajouter une paire ? Pourquoi ne pas lancer une | exception ? Peux-tu montrer le genre de codes typiques que tu écrirais avec ça ? Pourquoi cela est-il meilleur qu'utiliser find() ou binary_search() ? -- Gaby |
|
|
|
#9 |
|
Messages: n/a
Hébergeur: |
Fabien LE LEZ a écrit :
> Data::const_iterator it= data.find (cle); Merci beaucoup, c'était essentiellement const_iterator qui me manquait. Je n'en connaissait pas l'utilité. -- Mickaël Wolff aka Lupus Michaelis http://lupusmic.org |
|
|
|
#10 |
|
Messages: n/a
Hébergeur: |
On Apr 21, 3:03 am, Fabien LE LEZ <grams...@gramster.com> wrote:
> On Mon, 21 Apr 2008 02:53:09 +0200, Mickaël Wolff : > > Certes, mais pourquoi ajouter une paire ? Pourquoi ne pas > > lancer une exception ? > L'opérateur non-const [] ajoute une paire si nécessaire, pour qu'on > puisse écrire : > map<int,int> m; > m[3]= 5; > Une conséquence immédiate (et peut-être malheureuse) est que le code > ci-dessous ajoute également une paire dans le map<> : > map<int,int> m; > cout << m[4]; > En C++, le mot-clé "const" sert avant tout à avoir un message > d'erreur (à la compilation) si on modifie un élément qui, > logiquement, n'a pas à être modifié. En particulier, l'ajout > ou le retrait d'un "const" (tout comme de "public"/"private") > ne devrait pas changer sensiblement le comportement du > programme. À condition, évidemment, que le code reste légal. > Ainsi, si j'écris > map<int,int> m const; > cout << m[4]; > j'ai une erreur à la compilation. C'est logique, map: perator[] n'a> pas été prévu pour être utilisé comme ça. > Si, par l'ajout du "const", le comportement du programme avait été > silencieusement modifié, avec à la clé une exception lancée de temps > en temps, ce serait un cauchemar pour le programmeur. C'est un bon point. En fait, tel que je le vois, l'operator[] est un surplus, qui ne fait pas partir de l'abstraction de base de std::map. L'abstraction de base, c'est find, insert et erase. Selon l'utilisation, tu veux des comportements différents si l'élément récherché ne s'y trouve pas : l'insérer avec une valeur par défaut, renvoyer un pointeur plutôt qu'une référence, avec un pointeur null, lever une exception, ou même l'échec d'une assertion. Tout ce qui se laisse implémenter facilement à partir des trois opérations de base. L'existance de l'opérateur [], c'est simplement pour permettre l'utilisation du map de la même manière qu'on utilise son équivalent en AWK ou en perl. Si c'est ce que tu veux, tu ne déclares pas le map const, et tu l'utilises exactement comme en AWK ou en perl. Sinon, tu fais un petit wrapper pour donner la sémantique que tu veux. -- James Kanze (GABI Software) email:james.kanze@gmail.com Conseils en informatique orientée objet/ Beratung in objektorientierter Datenverarbeitung 9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34 |
|
|
|
#11 |
|
Messages: n/a
Hébergeur: |
Fabien LE LEZ wrote:
> En C++, le mot-clé "const" sert avant tout à avoir un message d'erreur > (à la compilation) si on modifie un élément qui, logiquement, n'a pas > à être modifié. En particulier, l'ajout ou le retrait d'un "const" > (tout comme de "public"/"private") ne devrait pas changer sensiblement > le comportement du programme. > > Ainsi, si j'écris > > map<int,int> m const; > cout << m[4]; > > j'ai une erreur à la compilation. C'est logique, map: perator[] n'a> pas été prévu pour être utilisé comme ça. > Si, par l'ajout du "const", le comportement du programme avait été > silencieusement modifié, avec à la clé une exception lancée de temps > en temps, ce serait un cauchemar pour le programmeur. Notons aussi qu'en C++09(?) on prévoit pallier à ce manque constaté par des utilisateurs de std::map en y ajoutant des fonctions membre : T& at(const key_type& x); const T& at(const key_type& x) const; qui lèvent toujours une exception std: ut_of_range si la clé ne setrouve pas déjà dans le map. Falk |
|
|
|
#12 |
|
Messages: n/a
Hébergeur: |
Gabriel Dos Reis a écrit :
> Peux-tu montrer le genre de codes typiques que tu écrirais avec ça ? > Pourquoi cela est-il meilleur qu'utiliser find() ou binary_search() ? Je l'utilisais par soucis de concision, pour rendre le code plus lisible. Il était logique dans ce cas là qu'une exception soit levée, comme c'est le cas en javascript. Autre exemple, en Java et PHP, null est renvoyé. Certes, ce n'est pas possible en C++, c'est pour ça que je me disais que l'exception était incontournable. Le problème de find est qu'en fonction du conteneur, il ne renvoie pas la même chose. Un itérateur tantôt sur le type stocké (vector, deque), tantôt sur une paire (map), alors que operator[] renvoie toujour une référence sur le type. Au final, ce n'est pas un problème, une fois qu'on sait à quoi correspond exactement std::map<>: perator[] ![]() Merci à tout ceux qui ont apporter des pièces pour comprendre ma compréhension erronée. -- Mickaël Wolff aka Lupus Michaelis http://lupusmic.org |
|
|
|
#13 |
|
Messages: n/a
Hébergeur: |
Mickaël Wolff <mickael.wolff@laposte.net> writes:
| Gabriel Dos Reis a écrit : | | > Peux-tu montrer le genre de codes typiques que tu écrirais avec ça ? | > Pourquoi cela est-il meilleur qu'utiliser find() ou binary_search() ? | | Je l'utilisais par soucis de concision, pour rendre le code plus | lisible. Comme je l'avais suggéré plus haut, peux-tu montrer le genre de codes typiques que tu écrirais pourqu'on puisse juger si cela serait plus concis et plus lisible ? [...] | Le problème de find est qu'en fonction du conteneur, il ne renvoie pas | la même chose. Un itérateur tantôt sur le type stocké (vector, deque), | tantôt sur une paire (map), alors que operator[] renvoie toujour une | référence sur le type. le type stocké est donné par value_type -- donc, cela renvoie bien la même chose pour vector, list, et map. Au contraire, c'est operator[] qui ne renvoie pas toujours une référence sur value_type. -- Gaby |
|
|
|
#14 |
|
Messages: n/a
Hébergeur: |
On Tue, 22 Apr 2008 02:10:13 +0200, Mickaël Wolff :
>Le problème de find est qu'en fonction du conteneur, il ne renvoie pas >la même chose. C'est vrai que les interfaces des types de la STL ne sont pas toujours idéales. Mais il faut considérer les conteneurs standard comme des briques de base ; tu peux facilement les encapsuler dans tes propres classes, pour fournir l'interface qui te convient. >alors que operator[] renvoie toujour une référence sur le type. Attention, vector<>: perator[] ne lance pas non plus d'exception. |
|
|
|
#15 |
|
Messages: n/a
Hébergeur: |
On Apr 22, 5:11 am, Fabien LE LEZ <grams...@gramster.com> wrote:
> On Tue, 22 Apr 2008 02:10:13 +0200, Mickaël Wolff : > >Le problème de find est qu'en fonction du conteneur, il ne renvoie pas > >la même chose. > C'est vrai que les interfaces des types de la STL ne sont pas toujours > idéales. Mais il faut considérer les conteneurs standard comme des > briques de base ; tu peux facilement les encapsuler dans tes propres > classes, pour fournir l'interface qui te convient. > >alors que operator[] renvoie toujour une référence sur le type. > Attention, vector<>: perator[] ne lance pas non plus d'exception.Il se présente un peu différement : il est assez facile de démander d'abord si l'indice s'y trouve (« i < v.size() »). Du coup, sinon, c'est un comportement indéfini, c-à-d un core dump garanti dans toute implémentation qui se respecte. En revanche, ça ne fait pas partie de la STL, mais on pourrait bien imaginer un concepte « Indexable » pour une collection qui 1) supporte l'opérateur [] (const et non const) et 2) a une fonction supplémentaire « contains( index ) » ou « isValid( index ) » ou quelque chose du genre. Si on admet que le comportement de l'opérateur [] est indéfini si « ! contains() », on n'a besoin que d'ajouter contains() à std::map et à std::vector pour qu'ils soient conforme tous les deux -- indéfini laissant la liberté de définir dans des cas particulier. Et en étendant ce concepte : at() lève une exception si l'indice n'est pas valide, et get() renvoie un pointeur, avec un pointeur null si l'indice n'est pas valide. -- James Kanze (GABI Software) email:james.kanze@gmail.com Conseils en informatique orientée objet/ Beratung in objektorientierter Datenverarbeitung 9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34 |
|
|
|
#16 |
|
Messages: n/a
Hébergeur: |
Gabriel Dos Reis a écrit :
> Comme je l'avais suggéré plus haut, peux-tu montrer le genre de codes > typiques que tu écrirais pourqu'on puisse juger si cela serait plus > concis et plus lisible ? class message { typedef std::map<std::string,std::string> map ; public: std::string & get_value(std::string key) const throw(std::range_error) { // return m_value_list[key] ; // ça marche pas © map::const_iterator s = m_value_list.find(key) ; if(s == m_value_list.end()) throw std::range_error("message::get_value()") ; return s->second ; } protected: map m_value_list ; } ; > le type stocké est donné par value_type -- donc, cela renvoie bien la > même chose pour vector, list, et map. Au contraire, c'est operator[] > qui ne renvoie pas toujours une référence sur value_type. Ben non ![]() -- Mickaël Wolff aka Lupus Michaelis http://lupusmic.org |
|
|
|
#17 |
|
Messages: n/a
Hébergeur: |
Mickaël Wolff <mickael.wolff@laposte.net> writes:
| Gabriel Dos Reis a écrit : | | > Comme je l'avais suggéré plus haut, peux-tu montrer le genre de codes | > typiques que tu écrirais pourqu'on puisse juger si cela serait plus | > concis et plus lisible ? | | class message | { | typedef std::map<std::string,std::string> map ; | public: | std::string & get_value(std::string key) const | throw(std::range_error) | { | // return m_value_list[key] ; // ça marche pas © | | map::const_iterator s = m_value_list.find(key) ; | if(s == m_value_list.end()) | throw std::range_error("message::get_value()") ; | | return s->second ; | } | | protected: | map m_value_list ; | } ; Ça c'est le code qui implémente l'équivalent de operator[] ; ce que je demandait, c'est exemple d'utilisation. | | | > le type stocké est donné par value_type -- donc, cela renvoie bien la | > même chose pour vector, list, et map. Au contraire, c'est operator[] | > qui ne renvoie pas toujours une référence sur value_type. | | Ben non ![]() Comment ça non ? -- Gaby |
|
|
|
#18 |
|
Messages: n/a
Hébergeur: |
On Apr 22, 8:15 pm, Gabriel Dos Reis <g...@cs.tamu.edu> wrote:
> Mickaël Wolff <mickael.wo...@laposte.net> writes: > > Gabriel Dos Reis a écrit : > > > Comme je l'avais suggéré plus haut, peux-tu montrer le > > > genre de codes typiques que tu écrirais pourqu'on puisse > > > juger si cela serait plus concis et plus lisible ? > > class message > > { > > typedef std::map<std::string,std::string> map ; > > public: > > std::string & get_value(std::string key) const > > throw(std::range_error) > > { > > // return m_value_list[key] ; // ça marche pas © > > > > map::const_iterator s = m_value_list.find(key) ; > > if(s == m_value_list.end()) > > throw std::range_error("message::get_value()") ; > > > > return s->second ; > > } > > protected: > > map m_value_list ; > > } ; > Ça c'est le code qui implémente l'équivalent de operator[] ; Pas tout à fait, vue qu'il fonctionne sur un objet const, et n'insère rien. Comme il me semble avoir déjà dit, selon l'utilisation, le comportement voulu dans l'absense d'une entrée varie. Dans un certain sens, l'abstraction de base de std::map ne fait pas de choix : elle donne les fonctions habituelles find, insert et erase, pour que tu implémentes ce qu'il te faut au-dessus (comme ici). AMHA, l'opérateur [] ne fait pas réelement partie de l'abstraction de base de std::map ; il est présente plutôt par commodité, pour permettre son utilisation comme un tableau AWK ou perl, dans le cas où ça convient. > ce que je demandait, c'est exemple d'utilisation. C'est un bon exemple d'une utilisation typique de l'abstraction de base, qui est, à mon avis, conçue précisement pour supporter ce genre d'utilisation. > > > le type stocké est donné par value_type -- donc, cela > > > renvoie bien la même chose pour vector, list, et map. Au > > > contraire, c'est operator[] qui ne renvoie pas toujours > > > une référence sur value_type. > > Ben non ![]() > Comment ça non ? Le problème, je crois, c'est que la STL considère std::map comme une collection, or que la plupart du temps, on l'utilise comme une application (au sens mathématique). Et dans ces utilisations, « value_type » a une autre signification ; pense à « attribute-value pair », un concepte bien connu. À cet égard, la bibliothèque Java, qui distingue entre le concepte de collection et celui de map, correspond plus aux utilisations courantes. Ceci dit, Java fournit aussi un type Map::Entry comme partie de l'interface contractuelle, avec aussi la possibilité d'itérer sur ces entrées. C-à-d que pragmatiquement, il doit y avoir aussi des cas où on veut bien le considérer comme une collection (ce que Java résoud par des « view »). Peut-être ce qui manque à la STL, ici, c'est ce concepte de « map » (ou « indexable », si on veut y inclure std::vector). En attendant, en revanche, il n'est pas trop difficile à implémenter soi-même, sur les bases qu'on a. -- James Kanze (GABI Software) email:james.kanze@gmail.com Conseils en informatique orientée objet/ Beratung in objektorientierter Datenverarbeitung 9 place Sémard, 78210 St.-Cyr-l'École, France, +33 (0)1 30 23 00 34 |
|
|
|
#19 |
|
Messages: n/a
Hébergeur: |
James Kanze <james.kanze@gmail.com> writes:
| On Apr 22, 8:15 pm, Gabriel Dos Reis <g...@cs.tamu.edu> wrote: | > Mickaël Wolff <mickael.wo...@laposte.net> writes: | | > > Gabriel Dos Reis a écrit : | | > > > Comme je l'avais suggéré plus haut, peux-tu montrer le | > > > genre de codes typiques que tu écrirais pourqu'on puisse | > > > juger si cela serait plus concis et plus lisible ? | | > > class message | > > { | > > typedef std::map<std::string,std::string> map ; | > > public: | > > std::string & get_value(std::string key) const | > > throw(std::range_error) | > > { | > > // return m_value_list[key] ; // ça marche pas © | > > | > > map::const_iterator s = m_value_list.find(key) ; | > > if(s == m_value_list.end()) | > > throw std::range_error("message::get_value()") ; | > > | > > return s->second ; | > > } | | > > protected: | > > map m_value_list ; | > > } ; | | > Ça c'est le code qui implémente l'équivalent de operator[] ; | | Pas tout à fait, vue qu'il fonctionne sur un objet const, et | n'insère rien. Il voulait un operator[] qui n'insère pas, mais lève une exception pour un object const. N'est-ce pas ce que le code ci-dessus ? Ma question est comment utilise-t-il operator[] (ou getValue()) typiquement pour si cela ressemble à l'utilisation tyoique de vector<>: perator[].| Comme il me semble avoir déjà dit, selon l'utilisation, le | comportement voulu dans l'absense d'une entrée varie. Dans un | certain sens, l'abstraction de base de std::map ne fait pas de | choix : elle donne les fonctions habituelles find, insert et | erase, pour que tu implémentes ce qu'il te faut au-dessus (comme | ici). J'ai bien compris cela, merci. Ma question est différente. De mon point de vue, pour arguer que operator[] doit exister dans la STL (qui rappelonsl-e, au cas où, fournit un cadre *générique*) et renvoyer une exception, je voudrais voir du code pour comparer le style d'utilisation et ce qu'il a montré ci-dessus ne correspond pas à ce je demandais. Par example prend la fonction at(). Elle a été ajoutée, mais si tu regardes bien son utilisation typique reseemble à ce qu'on fait avec vector try { // attempt at() } catch(const out_of_range&) { // oops someone wasn't paying attention. } De ce point de vue là, il devient tenable d'arguer de similarité. Mais si tu *ne peux pas* montrer du code typique pour les utilisations sont similaires, l'argument qu'il faudrait ajouter operator[] parce qu'un autre conteneur l'a est faible. | AMHA, l'opérateur [] ne fait pas réelement partie de | l'abstraction de base de std::map ; il est présente plutôt par | commodité, pour permettre son utilisation comme un tableau AWK | ou perl, dans le cas où ça convient. Ça c'est du blah blah blah, je veux voir du code d'utilisation pour juger. [...] | > ce que je demandait, c'est exemple d'utilisation. | | C'est un bon exemple d'une utilisation typique de l'abstraction | de base, qui est, à mon avis, conçue précisement pour supporter | ce genre d'utilisation. Non, désolé. | > > > le type stocké est donné par value_type -- donc, cela | > > > renvoie bien la même chose pour vector, list, et map. Au | > > > contraire, c'est operator[] qui ne renvoie pas toujours | > > > une référence sur value_type. | | > > Ben non ![]() | | > Comment ça non ? | | Le problème, map<Key,T>::value_type , c'est une pair<const Key,T>, et map<Key,T>: perator[] renvoie T&. vector<T>::value_type c'est T,et vector<T>: perator[] renvoie une référence (const ou non) sur T.Si tu affirmes « ben non », alors tu dois utiliser quelque chose qui n'est pas C++. -- Gaby |
|
![]() |
| Outils de la discussion | |
|
|