Discussion :

[n0mad]

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 :

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
 
#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

[ram-0000]

L'erreur est ici :

Code :

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

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 :

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
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 ...

[n0mad]

Merci bcp ram-0000 ! :-)

[diogene]

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 :

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

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 :

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

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
....

[flov91]

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

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
 
 
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;
}

[orfix]

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

Peut être :

Code :

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

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