Calculer le nombre de caractères dans un Mot

Version imprimable

22/11/2018, 09h08
joujou98

Calculer le nombre de caractères dans un Mot
Bonjour,
On nous demande de construire une fonction qui retourne le nombre d'occurrence du caractère C dans une chaîne S
Voilà comment j'ai fait
Code:

1 2 3 4 5 6 7 8 9 10 int freqchar ( char*M,char C ) { int nb=0,l ; l=strlen(M) ; for( int i=0;i<l;i++) { If ( M[i]== C) nb++ ; } return nb ; }
Mais ça n'a pas marché dans le main()
Aidez moi svp
22/11/2018, 09h25
Sve@r

Bonjour

Citation:

Envoyé par joujou98

qui retourne le nombre d'occurrence du caractère C dans une chaîne S

Donc si la chaine c'est "S" dans l'énoncé, pourquoi toi tu l'appelles "M" ? Franchement c'est déjà rajouter de la complexité pour rien.

Citation:

Envoyé par joujou98

for( int I=0;I<=l-1;I++)

Tu ne penses pas qu'écrire I < l est plus efficace que ce I<=l-1 ???
Et évites les noms de variables en majuscules, ça nuit à la lisibilité. Surtout quand l'une c'est l et que l'autre c'est I !!!

Citation:

Envoyé par joujou98

Mais ça n'a pas marché dans le main()

:cristal:
22/11/2018, 09h27
foetus

:tagcode: :tagcode:

Citation:

Envoyé par joujou98

Int nb=0,l ;

Ce type Int n'existe pas. Édit: c'est la publication sans balises qui a mis en majuscule les premier mot des lignes

Citation:

Envoyé par joujou98

l=strlen(M) ;

C'est inutile lorsqu'on sait ce qu'est une chaîne de caractères en C.

Citation:

Envoyé par joujou98

Return nb ;

Ce mot clef Return n'existe pas. Édit: c'est la publication sans balises qui a mis en majuscule le premier mot des lignes

Citation:

Envoyé par joujou98

Mais ça n'a pas marché dans le main()

:cristal: quelle-est l'erreur ?
22/11/2018, 09h29
joujou98

-_- les majuscules parce que je viens de publier avec mon smartphone ... Je sais ce que vous dites mais le problème n'est pas dans les majuscules -_-

Je ne sais pas j'ai toujours un problème en testant une fonction dans le main () ... Même si une fonction juste j'arrive pas à la compiler correctement
22/11/2018, 09h52
Sve@r

Citation:

Envoyé par joujou98

Je sais ce que vous dites mais le problème n'est pas dans les majuscules -_-

Bien évidemment que non il n'est pas dans les majuscules (enfin si c'est juste un souci de transcription smartphone). Ca c'est juste un détail. Ta fonction est correcte et c'est ce qu'on essaye de te faire comprendre. Elle n'est pas parfaite, mais au-moins elle tourne correctement et renvoie le bon résultat.

Citation:

Envoyé par joujou98

Je ne sais pas j'ai toujours un problème en testant une fonction dans le main () ... Même si une fonction juste j'arrive pas à la compiler correctement

Mais putain tu vas nous le montrer ton main() ? Deux fois qu'on te dit qu'on ne peut pas le deviner !!! Surtout que si ça ne compile pas, alors il doit y avoir des messages d'erreur du compilateur. A moins que tu penses que ça non plus ça ne sert à rien :weird:

Voilà le main()

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
int main()
{
char *ch; 
int n=0 ; 
ch = (char*)malloc(10*sizeof(char));
if(fgets(ch,10,stdin)==NULL)
{
return EXIT_FAILLURE ;
}
printed("%s",ch);
n= freqchar(&ch,p) ; 
return 0 ; 
}

22/11/2018, 10h58
foetus

Et si tu affichais le retour de ta fonction : je dis cela mais vraiment au hasard :whistle:
22/11/2018, 11h04
Sve@r

Citation:

Envoyé par joujou98

n= freqchar(&ch,p)

Donc en dehors de la très intéressante remarque de foetus (qui a eu un :plusser: parce qu'il m'a beaucoup fait rire), c'est quoi le type de "ch" et par voie de conséquence c'est quoi le type de "&ch" ?
Allez, je te donne la réponse parce que sinon on en aura pour des semaines : "ch" est de type "char étoile" (tu l'as écrit char *ch). Donc "&ch" est de type "char étoile étoile" (l'adresse d'un truc est toujours du type "truc étoile" donc l'adresse d'un "char étoile" est de type "char étoile étoile") . Or, la fonction attend un "char étoile" donc un "char étoile étoile" ne convient pas donc "&ch" ne convient pas.

Voilà. La prochaine fois tu liras les messages du compilateur qui t'ont dit exactement la même chose. Et pense aussi à libérer tes allocations quand tu n'en as plus besoin (déjà que ici tu aurais écrit char ch[10] tu aurais eu la même chose sans avoir besoin de passer par cette allocation inutile... :roll:). Et pense aussi à indenter proprement, ne serait-ce que pour avoir un code tout autant agréable à lire que facile à faire évoluer.
22/11/2018, 14h09
Bousk

Et p sort d'où ? C'est une variable globale ? Un magicien a modifié ton code ?
Allouer dynamiquement une taille fixe aussi petite est inutile sauf à ajouter de la complexité.
Un code identé est plus lisible et donc plus simple à debugger.
22/11/2018, 15h14
Sve@r

Citation:

Envoyé par Bousk

Et p sort d'où ? C'est une variable globale ? Un magicien a modifié ton code ?

Tiens, c'est vrai, je l'ai raté celui-là :lol: