surcharge d' operateur

Version imprimable

bonjour j"aimerais faire le somme de deux vecteurs en utilisant une surcherge d'operateur en C++ voici ma methode
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 vecteur vecteur::operator+(const vecteur& vecter) { int i; int *tabl; tabl=new int[Nblignes]; //les valeurs de vecteur1 sont chargés dans tabl for(i=0;i<Nblignes;i++) { tabl[i]=tab[i]; } // somme des elements de tabl et vecteur2 for(i=0;i<Nblignes;i++) { tabl[i]=vecter.tab[i]; } //Création d'un type de retour vecteur resultat (*tabl); //retour du resulatt return resultat; }
je sais pas si elle est correcte

l'instruction qui appele cette methode est

resultat=vecteur1+vecteur2;

quand je compile j'ai cette unique erreur

invalid operands of types `vecteur*' and `vecteur*' to binary `operator+'

28/12/2009, 08h06
koala01

Salut,

Pourrait-on avoir la définition de ta classe vecteur, et l'implémentation de tes différents constructeurs :question:

Ceci dit, il y a une ligne qui m'intrigue au plus haut point... c'est

Code:

vecteur resultat (*tabl);

En effet, tu prend la peine de créer un tableau et de le remplir, tout cela pour... ne transmettre que le premier élément du tableau au constructeur de ta classe...

Cherchez l'erreur :D8O

fichier.h

voici mon fichier.h

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#ifndef _VECTEUR_H_
#define _VECTEUR_H_
#define min  1
#define max  100
class vecteur
 
{
   int Nblignes ;
   int *tab;
  //tab=new int[Nblignes];
 
   public:
    vecteur(int n);
    void lire();
    void afficher();
    vecteur operator+(const vecteur &vecter);
 
 
    ~vecteur();
};
#endif

Mon fichier .cpp

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
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
 
#include <iostream>
#include "vecteur.h"
using namespace std;
 
 
vecteur:: vecteur (int n)
 { int i;
  cout<<"\n\n Construction du vecteur \n";
 
    do
    { if ((n < min)||(n > max))
       { 
         cout <<" \n\n le nombre d'elements du vecteur doit etre compri entre "<<min <<" et " <<max;
         cout << "\n \nVeuillez recommencer \n";
         cout << "\n \nNombre d elements du vecteur ";
         cin>>n;
       }
    }while((n<min)||(n>max));
 
     cout <<" \n\nle nombre d'elements du vecteur est : "<<n<<"\n";
 
    tab=new int[n];
 
      for(i=0;i<n;i++)
      tab[i]=0;
 
     for(i=0;i<n;i++)
     cout <<" \n\nl element " <<i+1 <<" du vecteur est : " <<tab[i]<<"\n";
   }
 
   void  vecteur::lire()
   { int i;
         cout << "\n \n Entrer le nombre de lignes  du vecteur\n\n ";
         cin>>Nblignes;
     tab=new int[Nblignes];
         for(i=0;i<Nblignes;i++)
          {
            cout<< "\n \n Entrer l element " << i+1<< "  du vecteur\n\n ";
            cin>>tab[i];
         }
   }
 
   void vecteur::afficher()       
   { 
     int i;
        for(i=0;i<Nblignes;i++)
        cout <<" \n\nl element " <<i+1 <<" du vecteur est : " <<tab[i]<<"\n";
   }
 
vecteur vecteur::operator+(const vecteur& vecter)
  { int i;
      int *tabl;
      tabl=new int[Nblignes];
 
    //les valeurs de vecteur1 sont chargés dans tabl
 
      for(i=0;i<Nblignes;i++)
      {
        tabl[i]=tab[i];
      }
 
    // somme des elements de tabl et vecteur2
      for(i=0;i<Nblignes;i++)
      {
        tabl[i]=vecter.tab[i];
      }
 
    //Création d'un type de retour
vecteur resultat (*tabl);
 //retour du resulatt
return resultat; 
  }
 
vecteur:: ~vecteur ()
 {
    cout<<"\n\n Destruction du vecteur \n";       
  }

Mon prog principal

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
#include <iostream>
#include "vecteur.h"
using namespace std;
 
main()
 
 {
   vecteur *vecteur1=new vecteur(3);
 
   vecteur1->lire();
   vecteur1->afficher();
 
   vecteur *vecteur2=new vecteur(3);
 
   vecteur2->lire();
   vecteur2->afficher();
   vecteur *resultat=new vecteur(3);
 
   resultat=vecteur1+vecteur2;
   delete vecteur1;
 }

28/12/2009, 10h05
koala01
oui, ben, c'est encore pis que ce que j'aurais pu penser...

Reprends, si tu veux bien, ta logique pour réfléchir trente secondes...

Dans ton opérateur, tu:
crée un tableau dynamique
1. le rempli avec les éléments du tableau courent
2. additionne les valeurs du nouveau tableau une à une avec celles du tableau de l'élément passé en paramètre, jusqu'ici, tout va très bien... mais après
3. tu passe le premier élément de ce tableau au constructeur de ta classe vecteur qui... le considère comme... le nombre d'élément à allouer au tableau...
Heuu... ca te semble logique :question:

N'a tu pas un tout petit peu l'impression qu'il y a comme qui dirait un problème :question:
28/12/2009, 11h26
patricx

tableau
Code:

1 2 3. tu passe le premier élément de ce tableau au constructeur de ta classe vecteur qui... le considère comme... le nombre d'élément à allouer au tableau...
je vois pas laquelle de mes instructions le fait

Apres l'addition je crée un type de retour (resultat) et je j'utilise pour retourner le tableau à partir du pointeur vers le premier element
28/12/2009, 17h11
Astraya
euh... :mouarf:
Code:

1 2 3 4 5 6 7 8 9 10 int i; int *tabl; tabl=new int[Nblignes]; //les valeurs de vecteur1 sont chargés dans tabl for(i=0;i<Nblignes;i++) { tabl[i]=tab[i]; }
N'a aucun interêt, sauf si tu veux modifier tab... mais dans un operator + les deux membres doivent être constant.
De plus
Code:

1 2 3 4 5 for(i=0;i<Nblignes;i++) { tabl[i]=vecter.tab[i]; }
ça veut dire que tabl[] = vecter.tab[].... alors pourquoi l'avoir remplis avant avec le membre de droite? tu l'efface avec le membre de gauche...
Et selon ton code tu devrais retourné vecteur v(NbLignes + vecter.GetNbLignes()).. car effectivement tu ne revois jamais de pointeur... le type de ta fonction est vecteur... tu doit retourné une copie.
28/12/2009, 17h22
koala01
Citation:
Envoyé par patricx

Code:

1 2 3. tu passe le premier élément de ce tableau au constructeur de ta classe vecteur qui... le considère comme... le nombre d'élément à allouer au tableau...

je vois pas laquelle de mes instructions le fait

Apres l'addition je crée un type de retour (resultat) et je j'utilise pour retourner le tableau à partir du pointeur vers le premier element
Oui, mais réfléchis un peu à ce que fait

Code:

Tableau resultat(*tabl);

*tabl indique que tu transmet au constructeur... ce qui est pointé par le pointeur nommé tabl...

En l'occurence, il s'agit... du premier élément du tableau (cela revient strictement au même que si tu avais utilisé tabl[0])...

Ce premier élément de tabl, tu le transmet au constructeur de ta classe Vecteur, dans lequel on lit:
Code:

1 2 3 4 vecteur:: vecteur (int n) { if ((n < min)||(n > max)) cout <<" \n\n le nombre d'elements du vecteur doit etre compri entre "<<min <<" et " <<max;
qui nous ramènent aux directives préprocesseurs
Code:

1 2 #define min 1 #define max 100
... C'est quand même pas moi qui l'ai inventé, ce code là ;)

Tu vois donc bien que ta ligne Vecteur resultat (*tabl) transmet le premier élément du tableau comme... nombre d'élément au constructeur de ta classe vecteur...

CQFD...
30/12/2009, 07h09
patricx

typre de retour

voici l'ordre dans lequel je m'inscrit
//Création d'un type de retour
vecteur resultat (*tabl);

//retour du resulat
return resultat;

Depuis j cherche mais j pariens à rien

quand je compile dans mon main j'ai cette ligne
resultat=vecteur1+vecteur2;
qui me pose cette erreur
invalid operands of types `vecteur*' and `vecteur*' to binary `operator+'
30/12/2009, 07h15
Astraya

resultat=vecteur1+vecteur2;
Ce que tu fais c'est
resultat = adresse du pointeur du vecteur1 + adresse du pointeur du vecteur2
normal qui ne compile pas!
Essaye avec:
resultat= (*vecteur1) + (*vecteur2);
30/12/2009, 10h44
patricx

resultat= (*vecteur1) + (*vecteur2);

me donne un autre message qui est

cannot convert `vecteur' to `vecteur*' in assignment
30/12/2009, 14h49
koala01
C'est vraiment à se demander si tu comprend le code que tu écris ou si tu le fais exprès...

Je reprend donc ton code "ligne par ligne" pour t'indiquer les différents problèmes et ce que font ces liges:

1-
Code:

1 2 int *tabl; tabl=new int[Nblignes];
créent dynamiquement un tableau de NbLignes entiers nommé tabl

2-
Code:

1 2 3 4 for(i=0;i<Nblignes;i++) { tabl[i]=tab[i]; }
copient les valeurs des éléments de tab dans tabl, jusque là, tout va bien... Mais les choses se corsent après:

3-
Code:

1 2 3 4 for(i=0;i<Nblignes;i++) { tabl[i]=vecter.tab[i]; }
copient le contenu de vecter.tab dans tabl...

C'est à dire que cela remplace les valeurs des éléments de tabl que tu viens justement de donner... par les éléments de vecter.tab...

Ce n'est pas ce que tu veux faire: tu voudrais surement additionner les deux valeurs...

4-

Code:

vecteur resultat (*tabl);

Cette ligne crée un vecteur en appelant vecteur::vecteur(int), en lui passant... la valeur du premier élément de tabl (ce qui revient au même que de lui passer tabl[0])...

A partir d'ici, on entre donc dans la fonction vecteur::vecteur(int)... qu' est le constructeur ;)

