Discussion :

[EvaristeGaloisBis]

Bonjour a tous,

Je n'arrive pas a comprendre la raison qui fait que le compilateur plante lorsque je cherche a désallouer la mémoire.

J'ai voulu créer un tableau d'objet (volontairement sans utiliser la classe vector), pour cela je crée un pointeur sur ma classe et je la dimensionne sans probleme, mais au moment de la suprrimer le delete plante sans que j'en saisisse la raison...

Voici le code:

Code Point.h :

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

1
2
3
4
5
6
7
8
Class Point
{
private :
    long x,y;
public:
    Point(void);
    ~Point(void);
}

Code main.cpp :

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
#include "Point.h"
#include <string>
 
using namespace std;
 
int main(void)
{
Point * pMap;
long lSize;
string stErrMsg("");
 
    if (!GetMapSize(lSize,stErrMsg))
        return PromptErrMsg(stErrMsg);
 
    pMap=new Point[lSize];
    if (pMap==NULL)
        return 0;
 
 
/* tout un code dummy pour initialiser le tableau de points
 et faire ce que je veux avec */
 
    delete[] pMap;
 
    return 1;
}

Je ne comprends pas pourquoi mon code est rejeté. =(
En débuguant en pas a pas, j'ai bien que pMap est différent de NULL avant le delete. Il me semble être cohérent avec l'idée d'allocation et désallocation et surtout être syntaxiquement correct.
Juste pour avancer un peu j'ai taché de remplacer delete[] par delete et ça plante de la même manière.

Est ce que quelqu'un aurait une idée de l'origine de mon erreur? =(

Merci pour vos réponses (ou du moins vos lectures)

E.G

[Joker-eph]

Faudrait voir ce que fait le code "dummy", il doit y avoir quelque chose de pas net :-)

Si tu enlève le code "dummy" il y a toujours l'erreur ?

[Neckara]

Bonjour,

Code :

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

1
2
3
pMap=new Point[lSize];
    if (pMap==NULL)
        return 0;

Le if est inutile, en cas d’échec de l'allocation, new lance une exception.
Sauf compilateur très anciens mais dans ce cas là autant changer de compilateur.

Si tu veux vraiment que new retourne NULL, utilise :

Code :

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

pMap = new (std::nothrow) Point[Size]

Cf cppreference

[EvaristeGaloisBis]

Salut,
Merci pour vos réponses.

Alors Joker Eph, lors de mes investigations j'avais commenté tout le bloc dit Dummy. et rien n'y faisait. =(
Et encore plus étrange, ce matin je réessaie sans rien changer a mon code et ça passe sans erreur. Le pointeur est bien desalloué etc... je ne comprends pas vraiment ce qui a pu arriver. Et surtout ne laisse présager rien de bon sur mon code :-/
Je continue a chercher l'origine du problème...

Merci NeckAra, je gèrerai maintenant l'erreur d'allocation avec un try{} catch(badalloc){} .

[Bousk]

Plusieurs remarques sur le code:
- il est inutile d'initialiser une std::string avec une chaîne vide, le constructeur par défaut le fera
- à l'inverse, il peut être intéressant d'initialiser un pointeur avec NULL (ou nullptr), un delete sur null n'aura aucun effet
- la déclaration des variables ne devrait être faite que lors de son utilisation, il n'est pas nécessaire de les déclarer toutes en début de bloc (ceci est une ancienne façon de faire en C)