Discussion :

[patricx]

bonjour j"aimerais faire le somme de deux vecteurs en utilisant une surcherge d'operateur en C++ voici ma methode

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
 
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+'

[koala01]

Salut,

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

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

Code :

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

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

[patricx]

voici mon fichier.h

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
 
#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 :

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
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 :

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
 
#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;
 }

[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

N'a tu pas un tout petit peu l'impression qu'il y a comme qui dirait un problème

[patricx]

Code :

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

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

[Astraya]

euh...

Code :

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

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 :

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

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.

[koala01]

Code :

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

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 :

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

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 :

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

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 :

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

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...