besoin d'un coup de pouce pour strcmp

Version imprimable

03/01/2007, 02h50
echantillon

besoin d'un coup de pouce pour strcmp
bonjour et bonne année a tous ;°)

comme le titre l'indique: je bloque! mais je +- débute donc je pense qu'il s'agit sûrement d'un problème grossier qui m'échappe.

voici mon code :
Code:

1 2 3 4 5 6 7 8 9 char * mot = ""; FILE * f; f = fopen("test","r"); fscanf(f,"%s",&mot); fclose(f); if (!strcmp(mot,"bonjour")) ... //me renvoie une erreur de segmentation.
voilà! je vous assure que j'ai écopé les man pages sans trouver de réponse.
je ne sais vraiment pas ou je me plante.
03/01/2007, 07h47
scol30

ton init de "mot"

Code:

char* mot

est faite sur une chaine constante. Pour que ton code fonctionne il te faut plutot faire un malloc.
Ensuite ton fscanf n'a pas besoin d'une reference sur mot:

Code:

fscanf(f,"%s",mot);

et sa fonctionnera mieux.

Stef
03/01/2007, 10h48
Emmanuel Delahaye
Citation:
Envoyé par echantillon

bonjour et bonne année a tous ;°)

comme le titre l'indique: je bloque! mais je +- débute donc je pense qu'il s'agit sûrement d'un problème grossier qui m'échappe.

voici mon code :

Code:

1 2 3 4 5 6 7 8 9 char * mot = ""; FILE * f; f = fopen("test","r"); fscanf(f,"%s",&mot); fclose(f); if (!strcmp(mot,"bonjour")) ... //me renvoie une erreur de segmentation.

voilà! je vous assure que j'ai écopé les man pages sans trouver de réponse.
je ne sais vraiment pas ou je me plante.
mot est un pointeur vers une chaine constante.

Tu commets donc 3 infractions entrainant un comportement indéfini :
- Tu passes l'adresse d'un pointeur sur char alors que fscanf() avec "%s" attend l'adresse d'un char
- Même si tu retires le '&'
  - Tu passes l'adresse du premier élément d'un tableau de char non modifiable (on ne peut donc pas écrire dedans)
  - Même si il était modifiable, ce tableau ayant une talle de 1, il ne peut recevoir autre chose qu'une chaine vide ("").

Salut,

Je pense que je ferais comme cela:

Code:

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
 
#include <stdio.h>
#include <stdlib.h>
 
int main(void)
{
    int i; /* indice de boucle */
    char s_mot[50];
    FILE *f;
 
    f = fopen("test", "r");
    /* On contrôle toujours la valeur de retour de fopen*/
    if (f == NULL)
    {
        fprintf(stderr, "Impossible d'ouvrir le fichier test!\n");
        exit(EXIT_FAILURE); 
    }
 
    /* fgets est la fonction de choix pour la saisie de chaînes de caractères*/
    fgets(s_mot, sizeof s_mot, f);
    fclose(f);
 
    for (i = 0; s_mot[i] != '\n' && s_mot[i] != '\0'; ++i)
    {
        continue; /* Le corps de la boucle est vide*/
    }
    if (s_mot[i] == '\0')
    {
        /* La saisie est incomplête: sizeof s_mot est insuffisant */
    }
 
    if  (!strcmp(s_mot, "bonjour"))
    {
        /* si s_mot contient la chaîne "bonjour" */
    }
    else
    {
        /* ... */
    }
 
    return EXIT_SUCCESS;
}

La fonction fscanf (et scanf) est une fonction exclusivement réservée aux experts, car très difficile à manipuler correctement. Pour une utilisation correcte de scanf et en particulier avec la saisie de chaîne de caractère, la lecture de ce tutoriel est fortement recommandée: http://xrenault.developpez.com/tutoriels/c/scanf/

Thierry

03/01/2007, 13h25
echantillon

Merci pour les réponse ;°)
alors:

=>Emmanuel :
"Tu commets donc 3 infractions entrainant un comportement indéfini :"

-..heu, me reste t' il des points sur mon permis de programmer !? :aie:

"Même si il était modifiable, ce tableau ayant une talle de 1, il ne peut recevoir autre chose qu'une chaine vide ("") "

-Ducoup, je ne devrais pas avoir le droit de faire ceci :
char * p = "";
p = "bonjour";
?
pourtant gcc ne me dit rien et ça fonctionne .

Sinon merci pour la clarté de ta réponse. je pense avoir définitivement compris mes erreurs précédentes.

=>mujigka :
Etant donné le nombre important de mots (une 100aine) dont je dois tester la présence dans mon fichier (je peux éventuellement vous donner des précisions sur mon prog) je crois que je vais quand même utiliser fscanf; tout en faisant très attention.
03/01/2007, 13h41
Emmanuel Delahaye
Citation:

Envoyé par echantillon

"Même si il était modifiable, ce tableau ayant une talle de 1, il ne peut recevoir autre chose qu'une chaine vide ("") "

-Ducoup, je ne devrais pas avoir le droit de faire ceci :
char * p = "";
p = "bonjour";
?
pourtant gcc ne me dit rien et ça fonctionne .

Ben oui, pourquoi on ne pourrait pas faire ça ? On peut faire pointer un pointeur n'importe où, du moment que le type est compatible et que la zone est valide...

Tu confonds peut être avec
Code:

1 2 3 char * p = ""; strcpy (p, "bonjour");
Ce qui est un comportement indéfini.

Rappel : http://emmanuel-delahaye.developpez.com/notes.htm#ub

Remarque. La zone pointée par p éant non modifiable, il est recommandé d'écrire :
Code:

1 2 3 char const * p = ""; strcpy (p, "bonjour");
Là, un compilateur bien réglé devrait signaler le problème de types incompatibles, ce qui pourrait éviter une erreur...
03/01/2007, 14h43
echantillon
"Ben oui, pourquoi on ne pourrait pas faire ça ?"
overdose de théine. désolé.

La vrai raison c'est qu'il faut que je revoie les pointeur en c.
Feu mon roulage de bosse en pointeurs pascalien dont je suit originaire (ha.. les belles années légos..) mais le c est plus coriace, j' eu été prévenu.

Sinon voici la correction de mon code:
Code:

1 2 3 4 5 6 7 8 char * mot = ""; mot = (char*)malloc(sizeof(char)*50); FILE * f; f = fopen("test","r"); fscanf(f,"%s",mot); fclose(f); if (!strcmp(mot,"bonjour")) system("echo bonjour");
cette fois je pense que c'est bon.
merci pour tout ;°)
et rebonne année a vous tous;°))

Citation:

Envoyé par echantillon

Sinon voici la correction de mon code:

Beaucoup de choses inutilement compliquées. Beaucoup de choses essentielles oubliées...

Code:

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
int main (void)
{
   char const *fname = "test.txt";
   char *mot = malloc (sizeof *mot * 50);
 
   if (mot != NULL)
   {
      int err = 1;
      FILE *f = fopen (fname, "r");
 
      strcpy (mot, "");
 
      if (f != NULL)
      {
         err = fscanf (f, "%s", mot) != 1;
         fclose (f), f = NULL;
      }
      else
      {
         perror (fname);
      }
 
      if (!err)
      {
         if (strcmp (mot, "bonjour") == 0)
         {
            system ("echo bonjour");
         }
      }
      free (mot), mot = NULL;
   }
   return 0;
}

Pose des questions si tu ne comprends pas.

03/01/2007, 18h24
Skyounet

Pourquoi une allocation dynamique pour un taille fixe ?
03/01/2007, 18h36
Emmanuel Delahaye

Citation:

Envoyé par Skyrunner

Pourquoi une allocation dynamique pour un taille fixe ?

Peut être en vue d'une réallocation...
:dehors:
03/01/2007, 19h02
echantillon

"Beaucoup de choses inutilement compliquées." là je ne vois pas.
"Beaucoup de choses essentielles oubliées..." ça je comprend très bien quand je vois la rigueur de ton code.

Mais est-ce vraiment nécessaire de prendre autant de précautions ? (je connais la réponse)
Connais-tu beaucoup de monde qui soit aussi rigoureux dans les moindre détails? débutant compris, surtout ? Qu'en penses-tu !?

Quand j'écope forums et tuto a la recherche d'exemple je ne vois jamais de code aussi parano/propre.

..enfin tu comprends mes craintes.
Si je dois coder comme ça, je vais fournir une ligne toute les huit heures, tripler mon nombre d'erreurs et fournir un code que j'aurais du mal a relire.

Mais je veux bien le faire. ..
03/01/2007, 19h36
Emmanuel Delahaye
Citation:

Envoyé par echantillon

"Beaucoup de choses inutilement compliquées." là je ne vois pas.

de mémoire...
- des définitions de variables initialisées ou non avec plus loin leur initialisaton véritable...
- des casts inutiles
- sizeof(char) vaut 1 par définition...
Citation:

"Beaucoup de choses essentielles oubliées..." ça je comprend très bien quand je vois la rigueur de ton code.

Bof, c'est du code industriel normal. C'est pas long à coder quand on sait ce qu'on fait. Il faut de l'habitude ...

Citation:

Mais est-ce vraiment nécessaire de prendre autant de précautions ? (je connais la réponse)

Ben oui, sinon je ne le ferais pas.

Citation:

Connais-tu beaucoup de monde qui soit aussi rigoureux dans les moindre détails? débutant compris, surtout ? Qu'en penses-tu !?

Je pense que plus on apprend tôt à coder correctement tôt, meilleure sera la vie des informaticiens. J'ai passé des jours et des nuits (littérallement) à reprendre du code fait à l'arrache par des soi-disantes SSII :
- Ca ne fonctionne pas
- Ca fait perdre du temps à celui doit réellement sortir le produit et de l'argent à la boite.

Citation:

Quand j'écope forums et tuto a la recherche d'exemple je ne vois jamais de code aussi parano/propre.

Tu ne dois pas aller dans les bons forums...

Citation:

..enfin tu comprends mes craintes.
Si je dois coder comme ça, je vais fournir une ligne toute les huit heures, tripler mon nombre d'erreurs et fournir un code que j'aurais du mal a relire.

Mais je veux bien le faire. ..

Un codeur confirmé produit 10 à 25 lignes de code validé par jour.

Citation:

Envoyé par echantillon

"Beaucoup de choses inutilement compliquées." là je ne vois pas.
"Beaucoup de choses essentielles oubliées..." ça je comprend très bien quand je vois la rigueur de ton code.

Mais est-ce vraiment nécessaire de prendre autant de précautions ? (je connais la réponse)
Connais-tu beaucoup de monde qui soit aussi rigoureux dans les moindre détails? débutant compris, surtout ? Qu'en penses-tu !?

Quand j'écope forums et tuto a la recherche d'exemple je ne vois jamais de code aussi parano/propre.

..enfin tu comprends mes craintes.
Si je dois coder comme ça, je vais fournir une ligne toute les huit heures, tripler mon nombre d'erreurs et fournir un code que j'aurais du mal a relire.

Mais je veux bien le faire. ..

Il ne fait que vérifier que les appels aux fonctions susceptibles d'échouer.
malloc échoue rarement mais il peut echouer.
fopen peut aussi échouer (fichier non présent par exemple)

Je pense savoir pourquoi tu trouves ce code compliqué, en fait Emmanuel respecte le : une seule sortie par fonction.

De ce fait il multiplie les imbrications.

Une autre facon de voir son code est la suivante.
Code:

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 #include <stdio.h> #include <stdlib.h> #include <string.h> int main (void) { char const *fname = "test.txt"; char *mot = malloc (sizeof *mot * 50); FILE *f = fopen (fname, "r"); if (mot == NULL) { fprintf(stderr, "Malloc failure !\n"); exit(EXIT_FAILURE); } if(f == NULL) { perror("file()"); exit(EXIT_FAILURE); } strcpy (mot, ""); if(fscanf (f, "%s", mot) == 1) { if (strcmp (mot, "bonjour") == 0) { system ("echo bonjour"); } } fclose(f), f = NULL; free(mot), mot = NULL; return 0; }
D'ailleurs ya un problème dans ton code Emmanuel
Code:

1 2 err = fscanf (f, "%s", mot) != 1;
Manque le if je crois.

03/01/2007, 19h40
Emmanuel Delahaye

Citation:

Envoyé par Skyrunner

D'ailleurs ya un problème dans ton code Emmanuel

Code:

err = fscanf (f, "%s", mot) != 1;

Manque le if je crois.

Bah non, même en BASIC, c'est vrai. Une expression retourne 0 ou 1.
03/01/2007, 19h46
Skyounet

Citation:

Envoyé par Emmanuel Delahaye

Bah non, même en BASIC, c'est vrai. Une expression retourne 0 ou 1.

Nop c'est bon j'ai rien dis. J'ai pas réfléchi. Du moins je voyais pas la chose comme ça.
04/01/2007, 19h23
echantillon

bonjour ;

En y réfléchissant il y a des choses que ne comprend pas:

char const *fname = "test.txt";
Pourquoi définis-tu une constante pour une utilisation si courte?
Ou quel est l'amélioration par rapport aux lignes correspondantes de mon programme ?
Un ami m'a demandé pourquoi tu n'utilisait pas #define ?

char *mot = malloc (sizeof *mot * 50);
Ne peut-on pas mettre malloc(50) pour allouer 50 octets/caractères a mot, tout en évitant les erreurs éventuelles renvoyées par malloc?

strcpy (mot, "");
Est-ce pour être sur que mot != NULL ?
Ne peut-on pas écrire char *mot = malloc (sizeof *mot * 50) = ""; ?

Je crois que c'est tout ;°)

Pondre 3 ligne par jour ne me dérange absolument pas tant que j'ai l'impression de bien faire.
C'est pour ça que tout ce que vous me dites, ce genre de conseils, là maintenant, je trouve ça hyper important. Je m' en délecte.
Ca vaut 200 exemple.
04/01/2007, 19h56
Emmanuel Delahaye

Citation:

Envoyé par echantillon

char const *fname = "test.txt";
Pourquoi définis-tu une constante pour une utilisation si courte?

Parce que j'utilise le nom 2 fois. (fopen() et perror())

Citation:

Un ami m'a demandé pourquoi tu n'utilisait pas #define ?

Pourquoi pas. Question de style.

Citation:

char *mot = malloc (sizeof *mot * 50);
Ne peut-on pas mettre malloc(50) pour allouer 50 octets/caractères a mot, tout en évitant les erreurs éventuelles renvoyées par malloc?

Si un jour tu passes en unicode avec des wchar_t, la maintenance est réduite au minimum avec mon code :
wchar_t *mot = malloc (sizeof *mot * 50);
Hop, terminé.

Citation:

strcpy (mot, "");
Est-ce pour être sur que mot != NULL ?

J'ai rien compris. Cette instruction permet de placer une chaine vide (un 0) dans le tableau. Je ne connais plus le contexte...

Citation:

Ne peut-on pas écrire char *mot = malloc (sizeof *mot * 50) = ""; ?

C'est précisément ce qui a été fait, non ?
04/01/2007, 20h43
echantillon

Bon, je crois que ça ira comme ça.

Je vais me replonger dans mes lignes et reprendre tout ce qui me paraît nécessaire.

Au moins je n'aurai pas qu'appris a utiliser fscanf.

A bientôt

!