Erreur class has not been declared

Résolu

velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention -
velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention - 25 juil. 2011 à 16:08

Bonjour à tous,

je suis en train de faire un petit exercice en c++ et je rencontre un problème qui m'empêche de terminer. voici le code :

//fichier ProfDeGym.h

#ifndef H_ProfDeGym
#define H_ProfDeGym

using namespace std; 

#include "TourDePiste.h"

class ProfDeGym{

public :
	void tourFini(TourDePiste&);

};

void ProfDeGym::tourFini(TourDePiste& tour){
	if(tour.nbTours>0)
		tour.nbTours--;
}

#endif

//fichier TourDePiste.h

#ifndef H_TourDePiste
#define H_TourDePiste

using namespace std;

class TourDepiste{

	int getTours(){return nbTours;}
	void setTours(int & nbTours){this->nbTours=nbTours;}
	friend void tourFini();

private:
	int nbTours;
};

#endif

//fichier ProfDeGym.cpp

#include <ProfDeGym.h>

//fichier TourDePiste.cpp

#include <TourDePiste.h>

//fichier main.cpp

#include "TourDePiste.h"
#include "ProfDeGym.h"

int main(){

	TourDePiste tour = new TourDePiste();
	ProfDeGym();

	tour.setTours(10);
	for(int i=0;i<5;i++){
		ProfDeGym.TourFini(tour);
	}

	cout << "nbTour" << tour.getTours(); << "\n";

return  0;
}

quand je compile le main sous ubuntu, j'ai les erreurs suivantes:
velocity@velocity-laptop:~/Documents/proj1$ gcc -o main.o main.cpp
In file included from main.cpp:2:
ProfDeGym.h:11: error: `TourDePiste' has not been declared
ProfDeGym.h:15: error: variable or field `tourFini' declared void
ProfDeGym.h:15: error: `TourDePiste' was not declared in this scope
ProfDeGym.h:15: error: `tour' was not declared in this scope
main.cpp: In function `int main()':
main.cpp:6: error: `TourDePiste' was not declared in this scope
main.cpp:6: error: expected `;' before `tour'
main.cpp:9: error: `tour' was not declared in this scope
main.cpp:11: error: expected unqualified-id before `.' token
main.cpp:14: error: `cout' was not declared in this scope
main.cpp:14: error: expected primary-expression before `<<' token

merci d'avance,

0

Afficher la suite

A voir également:

Class has not been declared c++
Bios has been reset - Forum BIOS
Warning your chassis has been opened ✓ - Forum Matériel & Système
No boot disk has been detected or the disk has failed - Forum Disque dur / SSD
Signal cable not connected ✓ - Forum Ecran
Nvcontainer windows class - Forum Carte graphique

8 réponses

Réponse 1 / 8

velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention 6

En premier essai, j'ai utilisé eclipse mais il ne compile pas, puis j'ai passé à Code::Blocks, là aussi j'ai plusieurs erreurs d'include et autres que j'ai pas compris et puisque j'ai pas beaucoup de temps j'ai choisi d'utiliser ubuntu sous virtual Box et voilà le résultat !

KX Messages postés 16761 Date d'inscription Statut Modérateur Dernière intervention 3 020

Si tu es sous Windows, Dev-C++ est pas mal, pas la peine de virtualiser pour ça !

Réponse 2 / 8

KX Messages postés 16761 Date d'inscription Statut Modérateur Dernière intervention 3 020

Tu as l'air de faire du Java plus que du C++

En C++ le new() a une signification différente, et tu ne dois pas t'en servir ici pour initialiser un objet, la syntaxe étant de mettre directement entre parenthèses les paramètres du constructeur (que tu n'as d'ailleurs pas fait)

Si tu fais des .h et des .cpp, autant mettre les bons codes aux bons endroits !

Tu ne peux jamais utiliser un attribut private d'une autre classe (nbTours ici)
Et le "this" bien qu'il existe ne permet pas de faire ce que tu veux ici... Tu ne peux pas utiliser un nom de paramètre identique au nom d'un attribut sans qu'il n'y ait d'ambiguïté.

De plus vu que tu ne sais pas te servir des using namespace (tu les mets partout sauf là où il faut) autant ne pas les mettre du tout !

