Discussion :

[psgman113]

Bonjour,

j'utilise une fonction qui construit une chaine de caractères pointée par un pointeur et qui la retourne , je veux l'afficher ensuite dans le main mais là probleme!
Ya t'il un souci dans ma gestion de chaines?
Merci
Je mets un bout de code (pas besoin de compiler)

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
 
 
char  *codage_msg(char *message); // declaration de fonction
char *message_code=NULL; 
 
int main()
{
 
 message_code= codage_msg(message); 
  printf("%s\n",message_code); // ca n'affiche plus la chaine
 }
 
 
 
 
char * codage_msg(char *msg)
{
char *msg_code;
 
// je prends une chaine *msg en entrée 
et je construis une autre chaine *msg_code
 
  printf("%s\n",message_code); // ca affiche bien la chaine!!
 
return msg_code
 
}

[souviron34]

Je mets un bout de code (pas besoin de compiler)

vivi on avait bien compris

Ben tu l'alloues bien ta chaîne msg_code ??

[psgman113]

comment faire?

[thewho]

Bonjour,

comment faire?

As-tu seulement réfléchi une ou 2 secondes à ce que tu as écrit comme code ?

Sans blague, c'est un des trucs les plus étranges que j'aie vu jusqu'ici.

Et ça ne doit pas compiler, même en ajoutant les en-têtes nécessaires.

[fearyourself]

comment faire?

2 solutions pour allouer de la mémoire :

- Statique : tu déclares une variable dans ta fonction qui peut contenir de l'information. Exemple :

Code :

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

1
2
3
4
5
6
 
int i;
char c;
int t[16];
char s[32];
char *ptr;

Comme tu vois, i, c sont des types simples. En les déclarant, on a alloué de la place en mémoire pour contenir leur valeur.

Pour les tableaux, on alloue aussi de l'espace pour eux. Ainsi, la déclaration de t a alloué 16 cases d'entiers.

Par contre, ptr est un pointeur. En déclarant ptr, finalement nous avons alloué de l'espace pour un pointeur vers un caractère et non une chaîne de caractères comme ce qui est fait pour s.

Le souci de l'allocation statique est qu'on ne peut pas retourner un pointeur vers une zone statique car, à la fin de la fonction, cette zone mémoire est récupérée.

NB: Juste parce qu'on déclare une variable ne veut pas dire que le compilateur va vraiment allouer de la mémoire pour elle. Il faudra s'en servir pour que cela soit vraie (du moins dans un compilateur un tant soit peu récent).

- L'allocation dynamique : on passe par une fonction : malloc.

La zone mémoire retournée par malloc est à nous et on peut en faire ce qu'on veut. Elle n'est pas perdue lorsque la fonction se termine comme dans le cas de l'allocation statique, donc on peut retourner une adresse allouée dynamiquement. C'est là tout son avantage.

Par contre, il faut prendre des précautions :

- Toujours vérifier le retour de malloc
- Ne pas perdre le pointeur retourner par malloc car il faudra faire une désallocation avec la fonction free



Enfin, dans ton cas, vu que tu veux retourner une chaîne de caractères, tu dois l'allouer dynamiquement. La seule autre solution serait de passer en paramètre la zone allouée pour la chaîne. Ceci permettrait de passer une zone allouée statiquement.

Jc

[psgman113]

message_code est une variable globale alors qu'il n'y a aucune justification pour cela (à corriger absolument)

J'ai besoin d'utiliser des variables globales dans mon programme!

Je ne vois vraiment pas comment m'en débarasser.

J'ai plusieurs fonctions qui doivent manipuler une chaine de caractères commune,
Que me proposez vous?

[Thierry Chappuis]

J'ai besoin d'utiliser des variables globales dans mon programme!

Je ne vois vraiment pas comment m'en débarasser.

J'ai plusieurs fonctions qui doivent manipuler une chaine de caractères commune,
Que me proposez vous?

Les cas où il est absolument nécessaire d'utiliser une variable globale sont rares, du moins dans ma pratique de tous les jours. Si tu as plusieurs fonctions qui doivent manipuler des chaines de caractère, passe à chaque fois les chaînes en paramètre des fonctions en question, c'est fait pour ça.

Thierry

[Thierry Chappuis]

Salut,

Je vois au moins les problèmes suivants:

• message_code est une variable globale alors qu'il n'y a aucune justification pour cela (à corriger absolument)
• Plus grave encore, message_code n'est jamais initialisé
• message n'est ni défini, ni initialisé et est passé à la fonction codage_msg(), dont le paramètre formel msg n'est jamais utilisé.
• La fonction codage_msg renvoie msg_code qui n'est également jamais initialisé

A ceci, on peut ajouter que main doit retourner un entier. Par convention, on renvoie l'entier 0 si l'exécution n'a pas connu de problème. S'il n'y a pas d'instruction return à la fin de main, je crois que la valeur entière 0 est retournée par défaut, mais par soucis de cohérence, il vaut mieux terminer main par return 0 ou return EXIT_SUCCESS.

Pour terminer, la forme de main n'est pas une forme valide en C. Deux formes sont acceptées:

Code :

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

1
2
3
4
5
int main(void)
 
/* ou */
 
int main(int argc, char *argv[])

Le meilleur conseil que j'ai à te donner est de te replonger dans tes cours de C pour approfondir les chapitres sur les chaînes de caractères, les pointeurs, les tableaux, les fonctions, et l'allocation dynamique (beau programme).

Meilleures salutations

Thierry

[Médinoc]

Il y a un début d'explication ici sur la différence entre "char*" et chaîne de caractères, mais cet article-là ne parle pas encore d'allocation...
http://emmanuel-delahaye.developpez.....htm#char_star