Structure cmake en c++

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:

---

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:

---

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:

---

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:

---

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:

---

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:

---

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

Pièce jointe 430437 et je precise que mes fichiers sont bien structurer.
merci d'avance.

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

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

Code:

---

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:

---

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

---

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

Code:

---

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:

---

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 )

---

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