Discussion :

[PRomu@ld]

Bonjour,

Vous pouvez postez ici vos avis, remarques, suggestions sur l'article suivant :

http://rperrot.developpez.com/articles/c/allocationC/

NB: Merci de lire ce message avant de poster.

[neow_]

Bonsoir,

Juste pour vous signaler une petite erreur dans un code de votre tutoriel :

Code :

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
int * tabentier;
/* Création d'un tableau de 3 entiers */
tabentier = calloc ( 3 , sizeof(int) );
 
tabentier[0] = 1;
tabentier[1] = 2;
tabentier[2] = 3;
 
/* Ajout d'un element au tableau */
tabentier = realloc (tabentier, 4 * sizeof(int) );
 
tabentier[3] = 4;
 
for ( i = 0 ; i < 3 ; i++ )
{
     printf(" tabentier[%d] = %d \n", i , tabentier[i] );
}

La condition de la boucle for est mauvaise : elle s’arrêtera au troisième élément du tableau, et n'affichera donc pas tabentier[3] = 4, comme attendu.

Merci.
Bonne continuation,
neow_

[minfo21]

Bonsoir,

la solution :

Code :

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
int * tabentier;
/* Création d'un tableau de 3 entiers */
tabentier = calloc ( 3 , sizeof(int) );
 
tabentier[0] = 1;
tabentier[1] = 2;
tabentier[2] = 3;
 
/* Ajout d'un element au tableau */
tabentier = realloc (tabentier, 4 * sizeof(int) );
 
tabentier[3] = 4;
 
for ( i = 0 ; i < 4 ; i++ )
{
     printf(" tabentier[%d] = %d \n", i , tabentier[i] );
}

maintenant la boucle s'amarche bien !

Bonne chance,
neow_

[Melem]

Je trouve le tutoriel excellent. Cependant, il y a un problème avec les macros MALLOC, CALLOC et REALLOC. Prenons ce code tiré du tutoriel par exemple :

Code :

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
int ** matrice , i;         /* un tableau à deux dimension */
 
/*allocation d'un tableau de trois tableaux d'entiers */
MALLOC(matrice,int*,3);    
 
for ( i = 0 ; i < 3 ; i ++ )
{
     /* allocation d'un tableau de tableau */
     CALLOC(matrice[i],int,3); 
}
 
/*remplissage d'une matrice diagonale*/
for ( i = 0 ; i < 3 ; i++ )
{
      matrice[i][i] = 1;
}
 
for(i = 0 ; i<3 ; i++)
{
     FREE(matrice[i]);
}
FREE(matrice);

Si CALLOC echoue disons alors qu'on en est à i = 2, matrice[2] va certes être libéré mais le programme se termine tout de suite : matrice[1], matrice[0] et matrice n'ont pas été libérés ! Il faut donc soit les retirer du tutoriel, soit les corriger.

[neow_]

minfo21 > Merci, mais j'avais déjà résolu le problème, c'était juste pour signaler l'erreur.

[Thierry Chappuis]

Je trouve ce didacticiel de très bonne qualité. Je reste toutefois sur ma fin par rapport aux macros proposées dans la section 4.2 qui cachent des exit() sauvages qui ne me plaisent pas. L'utilisation de tels points de sortie est certes souvent utilisé dans un contexte d'enseignement. Il y a toutefois certainement de meilleures façons de traiter ce type d'erreurs d'allocation dynamiques de mémoire dans la vraie vie. C'est à priori à mon avis une mauvaise pratique et je me demande s'il est opportun en tout cas de cacher ces points de sortie dans une macro. Typiquement, dans une fonction de création de matrice, ma stratégie sera, en cas d'erreur d'allocation, de faire le ménage (i.e. libérer tout ce qui a été allouer), puis de faire remonter l'erreur d'allocation, éventuellement jusqu'à main() qui sera la seule fonction autorisée à quitter l'application.

Avec mes meilleures salutations

Thierry

[leMédaillon]

Par medinoc

Si realloc() ne retourne ni NULL ni tabentier, ça veut dire:

1. Le pointeur retourné pointe sur le buffer voulu,
2. tabentier n'est plus un pointeur valide (realloc() a fait un free() dessus).

Je comprends beaucoup mieux. J'en conclue donc que PRomu@ld avait fait le bon choix et je rectifierais en conséquence cette erreur que je fais souvent dans mes codes avec realloc. Bonne soirée à vous !