Discussion :

[joujou98]

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 :

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

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

[Sve@r]

Bonjour

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.

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 !!!

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

[foetus]

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

l=strlen(M) ;

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

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

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

quelle-est l'erreur ?

[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

[Sve@r]

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.

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

[joujou98]

Code :

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

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 ; 
}

[foetus]

Et si tu affichais le retour de ta fonction : je dis cela mais vraiment au hasard

[Sve@r]

n= freqchar(&ch,p)

Donc en dehors de la très intéressante remarque de foetus (qui a eu un 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... ). Et pense aussi à indenter proprement, ne serait-ce que pour avoir un code tout autant agréable à lire que facile à faire évoluer.