Duplicate symbol error

Version imprimable

Bonjour,

j'ai le petit morceau de programme suivant, dans lequel j'appelle différentes fonctions, en fonction du type à traiter. J'appelle une des fonctions SegmentInterX ou SegmentX (en fonction de mon problème), qui doit s'occuper de dispatcher le traitement à la bonne fonction (HistX, InterHistX, AvX ou FmX).
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 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 #ifndef DILATE_H #define DILATE_H namespace mylib { namespace Dilate { void HistX(const UINT8 *src, const int sewidth, UINT8 *res) {...} void InterHistX(const UINT8 *src, const int sewidth, UINT8 *res) {...} template<typename T> void FmX(const T *src, const int sewidth, T *res) {...} template<typename T> void AvX(const T *src, const int sewidth, T *res) {...} template<typename T> void SegmentX(const T *src, const int sewidth, T *res) { if ( sewidth <= 13 ) AvX(src, sewidth, res) ; else if ( std::is_same<T, UINT8>::value ) HistX(src, sewidth, res) ; else FmX(src, sewidth, res) ; } template<typename T> void SegmentInterX(const T *src, const int sewidth, T *res) { if ( std::is_same<T, UINT8>::value ) if ( sewidth <= 13 ) AvX(src, sewidth, res) ; else InterHistX(src, sewidth, res) ; else if ( sewidth <= 13 || channel != 1 ) AvX(src, sewidth, res) ; else FmX(src, sewidth, res) ; } } // namespace Dilate } // namespace mylib #endif /* DILATE_H */
Mais lorsque je compile, j'ai une "Duplicate symbols error" pour les deux fonctions InterHistX et HistX.

Pourquoi ?
Pourquoi sur ces fonctions en particulier et pas sur les deux autres ?
Pourquoi si je transforme ces fonctions en template (ce qui est inutile car elles ne peuvent traiter que des UINT8) ET que je fais un cast (UINT8*) lors de l'appel dans les fonctions SegmentInterX et SegmentX, alors tout fonctionne correctement ? (mais je trouve ça môche !!!)

Merci par avance !!!

23/09/2015, 09h58
dragonjoker59

Tes deux fonctions incriminées ne sont ni inline, ni template (donc inlinées), ni statiques (portée locale au fichier), donc pour deux fichiers .cpp incluant ton header, le compilateur va générer deux fois le code nécessaire à la fonction HistX et pareil pour la fonction InterHistX.
Au moment du link il se trouve donc avec ces 2 fonctions dupliquées, et il te le dit donc.

Plusieurs solutions: inliner tes fonctions, les rendre statiques, ou les déclarer dqns ton header et les implémenter dans un seul cpp. (je te conseille la dernière solution)
23/09/2015, 19h24
ToTo13

Merci pour ta réponse !

Citation:

Envoyé par dragonjoker59

les déclarer dans ton header et les implémenter dans un seul cpp. (je te conseille la dernière solution)

J'avais essayé... mais j'ai dû faire une bêtise. Je vais retester.
J'ai pour l'instant utilisé une autre solution : déclarer les fonctions template, utiliser un static_assert à l'intérieur de la fonction et un cast dans la fonction qui appelle. C'est un peu moche, mais c'est la seule chose qui compilait correctement.
23/09/2015, 21h33
dragonjoker59

Citation:

Envoyé par ToTo13

Merci pour ta réponse !

J'avais essayé... mais j'ai dû faire une bêtise. Je vais retester.
J'ai pour l'instant utiliser une autre solution : déclarer les fonctions template, utiliser un static_assert à l'intérieur de la fonction et un cast dans la fonction qui appelle. C'est un peu moche, mais c'est la seule chose qui compilait correctement.

J'aimerais bien voir ça... Ca sent l'horreur...

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#ifndef DILATE_H
#define DILATE_H
 
namespace mylib
{
namespace Dilate
{
 
template<typename T> void HistX(const T *src, const int sewidth, T *res)
	{
	static_assert(std::is_same<T, UINT8>::value, "") ;
	...
	}
 
template<typename T> void InterHistX(const UINTT8 *src, const int sewidth, T *res)
	{
	static_assert(std::is_same<T, UINT8>::value, "") ;
	...
	}
 
template<typename T> void FmX(const T *src, const int sewidth, T *res) {...}
 
template<typename T> void AvX(const T *src, const int sewidth, T *res) {...}
 
 
 
template<typename T> void SegmentX(const T *src, const int sewidth, T *res)
	{
	if ( sewidth <= 13 ) AvX(src, sewidth, res) ;
	else 
		if ( std::is_same<T, UINT8>::value ) HistX((UINT8*)src, sewidth, (UINT8*)res) ;
		else FmX(src, sewidth, res) ;
	}
 
template<typename T> void SegmentInterX(const T *src, const int sewidth, T *res)
	{
	if ( std::is_same<T, UINT8>::value )
		if ( sewidth <= 13 ) AvX(src, sewidth, res) ;
		else InterHistX((UINT8*)src, sewidth, (UINT8*)res) ;
	else
		if ( sewidth <= 13 || channel != 1 ) AvX(src, sewidth, res) ;
		else FmX(src, sewidth, res) ;
	}
 
} // namespace Dilate
} // namespace mylib
#endif	/* DILATE_H */

