probleme de compatibilité avec une variable et une fonction

**Francky44003** · 02/10/2010, 12h17

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** · 02/10/2010, 12h34

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** · 02/10/2010, 12h53

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** · 02/10/2010, 13h31

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** · 02/10/2010, 13h41

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** · 02/10/2010, 13h43

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

**jeroman** · 02/10/2010, 18h03

Tu peux te baser sur quelque chose de ce genre :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <time.h>
 
#define TAILLE_MOT_MAX 128
 
char * choisirUnMot ( FILE * fichier , char * motAchercher , int nombreMots )
{
    char mot[TAILLE_MOT_MAX];
    int numero_ligne;
    int ligne_courante;
 
    numero_ligne = (int) ((double)rand() / (double)RAND_MAX * nombreMots) ;
    ligne_courante = 0 ;
 
    rewind(fichier);
 
    while ( fgets(mot,TAILLE_MOT_MAX,fichier) != NULL )
    {
        if ( ligne_courante == numero_ligne )
        {
            char * position_caractere ;
 
            position_caractere = strchr(mot,'\r');
            if ( position_caractere != NULL )
                *position_caractere = '\0' ;
            position_caractere = strchr(mot,'\n');
            if ( position_caractere != NULL )
                *position_caractere = '\0' ;
 
            strncpy(motAchercher,mot,TAILLE_MOT_MAX);
            return motAchercher ;
        }
        ligne_courante++;
    }
 
    return NULL;
}
 
#define NOMBRE_MOTS 10 /* à adapter selon le nombre de mots dans le fichier texte */
 
int main( void )
{
    char motAchercher[TAILLE_MOT_MAX] = "";
    FILE * fichier ;
    int i;
 
    fichier = fopen ( "mots.txt" , "rb" );
    if ( fichier == NULL )
        return 0 ;
 
    srand(time(NULL));
    rand(); /* j'en mets un ici volontairement (la première valeur est toujours quasiment identique chez moi :o ) */
 
    for ( i=0 ; i<50 ; i++) /* tire 50 mots au sort */
    {
        if ( choisirUnMot(fichier , motAchercher , NOMBRE_MOTS) != NULL )
        {
            printf("Mot : %s\n" , motAchercher);
        } else
        {
            printf("Erreur\n");
        }
    }
 
    fclose (fichier);
 
    return 0;
}

**Francky44003** · 02/10/2010, 19h18

merci, je vais regarder sa.
Moi j'ai reussi a faire une fonction bcp plus longue/barbare, ou je copie caractere par caractere, la voici :

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
void choisirUnMot (FILE *file, int numeroLigne, char *tableau)
{
 
    int hasard = rand()%numeroLigne;
 
    int  caractere = 0,i = 0, nbLettre = 0;
    char  motAtransferer[30] = "";
    rewind(file);
 
 
    if ( file != NULL && numeroLigne != 0)
    {
        while ( (caractere = fgetc(file))!= EOF && hasard != 0)
        {
 
            if (caractere == '\n')
            {
                hasard--;
            }
 
        }
 
        fgets(motAtransferer, 30, file);
 
        nbLettre = strlen(motAtransferer);
 
 
        while (i < nbLettre)
        {
            tableau[i] = motAtransferer[i];
            i++;
 
            if (i == nbLettre)
            {
                *(tableau + i) = '\0';
            }
        }
 
    }
}

je sais c'est un peu moche mais c'est la seule solution que j'ai trouver, je vais etudier le code que tu ma donner pour améliorer le mien

MERCI A TOI

**dzada** · 02/10/2010, 20h47

salut

juste une remarque de base,

pourquoi tu prend un int pour ton fgetc ?
bien sur quand tu va stocker un char (8bits) dans un int (32) ca va aller
mais ensuite quand tu vas le réutiliser?
le compilo est obligé de supposer que tu sais que les 24 autres bits seront perdus.

**Sve@r** · 02/10/2010, 22h05

Envoyé par momo-ben2A

Code :

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

1
2
3
 
    rewind(file);
    if ( file != NULL && numeroLigne != 0)

Moi, je me demande pourquoi tu testes file != NULL.
Non. En fait, je me demande surtout, si jamais il pouvait arriver que file soit égal à null, comment se comporterait le rewind() situé juste au dessus...

Accessoirement, traiter le fichier par fgetc() et compter les '\n' pour les lignes marche... mais utiliser fgets(), qui lit une ligne entière, est beaucoup plus aisé !

**jeroman** · 02/10/2010, 22h47

Envoyé par dzada

pourquoi tu prend un int pour ton fgetc ?

Il faut prévoir un éventuel EOF comme retour de fonction :

fgetc() lit le caractère suivant depuis le flux stream et renvoie ce caractère, lu sous forme unsigned char, puis transformé en int, ou EOF en cas d'erreur ou de fin de fichier.

**dzada** · 03/10/2010, 03h10

exact, ca fait vraiment trop longtemp que j'ai arrêté de penser C je devrais me taire.

**Francky44003** · 03/10/2010, 08h45

merci a tous pour vos conseils, que je mettra en pratique

PS : j'ai vraiment fait des erreurs betes ....

**Francky44003** · 03/10/2010, 12h20

voila ma procedure modifier :

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
void choisirUnMot (FILE *file, int numeroLigne, char *tableau)
{
 
    int hasard = rand()%numeroLigne;
    char  motAtransferer[30] = "";
 
    if ( file != NULL && numeroLigne != 0)
    {
        rewind(file);
        while ( hasard != 0)
        {
            hasard--;
            fgets(motAtransferer, 30, file);
        }
 
        strcpy(tableau, motAtransferer);
 
    }
}

un seul petit probleme : a chaque fois qu'on boucle, fgets va ecraser le contenu de motAchercher (les mots precedents)

PS : merci a tous encore pour vos explications
PS2 : j'en profite également pour vous demander si vous ne connaissez pas des livres en C de niveau intermediaire car j'aimerai bien approfondir mes connaissances sur ce langage

**diogene** · 03/10/2010, 12h33

un seul petit probleme : a chaque fois qu'on boucle, fgets va ecraser le contenu de motAchercher (les mots precedents)

C'est quoi motAchercher ?
Si c'est motAtransferer, quelle importance d'écraser les mots que l'on n'utilise pas ?

Code :

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

1
2
3
4
    int hasard = rand()%numeroLigne;
    char  motAtransferer[30] = "";
 
    if ( file != NULL && numeroLigne != 0)

C'est trop tard qu'intervient le test numeroLigne != 0. Si numeroLigne == 0, le programme a déjà planté pour division par 0 ici :

Code :

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

 int hasard = rand()%numeroLigne;

Pour bien te former au C regarde par ici : Cours et tutoriels programmation C et FAQ C.

**Sve@r** · 03/10/2010, 15h18

Envoyé par momo-ben2A

voila ma procedure modifier :

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
void choisirUnMot (FILE *file, int numeroLigne, char *tableau)
{
 
    int hasard = rand()%numeroLigne;
    char  motAtransferer[30] = "";
 
    if ( file != NULL && numeroLigne != 0)
    {
        rewind(file);
        while ( hasard != 0)
        {
            hasard--;
            fgets(motAtransferer, 30, file);
        }
 
        strcpy(tableau, motAtransferer);
 
    }
}

un seul petit probleme : a chaque fois qu'on boucle, fgets va ecraser le contenu de motAchercher (les mots precedents)

Ben oui. Si tu veux pas les perdre alors te faut définir un "tableau" de mots avec, à chaque fois, fgets qui remplit une nouvelle case du tableau. Bien entendu te faut un tableau assez grand donc avec le nb de mots connu à l'avance ou alors obligé de jouer avec les mallocet les realloc. Bref faut te donner les moyens...

Accessoirement, fgets() copie la ligne (enfin le mot) dans la variable "motAtransferer". Puis, sans que t'aies fait un traitement quelconque de ce mot, tu copies "motAtransferer" dans la variable "tableau". Bref la variable "motAtransferer" ne sert pas à grand chose. Comme si, chaque fois qu'on te rend la monnaie, tu mette l'argent dans la poche gauche puis, ensuite, tu transfères l'argent de la poche gauche vers la poche droite...

probleme de compatibilité avec une variable et une fonction

C

Discussions similaires

Partager

Partager