affectation de classes generiques

Version imprimable

07/03/2005, 23h14
Mehdi Feki

affectation de classes generiques
Bonjour tout le monde

Soit une classe generique
Code:

1 2 3 4 5 6 7 template <class Telem> class Vecteur { //declaration de la classe !! }
et j'ai deux instances
Vecteur<int> v1;
et
Vecteur<float> v2;

Comment puis-je affecter la deuxieme instance a la premiere ??
v1=v2;

merci de votre aide !!!
08/03/2005, 00h23
jmv
salut,
Impossible de te repondre sans connaitre ta classe Vecteur.

Avec la classe vector de la STL je ferais comme ça :
Code:

1 2 3 v1.resize(v2.size()); for(int i=0; i<v1.size(); i++) v1[i] = (int)v2[i];

voici ma classe :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
template <class Telem=int,int N=0,Telem zero=0>
class Vector  
{
public:
 
Vector(){for(int i=0;i<N;i++){tab[i]=zero;}};
Vector<Telem,N,zero> operator+(const Vector<Telem,N,zero> &V);
Vector<Telem,N,zero>& operator=(const Vector<Telem,N,zero> &V);
Telem operator[](int i);
virtual ~Vector();
 
private:
 
Telem tab[N];
 
};

l'idée est que lors de la'affectation le cast se fait implicitement !!!

08/03/2005, 02h03
JolyLoic

Voir le thread " Classe Matrix<T>" où ce sujet a été discuté
08/03/2005, 11h52
jmv
j'adapte mon code précédement posté à ta classe :
Code:

1 2 3 4 5 6 const int size=10; // par exemple Vecteur<int,size> v1; Vecteur<double,size> v2; ... for(int i=0; i<size; i++) v1[i] = (int)v2[i];
(code non testé :oops: )

Il faut que tu modifie ton operator[], il doit renvoyer une référence sinon on ne peut pas écrire v1[i]=...

Code:

Telem& operator[](int i);
08/03/2005, 16h32
Mehdi Feki

merci pour vos reponses !!

jmv merci pour la correction de la reference :wink:
mais je dois mettre ton code dans une methode mais moi j e veux affecter directement tu vois !!!!

JolyLoic : g consulté le lien du thread mais j'ai rien compris de la solution.
est ce que je dois creer un constructeur de conversion ??? !!!
genre :

Code:

Vector<float> operato Vector<int>(){ }

si oui qu'est ce que je dois ecrire dedans

merci encore !!!!

Tu dois effectivement ecrire un constructeur de conversion (et non un operateur de cast comme ce que tu as écris)
Code:

1 2 3 4 5 6 7 8 9 10 template<class T> class Vecteur { template <class U> Vecteur(const Vecteur<U>& v) { // Conversion et copie des éléments de v dans this } // ... };

09/03/2005, 09h15
jmv

Citation:

Envoyé par mehdi_tn

j'ai deux instances
Vecteur<int> v1;
et
Vecteur<float> v2;
Comment puis-je affecter la deuxieme instance a la premiere ??
v1=v2;

Citation:

Envoyé par Loulou24

Tu dois effectivement ecrire un constructeur de conversion ...

ça ne serait pas plutôt un operator= qu'il faudrait définir ici ???
09/03/2005, 11h02
Laurent Gomila
Non, car on pourrait avoir ce genre d'abberation :
Code:

1 2 3 Vecteur<int> v1; Vecteur<double> v2 = v1; // Erreur (appel d'un constructeur, qui n'existe pas) v2 = v1; // Ok (appel de l'opérateur d'affectation)
En général, on ne code toujours qu'un opérateur =, prenant en paramètre un objet de même type, et on laisse le soin aux constructeurs d'effectuer les diverses conversions.
09/03/2005, 12h27
jmv

Oui, c'est vrai, ça marche aussi comme ça.

et moi qui croyais etre un bon programmeur !!! :oops:

deja pour implementer la solution dans ma classe Vector c l'embrouille parce qu'elle prends 3 parametres dans la construction generique

Code:

template <class Telem=int,int N=0,Telem zero=0>

et j'ai eu plusieurs erreurs lors de la compilation que je posterais si je n'ai pas reussi à les resoudre.
Enfin j'ai creé une autre classe plus simple qui prend seulement le type generique et voici ma classe appliquée à la solution :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 template <class U> class Vecteur { public: Vecteur(int taille){tete=0;lataille=taille;tab= new U[taille];}; template <class V> Vecteur(const Vecteur<V> & v) { for(int i=0;i<lataille;i++) tab[i]=(U)v.getTab()[i]; }; void insert(U val){tab[tete]=val;tete++;}; U* getTab(){return tab;}; virtual ~Vecteur(); private: int lataille; int tete; U *tab ; };
j'ai été obligé d'utiliser le getter getTab car si je fai com ca

Code:

for(int i=0;i<lataille;i++)tab[i]=(U)v.tab[i];}

j'aurais cette erreur:

error C2248: 'tab' : cannot access private member declared in class 'Vecteur<int>'
vecteur.h(23) : see declaration of 'tab'
test.cpp(12) : see reference to function template instantiation '__thiscall Vecteur<float>::Vecteur<float>(const class Vecteur<int> &)' being compiled

Mais maintenant en utilisant getTab j'ai cette erreur:
getTab' : cannot convert 'this' pointer from 'const class Vecteur<int>' to 'class Vecteur<int> &' 8O 8O

J'ai une petite confusion dans ma tete :
lors de l'appel de ce constructeur est-ce que le pointeur this est referencé par v ?? Sinon pourquoi l'erreur surgit !!!

Merci d'avance

Citation:

error C2248: 'tab' : cannot access private member declared in class 'Vecteur<int>'

Oui, car Vecteur<U> et Vecteur<V> sont deux classes bien distinctes, qui n'ont donc aucun privilège l'une sur l'autre.
Typiquement pour les accesseurs de ce genre, il faut deux versions : une const et une non-const. Ici il te manque la version const de getTab() pour faire fonctionner ton code (car le vecteur duquel tu appelles cette fonction est lui-même const).

Citation:

deja pour implementer la solution dans ma classe Vector c l'embrouille parce qu'elle prends 3 parametres dans la construction generique

Tu as plusieurs solutions, il suffit de déterminer quels paramètres templates les 2 vecteurs doivent avoir en commun pour pouvoir être copiés :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 template <class Telem=int,int N=0,Telem zero=0> class Vecteur { public : template <class Telem2,int N2,Telem2 zero2> Vecteur(const Vecteur<Telem2, N2, zero2>& v); // Ou alors template <class Telem2> Vecteur(const Vecteur<Telem2, N, zero>& v); // Ou encore... template <class Telem2, Telem2 zero2> Vecteur(const Vecteur<Telem2, N, zero2>& v); };
Si ton vecteur doit posséder le même N et le même zero que celui que tu vas lui affecter, alors il faudra utiliser la 2eme version. Si par contre il n'y a aucune contrainte (ie. n'importe quel Vecteur<X, Y, Z> peut être copié dans un Vecteur<A, B, C>) alors tu peux utiliser la 1ere, etc...

Merci loulou pour tes reponses mais ca ne veut encore pas fonctionner !!!
:( :( :( :( :( :(

Citation:

Typiquement pour les accesseurs de ce genre, il faut deux versions : une const et une non-const

normalement on ne peut pas surcharger les fonctions sans modifier les arguments !!! ???

donc je me re explique :
voici ma class comme tu m'a dis de faire :

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
 
template<class U>
class Vector  
{
public:
	Vector(int lataille)
                 {
                      tete=0;
                      taille=lataille;
                      tab=new U[lataille];
                  };
	template <class V> Vecteur(const Vector<V> & v)
                {
                      for(int i=0;i<lataille;i++)
                             tab[i]=(U)v.getTab()[i];
                };
	void insert(U val)
                {
                     tab[tete]=val;
                     tete++;
                  };
	U* getTab() const 
                  {
                       return tab;
                   };
	U* getTab()
                  {
                       return tab;
                   };
	virtual ~Vector()
                    {
                       delete [] tab;
                     };
private:
int taille;
int tete;
U * tab;
 
};

code edité apres la correction de JolyLoic .

et voila mon main :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
int main(int argc, char* argv[])
{
Vector<int> v1(5);
Vector<float> v2(5);
v1.insert(2);
v2.insert(2.6f);
 
v1=v2;
 
return 0;
}

et l'erreur est :

binary '=' : no operator defined which takes a right-hand operand of type 'class Vector<float>' (or there is no acceptable conversion)

Pourtant j'ai bien suivi ce que t'as indiqué et mon programme a encore un probleme lors du cast. Peut etre que t'as oublié quelques choses !!!!!

STP LOULOU essai de resoudre mon probleme je comptes vraiment sur toi !!

10/03/2005, 22h38
JolyLoic
Citation:

Envoyé par mehdi_tn

Merci loulou pour tes reponses mais ca ne veut encore pas fonctionner !!!
:( :( :( :( :( :(

Citation:

Typiquement pour les accesseurs de ce genre, il faut deux versions : une const et une non-const

normalement on ne peut pas surcharger les fonctions sans modifier les arguments !!! ???

As-tu essayé ?
Le code suivant est autorisé :
Code:

1 2 3 4 5 class A { int f(); int f() const; };
Si tu veux une image pour mieux fixer les idées, le code que j'ai écrit revient à peut près au code suivant :
Code:

1 2 3 4 5 6 7 // Pas du C++ class A { int f(A* this); int f(A const * this); };
Et là, tu retrouve ta règle un peu simplifiée des arguments différents.

Typiquement, pour un vecteur de T, on définira un operator[] qui retournera un T&, et un operator[] const qui retournera un T const &.
10/03/2005, 23h30
Mehdi Feki

merci JolyLoic.
J'ai bien compris pourquoi le compilateur accepte const sans changer d'argument mais le probleme et que l'affectation ne fonctionne pas encore !!! Tu vois l'erreur que j'ai !!! :cry:
11/03/2005, 14h00
Laurent Gomila

Code:

template <class V> Vecteur(const Vector<V> & v)

Bête erreur de frappe : tu as "Vecteur" au lieu de "Vector".
12/03/2005, 16h05
Mehdi Feki

:oops: :oops: :oops: :oops: :oops:
La honte ( sans commentaire pour la faute de frappe)
Merci loulou pour ton aide mon probleme est enfin resolu.
T'es mon nouveau idole du langage C++ ( apres Mr stroustrup "le createur de C++ pour ceux qui ne le conaissent pas").
Envoie moi ta photo pour l'accrocher dans ma chambre :wink: :o