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