Héritage de classes.

**Berzerk_** · 08/08/2006, 18h46

Bonjour,

Je suis toujours sur le projet pour lequel j'avais demandé de l'aide dans ce thread: http://www.developpez.net/forums/sho...d.php?t=154525

Après avoir un peu avancé, je suis face à un nouveau problème.

A la compilation j'obtiens cette erreur:

c:\Documents and Settings\Fabrice\Bureau\Project\C++\GATest\GADiploid.h(30) : error C2504: 'GABinaryString' : classe de base non définie

Celle-ci concerne le code suivant:

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
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
#ifndef _ga_binstr_h_
#define _ga_binstr_h_
 
#include <string.h>
#include <ga/gatypes.h>
#include <ga/garandom.h>
#include <ga/GABinStr.h>
#include <ga/GAGenome.h>
 
 
#define GA_BINSTR_CHUNKSIZE 32	  // size of the chunks of bits we allocate
 
 
class GADiploid : public GABinaryString, public GAGenome
{
	public:
		GADiploid(unsigned int s);
		GADiploid::GADiploid(const GADiploid& orig);
 
		virtual ~GADiploid();
		void copy(const GADiploid&);
 
	int resize(unsigned int);		// pass desired size, in bits
	int size() const {return sz;}
 
	short bit(unsigned int a, unsigned int c) const
	{
		if (c = '1')
			return(Chrom_1->bit(a));
		else 
			return(Chrom_2->bit(a));
	}
 
	short bit(unsigned int a, short val, unsigned int c)
	{	// set/unset the bit
		if (c = '1')
			return(Chrom_1->bit(a, val));
		else
			return(Chrom_2->bit(a, val));
	}
 
	int equal(const GADiploid & b,
		unsigned int r, unsigned int x, unsigned int l, unsigned int c) const
	{
		if (c = '1')
			return(Chrom_1->equal(b.Chrom_1[0], r, x, l));
		else
			return(Chrom_2->equal(b.Chrom_2[0], r, x, l));
	}
 
	void copy(const GADiploid & orig,
			unsigned int r, unsigned int x, unsigned int l, unsigned int c)
	{
		if (c = '1') Chrom_1->copy(orig.Chrom_1[0], r, x, l);
		else		 Chrom_2->copy(orig.Chrom_2[0], r, x, l);
	}
 
	void move(unsigned int r, unsigned int x, unsigned int l, unsigned int c)
	{
		if (c = '1') Chrom_1->move(r, x, l);
		else		 Chrom_2->move(r, x, l);
	}
 
	void set(unsigned int a, unsigned int l, unsigned int c)
	{
		if (c = '1') Chrom_1->set(a, l);
		else		 Chrom_2->set(a, l);
	}
 
	void unset(unsigned int a, unsigned int l, unsigned int c)
	{
		if (c = '1') Chrom_1->unset(a, l);
		else		 Chrom_2->unset(a, l);
	}
 
	void randomize(unsigned int a, unsigned int l, unsigned int c)
	{
		if (c = '1') Chrom_1->randomize(a, l);
		else		 Chrom_2->randomize(a, l);
	}
 
	void randomize(unsigned int c)
	{
		if (c = '1') Chrom_1->randomize();
		else		 Chrom_2->randomize();
	}
 
	GABinaryString *Chrom_1;
	GABinaryString *Chrom_2;
	//GABinaryString Mask;
 
	protected:
	unsigned int sz;		// size of chrom
	unsigned int SZ;		// size of the memory allocated
	unsigned int csz;		// size of chunks we allocate
 
};
 
#endif

Je travaille sur une bibliothèque que je cherche à étendre. J'ai repris l'un des fichiers de cette bibliothèque et l'ai modifié pour obtenir le code qui précède.

Pour résoudre l'erreur j'ai essayé de commenter le ifndef:

Code :

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

1
2
3
4
5
6
//#ifndef _ga_binstr_h_
//#define _ga_binstr_h_
 
[...]
 
//#endif

L'erreur disparaît, mais j'obtiens maintenant cette erreur:

c:\Documents and Settings\Fabrice\Bureau\Project\C++\GATest\GADiploid.cpp(35) : error C2512: 'GABinaryString' : aucun constructeur par défaut approprié disponible

Celle-ci concernant ce fichier:

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ga/gaerror.h>
#include <GADiploid.h>
 
 
 
