Discussion :

[lezebulon]

Bonjour,
j'essaie pour la premiere fois de compiler en même temps plusieurs fichiers (un cpp par classe et un header), mais j'ai un soucis que je ne sais pas résoudre.

J'ai pour faire un test rapide une fonction globale appellée "load_image", que j'ai placé dans config.h qui contient aussi mes constantes
Mon fichier main inclu config.h
Mon fichier tileset.cpp pareil

Tous mes fichiers compilent bien, mais au niveau des liens j'ai l'erreur suivante :

Code :

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

1>TilesLayer.obj : error LNK2005: "struct SDL_Surface * __cdecl load_image(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >)" (?load_image@@YAPAUSDL_Surface@@V?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@@Z) déjà défini(e) dans main.obj

que je ne comprend pas trop : a priori j'ai mis des "ifndef .... endif" dans mes includes, donc je comprend pas trop pourquoi est-ce que ma fonction veut se redéfinir à 2 endroits différents.
Bien sûr si je n'inclut plus config.h dans mon 2eme fichier, la compilation ne marche plus....
Une idée?

[Chessmaster1966]

Peut-être une inclusion multiple de config.h. Je ne sais pas comment tu as déclaré tes directives de compilation ifndef...endif mais essayes ça si ce n'est pas déjà fait :

Code :

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

1
2
3
4
5
6
7
 
#ifndef CONFIG_H
#define CONFIG_H
 
...
 
#endif

[lezebulon]

c'est déjà fait, c'est pour ça que je comprends pas trop
Après j'ai tout fait au feeling, je ne connais pas du tout la méthode d'organisation quand on a plusieurs .cpp à compiler en même temps

[Arzar]

Bonjour,

lezebulon, quand tu dis :

J'ai pour faire un test rapide une fonction globale appellée "load_image", que j'ai placé dans config.h qui contient aussi mes constantes

On est d'accord que tu parles uniquement de la déclaration de la fonction dans config.h, car sa définition est bien dans config.cpp ?

[lezebulon]

Euh... non
En fait j'ai pas de déclaration séparée, juste la définition.
Et j'ai pas de config.cpp

En fait je suis parti d'un fichier de base .cpp codé rapido et j'ai essayé de le porter rapido aussi sur plusieurs fichiers, je sais pas trop quelle convention je dois prendre pour tout ce qui est global

[Lucien63]

Ta fonction ne serait défini deux fois,
une fois dans TilesLayer.cpp et
une fois dans main.cpp