Petite classe simple compile sous Visual mais pas g++.

Petite classe simple compile sous Visual mais pas g++. - C++ - Programmation

Marsh Posté le 08-10-2005 à 22:36:07    

Bonjour,
 
J'ai le classique problème des vecteurs qui me donne du fil à retordre car d'habitude je l'écris sous Visual C++ mais je dois maintenant le compiler avec g++ et ça ne donne pas le même résultat. Ca ne marche pas car les arguments ne sont pas vus du même type.
 

Code :
  1. // main.cpp
  2. #include "Vecteur.h"
  3. #include <iostream>
  4. using namespace std;
  6. int main()
  7. {
  8. Vecteur v10="(1,2,3)";
  9. return 0;
  10. }


 

Code :
  1. // Vecteur.h
  2. #include <iostream>
  3. using namespace std;
  5. class Vecteur
  6. {
  7. private:
  8. float* tab;
  9. int taille;
  10. public:
  11. Vecteur() {taille=0;tab=NULL;}
  12. Vecteur(Vecteur& );
  13. Vecteur(char*);
  14. Vecteur& operator =(Vecteur& );
  15. Vecteur& operator =(char*);
  16. Vecteur& operator +=(Vecteur& );
  17. Vecteur operator +(Vecteur& );
  18. float& operator [](int);
  19. friend ostream& operator <<(ostream&, Vecteur& );
  21. ~Vecteur() { delete[] tab;}
  22. };


 
Extrait de l'implémentation :

Code :
  1. // Vecteur.cpp
  2. #include "Vecteur.h"
  3. #include <iostream>
  4. using namespace std;
  6. Vecteur::Vecteur(Vecteur& vect)
  7. {
  8. int i;
  9. taille=vect.taille;
  10. tab=new float[taille];
  11. for(i=0;i<taille;i++)
  12.  tab[i]=vect.tab[i];
  13. }
  15. Vecteur::Vecteur(char* mytab)
  16. {
  17. int i,j=0;
  18. taille=(int)(strlen(mytab)-1)/2; // string.h ?
  19. tab=new float[taille];
  21. for(i=1;i<(int)strlen(mytab)-1;i++)
  22.  if(mytab[i]!=',')
  23.   tab[j++]=(float)mytab[i]-48; // Attendant de trouver un cast propre
  24. }
  26. Vecteur Vecteur::operator +(Vecteur& vect)
  27. {
  28. int i;
  29. Vecteur Resultat;
  30. Resultat.taille=vect.taille;
  32. Resultat.tab=new float[taille];
  33. for(i=0;i<vect.taille;i++)
  34.  Resultat.tab[i]=vect.tab[i]+tab[i];
  36. return Resultat;
  37. }
  39. Vecteur& Vecteur::operator =(Vecteur& vect)
  40. {
  41. int i;
  42. if(this!=&vect)
  43. {
  44.  if(taille!=0)
  45.   delete[] tab;
  46.  taille=vect.taille;
  47.  tab=new float[taille];
  48.  for(i=0;i<taille;i++)
  49.   tab[i]=vect.tab[i];
  50. }
  51. return *this;
  52. }


 
Et l'erreur :

10 C:\vecteurv\main.cpp no matching function for call to `Vecteur::Vecteur(Vecteur)'  
 note C:\vecteurv\Vecteur.h:13 candidates are: Vecteur::Vecteur(char*)  
 note C:\vecteurv\Vecteur.h:13                 Vecteur::Vecteur(Vecteur& )


 
Et ça marche parfaitement sous Visual. Bon certains diront que ce n'est pas une référence mais j'aimerais bien savoir comment faire fonctionne cela avec g++.
 
Merci !


Message édité par Master_Jul le 08-10-2005 à 22:44:11
Reply

Marsh Posté le 08-10-2005 à 22:36:07   

Reply

Marsh Posté le 08-10-2005 à 23:19:34    

g++ à raison.
Quelle version de visual ?
 
Vecteur v10="{1,2,3}" appelle le constructeur de copie.
vu que Vecteur dispose d'un constructeur Vecteur(char*) non explicit, tu autorise alors une conversion implicite char* <-> Vecteur.
Il y a donc création d'un objet Vecteur temporaire Vecteur("{1,2,3}" ). Or, ton constructeur de copie est déclaré comme cela : Vecteur ( Vecteur& ) ,ce qui est catastrophique. Or il est illégal d'assigner un temporaire à une référence non const, la norme est claire la dessus il m semble.
 
Donc tu sais ce qu'il te reste à faire ...

Spoiler :

mettre des const ! Utiliser la construction Vecteur v10("{1,2,3}" )

Reply

Marsh Posté le 09-10-2005 à 08:14:05    

Merci beaucoup pour ton explication très pédagogue ++fab, c'est comme ça que l'on progresse. Il s'agit de la dernière version .NET beta 2.
 
:hello:
 
Edit : Une petite précision s'il te plaît, qu'entends-tu par conversion non explicite ?
 
Quelle est la différence entre
vecteur v2("(1,2,3)" );
et vecteur v2="(1,2,3)";
 
(les deux semblent fonctionner depuis que j'ai ajouté le const au constructeur par copie)
 
Dans la première écriture, celle que tu m'as proposé, je comprends mieux ce qu'il se passe (passage par le constructeur de recopie). En fait, j'avais essayé de mettre un const, mais pour le char*. :sweat:

Message cité 1 fois
Message édité par Master_Jul le 09-10-2005 à 08:22:25
Reply

Marsh Posté le 09-10-2005 à 10:15:14    

HS : y a moins laid que
v1 = "(1,2,3)";
 
http://forum.hardware.fr/hardwaref [...] 0342-1.htm

Reply

Marsh Posté le 09-10-2005 à 23:11:01    

Master_Jul a écrit :

Quelle est la différence entre
vecteur v2("(1,2,3)" ); (1)
et vecteur v2="(1,2,3)"; (2)


 
(1) appelle le constructeur Vecteur( char* ) directement.
(2) passe par le constructeur de copie comme expliqué ci dessus.
 

Citation :

Edit : Une petite précision s'il te plaît, qu'entends-tu par conversion implicite ?


le cas (2). Si tu avais déclaré ton constructeur comme ceci : explicit Vecteur( char* );
La création du temporaire Vecteur("{1,2,3}" ) ne se serait pas faite automatiquement à partir de "{1,2,3}".
Si tu fournis un constructeur pouvant prendre un paramètre, et non précédé du spécificateur explicit, alors tu autorises une conversion implicite du type du paramètre vers le type Vecteur.

Reply

Marsh Posté le 10-10-2005 à 00:15:40    

Ok, merci beaucoup. :jap: Ca tombe bien, j'ai TP demain, cette aprem'. Je vais pouvoir expérimenter tout ça. :)

Message cité 1 fois
Reply

Marsh Posté le 10-10-2005 à 00:30:45    

Master_Jul a écrit :

Ok, merci beaucoup. :jap: Ca tombe bien, j'ai TP demain, cette aprem'. Je vais pouvoir expérimenter tout ça. :)


 
Alors tu vas probablement te rendre compte que ça ne fonctionne apparemment pas comme je te l'ai dis. Et pourtant, j'ai quand même raison ... pourquoi ?

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed