Retour d'une chaine grace à une fonction

**K0mpu73rn1kDaCS** · 25/01/2006, 22h32

Bonjour.
Je suis étudiant en BTS IRIS et nous travaillons à mon grand plaisir avec Visual C++.
J'ai un petit programme a faire en TP je suis à 2 doigts de finaliser celui-ci mais il y a encore quelquechose qui me pose problème.
Il s'agit de retourner une chaine de caractère que l'on obtiens d'une fonction
J'ai donc fait mes recherches et j'ai trouvé ceci dans les FAQ de ce site :
FAQ C - Section sur les fonctions

Comment créer une fonction qui retourne une chaîne de caractères ?
auteurs : Bob, LFE

Pour retourner une chaîne de caractères, il faut passer à la fonction un pointeur en paramètres et copier la chaîne dans la zone mémoire pointée (avec strcpy() par ex.).

Une autre façon de faire est de déclarer la fonction comme retournant un pointeur sur des caractères et d'allouer la mémoire dans la fonction sans la libérer dans celle-ci, mais dans la fonction appelante.

Voici d'autre part des extraits de mon code qui constitue une application en mode console :

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
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
 
int main(void)
{
	// Il s'agit là du code qui me permet de tester ma fonction
	int aff;
	char mot[2];
	char *pmot = &mot[0];
	cout << "Aff : " << endl;
	cin >> aff ;
	pmot = affecte(aff);
 
	cout << *pmot << *(pmot+1);
 
	getch();
	return 0;
}
 
char *affecte(int reste)
{
	char	syllabe[2];
	char	*psyllabe = &syllabe[0];
 
	switch (reste)
	{
	case 0 :
		*psyllabe = 'H';
		*(psyllabe+1) = 'o';
		break;
	case 1 :
		*psyllabe = 'H';
		*(psyllabe+1) = 'a';
		break;
	case 2 :
		*psyllabe = 'H';
		*(psyllabe+1) = 'e';
		break;
	case 3 :
		*psyllabe = 'H';
		*(psyllabe+1) = 'i';
		break;
	case 4 :
		*psyllabe = 'B';
		*(psyllabe+1) = 'o';
		break;
	case 5 :
		*psyllabe = 'B';
		*(psyllabe+1) = 'a';
		break;
	case 6 :
		*psyllabe = 'B';
		*(psyllabe+1) = 'e';
		break;
	case 7 :
		*psyllabe = 'B';
		*(psyllabe+1) = 'i';
		break;
	case 8 :
		*psyllabe = 'K';
		*(psyllabe+1) = 'o';
		break;
	case 9 :
		*psyllabe = 'K';
		*(psyllabe+1) = 'a';
		break;
	case 10 :
		*psyllabe = 'K';
		*(psyllabe+1) = 'e';
		break;
	case 11 :
		*psyllabe = 'K';
		*(psyllabe+1) = 'i';
		break;
	case 12 :
		*psyllabe = 'D';
		*(psyllabe+1) = 'o';
		break;
	case 13 :
		*psyllabe = 'D';
		*(psyllabe+1) = 'a';
		break;
	case 14 :
		*psyllabe = 'D';
		*(psyllabe+1) = 'e';
		break;
	case 15 :
		*psyllabe = 'D';
		*(psyllabe+1) = 'i';
		break;
	default :
		*psyllabe = 'x';
	}
 
	cout << "Syllabe : " << psyllabe << endl; // DEBUG // */
 
	return psyllabe ;
}

J'ai suivi l'exemple d'un livre sur C++ que je possède chez moi mais ça ne donne rien de convainquant.

