Discussion :

[tcharles]

bonjour à tous,
j'ai l'erreur suivante dans mon code que je ne comprends pas du tout

Code :

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

1
2
3
4
 
aFDPR.C:3355: no matching function for call to `CaFDPR::startNode(__gnu_cxx::__normal_iterator<Csegment*, std::vector<Csegment,
   std::allocator<Csegment> > >&)'
aFDPR.h:399: candidates are: Cnode CaFDPR::startNode(Csegment*)

les déclarations et la ligne de code 3355 sont les suivantes.

Code :

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

1
2
3
4
 
Cnode Cnode_11;
vector<Csegment>::iterator iter_Csegment1;
Cnode_11 = startNode(iter_Csegment1);

je dois aussi dire que c'est un code developpé au départ sur sun (solaris) ou cette erreur n'apparait pas, et qui est maintenant exécuté sur linux.

merci de me répondre

[JolyLoic]

J'ai l'impression (tu donne très peu d'infos, coupe un peu moins la prochaine fois, laisse au moins le message d'erreur en entier) que le code a été développé avec un compilateur où les itérateurs sur un vecteurs sont de simples pointeurs.

Or, même si c'est une possibilité offerte, rien ne l'oblige, et quand ce n'est pas le cas, et qu'une fonction attend un pointeur, on ne peut pas lui passer d'itérateur. Ecrit plutôt :

Code :

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

1
2
3
 
Csegment *segmentPtr = &v[0]; // a priori, je dirais qu'il faut l'initialiser, par exemple, j'ai initialiser sur le premier éléments d'un vecteur v, supposé non vide
Cnode_11 = startNode(iter_Csegment1);

[tcharles]

merci pour la reponse, mais je ne te comprend pas tres bien ton code

Code :

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

1
2
3
 
Csegment *segmentPtr = &v[0]; // a priori, je dirais qu'il faut l'initialiser, par exemple, j'ai initialiser sur le premier éléments d'un vecteur v, supposé non vide
Cnode_11 = startNode(iter_Csegment1);

sinon j'ai procede comme suit:

Code :

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

1
2
3
4
5
6
 
Cnode Cnode_11;
vector<Csegment>::iterator iter_Csegment1;
Csegment  *CsegmentPtr;
*CsegmentPtr = *iter_Csegment1;
Cnode_11 = startNode(CsegmentPtr);

et ca compile, je pense que c'est la ou tu voulais en venir, ou bien??

[Laurent Gomila]

Ta fonction attend un pointeur en paramètre, toi tu lui passais un itérateur. Pour obtenir un pointeur sur un élément (par exemple le premier) il suffit de prendre son adresse, d'où le code de Loïc.

Par contre celui-ci contient une petite coquille, voilà la version correcte :

Code :

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

1
2
Csegment *segmentPtr = &v[0];
Cnode_11 = startNode(segmentPtr);

Quant à ton code, il est incorrect car :
- Ton itérateur n'est pas initialisé
- Ton pointeur n'est pas initialisé
- Tu fais une copie de l'élément, ce qui n'est pas forcément souhaité

[tcharles]

merci pour vos réponses, en fait pour ce qui est de l'initialisation j'ai la boucle suivante:

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
 
vector<Csegment>  m_vecSegments;
vector<Csegment>::iterator iter_Cseg1;
vector<Csegment>::iterator iter_Cseg2;
Csegment *CsegPtr1, *CsegPtr2;
//check all Segments:
for(iter_Cseg1=m_vecSegments.begin();iter_Cseg1!=m_vecSegments.end();iter_Cseg1++)
    {
      iter_Cseg2 = iter_Cseg1;
      iter_Cseg2++;
      *CsegPtr1 = *iter_Cseg1;
      Cnode_11 = startNode(CsegPtr1);
       while(iter_Cseg2 != m_vecSegments.end())
      {
             ......
             if (......){
             *CsegPtr2 = *iter_Cseg2;
              Cnode_21 = startNode(CsegPtr2);
            }
      }
    }

voila à peu pret comment mon code se présente.
merci de vos suggestions

[loufoque]

C'est beau...

[Laurent Gomila]

Toujours les mêmes erreurs, ça va te péter à la tête à l'exécution

[loufoque]

Déjà, il devrait même pas y avoir de pointeurs dans ton truc.

Tu te rends compte quand que tu déréferences des pointeurs non initialisés ? Le genre de truc qui fait de suite une segfault (et encore, si t'es chanceux) ?

[tcharles]

mais dis donc, vous me foutez la trouille la. Pour ce qui est du pointeur il est demander par la procedure

Code :

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

1
2
 
Cnode CaFDPR::startNode(Csegment*)

C'est la raison pour laquelle je dois avoir un pointeur avant, sinon j'ai fait les initialisations suivantes je ne sais si elles donne un peu de sens a 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
 
 
vector<Csegment>  m_vecSegments;
vector<Csegment>::iterator iter_Cseg1;
vector<Csegment>::iterator iter_Cseg2;
Csegment *CsegPtr1  = &m_vecSegments[0];
Csegment *CsegPtr2 = &m_vecSegments[0];
 
//check all Segments:
for(iter_Cseg1=m_vecSegments.begin();iter_Cseg1!=m_vecSegments.end();iter_Cseg1++)
    {
      iter_Cseg2 = iter_Cseg1;
      iter_Cseg2++;
      *CsegPtr1 = *iter_Cseg1;
      Cnode_11 = startNode(CsegPtr1);
       while(iter_Cseg2 != m_vecSegments.end())
      {
             ......
             if (......){
             *CsegPtr2 = *iter_Cseg2;
              Cnode_21 = startNode(CsegPtr2);
            }
      }
    }

j'attend vos suggestions, merci

[loufoque]

Sans savoir ce que ton code est censé faire, c'est difficile.
De même, tu appelles une fonction alors qu'apparemment tu n'as aucune idée de ce qu'elle fait.

[tcharles]

le bout de code que je presente parour un ensemble de segments et pour chaque segment renvoie le noeud de depart (Cnode = startNode(Csegment*)).

[loufoque]

Ce qu'il te faut faire c'est simplement startNode(&*iter_Cseg1)