g++ : undefined reference to ...

Version imprimable

13/05/2004, 17h28
le_barbu

[Résolu] g++ : undefined reference to ...
voila, j'ai implémenté le design pattern "Observable" en C++ sous linux (gcc ou g++).

le code se compile bien mais j'ai voulu faire un test, j'ai donc écrit un fichier tel que:
Code:

1 2 3 4 5 6 7 #include "observable.h" int main() { Observable *o = new Observable(); }
et là, il me retourne:

/home/le_barbu/tmp/cchfCH2Q.o(.text+0x2b): In function `main':
: undefined reference to `Observable::Observable[in-charge]()'
collect2: ld returned 1 exit status

Il plante donc au niveau de l'édition des liens!

pourtant j'ai compilé avec:

g++ -I. main.cpp (j'ai bien "inclus" le répertoire où se trouvent les sources...)

quoi que c'est que cette erreur ?

merci d'avance.
13/05/2004, 18h00
Luc Hermitte

Tu n'aurais pas oublié de définir le constructeur ?

Re:

ben non, voici mon code:
(je sais qu'il n'est pas parfait mais je me remets au c++ donc je fais comme je peux pour l'instant :-) )
observable.h

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
 
#ifndef OBSERVABLE_H
#define OBSERVABLE_H
 
#include <vector>
 
using namespace std;
 
class Observer;
 
class Observable
{
private:
	vector<Observer *> *observers;
	bool stateChanged;
 
public:
	Observable();
 
	virtual ~Observable();
 
	void addObserver(Observer *o);
	void removeObserver(Observer *o);
 
	void notify(void);
	void notify(Observer *o);
 
	void changed(void);
};
 
class Observer
{
public:
	virtual void update(Observable *o);
};
 
#endif

et observable.cpp:

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
68
69
70
71
72
73
74
75
76
77
 
#include "observable.h"
 
Observable::Observable()
{
	observers = new vector<Observer *>;
	stateChanged=false;
}
 
Observable::~Observable()
{
	delete observers;
}
 
void Observable::changed()
{
	stateChanged = !stateChanged;
}
 
void Observable::addObserver(Observer *o)
{
	for (int i=0; i<observers->size(); i++)
	{
		if ( observers->at(i) == o)
		{
			return;
		}
	}
	observers->push_back(o);
}
 
void Observable::removeObserver(Observer *o)
{
	vector<Observer *>::iterator it = observers->begin();
 
	while ( it != observers->end() )
	{
		if ( *it == o)
		{
			observers->erase(it);
			return;
		}
		++it;
	}
}
 
void Observable::notify(void)
{
	if (! stateChanged)
	{
		return;
	}
 
	for (int i=0; i<observers->size(); i++)
	{
		observers->at(i)->update(this);
	}
}
 
void Observable::notify(Observer *o)
{
	if (! stateChanged)
	{
		return;
	}
 
	Observer *found;
	for (int i=0; i<observers->size(); i++)
	{
		found = observers->at(i);
		if (found == o)
		{
			found->update(this);
			return;
		}
	}
}

13/05/2004, 18h42
Luc Hermitte

L'autre hyppothèse est un problème d'ordre lors de l'édition des liens.
Sujet discuté ici: news://d6652001.0405032323.38d11f19@posting.google.com Regarde dans tout le fil d'où est tiré ce message-id.

Autrement. Les vecteurs ne sont pas fait pour être manipulés via des pointeurs. Il s'agit déjà d'une abstraction qui encapsule pour nous les problèmes de gestion de mémoire. Pas besoin de les réintroduire.

Sinon, dans le passé, j'avais posté une version assez personnelle de ce pattern (pour une toute autre question). http://www.developpez.net/forums/vie...=992700#992700
13/05/2004, 18h55
le_barbu

Re:

ok, merci bien je vais voir si je peux trouver de quoi répondre à ma question.

pour les pointeurs, je sais je vais trouver une solution plus adéquate.
13/05/2004, 19h11
Luc Hermitte

Pas besoin de chercher loin pour le vecteur. N'en fait pas un pointeur c'est tout. Un vecteur de pointeurs, OK ; un pointeur de vecteur, non.

Re:

decidement, j'ai un probleme!

c'est koi la différence entre:
Code:

1 2 3 Observable *o = new Observable(); // plante à la compilation Observable *o(); // ca compile!

13/05/2004, 21h30
Laurent Gomila

La différence est que la seconde ligne n'a pas besoin de connaitre la définition de la classe Observable, une déclaration anticipée suffit pour pouvoir la compiler. La 1ere ligne, au contraire, aura certainement besoin d'un #include "Observable.h".

PS : ta deuxieme ligne sera meme surement interprétée comme une declaration de fonction.

Re:

d'accord, mais comment je fais moi? lol

j'ai meme fait un test simple. voila:

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
 
//test.h
 
class Test
{
	public:
		Test();
};
 
//test.cpp
 
#include <iostream.h>
#include "test.h"
 
Test::Test()
{
	cerr << "Test()" << endl;
}
 
//main.cpp
 
int main()
{
	Test *t = new Test();
	return 0;
}

j'obtiens ca:

Code:

1
2
3
4
 
In function `main':
: undefined reference to `Test::Test[in-charge]()'
collect2: ld returned 1 exit status

c koi le probleme ?!

13/05/2004, 23h00
Luc Hermitte

Comment tu linkes ?
13/05/2004, 23h00
le_barbu

précision...
j'ai oublié, je compile test.cpp avec:
Code:

1 2 3 4 5 6 g++ -Wno-deprecated -c test.cpp //et main.cpp avec ceci: g++ main.cpp
j'oublie quelque chose ?
13/05/2004, 23h03
Luc Hermitte
oui.
Code:

1 2 3 g++ -Wall -pedantic -c test.cpp g++ -Wall -pedantic -c main.cpp g++ -Wall -pedantic main.o test.o -o main.exe
Et si tu ne veux pas être insulté au sujet d'anciens entêtes qui sont dépréciés, utilise <iostream> et n'oublie pas "std".

suite

donc je fais un:
Code:

1 2 3 4 5 6 7 8 9 10 11 g++ -Wall -pedantic main.cpp et j'obtiens toujours (sur l'utilisation de ma classe Test dans main.cpp): main.cpp: In function `int main()': main.cpp:5: warning: unused variable 't' /home/le_barbu/tmp/ccAJAKNF.o(.text+0x37): In function `main': : undefined reference to `Test::Test()' collect2: ld returned 1 exit status
j'ai aussi changé le <iostream.h> en <iostream> & utilisé "using namespace std;"
donc je n'ai plus de warning sur le fichier d'include iostream.h...

Mais toujours incapable de compiler le main !!!

au secours!

13/05/2004, 23h27
Luc Hermitte

Tu as mal recopié ce que j'ai écrit...
13/05/2004, 23h36
le_barbu

suite & fin [RESOLU]

merci bien!

ok, j'ai refait:

gcc -Wall -pedantic test.o main.o -o main

mais pourkoi faut-il lui donner tous les fichiers objet pour compiler ?

je débute en "compilation" alors ne m'en veuillez pas!

enfin, ce que je veux dire c'est pourkoi il ne les trouve pas tout seul vu qu'ils se trouvent dans le meme répertoire ?

en tout cas, MERCI BEAUCOUP :-)
13/05/2004, 23h54
Luc Hermitte

Il ne peut pas savoir quels fichiers il faut lier ensemble. Dans un répertoire, j'ai une 100aine de fichiers de tests, heureusement qu'il ne cherche pas à tous les mélanger ensembles.

Et gcc sans l'option qui va bien compile du C, c'est g++ qu'il te faudra utiliser pour lier.
14/05/2004, 07h23
le_barbu

merci

merci beaucoup!

pour l'histoire du linkage, c vrai heureusement qu'il ne cherche pas tout seul à tout lier...

des fois je me demande ou j'ai la tete lol

sinon, j'ai peut-etre mis gcc (je ne sais plus) comme commande dans mes postes mais j'utilise bien g++.

encore merci !