Quand je vérifie ma chaine avant de la retourner elle est comme je le désir. (Notez au passage la manière un peu idiote d'affecter la chaine de caractère mais c'est venu des resultats que j'ai pu avoir avec la valeur testée)

Problème il ne reste plus que le 1er caractère dans mon main. J'ai pensé à un problème de portée mais j'ai appliqué la même logique que l'exemple que j'ai tiré du manuel dont je parle plus haut, mon professeur a misé sur un mauvais positionnement du caractère NULL ( I mean \0). Quelqu'un pourrait-il m'expliquer le pourquoi (ça ne marche pas) et le comment (appliquer corrèctement le principe cité dans la FAQ - Qu'ai-je mal fait ? -) ?

D'avance merci.

**farscape** · 25/01/2006, 23h12

salut,
la question est de savoir ou tu te situe dans le langage :
tu fais du C ou du C++ ?
si c'est du c++ il serait beaucoup de voir ça avec l'objet string...

bon sinon dans ton code tu te torture la tete il n'y a pas trop d'interet de declarer un pointeur sur une chaine pour acceder a ses elements:

Code :

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

1
2
3
 
char mot[2];
   char *pmot = &mot[0];

tu peux faire mot[0] ou mot[1]...

pour ta fonction affecte, tu tombes dans le piege du debutant en C, tu retournes l'adresse d'une variable (syllabe) située dans la pile d'une fonction .
quand on sort de la fonction la pile est nettoyée syllabe n'est plus valide
donc ton retour (psyllabe) est incorrect et provoquera une erreur.
pour retouner l'adresse d'une chaine declarée dans une fonction celle ci doit etre déclarée static:

Code :

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

1
2
 
static char syllabe[2];

autre chose si tu envois psyllabe dans cout celle ci doit se terminée par un
\0
donc syllabe doit posséder 3 caracteres pour la place pour le zero
sinon probleme ....

Code :

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

1
2
 
static char syllabe[3];

autre option tu peux (comme signalé dans la faq C ) declarer un pointeur et allouer de l'espace mémoire,
le pointeur sera bien nettoyé en sortant de la fonction mais la zone memoire allouée existera toujours.

Code :

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

1
2
3
 
char *syllabe=new char [3];
syllabe[2]=0;

il faudra faire un delete [] de cette adresse pour liberer la zone...

**K0mpu73rn1kDaCS** · 26/01/2006, 00h09

Ok c'est ce à quoi je pensais pour le "allouer de la mémoire". On fait encore du c without classes. (en référence au premier nom de C++ : C with classes)

Je vais essayer les propositions que tu me donnes.

pour ta fonction affecte, tu tombes dans le piege du debutant en C, tu retournes l'adresse d'une variable (syllabe) située dans la pile d'une fonction .
quand on sort de la fonction la pile est nettoyée syllabe n'est plus valide

C'est donc bien ce que je pensais : un problème de portée.

Pour les chaines de caractères on sait qu'il faut les terminer avec le caractère NULL (\0) dans mon cas syllabe[2] quand je déclare une chaine le nombre entre crochet est en logique humaine ou informatique ? Dans ma tête syllabe[2] c'est déclarer un tableau qui peut avoir les indices 0, 1 et 2 donc par besoin de s'ennuyer avec le calcul du \0. Je vois que je me suis trompé donc je déclare syllabe[3] ( 3 caractères dans la chaine avec NULL compris)

Bon merci ça marche. Pas étonnant quand on est Most Valuable Professionnal, ça doit faire plaisir. Je ne met pas encore le tag [RESOLU] si certaines personnes voudraient apporter d'autres éléments à ce sujet. J'imagine qu'il y a d'autres choses à observer et à comprendre. Quelqu'un veut-il partager son expérience en la matière ?

Merci

**farscape** · 26/01/2006, 06h52

Envoyé par K0mpu73rn1kDaCS

Ok c'est ce à quoi je pensais pour le "allouer de la mémoire". On fait encore du c without classes. (en référence au premier nom de C++ : C with classes)

Je vais essayer les propositions que tu me donnes.

pour ta fonction affecte, tu tombes dans le piege du debutant en C, tu retournes l'adresse d'une variable (syllabe) située dans la pile d'une fonction .
quand on sort de la fonction la pile est nettoyée syllabe n'est plus valide

C'est donc bien ce que je pensais : un problème de portée.

Pour les chaines de caractères on sait qu'il faut les terminer avec le caractère NULL (\0) dans mon cas syllabe[2] quand je déclare une chaine le nombre entre crochet est en logique humaine ou informatique ? Dans ma tête syllabe[2] c'est déclarer un tableau qui peut avoir les indices 0, 1 et 2 donc par besoin de s'ennuyer avec le calcul du \0. Je vois que je me suis trompé donc je déclare syllabe[3] ( 3 caractères dans la chaine avec NULL compris)

Bon merci ça marche. Pas étonnant quand on est Most Valuable Professionnal, ça doit faire plaisir. Je ne met pas encore le tag [RESOLU] si certaines personnes voudraient apporter d'autres éléments à ce sujet. J'imagine qu'il y a d'autres choses à observer et à comprendre. Quelqu'un veut-il partager son expérience en la matière ?

Merci

re,
non syllabe[2] ,veux dire deux elements ,l'indicage commence toujours à 0 ,donc [0,1] ...

**K0mpu73rn1kDaCS** · 26/01/2006, 17h22

Donc ma logique est juste si je comprend bien.

Si je déclare syllabe[2] j'ai bien accès syllabe[0], syllabe[1] et syllabe[2].

**farscape** · 26/01/2006, 17h31

Envoyé par K0mpu73rn1kDaCS

Donc ma logique est juste si je comprend bien.

Si je déclare syllabe[2] j'ai bien accès syllabe[0], syllabe[1] et syllabe[2].

argh non 2 elements,l'indicage commence toujours à 0 donc [0,1]. sinon ça fait 3 elements !

**K0mpu73rn1kDaCS** · 27/01/2006, 00h18

OK merci bien alors.

Retour d'une chaine grace à une fonction

MFC

Vue hybride

Discussions similaires

Partager

Partager