Discussion :

[mulfycrowh]

Bonjour,

Je travaille sur un projet contenant plusieurs classes donc plusieurs .cpp et .h
J'ai créé de plus un fichier Toolbox.h incluant diverses fonctions que j'utilise dans les différents .cpp

Je me retrouve avec une erreur LNK1169 suivant LNK 2005

Code :

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

1
2
3
1>Instrument.obj : error LNK2005: "class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > __cdecl get_ExcelParameter(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >,int)" (?get_ExcelParameter@@YA?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@V12@H@Z) already defined in Directory.obj
1>Turn_For_You.obj : error LNK2005: "class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> > __cdecl get_ExcelParameter(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >,int)" (?get_ExcelParameter@@YA?AV?$basic_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@V12@H@Z) already defined in Directory.obj
1>D:\T4U Project\Debug\T4U Project.exe : fatal error LNK1169: one or more multiply defined symbols found

Apparemment, cela vient du fait de déclarations ou d'inclusions multiples.

Comment puis je m'en sortir ?

Merci à tous

[bacelar]

Vous ne devez avoir que les déclarations de fonctions dans les .h.

Les définitions de ces fonctions doivent être dans un cpp.

[mulfycrowh]

Voilà ce que je viens de faire :

j'ai mis #pragma once dans chaque fichier .h

Mais cela ne corrige p

[bacelar]

C'est une erreur de LINK, pas de compilation.
Vérifiez ma remarque.

[mulfycrowh]

Voilà ce que je viens de faire :

j'ai mis #pragma once dans chaque fichier .h

Mais cela ne corrige pas le pb.

Ce qu'il faut dire est que Toolbox.h ne contient QUE des fonctions incluant un template.

J'ai donc ajouté un template à la fonction que j'avais ajoutée dans Toolbox.h

Et cela compile très bien ...
Cela rejoint ce que vous dites

[bacelar]

get_ExcelParameter est une fonction ou une méthode.

Pourquoi est-elle définit dans Directory.cpp, Instrument.cpp et Turn_For_You.cpp.
Je dis bien défini et pas déclaré (elle l'est mais ce n'est pas le problème).

[mulfycrowh]

Toolbox.h ne contient que des fonctions indépendantes de toute classe. Elles sont toutes définies à partir d'un template.
Par contre, getExcelParameter ne l'est pas. Il faut peut être que je la place dans Toolbox.cpp ?

[bacelar]

Quand vous définissez un template (de classe ou de fonction) dans un .h, vous ne définissez pas du code au sens propre mais un patron de code.
C'est le compilateur qui définira du code à chaque instanciation du template (par instanciation d'un template, c'est renseigner les classes du patron).
C'est au compilateur de ne pas faire de double définition lors des instanciations de template.
Si getExcelParameter est une fonction "standard", sa définition génère du code. Si la définition est dans le .h, il y aura autant de génération de code que d'inclusion du .h dans un cpp différent, d'où le problème.

Alors, soit vous faite de getExcelParameter une fonction tempalte soit vous localisez sa définition dans un .cpp.

[mulfycrowh]

Merci beaucoup.