Discussion :

[Roming22]

Bonjour a tous,

J'ai le bout de code suivant, qui ajoute un element dans une liste contenue dans une Map, verifie la valeur qui y est mise, puis affiche l'ensemble du contenu de la map.

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
//inserer l'element
_recordRulesMap[record].push_back(ProcessingRule(offset,length,processing.c_str()[0],callback));
 
//Verfier la valeur de l'element insere
ProcessingRule rule=*(_recordRulesMap[record].rbegin());
DEBUG("Rule : " << record <<":"<< rule._offset <<"-"<< rule._length <<"/"<< rule._processing << "-"<<rule._callback<<"\n");
 
map<string, list<ProcessingRule> >::iterator itMap;
list<ProcessingRule>::iterator itList;
int number=0;
 
//Parcourir la map
for(itMap=_recordRulesMap.begin(); itMap!=_recordRulesMap.end(); itMap++)
{
    //Parcourir toute la liste
    for(itList=itMap->second.begin(); itList!=itMap->second.end(); itList++)
    {
         //Afficher la valeur de l'objet
         ProcessingRule myRule = *(itList);
         DEBUG("Rule "<<++number<<" found: " << (*itMap).first <<":"<< myRule._offset <<"-"<< myRule._length <<"/" << myRule._processing << "-"<< myRule._callback<<"\n");
 
     }
 
}

Il se trouve que l'output me donne la chose suivante si j'ai 2 objet ProcessingRule dans la map:

Code :

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

1
2
3
4
5
6
 Rule : 01:91-13/ -0
Rule 1 found: 01:8917048-0/ -0
 
Rule : 01:107-49/ -0
Rule 1 found: 01:8918560-0/ -0
Rule 2 found: 01:8918560-0/ -0

J'ai l'impression que je n'accede pas comme il faut a l'element lorsque je parse toute la map.

Je seche severe, parce qu'il ne me semble pas que je me sois planter dans les pointeurs... Or c'est tres surement le cas.

Merci beaucoup pour l'aide que vous pourriez m'apporter,
Roming22

[Laurent Gomila]

Et... Qu'est-ce qui est incorrect dans tout ça au juste ? Parce que le code autant que la sortie ont l'air bons.

Tu peux donner la sortie attendue ?

[Edit] Le Rule inséré dans la map ne correspond pas à l'original c'est ça ? Tu peux commencer par vérifier ton constructeur par copie et ton opérateur d'affectation.

[Roming22]

Merci pour ton commentaire

Ce que je ne comprends pas, c'est que juste après l'avoir inséré, j'ai bien les valeurs que j'attends quand je fais

Code :

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

ProcessingRule rule=*(_recordRulesMap[record].rbegin());

alors que trois ligne plus loin, je fais exactement la même chose (tout du moins ilme semble que c'est la même chose, si ce n'est pas le cas, je veux bien une petite explication)

Code :

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

ProcessingRule myRule = *(itList);

et là, paf, mes valeurs ne sont plus initialisées pareil.

De mémoire mon constructeur par copy est le suivant

Code :

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

1
2
3
4
5
6
7
8
 
ProcessingRule(const ProcessingRule& iRule)
{
  offset=iRule.offset;
  length-iRule.length;
  processing=iRule.processing;
  callback=iRulle.callback;
}

Je n'ai pas surchargé d'operateur d'affectation. Les quatres variables de la classe ProcessingRule sont de type int et char, et ne sont pas des pointeurs. Donc normalement, je n'ai pas besoin de le surcharger n'est-ce pas?

Voici l'output que je recherche

Code :

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

1
2
3
4
5
6
Rule : 01:91-13/ -0
Rule 1 found: 01:91-13/ -0
 
Rule : 01:107-49/ -0
Rule 1 found: 01:91-13/ -0
Rule 2 found: 01:107-49/ -0

Merci pour ton aide
Roming22

[Laurent Gomila]

length-iRule.length;

Faute dans le message ou dans le code ?

Sinon effectivement tu n'as pas besoin d'écrire l'opérateur d'affectation, d'ailleurs le constructeur par copie non plus.

[Roming22]

La faute est dans le message <img>

Par contre, mon constructeur par copie etait le suivant:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
ProcessingRule(const ProcessingRule& iRule)
{
  ProcessingRule(iRule.offset,iRule.length,iRule.processing,iRule.callback);
 
}
 
ProcessingRule (int offset, int length, char processing, int callback)
{
   _offset=offset;
   _length=length;
   _processing=processing;
   _callback=callback;
}

Visiblement, c'etait pas bon... Ca devait me construire un nouvel objet. Hors je pensais que ca me mettrait a jour mon objet.

J'ai donc changer mon constructeur par copie par celui que j'ai mis plus haut, et ca marche comme il faut.

Merci beaucoup pour ton aide.
Roming22

[Laurent Gomila]

Effectivement

http://c.developpez.com/faq/cpp/?pag...R_appel_chaine

Mais comme je te l'ai dit, ton constructeur par copie ne sert à rien.