Compter le nombre de chiffres dans un string

Version imprimable

Bonsoir,
Je n'arrive pas � comprendre pourquoi mon programme ne veut pas compter le nombre d'occurrences des chiffres dans une string.
La finalit� de mon programme veut pouvoir afficher par exemple "a : 5" si la phrase comporte 5 a, majuscule et minuscule non diff�renci�, pour �a �a marche. Mais impossible de compter les chiffres...

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
43
44
45
46
47
48
49
50
51
52
53
54
55
 int c = 0, count[26] = {0}, cpt=0;
    string phrase;
    ofstream fichiertexte;
    fichiertexte.open("output.txt", ios::out); //on ouvre le fichier en ecriture
    if (fichiertexte.bad())
    {
        cout<<"Impossible d'ouvrir le fichier."<<endl;
    } //permet de tester si le fichier s'est ouvert sans probleme
 
    cout<<"Entrer une phrase a inserer dans le fichier."<<endl;
    cin.ignore();
    getline(cin,phrase);
    fichiertexte<<phrase;
int taille=phrase.length();
int calcul=0;
 
    for (int i = 0 ; i<phrase.length() ; i++)
    {
        phrase[i] = tolower(phrase[i]);
    }
 
    cout<<"Output :"<<endl;
/*
    for(int k=0; k<10; k++) //de 0 à 9, donc ce que je veux rechercher
    {
        for (int z=0; z < taille; z++) //Je parcours tous les elements de ma phrase
        {
            if (phrase[z]==k) // Possible ?
            {
               calcul++;
            }
           cout<< k << " : " <<calcul<<endl;
        }
 
    }
*/
//OCCURR DES LETTRES
    while (phrase[c] != '\0')
    {
        if (phrase[c] >= 'a' && phrase[c] <= 'z' )
            count[phrase[c]-'a']++;
 
        c++;
    }
 
    for (c = 0; c < 26; c++)
    {
        if(count[c]!=0)
            cout<<(char)(c+'a')<<" : "<<hex<<count[c]<<endl;
        //On peut aussi faire :
        //printf("%c : %d\n",c+'a',count[c]);
    }
 
    fichiertexte.close();
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
/*
    for(int k=0; k<10; k++) //de 0 à 9, donc ce que je veux rechercher
    {
        for (int z=0; z < taille; z++) //Je parcours tous les elements de ma phrase
        {
            if (phrase[z]==k) // Possible ?
            {
               calcul++;
            }
           cout<< k << " : " <<calcul<<endl;
        }
 
    }
*/

Je ne comprends pas pourquoi cela ne marche pas. Pouvez vous m'�clairer svp?

Merci et bonne soir�e

Code:

1
2
3
4
5
6
7
8
9
10
11
12
for(int k=0; k<10; k++) 
{
    calcul = 0;
    for (int z=0; z < taille; z++)
    {
        if (phrase[z]== '0' + k)
        {
            calcul++;
        }
    }
    cout<< k << " : " <<calcul<<endl;
}

A l'arrache et que c'est sale.

13/01/2017, 20h06
kevbdx

Super �a marche un grand merci !
Est-ce qu'il y aurait du coup une m�thode plus propre ? J'ai essay� de faire pareil que pour les lettres mais j'obtiens des erreurs (lettres compt�es en trop)
Je d�bute en C++ du coup oui c'est un peu "sale" mais je vois pas d'autre moyen pour l'instant
14/01/2017, 11h50
Bousk

Il serait plus malin de parcourir la cha�ne et faire un switch pour chaque valeur cherch�e plut�t que de parcourir X fois la cha�ne selon le nombre de valeurs � trouver.
14/01/2017, 12h17
dalfab
C'est quoi propre, c'est quoi sale?

Le sale pourrait correspondre �
- du code difficile � lire
  - complexit� inutile
  - nommages trompeurs ou trop courts
  - commentaires insuffisants, inutiles ou trompeurs
  - pr�sentation et indentation farfelue
- non robuste
  - supporte mal les �volutions
  - d�pendant du contexte de compilation
- 'miraculeux' (c-a-d fonctionne malgr� des U.B.)
Code:

1 2 3 4 5 6 for ( char chif = '0' ; chif <= '9' ; ++chif ) { unsigned calcul{}; for ( auto car : phrase ) calcul += (car == chif); std::cout << chif << " : " << calcul << std::endl; }
Est-il plus propre ou plus sale?

On peut dire que le code pour les lettres est sale
Code:

1 2 3 4 5 6 int c = 0; while ( phrase[c] != '\0' ) { if (phrase[c] >= 'a' && phrase[c] <= 'z' ) count[phrase[c]-'a']++; c++; }
* comment �tre s�r simplement que le tableau count[] ne d�borde pas?
* et tout cas, phrase[] d�borde car dans une std::string le '\0' serait � l'indice phrase.length() qui est hors plage!

Par difficult� croissante :
Code compr�hensible par tout le monde :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 #include <cctype> // std::isdigit #include <cwctype> // std::iswdigit #include <string> size_t digitCount(const std::string& s) { size_t result = 0; for(size_t k = 0; k < s.length(); ++k) if(std::isdigit(s[k])) ++result; return result; } size_t digitCount(const std::wstring& s) { size_t result = 0; for(size_t k = 0; k < s.length(); ++k) if(std::iswdigit(s[k])) ++result; return result; }
Il faut pr�f�rer std::isdigit(s[k]) qui est plus idiomatique que s[k] >= '0' && s[k] <= '9'.
Code un peu plus technique avec l'ent�te <algorithm> et des lamdbas :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 #include <algorithm> // std::count_if #include <cctype> // std::isdigit #include <cwctype> // std::iswdigit #include <string> size_t digitCount(const std::string& s) { return std::count_if(s.begin(), s.end(), [](char c) -> bool { return std::isdigit(c); }); } size_t digitCount(const std::wstring& s) { return std::count_if(s.begin(), s.end(), [](wchar_t c) -> bool { return std::iswdigit(c); }); }
Quand on est familiaris� avec l'ent�te <algorithm> et les lambdas, cela se lit plus vite que les boucles for.
Code plus technique avec des templates et std::locale :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 #include <algorithm> // std::count_if #include <locale> // std::isdigit (with locale) #include <string> template<class charT> size_t digitCount(const std::basic_string<charT>& s, const std::locale& loc = std::locale()) { return std::count_if(s.begin(), s.end(), [&loc](charT c) -> bool { return std::isdigit(c, loc); }); } template<class charT> size_t digitCount(const charT* s, const std::locale& loc = std::locale()) { return digitCount(std::basic_string<charT>(s), loc); }
C'est la solution la plus flexible, car on peut donner en param�tre une locale avec laquelle les chiffres ne sont pas forc�ment les caract�res de '0' � '9'. Pour plus de d�tails, voir la doc de std::isdigit qui donne un exemple avec des caract�res japonais.
Remarque : Pour compter les lettres, c'est exactement pareil en rempla�ant std::isdigit par std::isalpha.

14/01/2017, 21h09
Detenu-1_0-J_6_4

Salut.

Pyramiddev je trouve votre solution un peu trop complexe pour un debutant surtout qu'il l'a pr�cis�.

la Solution premi�re serait effectivement plus adequate pour lui pour un d�but et puis il pourra venir regarder la votre:) quand il saura ce que c'est que les Template

Ca ressemblera plut�t � �a en C++:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 #include <algorithm> #include <string> #include <cctype> #include <iostream> int main() { std::string s{"ab2c"}; int tot = std::count_if( s.begin(), s.end(), isdigit ); std::cout << tot << std::endl; }
Il y a un pi�ge avec la fonction isdigit, o� pour ton cas la fonction C classique suffira je pense. Mais celle c++ (std::isdigit) demande une locale pour faire correspondre les chiffres �crits en diff�rentes langues.

Sinon tu peux �crire toi m�me tes fonctions "sp�ciales".
Par ex:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 bool isA( char c ) { return c == 'a' || c == 'A'; } int main() { std::count_if( s.begin(), s.end(), isA); //... }
Apr�s tu peux en faire des functors (un objet qui fait office de fonction o� tu pourrais passer la lettre cherch�e en param�tre ), des lambdas (�crire directement le code dans l'algo) o� que sais-je encore, mais tu ne connais ptet pas tout �a.