Discussion :

[BainE]

Bonjour,

je suis en train de refaire la classe vector de la STL (le prétentieux, non c'est juste pour pas perdre la main, pas la perdre plus en tout cas).

La j'ai un petit souci, j'ai codé ca :

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
 
   template < class T >
   Vector< T > Vector< T >::operator+( Vector< T > &right){
        Vector< T > vec ;
        vec.lst = lst;
 
        vec.lst.insert( vec.lst.end(), right.lst.begin(), right.lst.end() );
 
        return vec;
   }
 
   template < class T >
   Vector< T >& Vector< T >::operator=(Vector< T > &right){
        lst.clear();
 
        lst.insert( lst.begin(), right.lst.begin(), right.lst.end() );
 
        return *this;
   }

Ces deux methodes sont declaré comme ca dans le corps e la classe :

Code :

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

1
2
3
 
Vector operator+(Vector &right);
Vector& operator=(Vector &right);

dans mon mon de test des fois ca compil pas (ca c'est normal), mais je ne comprends pas et j aime pas trop

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
 
//En haut du main
//Pour que l'appel soit plus sympa
//D'ailleur au passage si vous connaiisez mieux que ca ;)
#define IntVecteur Vecteur<int>
 
int main( char** argc, int argv ){
   IntVecteur v1;
   IntVecteur v2;
   IntVecteur v3;
 
   //Je leur met quelques valeurs de test
   //...
 
   v3 = v1          // Ca ca marche
   V1 + v2          // Ca ca marche aussi
   v3 = v1 + v2   // La ca coince
}

et la on me dit :

error: no match for 'operator=' in 'autre =
Vector<T>::operator+(Vector<T>&) [with T = int]((&v))'
IntVector.h:138: error: candidates are: Vector<T>&
Vector<T>::operator=(Vector<T>&) [with T = int]

je ne comprends pas si vous pouviez me donner une explication

[Laurent Gomila]

Code :

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

1
2
//D'ailleur au passage si vous connaiisez mieux que ca ;)
#define IntVecteur Vecteur<int>

Code :

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

typedef Vecteur<int> IntVecteur;

Mais bon là t'as économisé deux caractères...

Code :

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

v3 = v1 + v2   // La ca coince

v1 + v2 génère un temporaire, qui ne peut pas être pris en paramètre par ton opérateur = puisque celui-ci prend une référence non constante. Tu dois prendre une référence constante, puisque tu ne modifies pas le paramètre à l'intérieur de la fonction. Pareil pour le paramètre de l'opérateur +. Ce dernier devrait également être const.

Code :

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

1
2
Vector operator+(const Vector &right) const;
Vector& operator=(const Vector &right);

[BainE]

Merci,

va falloir que je revoie les const

par contre tant que j'y suis j'ai un warning si tu as le temps d'y jeter un coup d'oeil

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
 
 In member function `void Vector<T>::resize(int)':
IntVector.h:72: warning: `std::list<T, std::allocator<_CharT> >::iterator' is
   implicitly a typename
IntVector.h:72: warning: implicit typename is deprecated, please see the
   documentation for details
 
//La fonction concerné :
     67 template < class T >
     68 void Vector< T >::resize(int newSize){
     69         list< T > tmp = lst;
     70         lst = *( new list<T>( newSize, 0 ) );
     71
     72         list< T >::iterator it_lst = lst.begin();
     73         list< T >::iterator it_tmp = tmp.begin();
     74
     75         while( it_tmp != tmp.end() ){
     76                 *it_lst++ = *it_tmp++;
     77         }
     78 }

Merci d'avance

[Laurent Gomila]

http://c.developpez.com/faq/cpp/?pag...LATES_typename

[Médinoc]

Essaie avec ceci:

Code :

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

typename list< T >::iterator it_lst = lst.begin();

Arrgh, grillé.

[Laurent Gomila]

Code :

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

lst = *( new list<T>( newSize, 0 ) );

Outch... Non seulement la liste allouée sur le tas ne sera pas libérée à la sortie de la fonction, mais en plus tu n'auras aucun moyen de le faire. D'autant plus que tu n'as absolument pas besoin d'allouer une liste sur le tas et de faire autant de copies.

[BainE]

Quel Talent Messieurs !!!

Je vous remercie

P.S. : Pour l'allocation oui c'est un peu (c'est un doux euphemisme)crade mais c'est ce que j'avais de plus rapide en tete, (vu que je simule un vecteur on doit pouvoir lui spécifier une taille)

Sur ce bon week end