Du bidouillage sans aucun doute, mais ça compile. Je vais retenter en ne faisant que les déclarations dans le header.

Je ne peux que te conseiller vivement d'enlever ces templates inutiles, et de faire beaucoup plus simple :

Dilate.h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#ifndef DILATE_H
#define DILATE_H
 
namespace mylib
{
namespace Dilate
{
 
	void HistX(const UINT8 *src, const int sewidth, UINT8 *res);
 
	void InterHistX(const UINT8 *src, const int sewidth, UINT8 *res);
 
	...
	...
	...
 
} // namespace Dilate
} // namespace mylib
#endif	/* DILATE_H */

Dilate.cpp

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#include "Dilate.h"
 
namespace mylib
{
namespace Dilate
{
 
	void HistX(const UINT8 *src, const int sewidth, UINT8 *res)
	{
	...
	}
 
	void InterHistX(const UINT8 *src, const int sewidth, UINT8 *res)
	{
	...
	}
 
} // namespace Dilate
} // namespace mylib

23/09/2015, 23h22
ToTo13

C'est bien mon intention (je teste dès que j'ai un moment).
J'avais essayé cette solution, mais j'avais dû faire une erreur car ça ne compilait pas :-(
24/09/2015, 02h11
ToTo13

Effectivement, il semblerait qu'il y ait maintenant un autre problème :

Citation:

Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentInterX(const T*, int, int, int, int, int, T*, int) [with T = short unsigned int]':
morphee_fastMorphee_DilateSegmentX.cpp:53:108: required from here
Dilate.h:324:25: error: cannot convert 'const short unsigned int*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentInterHistX(const UINT8*, int, int, int, int, int, UINT8*, int)'
else SegmentInterHistX(src, sizex, sizey, sizez, channel, sewidth, res, nbCPU) ;
^
Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentInterX(const T*, int, int, int, int, int, T*, int) [with T = int]':
morphee_fastMorphee_DilateSegmentX.cpp:71:108: required from here
Dilate.h:324:25: error: cannot convert 'const int*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentInterHistX(const UINT8*, int, int, int, int, int, UINT8*, int)'
Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentInterX(const T*, int, int, int, int, int, T*, int) [with T = float]':
morphee_fastMorphee_DilateSegmentX.cpp:89:108: required from here
Dilate.h:324:25: error: cannot convert 'const float*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentInterHistX(const UINT8*, int, int, int, int, int, UINT8*, int)'
Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentInterX(const T*, int, int, int, int, int, T*, int) [with T = double]':
morphee_fastMorphee_DilateSegmentX.cpp:107:108: required from here
Dilate.h:324:25: error: cannot convert 'const double*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentInterHistX(const UINT8*, int, int, int, int, int, UINT8*, int)'
Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentX(const T*, int, int, int, int, T*, int) [with T = short unsigned int]':
morphee_fastMorphee_DilateSegmentX.cpp:155:84: required from here
Dilate.h:315:52: error: cannot convert 'const short unsigned int*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentHistX(const UINT8*, int, int, int, int, UINT8*, int)'
if ( std::is_same<T, UINT8>::value ) SegmentHistX(src, sizex, sizey, sizez, sewidth, res, nbCPU) ;
^
Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentX(const T*, int, int, int, int, T*, int) [with T = int]':
morphee_fastMorphee_DilateSegmentX.cpp:172:89: required from here
Dilate.h:315:52: error: cannot convert 'const int*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentHistX(const UINT8*, int, int, int, int, UINT8*, int)'
Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentX(const T*, int, int, int, int, T*, int) [with T = float]':
morphee_fastMorphee_DilateSegmentX.cpp:189:89: required from here
Dilate.h:315:52: error: cannot convert 'const float*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentHistX(const UINT8*, int, int, int, int, UINT8*, int)'
Dilate.h: In instantiation of 'void FiReTiTiLiB::Dilate::SegmentX(const T*, int, int, int, int, T*, int) [with T = double]':
morphee_fastMorphee_DilateSegmentX.cpp:206:89: required from here
Dilate.h:315:52: error: cannot convert 'const double*' to 'const UINT8* {aka const unsigned char*}' for argument '1' to 'void FiReTiTiLiB::Dilate::SegmentHistX(const UINT8*, int, int, int, int, UINT8*, int)'

Il semblerait que le test std::is_same<T, UINT8>::value ne fonctionne pas correctement et donc que le dispatch est erroné :-(

[EDIT] Bien évidemment tout compile si je rajout un cast UINT8 dans les deux méthodes qui font les dispatch. Pourquoi est ce que ça ne compile pas sans les cast ?
24/09/2015, 09h19
prgasp77

Mais tu donnes un int* là où un unsigned char* (a/k/a UINT8*) est attendu ! Comment veux-tu que le compilateur sache quoi faire ?
Tu peux soit modifier l'appel, soit faire en sorte que tes fonctions puissent prendre des int*, par polymorphisme ou par template.
24/09/2015, 09h23
ToTo13

Citation:

Envoyé par prgasp77

Mais tu donnes un int* là où un unsigned char* (a/k/a UINT8) est attendu ! Comment veux-tu que le compilateur sache quoi faire ?

Comme je faisais le test avant, j'espérais qu'il soit capable de faire la différence.

Citation:

Envoyé par prgasp77

Tu peux sois modifier l'appel

Comment ?
En faisant un simple cast comme j'ai fait, ou alors y a t-il mieux ?

Citation:

Envoyé par prgasp77

par polymorphisme ou par template.

Ces fonctions ne peuvent traiter QUE des UINT8.
24/09/2015, 09h53
prgasp77

Citation:

Envoyé par ToTo13

En faisant un simple cast comme j'ai fait, ou alors y a t-il mieux ?

Un reinterpret_cast serait une très mauvaise idée, si tu veux modifier l'appel, et puisque tu passes par des pointeurs, tu dois déclarer une variable de type UINT8, lui assigner la valeur souhaitée depuis l'int, et appeler la fonction incriminée avec l'adresse de l'UINT8.

Citation:

Envoyé par ToTo13

Ces fonctions ne peuvent traiter QUE des UINT8.

Tu peux les encapsuler.
24/09/2015, 09h56
ToTo13

Désolé, je suis débutant en C++ et là j'avoue que j'ai du mal à comprendre ce que serait la solutions optimale à ce problème...

Voici une idée d'encapsulation. A adapter aux besoins. Penser à nommer correctement les variable en fonction de leur utilité.

Code:

1
2
3
4
5
6
7
8
9
10
11
void HistX(const UINT8 *src, const int sewidth, UINT8 *res);
 
template<class IntegerSource, class IntegerRes>
void HistX_(IntegerSource const& src, const int sewidth, IntegerRes& res)
{ 
    UINT8 u8src(src);
    UINT8 u8res(res);
    HistX(&u8src, sewidth, &u8res);
    // res = output?
    res = u8res;
}

Code:

1
2
3
4
5
6
7
template<typename T> void SegmentX(const T *src, const int sewidth, T *res)
{
	if ( sewidth <= 13 ) AvX(src, sewidth, res) ;
	else 
		if ( std::is_same<T, UINT8>::value ) HistX((UINT8*)src, sewidth, (UINT8*)res) ;
		else FmX(src, sewidth, res) ;
}

Il te suffit de spécialiser cette fonction, en fonction de T :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
template<typename T> void SegmentX(const T *src, const int sewidth, T *res)
{
	if ( sewidth <= 13 )
		AvX(src, sewidth, res) ;
	else 
		FmX(src, sewidth, res) ;
}
 
template<> void SegmentX< UINT8 >( const UINT8 *src, const int sewidth, UINT8 *res)
{
	if ( sewidth <= 13 )
		AvX(src, sewidth, res) ;
	else 
		HistX(src, sewidth, res);
}

Citation:

Envoyé par dragonjoker59

Il te suffit de spécialiser cette fonction, en fonction de T :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
template<typename T> void SegmentX(const T *src, const int sewidth, T *res)
{
	if ( sewidth <= 13 )
		AvX(src, sewidth, res) ;
	else 
		FmX(src, sewidth, res) ;
}
 
template<> void SegmentX< UINT8 >( const UINT8 *src, const int sewidth, UINT8 *res)
{
	if ( sewidth <= 13 )
		AvX(src, sewidth, res) ;
	else 
		HistX(src, sewidth, res);
}

Si je fais ça, j'ai à nouveau une "duplicate symbol error".

24/09/2015, 11h29
ternel

Alors, rajoute un inline (pour les spécialisations completes, je crois que c'est requis)
24/09/2015, 11h57
ToTo13

Merci à tous, ça fonctionne !