Discussion :

[julie_n3k0]

bonjour,

j'ai un probleme pour l'insertion d'objets dans un set...

alors voila à peu près mon code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
 
class A {
   public : 
      void insert(string s){
         set.insert(s); 
         //ici, s est bien dans set
      }
   private : 
      set<std::string> set;
}
 
class B {
   public :
      void createObject(string key, string s){
         map.find(key).insert(s);
         //mais ici il n'y est plus ...
      }
      void insert(string s, A a){
         map.insert(make_pair(s,a));
      }
   private :
      map<string, A> map;
}
 
main(){
   B b;
   A a;
   b.createObject("toto",a );
}

Alors voila le souci: lors de la création d'objet, j'insert dans le set de A un string lorsque l'on ets dans A::insert, le string ets bien inséré mais dès que l'on qui la méthode, le string n'y ets plus ...

Si vous avez besoin de plus d'explications demander, j'essaierai de reformuler mon problème.

Si quelqu'un pouvait m'expliquer ce phénomène et aussi comment y remédier, ce serai cool

Merci d'avance pour l'aide apportée

[koala01]

Salut, et bienvenue sur le forum

Avant tout, je souhaiterais te donner quelques conseils d'ordre général:

D'abord, lorsqu'il s'agit de fournir un argument à une fonction, de penser à le passer par référence (constante ou non) dés qu'il ne s'agit pas d'un type primitif.

En effet, lorsque tu passe un argument par valeur (ex: void foo(string s) ), il y a copie de l'objet transmis.

Cette copie risque de prendre énormément de ressources (en terme de mémoire et de temps) et peut même poser un certain nombre de problèmes car certaines classes ne sont pas destinées à être copiées.

La référence va agir comme un alias de l'objet à transmis et permettra donc d'en éviter la copie.

Par contre, comme il s'agit d'un alias de l'objet transmis, les modifications apportées à l'objet dans la fonction appelée seront répercutées dans la fonction appelante.

C'est la raison pour laquelle, si l'objet transmis n'est pas destiné à être modifié dans la fonction appelée, il est conseillé de le transmettre par référence constante.

L'avantage d'agir ainsi sera en plus de permettre de créer une variable temporaire non nommée, et donc de permettre un code proche de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
/* soit la fonction */
void foo(string const & s)
{
    ...
}
int main()
{
    foo("Salut toi"); /* il existe un constructeur de string prenant une
                       * chaine de caractères "C style"
                       */
    return 0;
}

En outre, je présumes (peut être à tord) que les types map, set et string que tu manipule sont ceux fournis par le standard et que tu as placé une directive using namespace std; quelque part...

Cette directive ne devrait jamais se trouver dans un fichier d'en-tête ( *.h ou *.hpp) car elle a la sale manie de se propager dans tous les fichiers dans lesquels le fichier d'en-tête est inclus, que ce soit de manière directe ou indirecte (elle sera appliquée dans le fichier qui inclus le fichier qui inclus le fichier qui... le fichier dans lequel la directive se trouve)

Mais, quoi qu'il en soit, il est souvent dangereux de créer une variable dont le nom est... celui qui correspond au nom du type manipulé...

Il serait donc des plus intéressants de trouver un nom plus opportun pour représenter les membres A::set et B::map...

Enfin, la réponse est finalement toute simple: ton code est tout bonnement incomplet et faux

En effet, la fonction membre de la classe map find va renvoyer un itérateur sur l'objet correpondant à la clé recherchée... s'il existe.

S'il n'existe pas, tu obtiendra un itérateur pointant sur... ce qui suit le dernier élément de la map et donc, tout à fait invalide.

De plus, il faut savoir que:

• Les itérateurs ont une sémantique de pointeurs (pour accéder aux données qu'ils représentent, il faut soit prendre "ce qui est pointé par" l'itérateur, soit utiliser la flèche "->" pour accéder à la donnée)
• Les itérateurs de la classe map représente un couple (une std::pair) cle / valeur (la cle étant accessible sous le nom de first et la valeur étant accessible par second).

Pour que ton code fonctionne, il faut donc:

1. vérifier que l'itérateur renvoyé par map.find() n'est pas équivalent à map.end()
2. si c'est le cas, il faut
   
   1. rajouter un nouvel objet dans map
   2. récupérer ce nouvel objet par find

une fois ceci fait, il faut donc veiller à accéder à la fonction insert() de ce qui est représenté par la valeur second de l'itérateur sur la map obtenu.

Le code serait donc proche de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
class B {
   public :
      void createObject(string key, string s){
         map<string,A>::iterator it= map.find(key);
         if(it = map.end())
         {
             insert(key,A());
             it=map.find(key);
         }
         (*it).second.insert(s);
      }
      void insert(string s, A a){
         map.insert(make_pair(s,a));
      }
   private :
      map<string, A> map;
}

[white_tentacle]

Pour que ton code fonctionne, il faut donc:

1. vérifier que l'itérateur renvoyé par map.find() n'est pas équivalent à map.end()
2. si c'est le cas, il faut
   
   1. rajouter un nouvel objet dans map
   2. récupérer ce nouvel objet par find

une fois ceci fait, il faut donc veiller à accéder à la fonction insert() de ce qui est représenté par la valeur second de l'itérateur sur la map obtenu.

C'est tout à fait exact, mais pourquoi ne pas lui dire que l'opérateur [] fait justement tout ça sans qu'on ait à s'en préoccuper

[koala01]

C'est tout à fait exact, mais pourquoi ne pas lui dire que l'opérateur [] fait justement tout ça sans qu'on ait à s'en préoccuper

Bonne question ... Peut être simplement parce que je n'y ait pas pensé

Je dois avouer que c'est un comportement qui m'oripile à ce point de l'opérateur [] que je ne l'utilise quasiment jamais...

Et voilà justement le cas où son utilisation s'avère intéressante

[julie_n3k0]

Bonjour,

Merci pour votre aide, effectivement, je modifiais des copies ...

Mais en tout cas t'inquiète pas koala01, j'ai bien un .cpp et un .hpp avec un using namespace std; juste dans le .cpp

J'ai bien tout corrigé et j'utilise des pointeurs partout

Merci encore pour votre aide, ça marche impeccablement !!

A plous sur ce forum !