Allocation dynamique dans une fonction

Bonjour,

Mon problème est "simple" : je déclare dans mon main un pointeur, et je souhaite que ce tableau soit initialisé (allocation + remplissage) dans une fonction.

Ci-dessous le code fonctionnel. J'entend par fonctionnel que j'obtiens bien le "ABC" en sortie, mais gcc me sort un warning quand je le compile avec le flag -Wall...n'étant pas un pro des pointeurs et n'aimant pas les warning, j'en appel à votre aide =)
Code:

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 #include <stdio.h> #include <stdlib.h> int ma_fonction(char *p) { *p = (char *) calloc(3, sizeof(char)); if( p == NULL ) return 0; p[0] = 'A'; p[1] = 'B'; p[2] = 'C'; return 3; } int main(void) { char *p; int i,j; i = ma_fonction(p); for(j=0; j<i; j++) printf("%c",p[j]); printf("\n"); return 1; }
L'erreur (plutot l'avertissement!) est : "warning: assignment makes integer from pointer without a cast".

Quelqu'un aurait-il une idée du pourquoi du comment ?

Merci,
n0mad

L'erreur est ici :
Code:

1 2 3 int ma_fonction(char *p) { *p = (char *) calloc(3, sizeof(char));
Tu passes à ta fonction un pointeur sur caractère
et tu affectes à l'objet pointé par ce pointeur (*p est un caractère) un pointeur sur caractère. Cela ne plait pas au compilateur que tu mette un pointeur dans un char.

Pour corriger cela,, il faut passer l'adresse de ton pointeur de caractère à la fonction :
Code:

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 int ma_fonction(char **p) { *p = (char *) calloc(3, sizeof(char)); if( *p == NULL ) return 0; (*p)[0] = 'A'; (*p)[1] = 'B'; (*p)[2] = 'C'; return 3; } int main(void) { char *p; int i,j; i = ma_fonction(&p); for(j=0; j<i; j++) printf("%c",p[j]); printf("\n"); return 1; }
Je ne suis pas très content de la forme "(*p)[0] = 'A'", peut être que quelqu'un aura mieux ...

Merci bcp ram-0000 ! :-)

Puique ta fonction crée le tableau, elle doit renvoyer l'adresse du tableau créé. Le plus simple serait d'utiliser la valeur de retour
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 char * ma_fonction(int dim) { char *p = calloc(dim, sizeof(char)); if( p != NULL ) { .....// initialisation } return p; } // main char *p = ma_fonction(3); if(p!= NULL) { ....
Si tu veux utiliser la valeur de retour pour autre chose, l'adresse obtenue doit être renvoyée par la liste des paramètres. Dans ce cas, il faut que le paramètre spécifie où la valeur doit être écrite donc l'adresse de destination.
Code:

1 2 3 4 5 6 7 8 9 10 int ma_fonction(char **p) // type de l'argument :adresse d'un pointeur sur char { *p = calloc(3, sizeof(char)); if( *p == NULL ) .... // main char *p; int i; i = ma_fonction(&p); // Adresse de p en argument ....

Si tu veux que tous soit fait dans t'as fonction tu peux aussi essayer ceci.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
 
char    *my_function(int *nb)
{
  char  *p;
 
  *nb = 3;
  p = malloc(*nb);
  p[0] = 'A';
  p[1] = 'B';
  p[2] = 'C';
  return p;
}
 
int     main(void)
{
  char  *p;
  int   nb;
 
  p = my_function(&nb);
  write (1, p, nb);
  printf("\n%d\n", nb);
  return 0;
}

Citation:

Envoyé par ram-0000

Je ne suis pas très content de la forme "(*p)[0] = 'A'", peut être que quelqu'un aura mieux ...

Peut être :
Code:

1 2 3 4 5 char *tab; <...> tab = *p; tab[0] = 'A'; tab[1] = 'B'; tab[2] = 'C';