delete ne marche pas ...aidez-moi :(

Bonjour a tous,
Voici mon code :
----
 for(int i= 1; i!= ArraySize+1 ;++i)
   delete []   ArrayPointer[i];
-----
ArrayPointer est de type char**. (un tableau a 2 dimension).
et donc, ArrayPointer[i] est un char*.
Tout est bien initialisé, le system a donné de la mémoire avec NEW, je peut meme lire les caractère que j'entre,
mais quand delete arrive...erreur.
Pourtant, avec la meme syntaxe, je peut lire les donné !!! (remplace delete par cout).
aidez-moi :(!!!

Pepsidrinker

C'est normal ça?

for(int i= 1
Tu commence à 1 donc ArreyPointer[0] n'est pas supprimé.

i!= ArraySize+1
Si il y a ArraySize pointeurs, tu essais de libérer la mémoire de 2 pointeurs inexistant car en dehors des bornes.

Essais:
for(int i = 0; i != ArraySize-1 ;++i)
   delete []   ArrayPointer[i];

C++ (@++)

oui je lavait deja essayer mais...meme probleme.
Voila exactement l'erreur ke j'ai:

         /* verify block type */
        _ASSERTE(_BLOCK_TYPE_IS_VALID(pHead->nBlockUse));
        Cette instruction est dans un autre fichier, qui contien l'opérateur delete.
       En fait, si le block métais pas valide, je ne serai aps capable de le lire avec cout....mais la je suis bel et bien capable.
     

Pepsidrinker

J'obtien le même type d'erreur quand j'essais de libérer 2 fois la mémoire pointé par un même pointeur:

char *test = new char[5];
delete[] test; delete[] test; // CRASH

 Si 2 ArrayPointer pointe au même endroit, c'est le crash assuré.

C++ (@++)

non ,le seul delete ds mon programme est dans cette instruction, et en plus, si je peut le lire avec cout, c'est qu'il n'est pas deleté...

Lerreur arrive au premier appelle de delete.

Pepsidrinker

Salut, tu peux mettre ton code à l'endroit ou tu fais les new stp

---

TuRn3r

Salut,
Moins d'erreurs avec chaine fixe :

#defineFIXED_STRING32

char (*pszArray)[FIXED_STRING];

pszArray = new char[10][FIXED_STRING];

delete[] pszArray; // UN SEUL DELETE !

Salut !

for(int i= 1; i!= ArraySize+1 ;++i)
   delete []   ArrayPointer[i];

=> Apparemment tu aimes plus VB que C/C++...

Commencer ta boucle par 1 au lieu de 0 : c'est bien du VB !!!!

Pour  comprendre ou est ton problème, il faut aussi que tu postes le code que fait les allocations....

Postes donc le code d'allocation, et on te dépatouillera tout ca ....

Aucun probleme.

Voici donc la fonction qui déclare et alloue de la mémoire (NEW)

----------------------------

ERROR CDatabase::CreateArray()
{
 ArrayPointer = new char*[ArraySize+1];
 if(ArrayPointer == NULL)
  return NOT_ENOUGH_MEMORY;
 for(int i= 1; i!= ArraySize+1; ++i)
 {
  ArrayPointer[i] = NULL;
  ArrayPointer[i] = new char[MAX_STRING_LENGHT];  //= 20 caractère
  ArrayPointer[i] = "NULL_CHARACTER"; //La valeur par defaut de la chaine...
 }
  ArrayCreated = true;
  return NO_ERROR

}

Pepsidrinker

Pepsidrinker,

Hum.... c'est bien ce que je disais... le C/C++, c'est pas comme VB !

ton code présente 3 soucis majeurs :

* le loop sur le tableau à partit de 1 : ArrayPointer[0] n'est pa initialisé et donc contient une valeur indeterminée et un acces à ArrayPointer[0] est un crash assuré.

 *   Pourquoi l'expression "for(int i= 1; i!= ArraySize+1; ++i) ". Depuis des décennies, cette expression est généralement écrite :
    for(int i= 0 ; i < ArraySize ; i ++ )...
    ...ton code est moins lisible, des opérations sont inutiles ( i < ArraySize génère moins de code machine que i!= ArraySize+1 )

* Mais la cerise sur le gateau et par ailleur l'origine du bug, c'est :

    ArrayPointer[i] = new char[MAX_STRING_LENGHT]; 
     -> tu alloue de la mémoire et du fait pointer ArrayPointer[i] dessus
    ArrayPointer[i] = "NULL_CHARACTER"
    ->t u fais maintenant pointer ArrayPointer[i] vers une constante dont la mémoire est automatiquement allouée par le compilo;
   delete []   ArrayPointer[i];
    -> tu cherche donc à désalouer "NULL_CHARACTER" , constante allouée par le compilo... C'est comme essayer de piquer la copine de Dark Vador...

remplace ArrayPointer[i] = "NULL_CHARACTER" par  :
        * ArrayPointer[0] = 0 ou
        * strcpy(ArrayPointer[i] , "NULL_CHARACTER") ou
        * strnpcy(ArrayPointer[i], "NULL_CHARACTER",MAX_STRING_LENGHT)


Un conseil, revois les bases des pointeurs en C/C++ !

Bonne soirée.... J'espère que cela a pu t'aider !

Bon courage...