Discussion :

[ziwiwi]

bonjour,
quelqu'un peut m'aiderrr j'ai un soucis pour renvoyer le nombre de lettres en majuscule, voila mon code en C ca se compile sauf que ca renvoie pas le nombre qu'il faut...Merci

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
#include <stdio.h>
#include <stdlib.h>
#define tmax 100
int main()
{   int m = 0;
    char chaine[tmax];
    printf("entrer chaine de caractere\n");
    scanf("%s",chaine);
    m = compterMajiscule(chaine);
    printf("%d",m);
    return 0;
}
//déclaration de la Fonction 
int compterMajiscule(int chaine[])
{
    char caractere=0;
    int i=0;
    int compteur=0;
    do
    {
       caractere = chaine[i];
       if ((caractere>=65)&&(caractere<=90))//j'ai utilisé le codage ascii
       {
           compteur+=1;
       }
       i++;
    }
    while(caractere != '\0');
 
    return compteur;
 
}

[Neckara]

Bonjour,




compteur+=1;, on a plus l'habitude de faire compteur++;.

if ((caractere>=65)&&(caractere<=90)), pourquoi ne pas utiliser une fonction déjà existante comme isupper ?

[ternel]

On est bien d'accord que %s dans le scanf ne lit qu'un seul mot?

http://en.cppreference.com/w/c/io/fscanf

[Neckara]

On est bien d'accord que %s dans le scanf ne lit qu'un seul mot?

http://en.cppreference.com/w/c/io/fscanf

Oui, de plus, il ne contrôle pas le nombre de caractère maximum à lire.
Je pense qu'un fgets serait plus approprié ou un scanf("%30s", ...) si tu ne veux que lire un mot à la fois.

[pythéas]

A mon avis le problème principal est le type du paramètre attendu par la fonction "compterMajiscule"

Les "int" prenant plus de "place" que les "char" ça fout complétement en l'air la lecture des caractères dans la boucle (tous ne sont pas lu) et donc le résultat

@ziwiwi :
si tu mets :

Code :

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

int compterMajiscule(char chaine[])

ça devrait beaucoup mieux fonctionner
si tu débutes, essaye de comprendre ce qui c'est passé, ça peut être un bon exercice.

[ternel]

Bien vu, j'aurai pu y passer du temps.