Discussion :

[tHE_fLAmMinG_mOE]

Bonjour,

Je suis en train d'�crire une classe de gestion de grands entiers, et j'aimerais red�finir les op�rateurs de base.

Seulement voil�, j'ai une erreur de linkage que je n'arrive pas � r�soudre (ou pas comme je veux en tout cas).

voici le code de ma classe :

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
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
 
#ifndef BIGINT_H
#define BIGINT_H
 
#include <iostream>
#include <string>
 
using namespace std;
 
namespace RSAPTut {
 
	class BigInteger {
	private:
		int size;	// le nombre de chiffres
		int sign;	// -1 si <0, 0 sinon
		unsigned short * number;	// chiffres du moins significatif au plus significatif
 
	public:
		// Constructeurs
		BigInteger();
		BigInteger(BigInteger * big);
 
 
		// Getters
		inline int getSign() { return this->sign;	}
		inline int getSize() { return this->size;	}
		unsigned short * getNumber() {	return this->number;	}
 
		// Setters
		inline void setSign(int sign) {	this->sign=sign;	}
		inline void setSize(int size) {	this->size=size;	}
		inline void setNumber(unsigned short * number, int size) {
			this->size = size;
			this->number = new unsigned short[size];	// on alloue la mémoire pour number
			for (int i=0; i<size; i++)
				this->number[i] = number[i];	// on recopie la valeur dans number
		}
 
		// Utilitaires
		void print();
		void stringToBigInteger(char * s);
		void setDigit(unsigned short d);
 
	};
 
 
 
	// Méthodes primitives
	[...]
 
	// Opérations de base
	BigInteger add(BigInteger M, BigInteger N);	// Addition
	[...]
 
	// Opérations modulaires
	[...]
 
	// Opérateurs
	BigInteger operator+(const BigInteger &m, const BigInteger &n)	{
		BigInteger r = new BigInteger( add(m,n) );
 
		return r;
	}
 
};
 
#endif

j'ai d�fini mon op�rateur + en dehors de ma classe mais dans un namespace.

En plus de ce code j'ai :

• l'impl�mentation dans un fichier s�par�, qui inclus ce .h et o� toutes les m�thodes sont impl�ment�es dans le namespace
• un fichier main pour tester

Si je compile tel quel, j'obtiens le message d'erreur :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
 
Compiling...
main.cpp
BigInteger.cpp
Linking...
main.obj : error LNK2005: "class RSAPTut::BigInteger __cdecl RSAPTut::operator+(class RSAPTut::BigInteger const &,class RSAPTut::BigInteger const &)" (??HRSAPTut@@YA?AVBigInteger@0@ABV10@0@Z) already defined in BigInteger.obj
main.obj : error LNK2005: "class RSAPTut::BigInteger __cdecl RSAPTut::operator+(class RSAPTut::BigInteger const &,class RSAPTut::BigInteger const &)" (??HRSAPTut@@YA?AVBigInteger@0@ABV10@0@Z) already defined in BigInteger.obj
Debug/RSA.exe : fatal error LNK1169: one or more multiply defined symbols found
Error executing link.exe.
 
RSA.exe - 3 error(s), 0 warning(s)

J'arrive � faire fonctionner l'op�rateur en le d�finissant dans le fichier ou se trouve le main :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
 
// Opérateurs
RSAPTut::BigInteger operator+(const RSAPTut::BigInteger &m, const RSAPTut::BigInteger &n)	{
	RSAPTut::BigInteger r = new RSAPTut::BigInteger( RSAPTut::add(m,n) );
 
		return r;
	}
 
int main()
{
[...]
}

Voil� donc je ne sais pas comment r�soudre ce probl�me (je ne souhaite pas mettre tous les op�rateurs dans le main.cpp), donc si quelqu'un sait comment faire ?
merci.

[Jean-Marc.Bourguet]

Tu d�finis des fonctions non inline dans un en-t�te. L'en-t�te devrait se contenter de les d�clarer.

[Matthieu Brucher]

Ca a l'air louche ce que tu fais dans ton op�rateur +. Tu assignes � un objet un pointeur vers un autre ?? Pas correct, �a. Pourquoi ne pas retourner directement add(m, n) ?

Bon, ensuite, pour ton op�rateur, tu devrais faire comme pour add, ne d�clarer que le prototype et mettre le corps de la fonction dans un autre fichier .cpp, genre BigInteger.cpp.

[tHE_fLAmMinG_mOE]

merci

j'ai corrig� pour le setNumber, et la bidouille dans le operator+ en fait ca venait d'une petite erreur que j'avais avant qui faisait que j'avais du bidouiller. je l'ai mis en inline :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

	inline BigInteger operator+(const BigInteger &m, const BigInteger &n) {	return add(m,n);	}

et �a marche nickel !

merci � vous.

[Sylvain Togni]

C'est normal, la d�finition de operator+ est dans le .h, ce qui fait qu'il est compil� plusieurs fois.

La solution est soit de mettre son code dans le .cpp soit d'utiliser inline.

[Matthieu Brucher]

La solution est soit de mettre son code dans le .cpp soit d'utiliser inline.

Sachant qu'inline n'est qu'une proposition, pas une obligation que le compilateur doit suivre.

[Sylvain Togni]

Sachant qu'inline n'est qu'une proposition, pas une obligation que le compilateur doit suivre.

�a d�pend, inline a deux sens :
- G�n�rer le code de la fonction en-ligne,
- Permettre � une fonction d'�tre d�finie dans plusieurs unit�s de compilation.
Le premier n'est pas une obligation, en effet. Le second, si.