Discussion :

[azez]

Bonjour,

je suis débutant et je veux comprendre comment utiliser les tableaux dynamiques 3D, j'ai fait un code qui construit un tableau 3D "A" de dimension axbxc ; a,b et c sont obtenus depuis un fichier taille.dat.

exemple de taille.dat :

Code :

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

2 3 4

mon code est le suivant :

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
    int  a,b,c,i,j,k;
    float ***A; // A est un tableau 3D de taille axbxc
    FILE *f_in;
 
    /*  Lecture des dimensions du tableau depuis un fichier "taille.dat" */
    f_in=fopen("taille.dat","r");
    if (f_in== NULL)
{
    printf("Impossible d'ouvrire le fichier taille.dat");
    exit(1);
}
fscanf(f_in, "%i %i %i\n",&a, &b, &c);
printf("a = %i; b = %i; c = %i\n",a,b,c); // affichage des valeurs de a, b ,c
    /* construction du tableau 3D A */
    A=malloc(sizeof*A*a);
 
     if ( A == NULL )
{
     printf("Allocation impossible \n");
     exit(1);
}
 
  for (i=0;i<a;i++)
  {
      A[i]=malloc(sizeof*A[i]*b);
      for (j=0;j<b;j++)
      {
          A[i][j]=malloc(sizeof*A[i][j]*c);
      }
  }
 
/* Initialisation de tous les éléments de A à 0 ensuite remplissage de A et affichage */
for (i=0;i<a;i++)
  {
      for (j=0;j<b;j++)
      {
          for (k=0;k<c;k++)
          {
 
               A[i][j][k]=0; // initialisation de tous les éléments de A
               A[i][j][k]=A[i][j][k]+0.5; // exemple d'operation sur les éléments de A
               printf("A[%i][%i][%i] = %f\n", i, j,k, A[i][j][k]); // affichage des élément du tableau
          }
 
      }
  }
free(A);
return 0;
}

le compilateur signale des erreurs :

Code :

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

1
2
3
4
 
line 20    error: invalid conversion from 'void*' to 'float***'
line 30    error: invalid conversion from 'void*' to 'float**' 
line 33    error: invalid conversion from 'void*' to 'float*'

merci de corriger mon code si c'est possible.

[Fendal]

Si je ne me trompe pas les problèmes correspondent aux lignes des mallocs.
Je dirais qu'il manque les parenthèses :
sizeof(A)*a
sizeof(A[i])*b
sizeof(A[i][j])*c

Je pense qu'il y a aussi un problème sur la dernière ligne car c'est la taille d'un pointeur vers un float qui est alloué et non la taille d'un float.
(A confirmer, personnellement je n'utilise jamais cette notation de sizeof avec les noms de variables)

[Thierry Chappuis]

Pour le message du compilateur, es-tu certain d'utiliser un compilateur C et non un compilateur C++. Si l'extension de ton fichier source est .C (majuscule), .cpp ou .cxx, il y a de grande chance qu'un compilateur C++ soit utilisé. Or, en C++, la convertion implicite d'un pointeur générique de type void * en n'importe quel autre type de pointeur n'est pas autorisée (alors qu'elle l'est en C). D'où les messages d'erreur...

En ce qui concerne le code proprement dit, je ne vois d'erreur flagrante. Je suis en train de tester...

Thierry

[azez]

grand merci à Fendal et mujigka,

oui c'est les lignes des mallocs la conversion de types ne se fait pas malgré que j'utilise l'EDI code::blocks v1.0 contenant un compilateur c ( l'extension de mon source est c miniscule)

j'ai corrigé mon code comme suit :

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
    int  a,b,c,i,j,k;
    float ***A; // A est un tableau 3D de taille axbxc
    FILE *f_in;
 
    /*  Lecture des dimensions du tableau depuis un fichier "taille.dat" */
    f_in=fopen("taille.dat","r");
    if (f_in== NULL)
{
    printf("Impossible d'ouvrire le fichier taille.dat");
    exit(1);
}
fscanf(f_in, "%i %i %i\n",&a, &b, &c);
printf("a = %i; b = %i; c = %i\n",a,b,c); // affichage des valeurs de a, b ,c
    /* construction du tableau 3D A */
    A = (float***)malloc(sizeof(float**) *a);
 
     if ( A == NULL )
{
     printf("Allocation impossible \n");
     exit(1);
}
 
  for (i=0;i<a;i++)
  {
      A[i] = (float**)malloc(sizeof(float*) *b);
      for (j=0;j<b;j++)
      {
          A[i][j] = (float*)malloc(sizeof(float)*c);
      }
  }
 
/* Initialisation de tous les éléments de A à 0 ensuite remplissage de A et affichage */
for (i=0;i<a;i++)
  {
      for (j=0;j<b;j++)
      {
          for (k=0;k<c;k++)
          {
 
               A[i][j][k]=0; // initialisation de tous les éléments de A
               A[i][j][k]=A[i][j][k]+0.5; // exemple d'operation sur les éléments de A
               printf("A[%i][%i][%i] = %f\n", i, j,k, A[i][j][k]); // affichage des élément du tableau
          }
 
      }
  }
free(A);
return 0;
}

et ça marche !!!

dernière question : est ce que je fais bien la libération de la mémoire ??

[stephl]

dernière question : est ce que je fais bien la libération de la mémoire ??

Non, free(A) ne suffit pas. A chaque malloc() doit correspondre un free(). Attention à l'ordre.

[azez]

Non, free(A) ne suffit pas. A chaque malloc() doit correspondre un free(). Attention à l'ordre.

alors je dois ajouter deux free() ?

c-à-d :

Code :

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

1
2
3
4
 
free(A[i][j]);     // est ce que ces deux free() doivent étre dans les  ...
free(A[i]);       // ...  boucles (i,j) où bien à la fin du code comme ça ?
free(A);

svp les deux free() contenant les indices i et j où dois je les mettre ?

merci.