Discussion :

[Douly]

Bonsoir,

Pour un projet que je dois effectuer en langage C, je dois bien structurer mon programme via diverses fonctions et je voulais savoir pourquoi ce petit bout de code ne fonctionnait pas.

Mon compilateur me dit que la fonction retourne l'adresse de la variable locale. Je débute en C et je m'y perds un peu avec les pointeurs.

Pourriez-vous m'aider à corriger mon erreur ?

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
#include <stdio.h>
#include <string.h>
#include <time.h>
#include <stdlib.h>
 
char *GenererCombiAleatoire();
int main()
{
    char solution[4];
    printf("Solution : %s",GenererCombiAleatoire());
}
char *GenererCombiAleatoire()
{
    char solution[4];
    int a=0;
            srand(time(NULL));
        for(a=1;a<=4;a++)
            {
            solution[a]=rand()%6;
            if (solution[a]==0)
            solution[a]='R';
            if (solution[a]==1)
            solution[a]='V';
            if (solution[a]==2)
            solution[a]='B';
            if (solution[a]==3)
            solution[a]='J';
            if (solution[a]==4)
            solution[a]='O';
            if (solution[a]==5)
            solution[a]='M';
            }
            return solution;}

[Pouet_forever]

En C une variable déclarée dans une fonction n'est "en vie" (présente) que dans cette fonction et nul par ailleurs. Arrivé à la fin de la fonction cette variable "meurt" et tu ne peux pas l'utiliser dans une autre fonction. (Tu peux toujours contourner le problème mais je n'en parlerai pas ça t'embrouilleras sûrement )

Pour les fonctions "normales" tu peux retourner une variable sans problème, mais pour les chaînes de caractères c'est plus subtil. Tu vas retourner un pointeur sur le premier élément de ta chaîne de caractère, mais ta chaîne de caractères n'est déclarée que dans ta fonction. Du coup elle est supprimée et dans la fonction appelante tu veux accéder à quelque chose qui n'existe plus ...

Le meilleur moyen est de passer la chaîne à traiter en paramètre

Autre petit truc :

Les indices de tableaux partent de 0 et vont jusqu'à n-1.
Dans ton cas tu as déclaré un tableau de 4 cases donc tu auras accès aux indices 0, 1, 2, 3.

Je te laisse modifier cette boucle en conséquence

Code :

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

for(a=1;a<=4;a++)

Je ne sais pas si tu as vu les switch mais dans ton cas c'est préférable je pense
La fonction main doit retourner quelque chose. La valeur 0 signifie que le programme s'est bien déroulé. Donc : return 0;

[Douly]

Le meilleur moyen est de passer la chaîne à traiter en paramètre

Ok, donc je devrais plutôt envoyer mon tableau de chaine de caractères à la fonction main?

Donc j'aurai une fonction de type :

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
 
char GenererCombiAleatoire(char solution);
int main()
{
//Appel de la fonction
return 0;
}
char GenererCombiAleatoire(char solution)
{
    char solution[4];
    int a=0;
            srand(time(NULL));
        for(a=0;a<=3;a++)
            {
            solution[a]=rand()%6;
            switch(solution[a])
            {
            case 0: solution[a]='R';
            break;
            case 1: solution[a]='V';
            break;
            case 2: solution[a]='B';
            break;
            case 3: solution[a]='J';
            break;
            case 4: solution[a]='O';
            break;
            case 5 : solution[a]='M';
            break;
            }
            }
}

Et pour appeler les valeurs dans mon main ? Car en gros je veux réutiliser ces valeurs pour les comparer ensuite à d'autres que l'utilisateur devra entrer (mais ça c'est pas vraiment le sujet du topic).

Merci pour ta réponse, j'avais pas pensé au switch.

[Melem]

FAQ : Comment retourner une chaîne de caractères ?

[Douly]

Ah oui, je n'avais pas bien regardé dans le FAQ la réponse à ma question y était. Après plusieurs test j'ai réussi à faire mon code de deux manières, avec le 'malloc' et avec la première méthode. Y a t-il une des deux méthodes qu'il faut privilégier ou bien c'est plus ou moins la même chose ?

En tout cas un grand merci à vous deux.

[souviron34]

tout dépend de l'usage et de ce que fait la fonction..

En général, si la fonction crée une chaîne, c'est mieux avec le malloc..

Si elle ne fait que modifier une chaîne in situ c'est mieux par paramètre..