J'oublie surement un certain nombre d'explications, mais étudier mon code devrait t'aider bien mieux que des dizaines de lignes d'explications...

TourDePiste.h

#ifndef H_TourDePiste
#define H_TourDePiste

class TourDePiste
{
private:
	unsigned int nbTours;

public:
	TourDePiste(const unsigned int);

	int getTours() const;
	void setTours(const unsigned int);

	void incrementer();
	void decrementer();

	unsigned int getTours();

	//friend void tourFini(); // à quoi ça sert ?
};

#endif

TourDePiste.cpp

#include "TourDePiste.h"

TourDePiste::TourDePiste(const unsigned int n=0)
{
	nbTours=n;
}

int TourDePiste::getTours() const
{
	return nbTours;
}

void TourDePiste::setTours(const unsigned int n)
{
	nbTours=n;
}

void TourDePiste::incrementer()
{
	nbTours++;
}

void TourDePiste::decrementer()
{
	if (nbTours>0)
		nbTours--;
}

unsigned int TourDePiste::getTours()
{
	return nbTours;
}

ProfDeGym.h

#ifndef H_ProfDeGym
#define H_ProfDeGym

#include "TourDePiste.h"

class ProfDeGym
{
public :
	static void tourFini(TourDePiste&);
};

#endif

ProfDeGym.cpp

#include "ProfDeGym.h"

void ProfDeGym::tourFini(TourDePiste& tour)
{
	tour.decrementer();
}

main.cpp

#include <iostream>

#include "TourDePiste.h"
#include "ProfDeGym.h"

int main()
{
	TourDePiste tour(10);

	for (unsigned int i=0; i<5; i++)
	{
		ProfDeGym::tourFini(tour);
	}

	std::cout << "nbTour = " << tour.getTours() << std::endl;

	return  0;
}

Réponse 3 / 8

velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention 6

Merci pour votre réponse KX, c'est gentil de votre part de me réecrire le code .

C'est vrai que c'est la première fois que je programme en c++ .
Pour le probleme de declaration de classe j'ai trouvé le problème dans le nom de classe TourDep(P)iste .
Pour les namespace, je sais que ca permet d'utiliser les librairy standard standard mais je ne sais pas s'il suffit de le mettre dans le main seulement alors je l'ai mis dans toutes les classes.
Pour la classe TourDePiste.cpp vide c'est parce que l'exercice demande que setTours et getTours soit inline alors il ne reste rien à ecrire dans TourDePiste.cpp .
Pour la fonction friend void tourFini() c'est la méthode de ProfDeGym qui doit décrémenter l'attribue nbTours de TourDePiste qui est private donc j'ai du la déclarer friend comme ca elle y accède.
Seulement je ne comprend pas pourquoi vous avez utilisé const pour la fonction setTour(const unsigned int n).

et merci encore pour votre aide précieuse.

KX Messages postés 16761 Date d'inscription Statut Modérateur Dernière intervention 3 020

Pour les "using namespace X", tu en as besoin pour éviter d'écrire X:: dans les classes qui en ont besoin. Par exemple tu pourrais faire un "using namespace ProfDeGym" dans la classe de test.
En fait tu peux très bien t'en passer, dans tous les je n'ai utilisé de std:: que pour le cout, et ça coute pas plus cher de marquer std::cout, et au moins ça te permettra de mieux comprendre d'où ça vient.

Si tes fonctions sont inline alors tu dois mettre le mot clé inline, c'est évident pour moi, mais tu ne les avais pas mis, je ne pouvais donc pas comprendre ce que tu voulais faire.
Quant au mot clé friend, je pense pas que tu en ai besoin, pour décrémenter nbTours, tu devrais utiliser la méthode TourDePiste::decrementer() que j'ai faite...

À chaque fois que je mets const, ça permet d'être sûr que la valeur que je passe en paramètre ne sera pas modifiée... De même pour l'objet this, lorsque je met const à la fonction. Exemples :

void TourDePiste::setTour(const unsigned int n); // (1)
int TourDePiste::getTours() const; // (2)

TourDePiste t(10);
int x=5;
t.setTour(x);  // avec le const (1) je suis assuré que x ne sera pas modifié
x=t.getTour(); // avec le const (2) je suis assuré que t ne sera pas modifié

velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention 6

d'accord pour les namespace, c'est un point clarifié et je les ai tous supprimés .
Pour le inline, j'ai lu dans un cours qu'on devrait le mettre si on défini la fonction dans un fichier header c_à_d en dehors de la classe mais si on la défini dans la classe même on n'a pas besoin de le mettre (pour le moment j'ai pas d'erreurs avec les fonctions inline).
Pour la fonction decrementer, l'exercice a decrit toutes les fonctions qu'il faut implémenter donc je pense qu'il ne faut pas que j'ajoute d'autres fonctions à part ceux qu'il a défini .
D'accord pour le const ca me rend heureux d'apprendre ce point important, j'y avais pas pensé qu'on pourrait modifier la donnée dans le setX() mais ca peut arriver .
Pourtant pour le friend j'arrive pas à le faire comme il faut, apparemment il y a quelque chose dans le code que j'ai oublié de mettre. voici le code à nouveau s'il vous plais :

//fichier main.cpp

#include "TourDePiste.h" 
#include "ProfDeGym.h" 

int main(){ 

 TourDePiste tour(); 
 ProfDeGym(); 

 tour.setTours(10); 
 for(int i=0;i<5;i++){ 
  ProfDeGym.TourFini(tour); 
 } 

 std::cout << "nbTour" << tour.getTours(); << "\n"; 

return  0; 
}

//fichier TourDePiste.h

#ifndef H_TourDePiste 
#define H_TourDePiste 

#include "ProfDeGym.h" 

class TourDePiste{ 

public: 
 unsigned int getTours() const {return nbTours;} 
 void setTours(const unsigned int n){nbTours=n;} 
 friend void ProfDeGym::tourFini(TourDePiste&); 

private: 
 unsigned int nbTours; 
}; 

#endif

//fichier ProfDeGym.h

#ifndef H_ProfDeGym 
#define H_ProfDeGym 


#include "TourDePiste.h" 

class ProfDeGym{ 

public : 
 void tourFini(TourDePiste&); 

}; 


#endif

//fichier ProfDeGym.cpp

#include "ProfDeGym.h" 

void ProfDeGym::tourFini(TourDePiste& tour){ 
 if(tour.nbTours>0) 
  tour.nbTours--; 
}

//fichier TourDePiste.cpp toujours vide

#include "TourDePiste.h"

le résultat de la compilation
velocity@velocity-laptop:~/Documents/proj1$ make
g++ -w -Wall -ansi -pedantic -c -o ProfDeGym.o ProfDeGym.cpp
In file included from ProfDeGym.h:5,
from ProfDeGym.cpp:1:
TourDePiste.h:11: error: `ProfDeGym' has not been declared
TourDePiste.h: In member function `void ProfDeGym::tourFini(TourDePiste&)':
TourDePiste.h:14: error: `unsigned int TourDePiste::nbTours' is private
ProfDeGym.cpp:4: error: within this context
TourDePiste.h:14: error: `unsigned int TourDePiste::nbTours' is private
ProfDeGym.cpp:5: error: within this context
make: *** [ProfDeGym.o] Error 1
[5]+ Done gedit TourDePiste.cpp

KX Messages postés 16761 Date d'inscription Statut Modérateur Dernière intervention 3 020

Pour les inline, il faut que tu mettes le mot clé inline, sinon ça ne sert à rien.
Pour la fonction decrementer, tu peux utiliser setTours(getTours()-1) si tu préfères ne pas faire d'autres méthodes, ça va sensiblement revenir au même.
Par contre pour le mot clé friend je ne suis toujours pas convaincu de son utilité dans ton code, je pense que c'est plutôt static qu'il faudrait mettre, et directement dans la classe ProfDeGym.

TourDePiste.h

#ifndef H_TourDePiste
#define H_TourDePiste

class TourDePiste
{
private:
	unsigned int nbTours;

public:
	inline unsigned int getTours() const
	{
		return nbTours;
	}

	inline void setTours(const unsigned int n)
	{
		nbTours=n;
	}
};

#endif

ProfDeGym.h

#ifndef H_ProfDeGym
#define H_ProfDeGym

