Discussion :

[awalter1]

Bonjour à tous,

Cela fait longtemps que je n'ai pas fait de C, et j'ai un doute dans le cas suivant : lorsque la procedure toto est terminée, son espace mémoire est désalouée et la valeur de tab peut être écrasée avant que l'appelant ne s'en serve ?
Ne faut t'il pas alors préciser "static char tab[10]" ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
char * toto() 
{
   char tab[10],
   strcpy(tab,"abcdef");
   return (&tab);
}
 
main() 
{
   printf("%s\n",toto());
}

Merci de votre éclairage

[untipy]

il me semble que par "static l'emplacement mémoire de la variable sera alloué lors de la compil et cela pour la durée total du programme
en fait dans cet exemple tu redeclare a chaque appel de la fonction un char

[Invité]

effectivement, retourner un pointeur sur une variable locale est tres maladroit.
static que etre une solution.

ce que tu propose est que l'apelant alloue le buffer et le passe en parametre de la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
void toto(char* buf, size_t tailleBuf) {
    strncpy(buf,"abcdef", tailleBuf);
}
 
void main(void) {
    char str[10] = {0};
    toto(str, 10)
    printf("%s\n", str);
}

[Gruik]

Salut,

effectivement, retourner un pointeur sur une variable locale est tres maladroit.
static que etre une solution.

ce que tu propose est que l'apelant alloue le buffer et le passe en parametre de la fonction :

Code :

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

1
2
3
4
5
6
7
8
9
 
void toto(char* buf, size_t tailleBuf) {
    strncpy(buf,"abcdef", tailleBuf);
}
 
main() {
    char str[10] = {0};
    printf("%s\n",toto(str, 10));
}

(attention je vais faire mon contre-iste)
Plusieurs trucs :
- toto ne retourne rien alors que tu utilises sa valeur de retour dans le printf (peut etre voulais tu retourner buf)
- il vaut mieux utiliser sizeof sur un tableau de caractere pour connaitre sa taille
- dans cet exemple, toto ne laisse pas la place pour un eventuel \0 final
- meme si dans cet exemple str est initialisé avec des \0, toto ne mets pas le \0 final (or d'apres la façon dont tu l'utilises, tu aurais escompté que ce qui est retourné est une chaine valide)
- le prototype de main est "int main(void)" et il retourne un entier indiquant l'exit status

[Invité]

@Gruik, tu as raison, c'etait un résidu de Copier-Coller du code de awalter1 qui trainait... c'est corrigé !

[Emmanuel Delahaye]

Cela fait longtemps que je n'ai pas fait de C, et j'ai un doute dans le cas suivant : lorsque la procedure toto est terminée, son espace mémoire est désalouée et la valeur de tab peut être écrasée avant que l'appelant ne s'en serve ?
Ne faut t'il pas alors préciser "static char tab[10]" ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
char * toto() 
{
   char tab[10],
   strcpy(tab,"abcdef");
   return (&tab);
}
 
main() 
{
   printf("%s\n",toto());
}

Déjà, c'est pas &tab, mais tab tout court. (Vu le type du retour, ce n'est pas l'adresse du tableau que tu retournes, mais l'adresse du premier élément).

Ensuite, effectivement, retourner l'adresse d'une variable locale n'a pas de sens, car elle est détruite après le return. Son usage entraine donc un comportement indéterminé.

La solution 'static' est techniquement possible, mais peu recommandée, car elle introduit une contrainte de fonctionnement qui rend cette fonction non ré-entrante.

Le bon usage est soit de passer l'adresse du premier élément d'un tableau (et éventuellement, son nombre d'éléments), soit d'allouer dynamiquement un tableau et d'en retourner l'adresse du premier élément.

[awalter1]

Merci à tous pour vos conseils