Regardons maintenant ce que fait vecteur::vecteur(int):

5-

Code:

int i;

déclare une variable de type entier nommée i, soit

Code:

cout<<"\n\n Construction du vecteur \n";

provoque un affichage... pas vraiment utile, mais bon...

6-
Code:

1 2 do {
Entre dans une boucle pour laquelle la décision d'y retourner sera évaluée en fin de boucle (en 10)

7-
Code:

1 2 if ((n < min)||(n > max)) {
Si n, qui correspond, dans le cas présent, à la valeur du premier élément de tabl, est plus petit que min et plus petit que max (définis respectivement à 1 et 100)

8-

Code:

cout <<" \n\n le nombre d'elements du vecteur doit etre compri entre "<<min <<" et " <<max;

nous affiche que le nombre d'éléments du vecteur doit être compris entre 1 et 100...

Tiens, je croyais que c'était la valeur du premier élément du tableau créé dans l'opérateur + :question::question:8O:koi:

Et tu nous le confirme encore bien, en affichant

9-
Code:

1 2 cout << "\n \nVeuillez recommencer \n"; cout << "\n \nNombre d elements du vecteur ";
et en attendant une introduction de la part de l'utilisateur

Code:

cin>>n;

Et tu fais cela...
10-
Code:

1 2 } }while((n<min)||(n>max));
jusqu'à ce que n soit effectivement compris entre min et max (respectivement entre 1 et 100)...
retourne en (6) si ce n'est pas le cas

