Discussion :

[aliwatte]

bonsoir tout le monde !
svp quelqu un pour m'expliquer pourquoi à la compilation le compilateur lance cette exception :

throwing an instance of 'std::bad_alloc'
what(): std::bad_alloc

j'ai compri que peut etre l'erreur été dans l'allocation de memoire j ai essayer quelque modification mais j'ai besoin d'explication d'un expert SVP .
voici le code :

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
#include <windows.h>
class texte{
    int taille;
    char *t;
public :
    texte(void);         // constructeur par defaut
    texte(const texte  & source);   // constructeur de copie
    ~texte(void);      // destructeur
};
 
texte::texte(void){ }
 
texte::texte(const texte &source){
    int taille = source.taille;
        t = new char  [taille + 1];
        strcpy(t, source.t);
}
 
texte::~texte(void) {delete t;}
 
int main(void){
    texte t1;
    texte t2(t1) ;
    return 0;
}

[Montag]

Salut,

Ton constructeur par défaut n'initialise aucune donnée membre. Donc taille vaut n'importe quoi. Quand tu copies t1 et que tu fais new char[taille+1] la valeur de taille+1 peut être énorme ce qui explique le lancement de l'exception.

Une autre remarque: dans le destructeur c'est delete[] qu'il faut utiliser et pas delete.

Sinon comme tu fais du C++ (et non du C) tu devrais utiliser std::string ça t'éviterait de faire ce genre d'erreur.

[koala01]

Salut,

c'est parce que tu ne définis pas la taille dans le constructeur par défaut...

Le résultat, c'est que la valeur du membre taille de l'instance t1 est... strictement n'importe quoi (les "crasses" d'une utilisation ultérieure de la mémoire)...

Cela signifie que ce peut tout aussi bien être égal à 0x00003 qu'à 0xFF58AB (voire à 0xFFFFFFFF)

Quoi qu'il en soit, la valeur de taille est supérieure à l'espace mémoire contigu qu'il est possible de trouver sur le système, ce qui fait que l'allocation de la mémoire échoue, et que new renvoie l'exception bad_alloc

[Edit] nous ne le répéterons jamais assez, lorsqu'il s'agit de gérer des chaines de caractères, le mieux reste toujours d'utiliser la classe string, fournie d'office par le standard, et disponible dans l'espace de noms std par inclusion du fichier d'en-tête <string>
[EDIT 2]grilled

[aliwatte]

Merci à tous pour l'aide que vous avez donner .
en effet je devais definir le constructeur . parce que la regle dit : s'il faut definir un constructeur de copie ,il faut definir avec l'operateur d'affectaion et le constructeur ,

Code :

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

1
2
3
4
texte::texte(){
    taille=0;
    return ;
 }

[3DArchi]

Bonjour,
Dans l'ordre:
1/ Tu dois aussi initialiser ton pointeur (t=NULL)
2/ Le return ne sert à rien.
3/ Tu devrais utiliser std::string comme mentionné précédemment.

[Médinoc]

Attends, j'ai du mal à comprendre ce thread.
C'est lors de la compilation que le compilateur lui-même lance une exception et se termine sur une erreur, ou bien c'est à l'exécution de ton programme que ton programme lance une exception et se termine en erreur ?

[aliwatte]

Attends, j'ai du mal à comprendre ce thread.
C'est lors de la compilation que le compilateur lui-même lance une exception et se termine sur une erreur, ou bien c'est à l'exécution de ton programme que ton programme lance une exception et se termine en erreur ?

c'etait a l'exécution du programme