Discussion :

[johhnh]

Bonjour

Débutant en c++ et plus largement dans le domaine de la programmation, j'avais jeudi en TP un programme de comptage de caractère à faire, le but était de faire un programme qui lit une suite de caractères au clavier et les analyse. La suite est terminée lorsque l’utilisateur tape un point (‘.’). On devait créer 3 sous prog' qui compter chaque un type de caractère.

Voilà mon prog :

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
bool est_un_chiffre (char a)
{   bool y;
    if (a>='0'&& a<='9')
    y = true;
    else
    y = false;
    return y ;
}
 
bool est_une_lettre (char a)
{   bool y;
    if (a>='a' && a<='z')
    y = true ;
    else if (a>='A' && a<='Z')
    y = true ;
    y = false ;
    return y ;
}
 
bool est_un_espacement (char a)
{   bool y;
    if (a==' ')
    y = true;
    else if (a=='\t')
    y = true;
    else if (a=='\n')
    y = true;
    else
    y = false;
    return y;
}
 
int main()
{
    //decla
  char caract;
  bool x,y,z ;
  int A,B,C ;
  A=0;
  B=0;
  C=0;
  cout<<"**Programme de comptage de caractere**"<<endl;
  cout<<" - Saisir une chaine de caractere a analyzer. La chaine doit se terminer par un point"<<endl;
do
{
  cin>>noskipws>>caract;
 
          x = (est_une_lettre(caract));
            if (x =true)
             {A=A+1;}
         y =(est_un_chiffre(caract)) ;
            if (y =true)
            {B=B+1;}
        z = (est_un_espacement(caract));
            if (z=true)
            {C=C+1;}
 
  cout<<"Le nombre de lettre est"<<A<<endl;
  cout<<"Le nombre de chiffre est"<<B<<endl;
  cout<<"Le nombre d'espacement est"<<C<<endl;
}
while (caract =! '.');
system("PAUSE");
}

Le problème est le suivant : le programme fonctionne mais cependant ne me compte pas les caractères, quoi que je rentre il me sort toujours 1 caractère pour chaque type.

Etant débutant je me rends compte qu'il y à un problème au niveau de l’exécution de la boucle mais je n'arrive pas à identifier mon erreur, il me semble que le programme rentre une fois dedans et en sort directement.
Si quelqu'un pouvait m'aiguiller et si vous avez des remarques sur des maladresse dans la conception du programme , n'hésitez pas

Merci.

[koala01]

Salut,

Pour répondre à ta question, tu essayes de faire rentrer une chaine de caractères (donc un tableau de caractères contigus) dans un caractère unique... Cherchez l'erreur

Sinon, tu te fais beaucoup de mal pour pas grand chose !

Regardes peut être du coté de std::isdigit, std::islapha, std::isalnum, std::isspace, std::string (surtout string::size()) et de std::getline.

Toutes ces fonctions et cette classe (std::string) ne pourront que t'être d'une aide particulièrement efficace

[Daïmanu]

Bonsoir, tu fais simplement quelques erreurs bêtes. Mais nous en sommes tous passé par la

Code :

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

1
2
do {}
while (caract =! '.');

L'opérateur "différent de" s'écrit != et non pas =! (ce qui signifierait littéralement "égale l'inverse de '.' ").

Code :

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

1
2
3
if (x=true)
if (y=true)
if (z=true)

Là encore, tu te trompes involontairement d'opérateur : tu utilises l'assignation et non la comparaison ( == ).

[johhnh]

@koala01 :

Effectivement je me fait beaucoup de mal pour rien, j'avais effectivement trouvé ces fonctions lors de mes recherches, mais dans l'exercice il n'est pas demandé de faire appel à ces fonctions, c'est surtout pour apprendre le c++ je pense.

@manudiclemente :

Effectivement, merci pour m'avoir fait remarquer ces erreurs

Voilà le prog' terminée et fonctionnel, si jamais sa peut aider quelqu'un un jour :

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
bool est_un_chiffre (char a)
{   bool y;
    if (a>='0'&& a<='9')
    y = true;
    else
    y = false;
    return y ;
}
 
bool est_une_lettre (char a)
{   bool y;
    if (a>='a' && a<='z'|| a>='A' && a<= 'Z')
    y = true;
    else
    y = false ;
    return y ;
}
 
bool est_un_espacement (char a)
{   bool y;
    if (a==' ' || a=='\t' || a=='\n')
    y = true;
    else
    y = false;
    return y;
}
 
int main()
{
    //decla
  char caract;
  bool x,y,z ;
  int A,B,C ;
  A=0;
  B=0;
  C=0;
  cout<<"**Programme de comptage de caractere**"<<endl;
  cout<<" - Saisir une chaine de caractere a analyzer. La chaine doit se terminer par un point"<<endl;
do
{
  cin>>noskipws>>caract;
          x = (est_une_lettre(caract));
            if (x ==true)
             {A=A+1;}
         y =(est_un_chiffre(caract)) ;
            if (y ==true)
            {B=B+1;}
         z = (est_un_espacement(caract));
            if (z==true)
            {C=C+1;}
}
while (caract != '.');
cout<<"Le nombre de lettre est"<<A<<endl;
cout<<"Le nombre de chiffre est"<<B<<endl;
cout<<"Le nombre d'espacement est"<<C<<endl;
system("PAUSE");
}

Je vous remercie encore pour votre aide !

Bonne journée.

[koala01]

@koala01 :

Effectivement je me fait beaucoup de mal pour rien, j'avais effectivement trouvé ces fonctions lors de mes recherches, mais dans l'exercice il n'est pas demandé de faire appel à ces fonctions, c'est surtout pour apprendre le c++ je pense.

Je pinaille, je sais...

Mais, tu dis qu'il n'est pas demandé de faire appel à ces fonctions. Cela ne veut pas forcément dire qu'il soit interdit d'y recourir

Si l'on se base sur le fait que tout ce qui n'est pas explicitement interdit est autorisé, pourquoi se faire du mal inutilement

[Jean-Marc.Bourguet]

Il faut utiliser isXXX, ces fonctions vont prendre en compte toutes les lettres supportées par le code utilisé (é par exemple) -- du moins si on le demande gentillement -- et fonctionner même quand on a 'I' < '}' < 'J' comme en EBCDIC (code qui est quand encore pas mal utilisé en volume, même si les plateformes où il l'est sont d'accès moins courant).