11-

Code:

cout <<" \n\nle nombre d'elements du vecteur est : "<<n<<"\n";

Une fois que tu es content, tu nous confirme encore bien que c'est le nombre d'éléments du vecteur...
12-

Code:

tab=new int[n];

Tu alloue dynamiquement la mémoire pour un nouveau tableau composé de n éléments

13-
Code:

1 2 for(i=0;i<n;i++) tab[i]=0;
Tu initialise les différents éléments du tableau à 0

14-
Code:

1 2 for(i=0;i<n;i++) cout <<" \n\nl element " <<i+1 <<" du vecteur est : " <<tab[i]<<"\n"
Et tu affiche la valeur de ces éléments... que tu viens d'initialiser à...0...

15-

Code:

}

On quite le constructeur:

Nblignes vaut strictement n'importe quoi, mais a peu de chances de valoir n

On perd la valeur de n, mais comme on ne sais plus trop s'il s'agissait du premier élément de tabl (comme le laisse supposer l'appel du constructeur) ou le nombre d'élément du tableau (comme le laisse supposer le code du constructeur), finalement, quelle importance :question:

Nous revenons ici dans la fonction operator +
16-
Code:

1 2 return resultat; }
Tu renvoies le vecteur nouvellement créé, qui contient surement quelque chose, mais qui ne contient surement ni les valeur de l'un des vecteur d'origine ni, et encore moins, des éléments représentant la somme des éléments respectifs de ces deux vecteurs...

