pb de dédoublement du même objet

pb de dédoublement du même objet - C++ - Programmation

Marsh Posté le 28-02-2005 à 10:39:13    

j'ai un problème dans le code suivant:
 

Code :
  1. typedef struct _SonContainer {
  2. ThreatNode *a_node;
  3. _SonContainer *next;
  4. } SonContainer;
  7. class ThreatNode {
  8. private:
  9. SonContainer *son;
  10. OPERATOR op;
  11. char name[NAME_SIZE];
  12. char comment[COMMENT_SIZE];
  13. int probability[PROBABILITY_SIZE];
  14. bool is_root;
  15. bool is_treated;
  16. public:
  17. ThreatNode(OPERATOR, bool is_root = false);
  19. OPERATOR getOperator();
  20. void setOperator(OPERATOR);
  21.     void printProbability();
  22. void setProbability(int[PROBABILITY_SIZE]);
  23. int getProbability(int);
  25. bool isRoot();
  26. void setIsRoot();
  28. bool isTreated();
  29. void setIsTreated();
  31. char* getName();
  32. void setName(char*);
  34. char* getComment();
  35. void setComment(char*);
  38. SonContainer* getSon();
  39. void addSon(ThreatNode*);
  41. int evaluate(int); // evaluate the probability
  44. void printNode();
  46. };


 
 

Code :
  1. int ThreatNode::evaluate(int k) {
  2. cout << "je print la prob de "<<this->getName()<<"\n";
  3. this->printProbability();
  4. SonContainer *a_son_container, *a_brother_container;
  5. ThreatNode *a_son, *a_brother;
  6. int prob, final_prob;
  8. if (this->isTreated())
  9.  return this->probability[k];;
  11. //a_son = this->son;
  12. a_son_container = this->son;
  14. if (a_son_container == NULL) {
  15.  if (this->op != LEAF)
  16.   printf("WARNING: the node %s should be LEAF\n", this->getName());
  17.  return this->probability[k];
  18. }
  20. a_son = a_son_container->a_node;
  21. final_prob = a_son->evaluate(k);
  22. a_son->setIsTreated();
  24. a_brother_container = a_son_container->next;
  26. while (a_brother_container != NULL) {
  27.  a_brother = a_brother_container->a_node;
  28.  prob = a_brother->evaluate(k);
  30.  a_brother->setIsTreated();
  32.  if (this->op == AND)
  33.   final_prob = min_prob(final_prob, prob);
  34.  else
  35.   final_prob = max_prob(final_prob, prob);
  37.  a_brother_container = a_brother_container->next;
  38. }
  39. this->probability[k] = final_prob;
  40. return final_prob;
  41. }


 
 
le pb vient d'ici:
 
dans mon main je fais:
ThreatNode *root;
...
...
root->evaluate(0);
root->evaluate(1);
 
le problème est le suivant: quand je fais root->evaluate(0) tout se passe bien bien, mais quand je fais root->evaluate(1), il ignore le traitement que j'ai fait sur l'objet root en faisant evaluate(0), comme si je dédoublais l'objet chose que je ne veux pas faire, je veux travaillé sur les mêmes objets.
 
si quelqu'un comprend.
Merci de m'expliquer

Reply

Marsh Posté le 28-02-2005 à 10:39:13   

Reply

Marsh Posté le 28-02-2005 à 10:46:39    

tiens du C

Reply

Marsh Posté le 28-02-2005 à 10:49:16    

Taz a écrit :

tiens du C


 
je sais, je suis pas un cador en C++ mais j'ai besoin d'un petit coup de main, car la je vois vraiment pas.
 
Merci

Reply

Marsh Posté le 28-02-2005 à 10:58:05    

Laisse moi deviner: tu t'attends à ce que evaluate(1) te renvoie le même résultat que le evaluate(0) qui précède, grace au calcul de la boucle en fin de fonction qui calcul une prob finale en fonction de tous les noeuds enfants, c'est bien ça ?

Reply

Marsh Posté le 28-02-2005 à 11:02:24    

pretorienx a écrit :

je sais, je suis pas un cador en C++ mais j'ai besoin d'un petit coup de main, car la je vois vraiment pas.
 
Merci

aucun rapport. tu te complique le travail tout seul. remplace tous tes char* et char[] par des std::string, ça fera déjà un problème en moins. Et je ne parle même pas des const

Reply

Marsh Posté le 28-02-2005 à 11:08:08    

Lam's a écrit :

Laisse moi deviner: tu t'attends à ce que evaluate(1) te renvoie le même résultat que le evaluate(0) qui précède, grace au calcul de la boucle en fin de fonction qui calcul une prob finale en fonction de tous les noeuds enfants, c'est bien ça ?


 
non c pas ça.
En faite evaluate(0) doit me renvoyer une proba pour une catégorie de personne et evaluate(1) pour une autre catégorie.  
Et pour ce faire chaque objet possède en donnée membre un tableau de proba, chaque case correspond a 1 type de personne.
Ce que je pensais c ke evaluate 0 va travailler sur tout mes objets, mais précisément sur la donnée membre probability[0],... et de même pour evaluate 1 qui devrait travailler sur probability[1] de tout mes objets.
mais je contaste que root->evaluate(0); puis root->evaluate(1)
ne fais pas ce je j'escompte: on dirait que c 2 root différents alors que c le même., il fait le traitement sur evaluate(0); et le fait sur evaluate(1) sans tenir compte que evaluate(0) a modifier la donnée membre probability[0].
 
Voila.
J'espère avoir été claire.
Un petit coup de main serait le bien venu

Reply

Marsh Posté le 28-02-2005 à 11:17:19    

Taz a écrit :

aucun rapport. tu te complique le travail tout seul. remplace tous tes char* et char[] par des std::string, ça fera déjà un problème en moins. Et je ne parle même pas des const


 
cela me simplifera la vie en quoi ?
et pour les const ?
 
je vois pas trop ce que cela changerait de mettre des std::string

Reply

Marsh Posté le 28-02-2005 à 11:21:52    

pretorienx a écrit :

cela me simplifera la vie en quoi ?
et pour les const ?
 
je vois pas trop ce que cela changerait de mettre des std::string


La correction ne va pas tarder à suivre...  :whistle:  


---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Marsh Posté le 28-02-2005 à 11:49:51    

je me casse

Reply

Marsh Posté le 28-02-2005 à 11:51:52    

Taz a écrit :

je me casse


 
 
je pensais que les gens de ce forum était là pour aider les autres, et pas pour se la peter quand ils savent et que les autres non ?

Reply

Marsh Posté le 28-02-2005 à 11:51:52   

Reply

Marsh Posté le 28-02-2005 à 12:10:24    

Taz a écrit :

je me casse


Non, allez, reviens, explique-lui.


---------------
Now Playing: {SYNTAX ERROR AT LINE 1210}
Reply

Marsh Posté le 28-02-2005 à 14:39:09    

aulieu de baragouiner du mauvais C, utilise std::string, std::list et tu verras que ton probleme sera deja plus clair :o

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed