Discussion :

[panda31]

Bonjour à tous.

J'ai un problème (ptetr tout bete pour vous gds gourous du C) avec ce code (entre autres... ) :

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
 
/* 
MODULE : CREER enregistrement du texte
Role : manipulation d'un enregistrement de 30 mots
Structures manipulees : enregistrement
Dependances : <stdio.h>
*/
#include <stdio.h>
#include "createRecord.h"
 
FILE * ptrFich;
 
 
/*
Nom : createRecord
Role : creer un enregistrement de 30 mots pour faciliter et accelerer le traitement du texte à indexer
Signature : FILE * -> ENREGISTREMENT *
Entrée : -
Sortie : ENREGISTREMENT * record
MAJ : -
Dependances : <stdio.h>
Algorithme : 
    DEBUT
        indice : entier, indice de parcours du tbl;
        indice<-0;
        Lire le fichier pointé par ptrFile;
        TANT_QUE indice<30 FAIRE
            indice++;
            record(indice)<-mot_courant;
        FIN_TANT_QUE
        Retourner record;
            
    FIN
 
 
*/
char * createRecord(FILE * ptrFile)
{
    int nb_mots;
    char tab_enr[50];
    char * mot;
    char caractere;
    char sep[]="|-><,(); :.\'\t\0 «»";
    nb_mots=0;
    if (ptrFile!=NULL)
    {    ptrFich=ptrFile;
    }
    while (!feof(ptrFich) || fgetc(ptrFich)!='\n')
    {    caractere=fgetc(ptrFich);
        int i=0;
        int bool=0;
        //on cherche si le caractère correspond à un séparateur dans le
        //tableau de séparateur
        while (i<16 || bool==1)
        {    if (caractere==sep[i])
                bool=1;
        }
        //si ce n'est pas un séparateur on écrit le mot
        if (bool==0)
        {    strcat(mot,caractere);
        }
        else 
        {    tab_enr[nb_mots]=mot;
            nb_mots ++;
        }
        i++;
    }
    tab_enr[nb_mots]='/n';
    return tab_enr;         
 
}

Voici les messages obtenus à la compilation :

gcc -W createRecord.c -o createRecord
createRecord.c:8:26: createRecord.h: No such file or directory
createRecord.c: In function `createRecord':
createRecord.c:60: warning: passing arg 2 of `strcat' makes pointer from integer without a cast
createRecord.c:63: warning: assignment makes integer from pointer without a cast
createRecord.c:68:19: warning: multi-character character constant
createRecord.c:68: warning: overflow in implicit constant conversion
createRecord.c:69: warning: function returns address of local variable

Je comprends pas... Certes ce ne sont "que" des warnings mais c inquiétant vu leur nature !

Pouvez-vous m'aider ?
Pourquoi les pointeurs de char ne sont ils pas acceptés comme cela devrait l'^etre au vu du man des fonctions utilisees.

Merci d'avance.

[homeostasie]

Pourquoi les pointeurs de char ne sont ils pas acceptés comme cela devrait l'^etre au vu du man des fonctions utilisees.

J'ai juste regarder la fonction strcat() et tu l'emploies mal.

Code :

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

1
2
 
strcat(mot,caractere);

Voici comment tes variables sont déclarées:

Code :

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

1
2
3
 
char * mot;
char caractere;

-> La variable "caractère" n'est pas un pointeur sur char.
-> L'espace mémoire pointé par le pointeur "mot" n'a pas été alloué: Utilisation de la fonction malloc() puis de free() pour libérer les ressources.

man strcat()

[millie]

En plus, ça va carrement planté pour la fonction utilisatrice.

Code :

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

char tab_enr[50];

Et tu retournes :

Code :

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

tab_enr

qui est une variable locale (c'est un pointeur). Donc qui ne sera pas accessible pour une fonction appelante.


Tu fais :

Code :

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

ptrFich=ptrFile

Je ne vois pas à quoi ça sert.

[millie]

Et encore une 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
      int i=0;
        int bool=0;
        //on cherche si le caractère correspond à un séparateur dans le
        //tableau de séparateur
        while (i<16 || bool==1)
        {    if (caractere==sep[i])
                bool=1;
        }
        //si ce n'est pas un séparateur on écrit le mot
        if (bool==0)
        {    strcat(mot,caractere);
        }
        else 
        {    tab_enr[nb_mots]=mot;
            nb_mots ++;
        }
        i++;

Ton i est réinitialisé à chaque tour de boucle... Je pense pas que c'est ce que tu souhaites.

[homeostasie]

Code :

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

1
2
3
 
tab_enr[nb_mots]=mot;
nb_mots ++;

Et là, je dirais qu'il faudrait que tu nous dises ce que ton programme est censé faire.
A mon avis, tu souhaites sauvegarder tout les mots que tu détectes. Si c'est bien le cas, tu dois définir un tableau à deux dimensions et utiliser la fonction memcpy().

[panda31]

Code :

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

1
2
3
 
tab_enr[nb_mots]=mot;
nb_mots ++;

Et là, je dirais qu'il faudrait que tu nous dises ce que ton programme est censé faire.
A mon avis, tu souhaites sauvegarder tout les mots que tu détectes. Si c'est bien le cas, tu dois définir un tableau à deux dimensions et utiliser la fonction memcpy().

Alors je reviens tout d'abord sur le but et le fonctionnement souhaité de mon programme.
Nous avons eu comme sujet :

Il s'agit de concevoir et coder un programme en C qui, a partir d'un fichier texte et d'une liste de mots "non-clés", produise un index indiquant le nombre d'occurences de chaque mot clé du texte, avec les numéros de ligne corresondantes.

• Si un meme mot clé apparait deux fois dans une meme ligne, le numéro de la ligne n'apparait qu'une seule foisdans l'index.
• Si un mot apparait plusieurs fois (par défaut max=50) dans le texte (nombtre configurable), il ne doit pas apparaitre dans l'index.
• L'index doit pouvoir etre visualisé ou listé "a la demande"
• Dans les deux cas, le résultat doit apparaitre classé par nombre d'occurences, par ordre alphabétique des mots clés.
• Prendre en compte le multilinguisme.

Comme vous pouvez le voir, c'est très restricitf et pas du tout adapté au langage C ! mais bon, la parole d'un prof est parole d'évangile !... enfin, ca reste a prouver.

Dans mon dernier poste, j'ai mis en PJ le diagramme SD de l'architecture. Voici en PJ les sources déjà faites (aucun .h pour le moment !).

Si ca tente quelqu'un de se plonger dans les méandres de notre inculture C-iste, bon courage et surtout merci beaucoup de la patience dont vous faites preuve !

Ma collègue de binome me fait vous dire que le problème principal semble résider dans l'utilisation de TOUTES les fonctions de manipulations de fichier et de chaines de caracteres.

Enfin, pourquoi un tableau deux dimensions? et problème de renvoi de tableau meme qd passé en paramètre par référence dans nos fonctions.

Merci encore et d'avance pour votre dévouement !

Vive le C

[millie]

-> L'espace mémoire pointé par le pointeur "mot" n'a pas été alloué: Utilisation de la fonction malloc() puis de free() pour libérer les ressources.

Si tu veux éviter d'utiliser un malloc.

Tu peux passer en paramètre ton pointeur tab_enr.

Code :

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

int createRecord(FILE * ptrFile, char* tab_enr, unsigned int tailletab)

Qui retourne EXIT_SUCESS si tout s'est bien passé ou EXIT_FAILURE si par exemple tab_enr était trop petit.

Une méthode pour retourner un char* sans passer par malloc serait de déclarer ta variable static, mais je te déconseilles fortement. (si tu réalises deux appels de suites, ton ancienne valeur sera écrasée).

[panda31]

Merci à tous de vos critiques si constructives.

Au sujet des initialisations de pointeurs à NULL, mon pb est que là où je bosse, cela me marque à la compilation que NULL n'est pas reconnu !

Autre chose, les .H manquant c'est normal vu que nous ne les avons pas écrits encore. Eh oui, on est deux sur le code. Perso, en C, tout ce que j'ai codé se rapportait à des nombres et des matrices de nombre donc les char, j'y pige pas grand chose. Mon truc à moi était d'interfacer avec un langage plus "simple" comme Python ()mais là, j'ai la contrainte du langage C imposé ! (grrrr) Le programme que je fais est censé indexer un texte à partir d'une liste de mots non-clés. Du coup, toutes les fonctionnalités sont "empaquetées" dans des modules spécialisés selon le diagramme SD fourni en PJ.

En perl, c'était simplissime à faire, mais en C... Personnellement, je préfère les langages haut niveau donc le C...

Merci encore de votre aide si précieuse, demain, je vois ce que je peux corriger sur le code.

[Médinoc]

NULL est supposé être défini dans <stdlib.h> au minimum.
(Edit: perdu, c'était <stddef.h>)
Si ce n'est pas le cas, vous pouvez toujours rajouter ceci dans votre code:

Code :

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

1
2
3
#ifndef NULL
#define NULL ((void*)0)
#endif

C'est une définition assez passe-partout de NULL, qui devrait marcher sur tout compilateur C qui supporte les pointeurs void*...

[Jean-Marc.Bourguet]

Tu veux dire <stddef.h>

NULL doit être défini dans
- stddef.h (l'entête à inclure de préférence pour avoir NULL)
- locale.h
- stdio.h
- stdlib.h
- string.h
- time.h
j'espère ne pas en avoir oublié.

[Emmanuel Delahaye]

En perl, c'était simplissime à faire, mais en C... Personnellement, je préfère les langages haut niveau donc le C...

Ne viens pas ici pour critiquer notre cher langage (il y a un forum pour ça). Rassure-toi, on est parfaitement au courant de ses défauts et des ses domaines d'application.

Savoir choisir le bon outil fait aussi partie de ce métier...

[Foobar1329]

Hello,

Bonjour à tous.

J'ai un problème (ptetr tout bete pour vous gds gourous du C) avec ce code (entre autres... ) :

Code :

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

1
2
3
4
5
6
7
8
9
 
/* 
MODULE : CREER enregistrement du texte
Role : manipulation d'un enregistrement de 30 mots
Structures manipulees : enregistrement
Dependances : <stdio.h>
*/
#include <stdio.h>
#include "createRecord.h"

Apparement, comme signalé à la compilation, gcc ne trouve pas createRecord.h, problème de casse (createrecord.h peut être) ou problème de chemin.

Code :

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

1
2
 
FILE * ptrFich;

C'est une globale, pas beau, mais bon...Il faut au moins l'initialiser.

Code :

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

1
2
 
FILE * ptrFich = NULL;

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
 
 
/*
Nom : createRecord
Role : creer un enregistrement de 30 mots pour faciliter et accelerer le traitement du texte à indexer
Signature : FILE * -> ENREGISTREMENT *
Entrée : -
Sortie : ENREGISTREMENT * record
MAJ : -
Dependances : <stdio.h>
Algorithme : 
    DEBUT
        indice : entier, indice de parcours du tbl;
        indice<-0;
        Lire le fichier pointé par ptrFile;
        TANT_QUE indice<30 FAIRE
            indice++;
            record(indice)<-mot_courant;
        FIN_TANT_QUE
        Retourner record;
            
    FIN
 
 
*/
char * createRecord(FILE * ptrFile)
{
    int nb_mots;
    char tab_enr[50];
    char * mot;
    char caractere;
    char sep[]="|-><,(); :.\'\t\0 «»";
    nb_mots=0;
    if (ptrFile!=NULL)
    {    ptrFich=ptrFile;
    }
    while (!feof(ptrFich) || fgetc(ptrFich)!='\n')
    {    caractere=fgetc(ptrFich);

Ici, il y a déjà pas mal de choses à dire. Initialiser les pointeurs et tant qu'on y est les tableaux.

Code :

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

1
2
char tab_enr[50] = { '\0' };
char * mot = NULL;

Ensuite, il ne faut pas utiliser feof() de cette façon. feof() indique si l'indicateur de fin de fichier a été positionné ou non. Or, il n'est d'une part pas évident que la fin de fichier soit atteinte lors d'un échec de lecture. D'autre part, il faut attendre qu'un échec de lecture survienne (i.e fgetc() a renvoyé EOF) aussi pour la vraie fin de fichier, ce qui fait qu'un tour de boucle supplémentaire est effectué. Il suffit de tester le retour de fgetc() et de voir si c'est EOF ou non. Le fgetc(ptrFich)!='\n' ne sert à rien.

int caractere;
while ( (caractere=fgetc(ptrFich) ) != EOF)
{
/*...*/
}
/* Test si l'échec de fgetc() est bien du à la fin de fichier, et pas une erreur de lecture autre */
if (feof(ptrFich)) { /* OK*/ }

Code :

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

1
2
3
4
5
6
7
8
9
10
 
 
        int i=0;
        int bool=0;
        //on cherche si le caractère correspond à un séparateur dans le
        //tableau de séparateur
        while (i<16 || bool==1)
        {    if (caractere==sep[i])
                bool=1;
        }

Et non
Je te laisse le soin de voir pourquoi.

Code :

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

1
2
3
4
5
while (i<16 && !bool)
{    if (caractere==sep[i]) {
                bool=1;
      }
}

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 
       //si ce n'est pas un séparateur on écrit le mot
        if (bool==0)
        {    strcat(mot,caractere);
        }
        else 
        {    tab_enr[nb_mots]=mot;
            nb_mots ++;
        }
        i++;

Et boum, mot n'est ni initialisé et ne pointe pas vers un bloc de mémoire alloué. Par ailleurs, caractere n'est pas une chaine de caracteres, marche pas. nb_mots n'est pas initialisé et ne vaut pas forcément 0 au début, le compilo lui affecte la valeur qui lui plaît, 4879 par exemple, au pif. Il ya un problème conceptuel important ici, une confusion caractere/mot. Ce que tu souhaites, c'est concatener plusieurs caracteres n'étant pas des séparateurs pour en faire un mot puis insérer ce dernier dans untableau de mots. tab_enr est un tableau de char, donc peut contenir une chaine de caractères et non plusieurs.
Pour être franc, il faut tout repenser et tout refaire.

Code :

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

1
2
3
4
5
6
7
 
 
    }
    tab_enr[nb_mots]='/n';
    return tab_enr;         
 
}

Voici les messages obtenus à la compilation :


Je comprends pas... Certes ce ne sont "que" des warnings mais c inquiétant vu leur nature !

Non, c'est normal, strcat() n'est par exemple pas utilisé correctement.
A+