#include "TourDePiste.h"

class ProfDeGym
{
public :
	inline static void tourFini(TourDePiste &tour)
	{
		unsigned int n = tour.getTours();
		if (n>0)
			tour.setTours(n-1);
	}
};

#endif

test.cpp

#include <iostream>

#include "TourDePiste.h"
#include "ProfDeGym.h"

int main()
{
	TourDePiste tour;
	tour.setTours(10);

	for (unsigned int i=0; i<5; i++)
		ProfDeGym::tourFini(tour);

	std::cout << "nbTour = " << tour.getTours() << std::endl;

	return  0;
}

Remarque : si un fichier est vide, tu peux le supprimer, ça ne changera rien.

velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention 6

j'ai suivi votre conseil et j'ai rajouté inline avant les fonction set et get .
friend void tourFini() est une exigence dans l'exercice pourriez vous m'aider à trouver le problème ?
c'est une méthode de ProfDeGym et elle doit accéder l'attribut private int nbTours de TourDePiste .

merci pour votre aide précieuse

Réponse 6 / 8

velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention 6

J'ai trouvé la solution pour le dernier problème c'est d'utiliser des forward declarations.
cette fois j'ai travaillé sur dev C++

//fichier ProfDeGym.h

#ifndef H_ProfDeGym  
#define H_ProfDeGym  

class TourDePiste; //forward declaration 

class ProfDeGym{  

      public:  
       void tourFini(TourDePiste& tour);        

}  

#endif

//fichier TourDePiste.h

#ifndef H_TourDePiste  
#define H_TourDePiste  

class ProfDeGym;  //forward declaration 

class TourDePiste{  
public :  
       inline unsigned int getTours() const {return nbTours;}  
       inline void setTours(const unsigned int n){nbTours = n;}  
       friend void ProfDeGym::tourFini(TourDePiste&);  
         
private:  
        unsigned int nbTours;  
};  

#endif

//fichier ProfDeGym.cpp

#include "ProfDeGym.h"  
#include "TourDePiste.h"  

void ProfDeGym::tourFini(TourDePiste& tour){  
     unsigned int n = tour.getTours();  
     if(tour.nbTours>0)  
          tour.setTours(n-1);  
}

//fichier TourDePiste.pp

#include "TourDePiste.h"  
#include "ProfDeGym.h"

mais maintenant j'ai l'erreur suivante :
In file included from main.cpp
redefinition of 'class ProfDeGym'
previous definition of 'class ProfDeGym'
[Build Error] [main.o] Error1

Réponse 7 / 8

java4ever

Bonjour,
Je te donne la solution ci-dessous, qui reprend ton code et tient principalement en deux mots : "forward declaration".
Ça permet de se servir d'un nom de classe dans une déclaration sans avoir inclu le h.

Par contre, pour faire marcher j'ai du modifier des déclarations qu'il faut que tu potasse un peu, si je puis me permettre
Les deux lignes du main qui déclaraient tout et ProfDeGym montrent que tu ne maîtrise pas le sujet des déclarations. Cherche des tuto d'initiation C++ et tu y gagneras
dernier point : ce code mple et fonctionne sous g++ (testé !)

//Le cde de velocituy remanié qui est compilable et fonctionne correctement en compilant avec g++
//fichier main.cpp
// J'ai rajouté include iostream (pour cout)
#include <iostream>
#include "TourDePiste.h"
// L'inclde ProfDegym n'est pas utile car déjà inclu dans TourDePiste.h, mais ça ne gêne pas
#include "ProfDeGym.h" 


int main(){ 


 TourDePiste tour;  // pas de parentheses après tour
 ProfDeGym profdeGym; // ici tu n'avais rien déclaré en faisant simplement ProfDeGym()
 
 tour.setTours(10); 
 for(int i=0;i<5;i++){ 
  profdeGym.tourFini(tour); // attention au majuscules minuscules 
 } 

 std::cout << "nbTour : " << tour.getTours() << '\n'; 

return  0; 
}

//fichier TourDePiste.h
// je n'est rien changé das ce fichier, je le remet pour mémoire
#ifndef H_TourDePiste 
#define H_TourDePiste 

#include "ProfDeGym.h"

