[g++] message erreur etrange

Version imprimable

Bonjour,

J'ai compilé un programme avec g++ sous linux et voici ce qu'il m'a repondu :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
/home/nico/tmp/ccJMYeTF.o(.text+0xadf): In function `main':
: undefined reference to `EnsExamples::~EnsExamples [in-charge]()'
/home/nico/tmp/ccJMYeTF.o(.text+0xb24): In function `main':
: undefined reference to `example::~example [in-charge]()'
/home/nico/tmp/ccJMYeTF.o(.text+0xb38): In function `main':
: undefined reference to `example::~example [in-charge]()'
/home/nico/tmp/ccJMYeTF.o(.text+0xb4a): In function `main':
: undefined reference to `example::~example [in-charge]()'
/home/nico/tmp/ccJMYeTF.o(.text+0xb86): In function `main':
: undefined reference to `EnsExamples::~EnsExamples [in-charge]()'
/home/nico/tmp/ccJMYeTF.o(.text+0xbe5): In function `main':
: undefined reference to `example::~example [in-charge]()'
/home/nico/tmp/ccJMYeTF.o(.text+0xc13): In function `main':
: undefined reference to `example::~example [in-charge]()'
/home/nico/tmp/ccJMYeTF.o(.text+0xc39): In function `main':
: undefined reference to `example::~example [in-charge]()'
collect2: ld returned 1 exit status

J'ai encore jamais vu ce type de message, et il me semble un peu ... etrange a mon gout.
Si quelqu'un voit d'ou ca vient merci de me le faire savoir.

Merci d'avance :wink:

17/09/2004, 23h14
alexrtz

Salut,

Cela ressemble à un problème d'édtition de liens.

Pourrais-tu mettre le code (ou un lien vers ton fichier source ou une archive si le code est très long) pour que l'on puisse tester?
18/09/2004, 02h47
sir_gcc

Je dirais la même chose, tu as du oublié d'include une bibliothèque, ou un truc du genre. Poste ton code.
18/09/2004, 09h38
Luc Hermitte

Dans mes souvenirs, il y a deux raisons pour cetype d'erreurs. L'une d'entre elle est l'oubli de définir (ou de lier avec la définition) des functions, genre le destructeur.

Si ce n'est pas ça, il faudrait que tu nous montres un exemple minimal mais complet qui reproduise cette erreur avec les lignes de commandes utilisées pour compiler.

Voici un bout de code qui me reproduit la meme erreur :

Code:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
 
#include <iostream>
 
using namespace std;
 
 
//struct of an example
struct example
{
  char *in;
  char *out;
 
  example(char [] , char []);
 ~example(void);
};
 
example::example(char ti[] , char to[])
{
  in = ti;
  out = to;
 
  return;
}
 
 
//struct of an array of examples
class EnsExamples
{
  example *pointex;
 
public :
  EnsExamples(example []);
  ~EnsExamples(void);
  void info (void);
};
 
EnsExamples::EnsExamples(example ex[])
{
  pointex = ex;
  return;
}
 
 
void EnsExamples::info(void)
{
  cout <<"There is "<<sizeof(pointex)<<"examples in this array"<<endl;
  return;
}
 
 
 
int main (void)
{
  char in1[1]={1};
  char out1[3]={1,0,1};
  char in2[1]={0};
  char out2[3]={0,0,0};
  example e1(in1,out1);
  example e2(in2,out2);
  example tab [2]={e1,e2};
 
  EnsExamples ex0(tab);
 
  ex0.info();
 
  return 0;
}

pour la compilation je fait tout simplement :

Code:

1
2
 
g++ test.cpp

Qui se minimalise à
Code:

1 2 3 4 5 6 7 8 9 struct toto { ~toto(); // pas de void en C++ }; int main() // pas de void en C++ { toto t; return 0; }
Et c'est bien ce que j'avais "hyppothésé".
Tu n'as pas défini les destructeurs.

18/09/2004, 12h24
infotron
Oups ! En effet j'avait oublié de définir mes destructeurs. (c'est a cause des pointeurs que ce ne marchait pas ?)
Voici comment je les ai definis :
Code:

1 2 3 4 5 6 7 8 9 10 11 example::~example(void) { if(in!=NULL)delete in; if(out!=NULL)delete out; } EnsExamples::~EnsExamples(void) { if(pointex!=NULL) delete pointex; }
Je sais si c'est la bonne méthode , en tout cas ca marche.

Merci beaucoup à ceux qui m'ont mis sur la voie.
:wink:
18/09/2004, 12h49
Luc Hermitte

Là par contre, ce code va t'exploser à la figure -- je ne dis pas ça parce qu'il n'est pas robuste, c'est un autre problème.

Tu veux libérer un tableau (non libérable qui existe à l'extérieur)
Un conseil passe à des types qui gèrent correctement la mémoire pour toi. std::string pour les chaînes de caractères et std::vector<> pour les tableaux.

PS: pas besoin de tester à null avant de libérer. Dans tous les cas delete fera aussi ce test.
18/09/2004, 17h02
infotron

Hum .... ok
donc en c++ , il n'y a pas d'intérêt à utilisé des tableaux, et il faudrait utilisé std::vector à chaque fois ? Pour le moment j'ai pris l'habitude d'utiliser les tableaux, et je n'utilise std::vector que lorsque j'en trouve l'utilité, mais je pourrais changer si il n'y a pas d'inconvénient à utiliser std::vector.
18/09/2004, 17h24
Luc Hermitte

Pas d'intérêt à haut niveau. Plutôt des désanvatages en fait aux tableaux dynamiques bruts.
En fait, on a tendance à éviter de confier la gestion brute de deux ressources à une même classe. Ce qui fait que l'on va fuir l'utilisation de pointeurs bruts. Dans le cadre des tableaux dynamiques, les vecteurs sont une assez bonne solution.
18/09/2004, 18h51
Selenite

Et il ne faut pas oublier les copains de std::vector que sont std::list, std::deque et std::set (et std::map, mais celui là n'est pas tout à fait pareil).

Merci pour ces précisions.

Citation:

Envoyé par Luc Hermitte

Un conseil passe à des types qui gèrent correctement la mémoire pour toi. std::string pour les chaînes de caractères et std::vector<> pour les tableaux.

J'ai converti mon bout de prog en utilisant std::vector

Code:

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
 
#include <iostream>
#include <vector>
 
using namespace std;
 
 
//struct of an example
class Example
{
  vector <bool> in;
  vector <bool> out;
 
public :
  Example(vector <bool> ,vector <bool>);
  ~Example(void);
};
 
Example::Example(vector <bool> ti , vector <bool> to)
{
  in = ti;
  out = to;
 
  return;
}
 
 
//struct of an array of examples
class EnsExamples
{
  vector <Example> ex;
 
public :
  EnsExamples(vector <Example>);
  ~EnsExamples(void);
  void info (void);
};
 
EnsExamples::EnsExamples(vector <Example> tex)
{
  ex = tex;
  return;
}
 
 
void EnsExamples::info(void)
{
  cout <<"There is "<<ex.size()<<"examples in this array"<<endl;
  return;
}
 
 
 
int main (void)
{
 
  return 0;
}

J'obtiens ce message :

Code:

1
2
3
4
5
 
 g++ test.cpp
/home/nico/tmp/cc5HXqsK.o(.gnu.linkonce.t._ZSt8_DestroyI7ExampleEvPT_+0xd): In function `void std::_Destroy<Example>(Example*)':
: undefined reference to `Example::~Example [in-charge]()'
collect2: ld returned 1 exit status

std::vector gere t-il vraiment correctement la mémoire ?

19/09/2004, 22h36
Laurent Gomila

Si tu déclares ton destructeur, il faut lui donner un corps. Ici tu n'en as plus besoin, enlève le donc de la déclaration de ta classe et tu ne devrais plus avoir d'erreur.

Citation:

std::vector gere t-il vraiment correctement la mémoire ?

Oui, c'est le but. Il gère la réallocation et la destruction de manière automatique et sûre.

Sinon petite remarque : pour des gros paramètres du genre vector, il vaut mieux les passer par référence constante.
19/09/2004, 22h47
infotron

Merci, en effet ca fonctionne. Désolé de vous avoir dérangés pour ca !

Quelles sont les avantages de passer les gros paramètres en référence constante ?
19/09/2004, 23h08
Laurent Gomila

Le passage par valeur entraîne une recopie du paramètre, c'est-à-dire que si ton vecteur contient 1000 éléments, un autre vecteur de 1000 élements sera créé... pour rien. Avec le passage par référence constante, tu ne paies qu'une copie de pointeur (en interne, toi tu ne voies rien), donc en général 4 octets, ce qui est une miette en comparaison.
19/09/2004, 23h14
infotron

Ok, encore merci :wink: