Discussion :

[Francky44003]

bonjour a tous je fais appelle a vous car j'ai besoin de votre aide concerant une fonction et une variable qui reçoit ce que la fonction renvoit.

Tout d'abord voici le code source :

variable :

Code :

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

1
2
char motAchercher[TAILLE] = ""
 motAchercher = choisirUnMot (fichier, nombreMots);

et la fonction qui est censé retourner le mot que l'on a piocher dans un fichier :

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
char  *choisirUnMot (FILE *file, int numeroLigne)
{
 
srand(time(NULL));
int hasard = rand()%numeroLigne;
 
 
    int  caractere = 0;
    char  motAretourner[30];
    rewind(file);
 
    if ( file != NULL && numeroLigne != 0)
    {
        while ( (caractere = fgetc(file))!= EOF && hasard != 0)
        {
 
            if (caractere == '\n')
            {
                hasard--;
                printf ("trouver n\n");
 
            }
 
 
        }
         fgets(motAretourner, 30, file);
         printf ("mot retourner %s\n", motAretourner);
         return motAretourner;
    }
    else
    {
        printf ("Une erreur s'est produite lors de la recherche d\'un mot\n");
        return EXIT_FAILURE;
    }
 
}

le compilateur m'indique comme erreur : "incompatible types in assignment"

Je ne vois pas pourquoi il y a une erreur car tout deux son de type char

PS : merci d'avance pour vos explications et aides

[jeroman]

Je ne sais pas sur quelle ligne il te met le warning, mais j'ai vu deux choses qui m'ont choqué :

Code :

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

return EXIT_FAILURE;

La fonction est censée renvoyer un pointeur sur char. EXIT_FAILURE est un entier. Ce serait plus logique de retourner NULL.

Code :

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

return motAretourner;

Il ne faut JAMAIS retourner l'adresse d'un objet local à une fonction. Une fois que tu sors de la fonction, l'objet créé n'existe plus et le contenu en mémoire (sur la pile, pour être précis) est susceptible d'être écrasé à n'importe quel moment. Du coup, l'adresse retournée pointe sur n'importe quoi.

Une chose aussi :

Code :

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

srand(time(NULL));

En principe, ceci doit être exécuté une seule fois. Or, si la fonction est appelée plusieurs fois, c'est pas terrible.
Le mieux, c'est de mettre ça en début de la fonction "main".

EDIT :

Code :

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

1
2
char motAchercher[TAILLE] = ""
motAchercher = choisirUnMot (fichier, nombreMots);

"motAchercher" est un tableau. Tu ne peux pas faire "tableau=adresse".

[Francky44003]

je te remercie pour tes remarques, qui m'aideront (a progresser) a l'avenir.
Je n'arrive pas a trouver de solution pour regler ce probleme, a part mettre le code dans le main.c mais sa va etre un grand bazar.
Je cherche mais je ne trouve pas.

Une solution de votre part peut etre ?

[jeroman]

Le problème, c'est que je ne comprends déjà pas ce que ton programme est censé faire réellement.

et la fonction qui est censé retourner le mot que l'on a piocher dans un fichier :

C'est-à-dire ?

[Francky44003]

c'est cette fonction justement

- on a comme parametre le fichier ou se trouve les mots et comme 2eme parametre le nombre de mots dans ce fichier

- ensuite on fait un rand avec le nombre de mots dans ce fichier, pour cela on fait moins 1 a hasard lorsqu'il rencontre ce caractere è\n' (donc changement de ligne)

- une fois celle ci arriver a 0 on fait appelle a fgets pour transferer le mot du fichier a la ligne X dans motAretourner

- et ensuite on fait un return motAchercher (mais bon c'est faux tu me la expliqué pourquoi)


voila

[Francky44003]

je veux en faite creer une sorte de pendu sur console.