IdentifiantMot de passe
Loading...
Mot de passe oublié ?Je m'inscris ! (gratuit)
Navigation

Inscrivez-vous gratuitement
pour pouvoir participer, suivre les réponses en temps réel, voter pour les messages, poser vos propres questions et recevoir la newsletter

C++ Discussion :

surcharge d'opérateur ne fonctionne pas


Sujet :

C++

  1. #1
    Membre éclairé

    Homme Profil pro
    développeur à la maison
    Inscrit en
    Septembre 2006
    Messages
    396
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Tarn et Garonne (Midi Pyrénées)

    Informations professionnelles :
    Activité : développeur à la maison

    Informations forums :
    Inscription : Septembre 2006
    Messages : 396
    Billets dans le blog
    16
    Par défaut surcharge d'opérateur ne fonctionne pas
    bonjour,

    je ne comprends pas ce qui s passe,
    ce code me dit

    Code : Sélectionner tout - Visualiser dans une fenêtre à part
    1
    2
    3
    4
    5
    6
    7
    8
    entrez le therme
    123
    et le deuxième?
    5
    la somme est:
    terminate called after throwing an instance of 'std::bad_alloc'
      what():  std::bad_alloc
    Abandon
    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
    nbVec operator+(nbVec a,nbVec b){
    	nbVec copie(a);
    	copie+=b;
    	return copie;
    }
     
    int main(){
    	nbVec exp,exp1,exp2;
    	cout<<"entrez le therme"<<endl;
    	exp1.saisie();
    	cout<<"et le deuxième?"<<endl;
    	exp2.saisie();
    	cout<<"la somme est:"<<endl;
    	exp=exp1+exp2;
    	exp.affiche();
    	cout<<endl;
    	return 0;
    }
     
    class nbVec{
    	private:
    		std::vector<int> nb;
    	public:
    		void saisie();
    		void affiche()const;
    		char getChifCar(int i);
    		int getChifNb(int i);
    		int getLong();
    		nbVec operator +=(nbVec operande);
    };
     
    void nbVec::saisie(){
    	string nbString;
    	int i;
    	nbString.clear();
    	nb.clear();
    	cin>>nbString;
    	for(i=nbString.size();i>0;i--)//nb[0] est le chiffre des unités
    		nb.push_back(nbString[i-1]-48);
    }
     
    void nbVec::affiche()const{
    	int i;
    	string nbString;
    	nbString.clear();
    	vector<int>vecTemp;
    	for(i=nb.size();i>0;i--)
    		nbString.push_back(nb[i-1]+48);
    	cout<<nbString;
    }
     
    nbVec nbVec::operator +=(nbVec operande){
    	int taille,schif,diff,i,retenue(0),rang;
    	nb.clear();
    	diff=nb.size()-operande.nb.size();
    	if(diff>0)
    		for(i=0;i<diff;i++)
    			operande.nb.push_back(0);
    	else if(diff!=0)
    		for(i=0;i<diff;i++)
    			nb.push_back(0);
    	taille=nb.size();
    	for(rang=0;rang<=taille;i++){
    		schif=nb[rang]+operande.nb[rang]+retenue;
    		if(schif<10){
    			nb.push_back(schif);
    			retenue=0;
    		}
    		else{
    			nb.push_back(schif-10);
    			retenue=1;
    		}
    	}
    	if(retenue==1)
    		nb.push_back(1);
    	return *this;
    }
    quelqu'un a une idée?

  2. #2
    Membre éclairé

    Homme Profil pro
    développeur à la maison
    Inscrit en
    Septembre 2006
    Messages
    396
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Tarn et Garonne (Midi Pyrénées)

    Informations professionnelles :
    Activité : développeur à la maison

    Informations forums :
    Inscription : Septembre 2006
    Messages : 396
    Billets dans le blog
    16
    Par défaut
    j'ai vu une erreur dans mon code: ne pas nb.clear()
    puis aussi supprimer les zéros qui commencent nb.

    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
    79
    80
    81
    82
    83
    84
    85
    86
    87
    88
    89
    90
    91
    92
    93
    94
    95
    96
    97
    98
    99
    100
    101
    102
    103
    104
    105
    106
    107
    108
    109
    110
    111
    112
    113
    114
    115
    116
    117
    118
    119
    120
    121
    122
    123
    124
    125
    126
    127
    128
    129
    130
    131
    132
    133
    134
    135
    136
    137
    138
    139
    140
    141
    142
    143
    144
    145
    /*
     * objets.hpp
     *
     *  Created on: 9 juin 2015
     *      Author: matser
     */
     
    #ifndef OBJETS_HPP_
    #define OBJETS_HPP_
    #include <vector>
     
    class nbVec{
    	private:
    		std::vector<int> nb;
    	public:
    		void saisie();
    		void affiche()const;
    		char getChifCar(int i);
    		int getChifNb(int i);
    		int getLong();
    		void normalise();
    		nbVec operator +=(nbVec operande);
    };
     
     
     
    #endif /* OBJETS_HPP_ */
    /*
     * objets.cpp
     *
     *  Created on: 9 juin 2015
     *      Author: matser
     */
    #include <vector>
    #include <string>
    #include <iostream>
    using namespace std;
    #include "objets.hpp"
     
     
    void nbVec::saisie(){
    	string nbString;
    	int i;
    	nbString.clear();
    	nb.clear();
    	cin>>nbString;
    	for(i=nbString.size();i>0;i--)//nb[0] est le chiffre des unités
    		nb.push_back(nbString[i-1]-48);
    }
     
    void nbVec::affiche()const{
    	int i;
    	string nbString;
    	nbString.clear();
    	vector<int>vecTemp;
    	for(i=nb.size();i>0;i--)
    		nbString.push_back(nb[i-1]+48);
    	cout<<nbString;
    }
     
    nbVec nbVec::operator +=(nbVec operande){
    	int taille,schif,diff,i,retenue(0),rang;
    	//nb.clear();
    	this->normalise();
    	operande.normalise();
    	diff=nb.size()-operande.nb.size();
    	if(diff>0)
    		for(i=0;i<diff;i++)
    			operande.nb.push_back(0);
    	else if(diff!=0)
    		for(i=0;i<diff;i++)
    			nb.push_back(0);
    	taille=nb.size();
    	for(rang=0;rang<=taille;i++){
    		schif=nb[rang]+operande.nb[rang]+retenue;
    		if(schif<10){
    			nb.push_back(schif);
    			retenue=0;
    		}
    		else{
    			nb.push_back(schif-10);
    			retenue=1;
    		}
    	}
    	if(retenue==1)
    		nb.push_back(1);
    	return *this;
    }
     
    void nbVec::normalise(){
    	while(nb[nb.size()-1]==0)
    		nb.pop_back();
    }
    /*
     * operations.cpp
     *
     *  Created on: 9 juin 2015
     *      Author: matser
     */
     
    #include "objets.hpp"
     
    nbVec operator+(nbVec a,nbVec b){
    	nbVec copie(a);
    	copie+=b;
    	return copie;
    }
     
     
    /*
     * experiencesLM.cpp
     *
     *  Created on: 9 juin 2015
     *      Author: matser
     */
     
    #include <iostream>
    using namespace std;
     
     
    #include "objets.hpp"
     
    nbVec operator+(nbVec a,nbVec b);
     
    int main(){
    	nbVec exp,exp1,exp2;
    	cout<<"entrez le therme"<<endl;
    	exp1.saisie();
    	cout<<"et le deuxième?"<<endl;
    	exp2.saisie();
    	cout<<"la somme est:"<<endl;
    	exp=exp1+exp2;
    	exp.affiche();
    	cout<<endl;
    	return 0;
    }
     
     
    entrez le therme
    123
    et le deuxième?
    5
    la somme est:
    terminate called after throwing an instance of 'std::bad_alloc'
      what():  std::bad_alloc
    quelqu'un a une idée?

  3. #3
    Membre éclairé

    Homme Profil pro
    développeur à la maison
    Inscrit en
    Septembre 2006
    Messages
    396
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Localisation : France, Tarn et Garonne (Midi Pyrénées)

    Informations professionnelles :
    Activité : développeur à la maison

    Informations forums :
    Inscription : Septembre 2006
    Messages : 396
    Billets dans le blog
    16
    Par défaut
    j'ai corrigé mon code et maintenant ça marche.
    par exemple, si on veux l'addition dont le résultat est dans nb3:

    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
    nbVec *nb3;
    	nb3=new nbVec;
    	nb3->nb.clear();
    	diff=nb.size()-operande.nb.size();
    	if(diff>0)
    		for(i=0;i<diff;i++)
    			operande.nb.push_back(0);
    	else if(diff!=0)
    		for(i=0;i<diff;i++)
    			nb.push_back(0);
    	taille=nb.size();
    	for(i=0;i<taille;i++){
    		schif=nb[i]+operande.nb[i]+retenue;
    		retenue=schif/10;;
    		nb3->nb.push_back(schif%10);
    	}
    	if(retenue==1)
    		nb3->nb.push_back(1);

  4. #4
    Expert éminent
    Avatar de Médinoc
    Homme Profil pro
    Développeur informatique
    Inscrit en
    Septembre 2005
    Messages
    27 395
    Détails du profil
    Informations personnelles :
    Sexe : Homme
    Âge : 41
    Localisation : France

    Informations professionnelles :
    Activité : Développeur informatique
    Secteur : High Tech - Éditeur de logiciels

    Informations forums :
    Inscription : Septembre 2005
    Messages : 27 395
    Par défaut
    Attention, operator+= devrait retourner une référence sur *this, pas une copie. Aussi, le paramètre pourrait probablement être une référence const, et certaines de tes méthodes (comme getLong()) devraient probablement être const également.
    SVP, pas de questions techniques par MP. Surtout si je ne vous ai jamais parlé avant.

    "Aw, come on, who would be so stupid as to insert a cast to make an error go away without actually fixing the error?"
    Apparently everyone.
    -- Raymond Chen.
    Traduction obligatoire: "Oh, voyons, qui serait assez stupide pour mettre un cast pour faire disparaitre un message d'erreur sans vraiment corriger l'erreur?" - Apparemment, tout le monde. -- Raymond Chen.

Discussions similaires

  1. [XL-2010] Les opérateurs de manipulation de chaines de caractères ne fonctionnent pas
    Par jpclabaux dans le forum Macros et VBA Excel
    Réponses: 9
    Dernier message: 31/07/2013, 19h22
  2. Réponses: 18
    Dernier message: 03/12/2010, 16h37
  3. constructeur surchargée qui ne fonctionne pas
    Par kate59 dans le forum XNA/Monogame
    Réponses: 1
    Dernier message: 13/06/2008, 22h30
  4. Un Hint sur un PopupMenu ne fonctionne pas !!??
    Par momox dans le forum C++Builder
    Réponses: 6
    Dernier message: 26/05/2003, 16h48
  5. ca ne fonctionne pas (generateur auto-incrémentant)
    Par tripper.dim dans le forum SQL
    Réponses: 7
    Dernier message: 26/11/2002, 00h10

Partager

Partager
  • Envoyer la discussion sur Viadeo
  • Envoyer la discussion sur Twitter
  • Envoyer la discussion sur Google
  • Envoyer la discussion sur Facebook
  • Envoyer la discussion sur Digg
  • Envoyer la discussion sur Delicious
  • Envoyer la discussion sur MySpace
  • Envoyer la discussion sur Yahoo