Discussion :

[adrbessy]

Bonjour,

Je débute sous C.
J'ai du mal à comprendre comment fonctionne la boucle while dans ce code.

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
#include <stdio.h>
#include <string.h>
#define CAR 'e'
#define LGMAX 132
 
int main()
{
    /*programme déterminant le nombre de lettres "e" */
    char texte[LGMAX+1];
    char * adr;
    int ncar;
 
    printf("Tapez un texte termine par return.\n");
    gets(texte);
    ncar=0;
    adr=texte;
 
    while (adr=strchr(adr,CAR))
    {
        ncar++;
        adr++;
    }
 
    printf("Votre texte comporte %d fois le caractere %c",ncar,CAR);
 
}

Merci beaucoup.

[imperio]

Alors la fonction strchr renvoit un pointeur sur la prochaine position du caractere passe en parametre. Cependant j'ai l'impression qu'il manque des parentheses, j'aurais pense que la condition aurait plutot du avoir cette tete :

Code :

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

while ((adr=strchr(adr,CAR)))

Apres je me trompe peut-etre... Donc la boucle continue tant qu'un pointeur non-NULL est retourne (c'est a dire tant que le caractere 'e' dans le cas present est trouve) et incremente de un la variable ncar pour compter le nombre d'occurence. Il incremente aussi le pointeur d'une case a chaque tour pour que strchr ne renvoie pas a chaque fois la meme position dans la chaine.

[Sve@r]

Bonjour

Cependant j'ai l'impression qu'il manque des parentheses, j'aurais pense que la condition aurait plutot du avoir cette tete :

Code :

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

while ((adr=strchr(adr,CAR)))

Ben tu vois bien que ces parenthèses sont inutiles. C'est uniquement si tu devais ensuite comparer "adr" avec autre chose style while ((adr=strchr(adr,CAR)) == xxx)...

Il incremente aussi le pointeur d'une case a chaque tour pour que strchr ne renvoie pas a chaque fois la meme position dans la chaine.

Presque. En fait, adr prend la position du premier caractère trouvé. Donc si ce caractère se trouve en position 50, adr passe alors immédiatement à cette adresse et jumpe ainsi 49 itérations inutiles.
Ensuite, effectivement, adr se décale d'un cran et passe à la position suivante pour pouvoir rechercher de nouveau le caractère dans le reste de la chaine....

[Médinoc]

Sur un bon compilateur, les parenthèses évitent un warning, car elles sont interprétées comme "oui, je sais, une affectation dans une expression conditionnelle, mais je sais ce que je fais, c'est normal". Mais je trouve qu'il est quand même plus lisible de rajouter explicitement !=NULL à la fin de l'expression.

[imperio]

Bonjour

Ben tu vois bien que ces parenthèses sont inutiles. C'est uniquement si tu devais ensuite comparer "adr" avec autre chose style while ((adr=strchr(adr,CAR)) == xxx)...

Quand je compilais ce code sans parenthèse, j'avais un warning donc je pense qu'elles sont utiles.

Presque. En fait, adr prend la position du premier caractère trouvé. Donc si ce caractère se trouve en position 50, adr passe alors immédiatement à cette adresse et jumpe ainsi 49 itérations inutiles.
Ensuite, effectivement, adr se décale d'un cran et passe à la position suivante pour pouvoir rechercher de nouveau le caractère dans le reste de la chaine....

En quoi ce que j'ai écrit est-il différent de ce que toi tu as écrit ? C'est plus court mais j'ai bien l'impression que c'est la même chose...

Sur un bon compilateur, les parenthèses évitent un warning, car elles sont interprétées comme "oui, je sais, une affectation dans une expression conditionnelle, mais je sais ce que je fais, c'est normal". Mais je trouve qu'il est quand même plus lisible de rajouter explicitement !=NULL à la fin de l'expression.

Je suis tout à fait d'accord avec toi concernant le "!=NULL". Un peu de lisibilité ne tue pas et fait gagner un temps fou quand on relit un code plutôt long.

[foetus]

En quoi ce que j'ai écrit est-il différent de ce que toi tu as écrit ? C'est plus court mais j'ai bien l'impression que c'est la même chose...

Lorsque tu dis "Il incrémente aussi le pointeur d'une case a chaque tour pour que strchr ne renvoie pas a chaque fois la meme position dans la chaine."

Sve@r ne pensait pas à adr++ mais à strchr(adr,CAR)

Je suis tout à fait d'accord avec toi concernant le "!=NULL". Un peu de lisibilité ne tue pas et fait gagner un temps fou quand on relit un code plutôt long.

:bof: Si c'est pour mettre un !=NULL qui va alourdir le test autant refaire la boucle.
Mais je connais la réaction de certains sur ce point

Code à l'arrache

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
 
    adr = strchr(adr, CAR);
 
    while (adr != NULL) {
        ncar++;
 
        adr++;
        adr = strchr(adr, CAR);
 
// or   adr = strchr(++adr, CAR);
    }
 
 
// or
 
    char c = adr[0];
 
    ncar = 0;
 
    for(int pos = 0; c != '\0'; pos++, c = adr[pos]) {
        if (c == CAR) { ncar++; }
    }

Édit: avec une boucle for (code si dessus). Mais il faut éventuellement une variable caractère en plus et on ne peut pas initialiser ncar à zéro dans la boucle (à moins que la variable pos existe déjà)

[-Eks-]

Les contributeurs à ce fil ont déjà répondu sur le code d'adrbessy, où il y a effectivement une affectation dans l'argument du while.

sans strchr, on peut faire :

Code :

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

1
2
3
    while (*adr)
        if (*adr++ == CAR)                                                                                                                  
            ncar++;

Ce qui (me) parait moins compliqué, tout en exploitant la compacité du C.

Eks