GADiploid::GADiploid(unsigned int s): GABinaryString(s)
{
	Chrom_1 = new GABinaryString(s);
	Chrom_2 = new GABinaryString(s);
	//resize(s);
 
	//Mask = new GABinaryString(s);
}
 
GADiploid::GADiploid(const GADiploid& orig)
{
	sz=0; SZ=0; data=(GABit *)0;
	copy(orig);
}
 
void
GADiploid::copy(const GADiploid& orig)
{
  if(&orig == this) return;
  Chrom_1->copy(orig.Chrom_1[0]);
  Chrom_2->copy(orig.Chrom_2[0]);
}
 
int
GADiploid::resize(unsigned int x)
{
  Chrom_1->resize(x);
  Chrom_2->resize(x);
 
  return(sz = x);
}

Le compilateur me réclame un constructeur par défaut alors que celui-ci semble bien défini dans le fichier suivant (fichier que j'ai récupéré dans la bibliothèque, sans modification):

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
#ifndef _ga_binstr_h_
#define _ga_binstr_h_
 
#include <string.h>
#include <ga/gatypes.h>
#include <ga/garandom.h>
 
#define GA_BINSTR_CHUNKSIZE 32	  // size of the chunks of bits we allocate
 
 
class GABinaryString {
public:
  GABinaryString(unsigned int s){
    csz=GA_BINSTR_CHUNKSIZE; sz=0; SZ=0; data=(GABit *)0;
    resize(s);
  }
  GABinaryString(const GABinaryString& orig){
    sz=0; SZ=0; data=(GABit *)0;
    copy(orig);
  }
  virtual ~GABinaryString(){delete [] data;}
  void copy(const GABinaryString&);
  int resize(unsigned int);		// pass desired size, in bits
  int size() const {return sz;}
 
  short bit(unsigned int a) const {
    return(data[a]);
  }
  short bit(unsigned int a, short val) {	// set/unset the bit
    return(data[a] = (val ? 1 : 0));
  }
  int equal(const GABinaryString & b,
	    unsigned int r, unsigned int x, unsigned int l) const {
    return(memcmp(&(data[r]),&(b.data[x]),l*sizeof(GABit))?0:1);
  }
  void copy(const GABinaryString & orig,
	    unsigned int r, unsigned int x, unsigned int l){
    memcpy(&(data[r]), &(orig.data[x]), l*sizeof(GABit));
  }
  void move(unsigned int r, unsigned int x, unsigned int l){
    memmove(&(data[r]), &(data[x]), l*sizeof(GABit));
  }
  void set(unsigned int a, unsigned int l){
    memset(&(data[a]), 1, l*sizeof(GABit));
  }
  void unset(unsigned int a, unsigned int l){
    memset(&(data[a]), 0, l*sizeof(GABit));
  }
  void randomize(unsigned int a, unsigned int l){
    for(unsigned int i=0; i<l; i++) 
      data[i+a] = (GABit)GARandomBit();
  }
  void randomize(){
    for(unsigned int i=0; i<sz; i++) 
      data[i] = (GABit)GARandomBit();
  }
 
protected:
  unsigned int sz;		// size of chrom
  unsigned int SZ;		// size of the memory allocated
  unsigned int csz;		// size of chunks we allocate
  GABit *data;			// the data themselves
};
 
#endif

Deux questions:

-A quoi sert le code #ifndefine... #define... #endif que j'ai mis en commentaire?
-Pour quoi le compilateur me réclame-t-il un constructeur par défaut?

Merci d'avance pour vos réponses.

**Matthieu Brucher** · 08/08/2006, 19h09

Ben non, il n'y a pas de constructeur par défaut défini..
Les "_ga_binstr_h_" servent à protéger une redéfinition, mais si tu utilises la même macro - _ga_binstr_h_ - pour 2 fichiers d'en-têtes, le deuxième sera comme inexistant.
Il te demande un constructeur par défaut parce que dans un des constructeur de la classe fille, il est appelé.

**hiko-seijuro** · 08/08/2006, 19h09

le code que tu as mis en commentaire permet d'assurer l'unicité du parcours du fichier. voir ici

pour ta deuxièyme question poste seulement la ligne concernée

car il faut chercher et ça prend du temps

**Caine** · 08/08/2006, 19h17

Bonjour,

Le bloc ifndef...define...endif sert à protéger le header contre l'inclusion multiple.

Soit un entête A.h, sans cette clause et sans .c.

Immaginons deux modules B.cpp et D.cpp qui inclus chacun A.h. B est compilé avant D.

Au moment où B est compilé, le compilateur défini les symboles de l'entête A.h.

Ensuite, D est compilé. Le compilateur veut aussi définir les symboles de A.h. Problème: ceux-ci existent déjà. La compilation échoue.

Si les clause ifndef/def/endif sont présentes, lors de la compilation de D, le symbole A_H est défini, donc le compilayeur cherche un #else, comme il n'y en a pas, il saute au #endif. Les symboles ne sont pas redéfinis. La compilation continue.

Pour la deuxième question, règle d'abord le premier problème de classe de base non définie.

**Berzerk_** · 08/08/2006, 19h38

Envoyé par Miles

Ben non, il n'y a pas de constructeur par défaut défini..
Les "_ga_binstr_h_" servent à protéger une redéfinition, mais si tu utilises la même macro - _ga_binstr_h_ - pour 2 fichiers d'en-têtes, le deuxième sera comme inexistant.
Il te demande un constructeur par défaut parce que dans un des constructeur de la classe fille, il est appelé.

Pour la 1ère question j'ai trouvé, c'est juste que j'ai recopié bêtement le fichier d'origine sans modifier le nom dans le #define, donc ça faisait automatiquement une double définition.

Pour la 2ème question, j'ai repris le modèle d'un fichier de la bibliothèque (qui compile parfaitement quand je ne rajoute pas mes fichiers maisons).

Mon fichier, GADiploid, s'inspire du fichier GA1DBinStrGenome qui hérite du fichier GABinStrGenome.

Voilà le code du fichier d'origine.

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
#include <stdio.h>
#include <stdlib.h>
#include <ctype.h>
#include <string.h>
#include <ga/gaerror.h>
#include <ga/garandom.h>
#include <ga/GA1DBinStrGenome.h>
#include <ga/GAMask.h>
 
#define SWAP(a,b) {unsigned int tmp=a; a=b; b=tmp;}
 
/* ----------------------------------------------------------------------------
   Genome class definition
---------------------------------------------------------------------------- */
// Set all the initial values to NULL or zero, then allocate the space we'll
// need (using the resize method).  We do NOT call the initialize method at
// this point - initialization must be done explicitly by the user of the
// genome (eg when the population is created or reset).  If we called the
// initializer routine here then we could end up with multiple initializations
// and/or calls to dummy initializers (for example when the genome is 
// created with a dummy initializer and the initializer is assigned later on).
GA1DBinaryStringGenome::
GA1DBinaryStringGenome(unsigned int len, 
		       GAGenome::Evaluator f, void * u) :
GABinaryString(len),
GAGenome(DEFAULT_1DBINSTR_INITIALIZER, 
	 DEFAULT_1DBINSTR_MUTATOR,
	 DEFAULT_1DBINSTR_COMPARATOR) {
  evaluator(f);
  userData(u);
  crossover(DEFAULT_1DBINSTR_CROSSOVER); // assign the default sexual crossover
  nx=minX=maxX=0;
  resize(len);
}
 
 
// This is the copy initializer.  We set everything to the default values, then
// copy the original.  The BinaryStringGenome creator takes care of zeroing
// the data and sz members.
GA1DBinaryStringGenome::
GA1DBinaryStringGenome(const GA1DBinaryStringGenome& orig) :
GABinaryString(orig.GABinaryString::size()),
GAGenome() {
  nx=minX=maxX=0;
  GA1DBinaryStringGenome::copy(orig);
}
 
GA1DBinaryStringGenome::~GA1DBinaryStringGenome() {
}

Le code des deux autres fichiers est donné dans le post précédent.
Voilà la partie incriminée par le compilateur (j'ai noté l'emplacement de l'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
GADiploid::GADiploid(unsigned int s): GABinaryString(s)
{
	Chrom_1 = new GABinaryString(s);
	Chrom_2 = new GABinaryString(s);
	//resize(s);
 
	//Mask = new GABinaryString(s);
}
 
GADiploid::GADiploid(const GADiploid& orig)
{/*** Erreur ici ***/
	sz=0; SZ=0; data=(GABit *)0;
	copy(orig);
}

Comme je disais, il ne devrait pas y avoir de problème de classe par défaut, puisqu'il fonctionne avec l'autre fichier. Et j'aimerais éviter de modifier GABinStr et GA1DBinStrGenome, puisqu'ils font partie de la bibliothèque d'origine.

**Matthieu Brucher** · 08/08/2006, 19h57

Regarde mieux.

**Berzerk_** · 08/08/2006, 20h00

Envoyé par Miles

Regarde mieux.

Euh je veux bien, mais ça fait... 5 jours que je regarde.

Une petite piste (fichier? fonction?), please?

**Matthieu Brucher** · 08/08/2006, 20h02

Je t'ai dit de regarder les constructeurs.

Ton constructeur par recopie :

Code :

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

1
2
3
4
5
GADiploid::GADiploid(const GADiploid& orig)
{/*** Erreur ici ***/
	sz=0; SZ=0; data=(GABit *)0;
	copy(orig);
}

L'exemple que tu as mal repris :

Code :

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

1
2
3
4
5
6
7
GA1DBinaryStringGenome::
GA1DBinaryStringGenome(const GA1DBinaryStringGenome& orig) :
GABinaryString(orig.GABinaryString::size()),
GAGenome() {
  nx=minX=maxX=0;
  GA1DBinaryStringGenome::copy(orig);
}

Le pire est que ton erreur est exactement là où tu l'as indiqué, et c'est exactement là où tu n'as pas mis la liste d'initialisation.

**Berzerk_** · 08/08/2006, 22h25

Merci, c'était ça en effet.

Comme de bien entendu, une erreur vient en chasser une autre.

Code :

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

1
2
GADiploid.obj : error LNK2019: symbole externe non résolu "public: virtual __thiscall GADiploid::~GADiploid(void)" (??1GADiploid@@UAE@XZ) référencé dans la fonction "public: virtual void * __thiscall GADiploid::`scalar deleting destructor'(unsigned int)" (??_GGADiploid@@UAEPAXI@Z)
Debug/GATest.exe : fatal error LNK1120: 1 externes non résolus

J'ai vérifié la solution référencée sur ce site, mais mes paramètres sont bons.
( http://c.developpez.com/faq/cpp/?pag...S_console_vcpp )

Rien d'intéressant sur Google non plus.

Vous avez une idée d'où ça pourrait venir (à priori du destructeur, mais ensuite...) ?

Edit:
Les soluces que je trouve me suggèrent de linker une dll, mais je ne vois pas ce qui pourrait me manquer, vu que je n'ai pas rajouté de fonctions par rapport à celles qu'utilise la librairie de base.

**Matthieu Brucher** · 08/08/2006, 22h42

Où est le corps de ton destructeur ?

...

**Berzerk_** · 08/08/2006, 23h03

Envoyé par Miles

Où est le corps de ton destructeur ?

Code :

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

virtual ~GADiploid();

Voici le destructeur incriminé. Je vois d'où peut venir le problème...
Dans une 1ère version, j'avais écrit un destructeur, mais il a disparu avec mes bidouilles.
Je viens de le remettre, et l'erreur a disparu. Mais peux-tu me dire s'il est valide?

La classe GADiploid crée lorsqu'elle est instantiée deux objets de type GABinaryString: Chrom_1 et Chrom_2.
J'active donc le destructeur de ces deux structures:

Code :

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

1
2
3
4
5
virtual ~GADiploid()
		{
			Chrom_1->~GABinaryString();
			Chrom_2->~GABinaryString();
		}

C'est bon comme ça?

**roulious** · 08/08/2006, 23h22

Envoyé par Berzerk_

La classe GADiploid crée lorsqu'elle est instantiée deux objets de type GABinaryString: Chrom_1 et Chrom_2.

Si Chrom_1 et Chrom_2 sont des données membres de ta classe GADiploid, il n'y a rien de particulier à faire, le destructeur est appelé automatiquement. En général, le destructeur se contentera de libérer les ressources allouées dans le constructeur. Par exemple, en allouant de la mémoire avec new dans le constructeur, tu la libéraras par delete ou delete[] dans le destructeur.

**Berzerk_** · 09/08/2006, 00h29

Envoyé par roulious

Si Chrom_1 et Chrom_2 sont des données membres de ta classe GADiploid, il n'y a rien de particulier à faire, le destructeur est appelé automatiquement. En général, le destructeur se contentera de libérer les ressources allouées dans le constructeur. Par exemple, en allouant de la mémoire avec new dans le constructeur, tu la libéraras par delete ou delete[] dans le destructeur.

C'est ce que je fais en appelant le destructeur de Chrom_1 et Chrom_2:

Code :

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

virtual ~GABinaryString(){delete [] data;}

Par contre, cela sert encore à quelque chose que je laisse le destructeur de GADiploid en virtual?

**Matthieu Brucher** · 09/08/2006, 08h15

Envoyé par Berzerk_

Code :

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

1
2
3
4
5
virtual ~GADiploid()
		{
			Chrom_1->~GABinaryString();
			Chrom_2->~GABinaryString();
		}

C'est bon comme ça?

Clairement non. C'est un delete si c'est un pointeur.

**Berzerk_** · 09/08/2006, 10h22

Envoyé par Miles

Clairement non. C'est un delete si c'est un pointeur.

D'accord, mais Chrom_1 et Chrom_2 créent aussi une structure data, donc il faut que je fasse appelle à leur destructeur.

Est-ce que faire:

Code :

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

1
2
3
4
5
		virtual ~GADiploid()
		{
			delete[] Chrom_1;
			delete[] Chrom_2;
		}

suffit (delete[] appelle automatiquement le destructeur de Chrom_1 et Chrom_2)?

Ou est-ce qu'il faut que j'écrive ceci:

Code :

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

1
2
3
4
5
6
7
		virtual ~GADiploid()
		{
			Chrom_1->~GABinaryString();
			Chrom_2->~GABinaryString();
			delete[] Chrom_1;
			delete[] Chrom_2;
		}

**Matthieu Brucher** · 09/08/2006, 10h29

http://c.developpez.com/faq/cpp/?pag...teur_explicite

**Berzerk_** · 09/08/2006, 10h52

Merci, donc réponse 1).

Question subsidiaire:

Je vois dans l'exemple:

Code :

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

t.~Test();

Or mon compilateur (Visual C++ 7.1) m'oblige (erreur à la compilation) à faire appel aux méthodes par:

Code :

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

t->~Test();

(c'est la correction qu'il me suggère)

Il n'y a que les membres d'une classe que je peux appeller par un "point" (notation qu'il m'impose également).

Puisque l'exemple propose une autre notation, est-ce normal que mon compilateur la refuse, où est-ce qu'il y a quelque chose qui m'échappe?

**Matthieu Brucher** · 09/08/2006, 10h56

Là, tu exagères, c'est le cours de base de C++ ou de C.
On utilise . quand on utilise l'objet directement et -> quand c'est un pointeur.

**Berzerk_** · 09/08/2006, 11h15

Envoyé par Miles

Là, tu exagères, c'est le cours de base de C++ ou de C.
On utilise . quand on utilise l'objet directement et -> quand c'est un pointeur.

D'accord merci, je ne me rappelais plus de cette nuance, désolé pour la question stupide.

Je mets Résolu et j'essaie d'avancer un peu sur mon projet.

Merci pour toutes tes réponses.

**Berzerk_** · 09/08/2006, 17h18

J'ai conscience que ce problème doit être assez ridicule, qu'il faut juste que je cast quelque chose, mais je ne vois pas quoi...

Voici mon code:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
	short bit(unsigned int a, unsigned int c) const
	{
		printf("c = %d  ", c); //delete
		switch(c)
		{
			case 1:		{printf("reading in 1: %d\n",Chrom_1->bit(a));
						return(Chrom_1->bit(a));}
						break;
			default:	{printf("writing in 2: %d\n",Chrom_2->bit(a));
						return(Chrom_2->bit(a));}
		}
	}

J'ai fait une routine de test qui me donne la sortie suivante:

c = 1 writing in 2: 0
c = 2 writing in 2: 1
c = 1 writing in 2: 1
c = 2 writing in 2: 0
c = 1 writing in 2: 0
c = 2 writing in 2: 1
c = 1 writing in 2: 1
c = 2 writing in 2: 0

J'ai essayé de mettre un (char) devant le c, de mettre le 1 entre ' ', mais rien n'y fait, il me sort toujours le writing in 2, donc le default.

Une idée de la correction à apporter?

Héritage de classes.

C++

Discussions similaires

Partager

Partager