Discussion :

[Spinoza23]

Bonjour à tous, voilà mon petit souci : J'aimerai lire un fichier, récupérer les informations en les triant... Pour ce qui est de la lecture et de la récupération pas de soucis, mais c'est pour le tri que les choses se compliquent. J'aimerai comparer deux chaines de caractères qui doivent et qui à priori, sont la plus part du temps identiques. Malheureusement, je ne sais pas pourquoi je ne rentre pas dans mon if de condition. Je vous montre le code ça sera certainement plus clair :

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 motcle[12];/* variable qui va contenir le mot cle de ligne lue */
    char val[18];  /* variable qui va contenir la valeur */
    char * sep;    /* variable qui va contenir le separateur */
    motcle[12]= '\0';
    val[18]= '\0';
    printf("Entree P202 \n");
    /* Recherche du separateur dans la chaine */
    printf("Valeur de ligne>%s<\n", ligne);
    sep = strchr(ligne, '=');
    printf("Valeur de sep>%d<\n", sep);
 
    /* Initialisation des variables avec les valeurs correspondantes */
    /*motcle = strncpy(l_T_cartes[i].Mot_cle, ligne, 11);*/
    /* val    = strcpy (l_T_cartes[i].Valeur, sep +1); */
    strncpy(motcle, ligne, 12);
    printf("Valeur de motcle >%s< \n", motcle);
    strcpy (val, sep +1);
    printf("Valeur de val >%s< \n", val);
 
    /*Test de la ligne et des differentes valeurs*/
 
    /* Verification du mot cle et action en consequences */
    if (strcmp(motcle, "UNIT DSNAME ")== 0)
    {
        printf("Lecture de %s \n", motcle);
        strncpy(l_T_cartes[0].Valeur, val, 6);
        l_T_cartes[0].gere   = 'Y';
        printf("Attribution de Unit : <%s> \n", l_T_cartes[0].Valeur);
        printf("Gestion de UNIT DSNAME : %c \n", l_T_cartes[0].gere);
        return(g_rc);
    }

Le " if (strcmp(motcle, "UNIT DSNAME ")== 0)" n'est donc pas valable, puisque je n'ai jamais l'affichage du "printf("Lecture de %s \n", motcle);"... Es ce que quelqu'un aurait une idée pour m'aider ou alors une erreur que je n'ai pas vu.

D'avance merci de votre aide

[Invité(e)]

Bonjour,

Il y a un soucis dans ton code :

Code :

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

if (strcmp(motcle, "UNIT DSNAME ")== 0)

Quand tu écris "UNIT DSNAME ", il faut penser "UNIT DSNAME \0" : de taille 13 ! Comme motcle n'est que de taille 12, il y a peu de chance que ta comparaison aboutisse.

Solution, passer à strncmp(..., ..., 12) ou retirer le dernier espace...

[frenchi]

Un petite remarque à confirmer,
la variable motcle est un tableau de 12 caractère

Code :

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

char motcle[12];/* variable qui va contenir le mot cle de ligne lue */

Mais tu fais:

Code :

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

motcle[12]= '\0';

Et donc, tu mets le 13ème caractère à '\0', pour moi tu es en dehors du tableau

Sinon, la taille de motcle est de 12 mais celle de "UNIT DSNAME " 13, c'est donc normal que tu ne passes jamais le if. Utilise strncmp sur les 12 premiers caractères.

[diogene]

Code :

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

1
2
3
4
5
    char motcle[12];/* variable qui va contenir le mot cle de ligne lue */
    char val[18];  /* variable qui va contenir la valeur */
...
    motcle[12]= '\0';
    val[18]= '\0';

Ces valeurs sont hors des tableaux (indices 0 à 11 et 0 à 17)

Code :

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

1
2
3
4
5
    sep = strchr(ligne, '=');
...
    strncpy(motcle, ligne, 12);
...
    strcpy (val, sep +1);

J'en déduis que motcle est en début de ligne et est suivi d'une suite de caractères. Par conséquent, le comportement de strncpy fait que la chaîne motcle ne sera pas terminée par un zéro terminal. (Il faut mettre celui-ci explicitement si la chaîne source est plus grande ou égale que le nombre de caractères demandés). Ceci fera échouer la comparaison

[Spinoza23]

Je vien juste d'y penser... Merci beaucoup... J'hésitait entre rajouter \0 dans ma chaine de comparaison, mais je n'avais pas pensé à strncmp... j'essaye et je vous dit.

[Spinoza23]

Oui tout à fait, motcle est en début de ligne... le fichier se présente de la manière suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
UNIT DSNAME =SYSDA
VOLUME1     =ETD000
TYPE ALLOC  =TRK
DISQ PRIMAI =00000030
DISQ SECOND =00000030
LRECL       =08205
DCB BLKSIZE =00000
BUFNO       =003
PREFIX      =TRAV.S028443.
NOM PROCMEF =IYCAJMF
NOM PROCRAN =IYCAJRN
TIMESTAMP   =O

[Melem]

Et donc, tu mets le 13ème caractère à '\0'

Sauf qu'il n'y a pas de 13ème caractère puisque la taille de motcle est 12. La solution ce n'est pas d'utiliser strncmp à la place de strcmp, c'est de donner la taille adéquate à chaque tableau.

[frenchi]

à Melem, oui c'est pour cela que j'ai rajouté qu'il était en dehors du tableau

[Melem]

Spinoza23, Vu la structure de ton fichier, tu ferais mieux de lire une ligne avec fgets (bien sûr) puis séparer les "composantes" à l'aide de strtok. Comme ça, tu n'auras même plus besoin d'uniformiser la taille de tes mot-clés.

[Spinoza23]

Spinoza23, Vu la structure de ton fichier, tu ferais mieux de lire une ligne avec fgets (bien sûr)

C'est bien comme cela que je procède, je te rassure.

Code :

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

g_ligne = fgets(buf_ligne, sizeof(buf_ligne), g_fp);

Pour le reste je vais tout de même tenter un strncmp... Il me parrait plus approprié... Mais si ça ne marche pas, j'essaye autre chose... :p

[Spinoza23]

Ci cool !!! Problème résolut donc avec le strncmp... ça ç suffit pour résoudre le problème. Merci de votre aide...