Salut,
Je débute en c++, et j'ai un petit problème avec le destructeur d'une classe.Voici le constructeur :  
 

int x, int y, char *nom sont des membres privés de ma classe.
Voici le destructeur :

Je suis sous visual C++. La compilation s'effectue sans problème. A l'exécution, le pgm plante lorsqu'il passe sur la ligne "delete []nom". Si qqun pouvait m'aider, merci.

Cherche même pas à comprendre, utilise std::string.

Avec std::string, ça tourne, je l'avais déjà essayé. Mais, je ne comprends pas pq l'autre manière ne fonctionne pas. Merci qd même.

C'est simple : les char *, c'est mal. Tu ne gagnes que ce genre de problèmes avec.

essaye
 
if(nom)
  delete []nom;
 
et deja ya des truc louches :
 
 
nom = new char [strlen(nom1 + 1)];
 
c'est pas plutot ?
 
nom = new char [strlen(nom1)+1];

Je n'avais pas vu l'erreur du strlen.  
Aves le test, ça ne fonctionne pas non plus.

char *nom=""
 
vire ça aussi

Bon, donne le code complet de ta classe, sinon on y arrivera pas. Est-ce qu'elle est dérivée ? Est-ce qu'il y a d'autres constructeur, ou as-tu redéfinis l'opérateur d'affectation ?
Comment te sers tu des objets ?
 

Je suis passé sur dev-cpp, et là ça fonctionne.  
Merci pour l'aide

si tu fais un vilain mix, tu le fais à peu près comme ça:
 

 
sinon tu vires ton "char* nom" de ta classe, et tu mets un std::string.
ça sera 10x plus propre.

ok merci bjone. Pour le std::string, je l'avais déjà essayé, avec succès, mais au cours, on nous demande parfois d'utiliser le char *.

j'en profites pour poser une question stupide (comme d'habitude...)
 
les std::string, c'est gourmand en place mémoire?  
 
(je demande parce que je ne les ai jamais utilisé, et si c'est pas gourmand, ça voudrait p'te le coup que je m'y mettes)

ouais, ben tu retournes en cours apprendre la forme de Coplien

Pas besoin de faire ça, "delete 0;" et "delete [] 0;" sont valides tous les deux.  
 
Par contre, certains (mauvais) compilos ne l'aiment pas, certes.

J'en profite pour faire une citation
 
"un programme qui plante est d'une utilité quasi-nulle"

"un programme dépassant les 512Ko ne rentre pas dans la PROM"
 
d'ou ma question (oui je sais, j'ai qu'à bosser sur un PC )

tu fait ton prototype avec un maximum d'objets déjà existants, après tu regardes comment optimiser tes ressources si tu es à cours...

ben mets-y toi

probleme: mon soft existe deja depuis un certain, je n'ai que 10ko de libre, donc avant de me taper les 450000 lignes à modifier, je me renseigne c'est tout

la question est:
 
tes chaines là, si tu est en rom, t'as besoin de les dupliquer ?

ben techniquement une classe ça te permet aussi de centraliser le code de tes traitement de strings alors, ça peut réduire ton .text

où tu peux avoir deux cas, un où l'on a une chaine en ROM et une autre en RAM ?

alors je gére des chaines en RAM zero-power, en RAM, plus quelques constantes compressées (enfin je crois) en ROM.
 
elles font generalement moins de 32 caracteres, et les opérations sont ultra-basiques: strcpy, strncpy et strcmp.
 
ça répond à vos questions ou pas?

je suis d'accord, mais si pour ça faut que j'embarque une lib de 10Ko, ça perd tres vite de son interet pour moi

ben ça pèse bien moins que ça. ça te permet de factoriser ton code et d'implémenter des mécanismes comme du COW

oki, j'essaierai à l'occas voir si ça passe ou pas.

Ca dépend des implémentations. Certaines utilisent le Copy On Write, d'autres ne font de l'allocation dynnamique que pour une longueur > 15, et toutes stockent la longueur de la chaine ce qui permet d'économiser les appels à strlen.
http://c.developpez.com/faq/cpp/?p [...] GS_lenteur

merci pour le lien
 
mais bon, le probleme est resolu, mon ancetre de compilo n'accepte pas l'include de STRING, il me fait un caca nerveux sur streambuf (probablement parce qu'il y a 9 caractères...)