erreur class has not been declared Résolu

Question

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

Biesmelle · Answer

Tu utilises quel logiciel pour programmer?

velocity · Answer

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 · Answer

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

velocity · Answer

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.

velocity · Answer

Je programme sous ubuntu avec gedit et je compile avec un makefile que je viens de créer.  
J'ai téléchargé Dev C++ que KX m'a proposé je vais l'essayer après que je termine cet exercice sous ubuntu .

voici le contenu de mon makefile :
CC=gcc
CXXFLAGS= -w -Wall -ansi -pedantic
LDFLAGS=

EXEC=Gym

all: $(EXEC)

Gym : ProfDeGym.o TourDePiste.o
	@$(CC) -o $@ $^ $(LDFLAGS)

ProfDeGym.o : TourDePiste.h

main.o : TourDePiste.h ProfDeGym.h

%.o : %.c
	@$(CC) -o $@ -c $< $(CXXFLAGS)

.PHONY: clean mrproper

clean : 
	@rm *.o

mrproper : clean
	@rm $(EXEC)

velocity · Answer

Apparamment le problème vient du fait que chaqu'une des deux classe ProfDeGym et TourDePiste fait un include vers l'autre mais je ne sais pas comment corriger ce problème.    
Vous  avez une idée ?

velocity · Answer

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

java4ever · Answer

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

Erreur class has not been declared

8 réponses

Discussions similaires

Newsletters