Discussion :

[le_barbu]

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 :

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

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.

[Luc Hermitte]

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

[le_barbu]

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 :

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

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
 
#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;
		}
	}
}

[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

[le_barbu]

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.

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

[le_barbu]

decidement, j'ai un probleme!

c'est koi la différence entre:

Code :

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

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

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

[le_barbu]

d'accord, mais comment je fais moi? lol

j'ai meme fait un test simple. voila:

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

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

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

c koi le probleme ?!

[Luc Hermitte]

Comment tu linkes ?

[le_barbu]

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

Code :

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

1
2
3
4
5
6
 
g++ -Wno-deprecated -c test.cpp
 
//et main.cpp avec ceci:
 
g++ main.cpp

j'oublie quelque chose ?

[Luc Hermitte]

oui.

Code :

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

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

[le_barbu]

donc je fais un:

Code :

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

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!

[Luc Hermitte]

Tu as mal recopié ce que j'ai écrit...

[le_barbu]

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

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

[le_barbu]

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 !