Discussion :

[ouste4863]

Bonjour tout le monde,
depuis plus de deux heures je n'arrive pas à trouver l'erreur qui empêche mon programme de compiler. Je tente pour la première fois de structurer mon programme avec cmake mais c'a pas l'air de bien réussir même si cela peut paraitre simple , mais je souhaite vivement vos aides. Je mets le code et âpres l'erreur:

le fichier CMakeLists.txt

Code :

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

1
2
3
4
5
6
7
8
9
10
11
CMAKE_MINIMUM_REQUIRED(VERSION 3.0)
if(NOT WIN32)
	set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")
ENDIF()
ADD_EXECUTABLE(tp1
src/main.cpp
src/stocker.cpp
src/stocker.h
src/produits.h
src/produits.cpp
)

les fichiers.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
#ifndef PRODUITS_H
#define PRODUITS_H
 
#include <iostream>
#include <string>
#include "stocker.h"
 
using namespace std
 
class Produits
{
private: 
	int ref;
	string designation;
public:
	Produits(int ref, string desig);
	~Produits();
	void affiche();
 
};
 
#endif

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
#ifndef STOCKAGE_H
#define STOCKAGE_H
 
#include <string>
#include "produit.h"
#include <vector>
 
using namespace std;
 
	class Stockage 
	{
	private:	
		int num;
		vector<Produits*> vProd;
		string type;
 
	public:
		Stockage(int num,string type);
		~Stockage();
		void ajouterProduit(Produit*);
		void afficheTout();	
	};
 
#endif

le premier étant le fichier produits.h et le second le fichier stocker.h
Ensuite les fichiers .cpp

stocker.cpp

Code :

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

1
2
3
4
5
6
7
8
9
#include "stocker.h"
#include <iostream>
#include<string>
using namespace std;
 
Stockage::Stockage(int num,string type){
	this->num=num;
	this->type=type;
}

produits.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
#include <iostream>
#include "produits.h"
#include "stockage.h"
 
using namespace  std;
 
Produits::Produits(int ref, string desc){
	ref=ref;
	designation=desc;
}
 
Produits::affiche(){
	cout<<"Ref:"<<ref<<"  designation:"<<designation<<endl;
}

et en fin le main

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include <iostream>
#include "produits.h"
#include "stocker.h"
 
using namespace std;
 
int main(int argc, char* argv[]){
	Stockage s(1,"viande");
	Produits pf1(1,"pf1");
 
	//pf1.affichage();
}

Et mon produit, il me dit que les types Produits et Stockage n'existent pas
je met une capture de mon écran quand je compile

et je precise que mes fichiers sont bien structurer.
merci d'avance.

[Bousk]

Salut, rien à voir avec CMake : tu as produit qui inclut stocker, qui inclut produit, qui inclut stocker, qui ....

https://cpp.developpez.com/faq/cpp/?...-une-a-l-autre

[ouste4863]

Merci Bousk pour la reponse, je m'en etais rendu compte et j'ai géré l'inclusion multiple avec les template... mais un autre problème on me dit ceci

principal.cpp: In function ‘int main(int, char**)’:
principal.cpp:8:12: error: variable ‘Stockage s’ has initializer but incomplete type
  Stockage s(1,"viande");
            ^
principal.cpp:9:12: error: variable ‘Produits p’ has initializer but incomplete type
  Produits p(1,"pf1");

lorsque j'essaie d'instancier mes classe. Je me demande ce qui manque à mes constructeurs qui sont définis dans mes fichier .cpp

[Bktero]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
CMAKE_MINIMUM_REQUIRED(VERSION 3.0)
if(NOT WIN32)
	set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11")
ENDIF()
ADD_EXECUTABLE(tp1
src/main.cpp
src/stocker.cpp
src/stocker.h
src/produits.h
src/produits.cpp
)

Ce fichier pourrait grandement être amélioré.

Déjà, pourquoi tester WIN32 ? Tu veux surtout tester que le compilateur est gcc (peut-être clang). Ton test devrait donc être :

Code :

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

if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU") ...

Ensuite, il y a bien mieux pour choisir la version du standard :

Code :

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

1
2
    set(CMAKE_CXX_STANDARD 11) # on veut du C++ 11
    set(CMAKE_CXX_STANDARD_REQUIRED ON) # c'est obligatoire

De plus, il manque quand même quelque de très important : il faut ajouter les options -Wall -Wextra !!!

Enfin, le plus important, tu devrais autant que possible ne pas modifier directement les variables telles que CMAKE_CXX_FLAGS. Cette façon de faire est une vieille technique aujourd'hui considérée comme mauvaise. Regarde ce que du côté de add_compile_options(), c'est ce dont tu as besoin Voir https://stackoverflow.com/questions/...oject/46330404

Au final, ton CMakeLists.txt devrait plutôt ressembler à ceci :

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
cmake_minimum_required(VERSION 3.0)
 
project(mon tp)
 
set(CMAKE_CXX_STANDARD 17)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
 
if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU")
    message("GNU GCC detected")
    add_compile_options(-Wall -Wextra)    
endif ()
 
ADD_EXECUTABLE(tp1
src/main.cpp
src/stocker.cpp
src/stocker.h
src/produits.h
src/produits.cpp
)

[ouste4863]

Merci beaucoup, ça y est c'est résolu.