Discussion :

[z980x]

Bonjour à tous,

je suis en train de travailler sur mon projet, et je n'ai pour l'instant fait aucune classe... J'ai donc décidé d'en faire une, mais n'étant pas chez moi, je n'ai pas les infos nécessaires...

Voici ma classe, dans un fichier .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
38
39
40
      #ifndef prendre_photo_h
#define prendre_photo_h

#include "creer_ville.h"
extern pt_ville VILLES;

class photo{
bool entrepot;
bool centrale;
bool cyber;
public:
//photo(entrepot=false, centrale=false,cyber=false);

void photo::prendre(string nom_vil) {
	while(VILLES) {
		if(VILLES->nom_ville == nom_vil) {
			if(VILLES->entrepot == true) this->entrepot = true; 
			else this->entrepot = false;

			if(VILLES->centrale == true) this->centrale = true; 
			else this->centrale = false;
			
			if(VILLES->cyber == true) this->cyber = true; 
			else this->cyber = false;
		}
	VILLES = VILLES->lien;
	}
}

void photo::afficher_photo(string nom_vil) {
	cout << "ici"<< endl;
	this->prendre(nom_vil); //Si on ne l'a fait pas avant, on va chercher les infos maintenant.
	cout << "La ville " << nom_vil ;
	if(entrepot) cout << " comporte un entrepot,"; 
	else cout << " ne comporte pas d'entrepot,";

	if(centrale) cout << " comporte une centrale,"; 
	else cout << " pas de centrale,";

	if(cyber) cout << " et comporte un centre cybern

Dans un fichier .cpp (ou j'ai inclus le .h), je fais ca:

Code :

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

photo::afficher_photo(nom_vil);

lors de la compil, j'ai cette erreur:

Code :

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

1
2
3
4
5
g++ -Wall   -c -o lancement_partie.o lancement_partie.cpp
lancement_partie.cpp: Dans function « void lancement_partie() »:
lancement_partie.cpp:40: error: cannot call member function `void
   photo::afficher_photo(const std::string&)' without object
make: *** [lancement_partie.o] Erreur 1

Comment corriger ce pb??

Merci !

Philippe

[hansaplast]

tu n'utilise pas une instance de ta classe, en gros quan tu utilise la notation :: tu demande a lancer une fonction.

la difference avec

Code :

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

photo->afficher_photo(nom_vil);

ou

Code :

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

photo.afficher_photo(nom_vil);

est que ta fonction ne doit pas utiliser this (aucune de ses variables memebres n'est initiamisée)

[Matthieu Brucher]

On n'a le droit d'appeler que des fonctions statiques avec cette méthode.

Pour plus de renseignements : http://c.developpez.com/faq/cpp/

[z980x]

hansaplast merci de ta réponse, mais je ne crois pas que ce soit, j'ai toujours appris avec les ::

Pour Miles, euh... Si on est pas autorisé à faire ca, alors comment faire,??

Merci

Philippe

[Herode]

hansaplast merci de ta réponse, mais je ne crois pas que ce soit, j'ai toujours appris avec les ::

Moi, je te garantis que hansaplast a raison. Peut-être envisager de changer de prof ?

1 - A l'intérieur d'une définition de classe, le nom de la classe est implicite en tant qu'espace de nommage. Il est donc inutile de le redéclarer :

Code :

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

1
2
3
class Photo {
   void prendre(string nom_vil) { [...] }
};

2 - Quand tu définis une classe, tu définis des méthodes "normales", qui sont appelées en passant par une instance de la classe, ou (inclusif) des méthodes statiques, qui n'ont pas besoin d'une instance de la classe pour être appelées.

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
class Photo {
  public : 
   void prendre(string nom_vil) { [...] }
   static void prendre_statique (string nom_vil) { [...] }
   void test();
};
 
void fct() {
  Photo ph;
  ph.prendre( "depuis une instance" );
  Photo* pph = new Photo();
  pph->prendre( "depuis un pointeur initialisé" );
  photo::prendre_statique( "depuis rien du tout" );
}
 
void Photo::test() {
  prendre( "de l'intérieur de la classe" );
  prendre_statique( "toujours à l'intéreur de la classe" );
  Photo::prendre_statique( "pour les bavards ou les amateurs de précision" );
}

Par défaut, une méthode est non statique.

[Laurent Gomila]

Tu devrais relire un bon cours de C++, c'est quand même la base.

En gros si tu définis une classe c'est pour ensuite en déclarer des instances, tu ne peux pas appeler une fonction membre sans instance, à moins que celle-ci soit déclarée statique.

Code :

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

1
2
3
4
5
6
7
8
9
10
class photo
{
    ...
};
 
// Je définis une instance de la classe photo
photo p;
 
// J'appelle sa fonction membre afficher_photo
p.afficher_photo(nom_vil);

[z980x]

Ah voila ! J'avais oublié... Merci !

[hansaplast]

oublie aps le tag "resolut"

[Matthieu Brucher]

hansaplast merci de ta réponse, mais je ne crois pas que ce soit, j'ai toujours appris avec les ::

Pour Miles, euh... Si on est pas autorisé à faire ca, alors comment faire,??

Merci

Philippe

http://www.developpez.net/forums/sho...03&postcount=3
Juste au-dessus....................