Discussion :

[infotron]

Bonjour,

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

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
 
/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

[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?

[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.

[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.

[infotron]

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

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
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 :

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

1
2
 
g++ test.cpp

[Luc Hermitte]

Qui se minimalise à

Code :

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

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.

[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 :

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

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.

[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.

[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.

[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.

[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).

[infotron]

Merci pour ces précisions.

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 :

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
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 :

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

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 ?

[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.

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.

[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 ?

[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.

[infotron]

Ok, encore merci