class TourDePiste { 

public: 
 unsigned int getTours() const {return nbTours;} 
 void setTours(const unsigned int n){nbTours=n;} 
 friend void ProfDeGym::tourFini(TourDePiste&); 

private: 
 unsigned int nbTours; 
}; 


#endif 
//fichier ProfDeGym.h

#ifndef H_ProfDeGym 
#define H_ProfDeGym 
// **** C'est la la modif la plus importante : il ne faut pas include le .h de TourDePiste sinon redondance cyclique

// La ligne ci-dessous se nomme une "forward declaration", ça permet de gérer les références croisées
class TourDePiste;

class ProfDeGym { 

public : 
 void tourFini(TourDePiste &lesToursDePiste); 
}; 

#endif 

//fichier ProfDeGym.cpp
// Ici, par contre, on a beoin de la "vraie" déclaration de TourDePiste, il faut donc l'include "TourDePiste.h"
// J'ai donc juste rajouté cette ligne.
#include "ProfDeGym.h" 
#include "TourDePiste.h"

void ProfDeGym::tourFini(TourDePiste& ptour) { 
 if(ptour.nbTours>0) 
  ptour.nbTours--; 
}

Cordialement

KX Messages postés 16761 Date d'inscription Statut Modérateur Dernière intervention 3 020

Je suis d'accord pour l'utilisation du friend comme cela, j'avoue avoir été un peu réticent à l'utiliser, car en général si on peut s'en passer (et là on le pouvait) autant l'oublier...
Mais dans la mesure où ici c'était imposé il n'y a plus vraiment le choix ^^

Cependant, il faudrait quand même remettre les mots-clés inline pour getTours et setTours sinon les fonctions en questions ne seront pas inline alors que c'est ce qu'on attend d'elles...

ProfDeGym.h

#ifndef H_ProfDeGym
#define H_ProfDeGym

class TourDePiste;

class ProfDeGym
{
public : 
	void tourFini(TourDePiste &tour);
};

#endif

TourDePiste.h

#ifndef H_TourDePiste
#define H_TourDePiste

#include "ProfDeGym.h"

class TourDePiste
{
private:
	friend void tourFini(TourDePiste &tour);
	unsigned int nbTours;

public:
	inline unsigned int getTours() const
	{
		return nbTours;
	}

	inline void setTours(const unsigned int n)
	{
		nbTours=n;
	}

	friend void ProfDeGym::tourFini(TourDePiste &tour);
};

#endif

ProfDeGym.cpp

#include "ProfDeGym.h" 
#include "TourDePiste.h"

void ProfDeGym::tourFini(TourDePiste &tour)
{
	if (tour.nbTours>0)
		tour.nbTours--;
}

Test.cpp

#include <iostream>

#include "TourDePiste.h"
#include "ProfDeGym.h"

int main()
{
	TourDePiste tour;
	tour.setTours(10);

	ProfDeGym prof;

	for (unsigned int i=0; i<5; i++)
		prof.tourFini(tour);

	std::cout << "nbTour = " << tour.getTours() << std::endl;

	return  0;
}

De plus je trouverais plus cohérent de définir un constructeur pour la classe TourDePiste de manière à attribuer une valeur initiale à nbTours (0 par exemple) car pour l'instant on s'oblige à utiliser setTours juste après le constructeur, alors que si on l'oublie getTours renverrait n'importe quoi !

velocity Messages postés 204 Date d'inscription Statut Membre Dernière intervention 6

Merci beaucoup pour votre réponse java4ever, j'ai essayé votre code avec Dev C++ et ca marche enfin, merci pour l'explication du forward declaration et le temps que vous a pris pour m'ecrire le code.

C'est vrai je ne suis pas très à l'aise avec le C++ et j'en ai un cours qui explique un peut les nouveau concepts et j'en apprend petit à petit tout en essayant les exercices.
Maintenant je peut passe à l'exercice suivant .

merci à tous pour votre aide .

Discussions similaires

#define et const double

Calcul de 2 vecteurs (C++)

c++ random (nbre aléa ) decimal entre 0 et 1

Héritage et polymorphisme C++

afficher en c++ le tableau

Microsoft Visual C++ Debug Library

Votre réponse

Discussions similaires