Discussion :

[jmjmjm]

Comment retourner une chaîne de caractères dans une fonction ??

j'ai une fonction qui lit un message socket et doit me renvoyer le message ??
Désolé ca doit etre basic mais je suis habitué au c++ et a string qui n'existe malheureusement pas en C et je dois faire le projet en C :s

[hegros]

Code :

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

1
2
3
4
5
char * MaFonc() {
  char buf[TAILLE];
  <..>
  return buf;
}

[gege2061]

Bonjour,

un p'tit exemple : Copier une chaîne de caractères

Une chaîne de caractères est un tableau de char terminé par un caractère nul. Il faut utiliser l'allocation dynamique de mémoire pour éviter les problème de destruction de variables locales (comme nous le montre hegros )

[jmjmjm]

oué desolé pour ce post je viens de reparcourir la FAQ et j'ai vue que c'etait dedans (je l'avais pourtant regarder avant de poster :-( ca doit etre la fatigue !!)

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
char * MaFonc() {
  char buf[TAILLE];
  <..>
  return buf;
}

Horrible !

[jmjmjm]

Salut Emmanuel, j'ai fait ca en m'inspirant de la FAQ est ce bon ??

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
 
char *LectureMSQ(key_t Cle,long type)
{
   char *Retour=NULL;
   message_t message;
   int msqid,length;
   message.type=type;
   length = sizeof(message_t)-sizeof(long);
 
   if ((msqid = msgget(Cle, IPC_CREAT | 0600))==-1) {
      perror("msgget");
      exit(EXIT_FAILURE);
   }
 
   if (msgrcv (msqid, &message, length, message.type, 0)>=0) {
      Retour=malloc((strlen(message.Nom)+1)*sizeof(Retour));
      if (Retour) strcpy(Retour,message.Nom);
   }
   else
      perror("msgrcv");
 
//   msgctl(msqid,IPC_RMID, NULL);                            /* destruction */
   return Retour;

Ma fonction sert a recevoir une file de message et a la retourner
et apres dans le main j'ai fait

Code :

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

strcpy(Ma_Chaine,LectureMSQ(x,x));

[Emmanuel Delahaye]

Ca devrait aller. L'appelant doit tester la valeur (NULL : ne pas utiliser) et libérer le bloc dont il a reçu l'adresse, après usage.

[jmjmjm]

oui en effet je n'ai pas liberer l'espace memoire mais comment faire ?? car ce n'ai pas une variable mais une fonction qui prend l'adresse non ?

[Emmanuel Delahaye]

oui en effet je n'ai pas liberer l'espace memoire mais comment faire ?? car ce n'ai pas une variable mais une fonction qui prend l'adresse non ?

Il faut mémoriser la valeur dans un pointeur, vérifier qu'elle est valide, l'utiliser, puis libérer le bloc pointé. Rien de très nouveau si tu sais utiliser fopen(), par exemple.

Code :

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

1
2
3
4
5
6
7
8
9
10
 
   T *p = f();
   if (p != NULL)
   {
      /* usage de p... */   
 
 
      /* quand c'est fini */
      free (p), p = NULL;
   }

[hegros]

Horrible !

Beh je peux l'utiliser de la sorte sans que cela provoque un comportement indeterminé (pas testé pas compilé)

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
#include <stdio.h>
 
char * MaFonc() {
  char buf[TAILLE];
  <..>
  return buf;
}
 
 
int main() {
 
  char buf[TAILLE];   
  printf("%s ", MaFonc());  //ceci ne pose pas de probléme si ?
  strcpy(buf,MaFonc());  //ceci ne pose pas de probléme aussi si ?  
  return 0 ;
}

[Emmanuel Delahaye]

Beh je peux l'utiliser de la sorte sans que cela provoque un comportement indeterminé (pas testé pas compilé)

Tu aurais dû le tester...

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
 
#include <stdio.h>
#include <string.h>
#define TAILLE 32
char *MaFonc ()
{
   char buf[TAILLE];
   strcpy (buf, "hello world");
   return buf;
}
 
int main (void)
{
   char buf[TAILLE];
   printf ("%s ", MaFonc ());   //ceci ne pose pas de probléme si ?
   strcpy (buf, MaFonc ());     //ceci ne pose pas de probléme aussi si ?
   printf ("%s ", buf);
   return 0;
}

Même gcc a vu le problème :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `MaFonc':
main.c:8: warning: function returns address of local variable
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 1 warnings

Quand au comportement...

Code :

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

1
2
3
 
é8Ã☺ é8Ã☺
Press ENTER to continue.

Ce problème est archi connu, je ne comprends pas que tu sois passé à coté, tu n'es pas un débutant que je sache...

[Médinoc]

Beh je peux l'utiliser de la sorte sans que cela provoque un comportement indeterminé (pas testé pas compilé)

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
#include <stdio.h>
 
char * MaFonc() {
  char buf[TAILLE];
  <..>
  return buf;
}
 
 
int main() {
 
  char buf[TAILLE];   
  printf("%s ", MaFonc());  //ceci ne pose pas de probléme si ?
  strcpy(buf,MaFonc());  //ceci ne pose pas de probléme aussi si ?  
  return 0 ;
}

1. Ben si. Les variables locales de printf() écrasent ton tableau.
2. Y'a des chances que là aussi.

Edit: Argh, grillé.

[hegros]

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
#include <stdio.h>
#include <string.h>
 
#define TAILLE 32
 
char * MaFonc() {
   char buf[TAILLE]={"Bonjour"};
  return buf;
}
 
 
int main() {
 
  char buf[TAILLE];
 
  printf("Avant strcpy %s \n", MaFonc());  //ceci ne pose pas de probléme si ?
  strcpy(buf,MaFonc());                     //ceci ne pose pas de probléme aussi si ?
  printf("Apres strcpy %s\n",buf);
 
  return 0 ;
}

Voici le résultat :

Code :

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

1
2
3
4
Avant strcpy
Apres strcpy Bonjour
 
Press ENTER to continue.

Sinon effectivement mon compilateur me dit :

Compiling: main.c
main.c: In function `MaFonc':
main.c:8: warning: function returns address of local variable
Linking console executable: C:\dev\console.exe
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 1 warnings

Sinon oui c'est un probléme archi connu je ne retournerai pas l'adresse d'une variable locale d'une fonction mais pour dire que même en faisant le test on arrive pas forcément au même résultat.

[Médinoc]

C'est donc bien un comportement indéfini, donc à fuir comme la télé éducative.

[Emmanuel Delahaye]

Sinon oui c'est un probléme archi connu je ne retournerai pas l'adresse d'une variable locale d'une fonction mais pour dire que même en faisant le test on arrive pas forcément au même résultat.

Ben oui. C'est le propre d'un comportement indéfini...

http://emmanuel-delahaye.developpez.com/notes.htm#ub

Je ne vois pas bien ce que tu cherches à prouver. Tu t'es trompé, c'est tout. Y'a pas de quoi en faire un fromage.

Code :

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

1
2
3
4
5
 
Avant strcpy À8Ã☺P "
Apres strcpy À8Ã☺P "
 
Press ENTER to continue.

[hegros]

Je ne vois pas bien ce que tu cherches à prouver. Tu t'es trompé, c'est tout. Y'a pas de quoi en faire un fromage.

Je ne cherche rien à prouver C'est une erreur de ma part d'avoir mis ce code certes ca m'apprendra de vouloir jouer aux apprentis sorciers et de vouloir retourner des adresses invalides

J'espère que le posteur a bien compris le probléme afin de ne plus le reproduire

[jmjmjm]

Ne vous inquietez pas j'ai compris et sa marche niquel merci a tous