Discussion :

[Merillym]

Bonjour,

Suite à cette discussion, http://www.developpez.net/forums/d85...-regdeletekey/ , je dois allouer dynamiquement un tableau de type char à deux dimensions pour y stocker X chaînes de caractères.

J'ai donc créé une fonction pour le faire :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
char **AllocateMemoryForBuffer(int lignes, int colonnes)
{
  char **TabRes2d;
  int i;
 
  TabRes2d = malloc((lignes+1)*(sizeof(char)));
  for (i = 0; i < colonnes+1; i++)
    TabRes2d[i] = malloc((colonnes+1)*(sizeof(char)));
 
  printf("\nlignes = %d\n", lignes+1);
  printf("colonnes = %d\n\n", colonnes+1);
 
  return (TabRes2d);
}

Les 'lignes+1' et 'colonnes+1' répondent au besoin de mon programme : je dois augmenter de 1 'lignes' et de 1 'colonnes'.

Déjà, j'aimerais savoir si mon allocation de tableau est correcte ou pas.

Le problème survient lors du remplissage du tableau : j'arrive à remplir les 5, 6 premières lignes, puis ça plante sur la septième. L'errorlevel de mon programme est -1073741819.

Le tableau 2D type char est défini dans une structure. J'alloue la mémoire comme ceci :

ptr_registre->TabOfSubkeysToDelete = AllocateMemoryForBuffer(ptr_registre->NbrSuKeys, ptr_registre->MaxSubKeySize);

La structure :

Code :

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

1
2
3
4
5
6
7
8
9
struct      s_registre
{
    ..............
    char **TabOfSubkeysToDelete;
};
 
struct      s_registre t_registre;
struct      s_registre *ptr_registre;
ptr_registre = &t_registre;

J'envoie un pointeur vers cette structure à la fonction chargée de remplir ce tableau et voici le code de remplissage ci-dessous :

Code :

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

1
2
3
4
5
6
    while (a < strlen(path_key))
      {
        ptr_registre->TabOfSubkeysToDelete[count][a] = path_key[a];
        a++;
      }
    ptr_registre->TabOfSubkeysToDelete[count][a] = '\0';

Et count est incrémenté de 1 à chaque fois, dès qu'on lit une autre clé.

Le remplissage du tableau commence bien mais plante à ptr_registre->TabOfSubkeysToDelete[6]. Si je n'incrémente pas count, c'est la première ligne qui est remplie à chaque fois, et là le programme plante lors du douzième remplissage, même erreur.

Ca fait bien 3h que je suis sur le problème, j'ai beau repartir à 0 niveau code, je retombe sur le même erreur : j'en déduis que je me suis bien planté quelque part.

Merci par avance de votre aide pour m'aider à trouver.

[Mac LAK]

Plutôt ça, en fait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
char **AllocateMemoryForBuffer(int lignes, int colonnes)
{
  char **TabRes2d;
  int i;
 
  TabRes2d = malloc((lignes+1)*(sizeof(char*)));
  for (i = 0; i < colonnes+1; i++)
    TabRes2d[i] = malloc((colonnes+1)*(sizeof(char)));
 
  printf("\nlignes = %d\n", lignes+1);
  printf("colonnes = %d\n\n", colonnes+1);
 
  return (TabRes2d);
}

Sinon, tu n'alloue pas un vecteur de char*, mais un vecteur de char... Qui est quatre fois plus petit en taille sur ta machine.

[Merillym]

Re Mak Lak,

Effectivement ça marche mieux.

Maintenant ça bloque lors du remplissage de la dernière ligne du tableau. Même code d'erreur.

Mais déjà ça ne bloque plus pendant, c'est déjà ça.

Si tu as une idée sur le pourquoi du plantage lors du remplissage de la dernière ligne, je suis preneur.

J'y suis presque

[Mac LAK]

Je viens de voir une erreur dans la fonction d'allocation, par contre :

Code :

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

  for (i = 0; i < colonnes+1; i++)

est invalide, il faut mettre

Code :

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

  for (i = 0; i < lignes+1; i++)

à la place.

Si tu as une idée sur le pourquoi du plantage lors du remplissage de la dernière ligne, je suis preneur.

Si le problème persiste, c'est sûrement un problème d'indices base zéro / base 1, c'est le cas le plus fréquent quand ça plante sur le dernier élément. Essaie de tracer les valeurs prises par ta variable count, et compares les à la valeur initiale passée pour lignes à ta fonction d'allocation.

Au fait, tu peux remplacer ton code de copie de chaîne suivant :

Code :

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

1
2
3
4
5
6
    while (a < strlen(path_key))
      {
        ptr_registre->TabOfSubkeysToDelete[count][a] = path_key[a];
        a++;
      }
    ptr_registre->TabOfSubkeysToDelete[count][a] = '\0';

par celui-ci :

Code :

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

    strcpy(ptr_registre->TabOfSubkeysToDelete[count],path_key);

Ce sera toujours ça de pris.

[Merillym]

Re,

Bon mystérieusement, plus de plantage du programme mais la dernière ligne du tableau n'est jamais remplie ( quand je la lis, il m'affiche n'importe quoi ), faut que je trouve pourquoi.

Si le problème persiste, c'est sûrement un problème d'indices base zéro / base 1, c'est le cas le plus fréquent quand ça plante sur le dernier élément. Essaie de tracer les valeurs prises par ta variable count, et compares les à la valeur initiale passée pour lignes à ta fonction d'allocation.

J'y ai pensé, je vais revérifier, ça me semblait bon, mais bon en même temps, ça fait tellement de fois que je repars de 0 que ça ne m'étonnerait pas que je fasse systématiquement les mêmes erreurs

Merci d'avoir repéré les deux erreurs dans mon allocation, j'aurai pas trouvé avant des heures je pense. Je vais relire un peu de théorie sur ce genre d'allocation.

Ce sera toujours ça de pris.

Vu et modifié

Je te tiens au courant quand j'ai trouvé ce dernier bug.

[Merillym]

Au fait, tu peux remplacer ton code de copie de chaîne suivant :

Code :

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

1
2
3
4
5
6
    while (a < strlen(path_key))
      {
        ptr_registre->TabOfSubkeysToDelete[count][a] = path_key[a];
        a++;
      }
    ptr_registre->TabOfSubkeysToDelete[count][a] = '\0';

par celui-ci :

Code :

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

1
2
 
    strcpy(ptr_registre->TabOfSubkeysToDelete[count],path_key);

Ce sera toujours ça de pris.

Je viens de le faire et... ma dernière ligne se remplie bien...

Le problème venait donc de ma boucle de copie :

Code :

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

1
2
3
4
5
6
while (a < strlen(path_key))
      {
        ptr_registre->TabOfSubkeysToDelete[count][a] = path_key[a];
        a++;
      }
    ptr_registre->TabOfSubkeysToDelete[count][a] = '\0';

Je suis curieux de voir où est l'erreur, je vais chercher, théoriquement, il n'y a pas de raison que ce genre de boucle ne marche pas. Je vais vérifier les indices de la boucle.

edit: Trouvé => il fallait réinitialiser a à 0 chaque fois... car c'est une fonction récursive. Et je viens de voir que j'avais juste déclaré a, mais pas initialisé à 0... donc a pouvait valoir n'importe quoi...

Problème résolu.

Un grand merci à toi Mak Lak !!!