Comme on quitte la fonction, on perd "toute trace de tabl, et nous avons, en plus, une jolie fuite mémoire...

Alors, si, en plus, on met (1) en relation avec (15), on se rend compte que Nbligne vaut... strictement n'importe quoi, mais ne représente surement pas le nombre d'éléments qu'il y a dans le tableau...

Bref, nous partons en couille intégrale dans toute la logique...
30/12/2009, 17h54
patricx

correction
je comprend mieux

pour y palier

je fais ceci

3- devient
Code:

1 2 3 4 5 for(i=0;i<Nblignes;i++) { tabl[i]+=vecter.tab[i]; }
à partir de 4- je vois pas trop comment palier à
je voudrais
Créer un type de retour c'est à dire retourner le tableau qui contient la somme des deux tableaux

et le retourner et c'est là ue reside tout mon probleme
Enfait je pas trop comment
30/12/2009, 20h02
Astraya

1_ Tu créer vecteur temp en premier
2_ Tu rajoute le membre de droite
3_ tu rajoute après le membre de droite le membre de gauche
4_ tu retourne temp
(si j'ai bien compris ce que tu veux)
30/12/2009, 20h42
koala01
Citation:

Envoyé par Astraya

1_ Tu créer vecteur temp en premier
2_ Tu rajoute le membre de droite
3_ tu rajoute après le membre de droite le membre de gauche
4_ tu retourne temp
(si j'ai bien compris ce que tu veux)

A ceci près que l'opérateur + dois renvoyer... un objet de type vecteur :P
Citation:
Envoyé par patricx

je comprend mieux

pour y palier

je fais ceci

3- devient

Code:

1 2 3 4 5 for(i=0;i<Nblignes;i++) { tabl[i]+=vecter.tab[i]; }
C'est déjà mieux :D

Citation:

à partir de 4- je vois pas trop comment palier à
je voudrais
Créer un type de retour c'est à dire retourner le tableau qui contient la somme des deux tableaux

et le retourner et c'est là ue reside tout mon probleme
Enfait je pas trop comment

Peut-être serait il intéressant d'envisager de... créer un constructeur supplémentaire :question:
30/12/2009, 21h02
patricx

_ Tu créer vecteur temp en premier
2_ Tu rajoute le membre de droite
3_ tu rajoute après le membre de droite le membre de gauche
4_ tu retourne temp
(si j'ai bien compris ce que tu veux)

4- est mon probleme meme je fais return *tabl ou return tabl ya pas resultat meme erreur

bon.....

Code:

1
2
3
4
5
6
7
8
vecteur vecteur::operator+(const vecteur& vecter)
{
    vecteur temp;
 
    //Tu additione les deux vecteurs
 
    return temp;
}

Si tu fais ça sa compilera...

bonjour et desolé mais ça me pose le meme probleme

voici mon code

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
vecteur vecteur::operator+(const vecteur& vecter)
  { int i;
      vecteur *tabl=new vecteur(3);
 
 
    //les valeurs de vecteur1 sont chargés dans tabl
 
      for(i=0;i<Nblignes;i++)
      {
        tabl[i]=tab[i];
      }
 
    // somme des elements de tabl et vecteur2
      for(i=0;i<Nblignes;i++)
      {
        tabl[i]=tabl[i]+vecter.tab[i];
      }
 
 
    //retour du resulatt
      return *tabl; 
  }