Discussion :

[FidoDido®]

Salut,
Je ne vous mets pas tout le code car c'est carrément trop long, mais j'ai une erreur que je n'arrive pas à m'expliquer.
J'ai un ensemble de classes qui interagissent entre elles. Dans une de mes méthodes, je déclare une instance d'une de mes classes, et j'effectue des calculs grâce aux attributs de l'instance.
A la sortie de cette méthode, l'instance est donc supprimée par le destructeur de la classe qui ressemble à ceci :

Code :

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

1
2
3
4
5
6
 
Interpol::~Interpol() {
        cout << "entree" << endl;
       delete[] coeff;
       cout << "sortie" << endl;
}

sachant que le .hpp de la classe est le suivant :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
class Interpol{
public:
     Interpol();
     ~Interpol();
     void calcul(double * t, doublue * y, int n);
 
private:
     double * coeff;
     int nbpoints;
};

le truc c'est que j'ai bien le message "entree" qui apparait, mais pas le "sortie"

Je ne comprends donc pas comment je peux planter sur une instruction comme celle-ci...

Merci pour votre aide !

[Laurent Gomila]

Tu veux dire que c'est le delete qui plante (si tu le retires tu vois "sortie" ?) ? Comment est manipulé ton tableau coeff dans la classe ? Pourquoi tu n'utilises pas un std::vector<double> ?

[r0d]

Bonjour,

une fois de plus, si tu avais utilisé un conteneur stantard (un std::vector ici en l'occurence), tu n'aurais sans doute pas rencontré ce problème.

Car si ça plante sur ton delete, c'est que ton tableau n'est pas valide au moment du delete [].
Es-tu certain qu'il a été alloué?
N'a-t-il pas été désalloué ailleurs? (avant ce delte [] )

edit:grilled

[FidoDido®]

si je retire le delete[] coeff ça plante qd même...

Donc je suppose que j'ai un pb de mémoire avant cela, mais je ne comprends pas d'où il peut venir.

Je n'ai pas utilisé de std::Vector pcq je ne connaissais pas...

[Laurent Gomila]

D'ailleurs, pour enfoncer encore un peu le clou (et peut-être que c'est la cause du plantage), ta classe gère une ressource brute qui ne peut être copiée trivialement, or elle ne définit ni constructeur par copie ni opérateur d'affectation. Résultat : si tu copies une instance de ta classe, son membre coeff va pointeur sur un tableau géré par une autre instance, et le second qui va tenter de le détruire va crasher.

[Laurent Gomila]

Donc je suppose que j'ai un pb de mémoire avant cela, mais je ne comprends pas d'où il peut venir.

Tu n'as pas un débogueur sous la main ?

[FidoDido®]

nope

[r0d]

Arf, sans débuggeur, ça va être difficile de débugguer un bug tel que celui-ci.
Le plus simple serait que tu remplaces ton tableau par un vecteur (voir cette faq), comme ça tu ne devrais plus avoir de problème (ça résous même le problème du constructeur par copie).