Discussion :

[bazoga]

salut les pro,

j'ai essayé d’écrire un programme simple en c permettant de calculer la fréquence d'apparition de caractères (majuscule de A à Z) mais un petit problème avec de le test d'arret :

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
 
int main(int argc , char* argv[]){
 
	int nb[25];
	int i;
	char c;
 
	for(i=0;i<26;i++)
		nb[i]=0;
 
	do{   
   		c  = getchar();
		if(c>64 && c<=90){
			nb[c-65]++ ;
		}
		printf("je suis la \n");
	}while(c != '\n');
 
	for(i=0;i<26;i++){
		if(nb[i]!=0)
			printf("%c == %d\n",(i+'A'),nb[i]);
	}
 
	return 0;
}

Merci d'avance

[Kirilenko]

Bonsoir,

En fait, ton problème est multicausal. Le soucis de départ est que que tu lis en dehors de ton tableau nb. N'oublie pas que les indices des tableaux vont de 0 à n-1. Quand i = 25, tu lis donc une case en dehors de ton tableau. Techniquement, cette situation conduit à un comportement indéterminé. Dans la pratique, pour ton cas, tu modifies la zone mémoire qui se situe juste après nb. Et là, pas de chance : c'est i qui se situe juste après nb en mémoire. Donc, à chaque fois que i = 25, tu vas réinitialiser i à 0 (note qu'en cas de comportement indéterminé, tout peut arriver). Cela provoque donc une boucle infinie.

Pour le reste de ton programme, il y a quelques habitudes pas terribles dont il convient de se débarrasser. Par exemple, utiliser des constantes numériques pour représenter des caractères manque singulièrement de lisibilité. Tu pourrais plutôt utiliser des constantes caractère ('a', 'z', etc.). Mieux encore, pour éviter de réinventer la roue, tu as, à ta disposition, la fonction isupper de <ctype.h>.

Bon codage !