Discussion :

[rikudou]

salut,
voila j'ai commencé a réalisé ce programme mais je me suis coincé :/
Voila ce que j'ai :

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
33
34
35
36
37
38
#include <stdio.h>
#include <string.h>
#define Max 200
 
void lirechaine (char Tab[Max])
{
    int i=0;
    char c;
    c=getchar();
    while((c!='\n') && (i<Max))
    {
        Tab[i++]=c;
        c=getchar();
    }
    Tab[i]='\0';
}
int main()
{
    char Tab[Max], TAB[Max];
    int x=0, i, j, k;
    printf("entrez une chaine de caracteres :\n");
    lirechaine(Tab);
    for(i=0; i<strlen(Tab); i++)
    {
            for (j=0; j<strlen(Tab); j++){
            if (Tab[i]==Tab[j])
            x++;
            for (k=0; k<i; k++){
            TAB[k]=Tab[k];
            }
            for (k=i; k<strlen(Tab); k++){
            TAB[k]=Tab[i+1];
            }
        }   
        printf("%d fois la lettre %c\n", x, Tab[i]);
        x=0;
    }
}

pouvez vous le corriger et le compléter, Merci d'avance.

[Neckara]

Sois un peu plus précis s'il te plait.

Ton programme compte le nombre de lettre ou de caractères?
Est-ce qu'il compte le nombre de lettres/caractères différents ou ils les compte tous?

Dis-nous aussi ce qui te coince : une erreur? un soucis algorithmique?

Soigne aussi l'indentation.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
for(i=0; i<strlen(Tab); i++)
    {
            for (j=0; j<strlen(Tab); j++){
                   if (Tab[i]==Tab[j])
                           x++;
                   for (k=0; k<i; k++)
                          TAB[k]=Tab[k];
                   for (k=i; k<strlen(Tab); k++)
                          TAB[k]=Tab[i+1];
        }

Dans ta boucle de lecture, si i == Max tu sors mais tu auras en dessous

Code :

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

1
2
 
tab[Max] = '\0';

Tu risques donc un segfault;

Sinon, à ta place, je trierais la chaîne de caractère par ordre croissant.
Ensuite, il ne reste plus qu'à faire un parcours en incrémentant une variable dès qu'on passe à une lettre/caractère différent.
Ce sera plus simple et plus rapide.


EDIT : je pense avoir compris ce qu'il te bloque :

lorsque tu t'aperçoit que tab[i] == tab[j];
Au lieu d'utiliser un TAB[];
fait un simple :

Code :

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

1
2
3
4
5
6
7
8
 
int k = j;
do
{
        tab[k] = tab[k+1];
        k++;
}
while (tab[k+1] != '\0')

[anacharsis]

salut !

s'il s'agit d'un exercice pourquoi ne pas utiliser un simple

Code :

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

gets(Tab)

pour compter des lettres tu n'as besoin que de 256 compteurs, un par lettre, que tu initialises à 0 avant de lire ta chaîne, lettre par lettre, en incrémentant le compteur correspondant dans une simple boucle.
commence par un programme qui fonctionne pour les 127 premiers caractères (sans accent), puis, si nécessaire, tiens compte des caractères < 0 (mais ça complique un peu la sauce).

A+

[aslo92]

Bonjour,

la solution d'anacharsis est la meilleure à mon avis.
J'aurai fait la même chose si j'avais eu à le coder en utilisant un tableau de 256 int pour stocker le nombre d'occurences de chaque lettre.

Par contre si je peux me permettre une remarque sur ton code:

Evites de déclarer des variables différentes en les nommant : TAB, tab.
On pourrait aussi ajouter Tab, tAb, taB, etc ...
Ca n'améliore pas la lecture du code et c'est source d'erreurs difficiles à trouver par la suite!

A+

[nabil_cube]

je n'ai pas tres bien compris se que tu veux faire, mais je crois que tu veux compter pour chaque lettre de ta chaine le nombre de fois qu'elle est cité dans la chaine:

genre pour la chaine : "manger manger" t'aura :
la lettre m est cité 2 fois.
la lettre a est cité 2 fois.
la lettre n est cité 2 fois.
la lettre g est cité 2 fois.
la lettre e est cité 2 fois.
la lettre r est cité 2 fois.

ton code est déjà bon a quelque détail prés, bien sure ta d'autre façon beaucoup plus élégante mais je croix que le but de l’exercice est l’algorithmique

je t'ai mis des commentaire.

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
 
#include <stdio.h>
#include <string.h>
#define Max 200
 
void lirechaine (char Tab[Max])
{
int i=0;
char c;
c=getchar();
while((c!='\n') && (i<Max))
{
Tab[i++]=c;
c=getchar();
}
Tab[i]='\0';
}
int main()
{
char Tab[Max];	// suprime le deuxieme tableau qui ne serta a rien a mon avis 
int x=0, i, j, k,repete = 0;	// rajout d'une variable pour tester la répétition 
printf("entrez une chaine de caracteres :\n");
lirechaine(Tab);
for(i=0; i<strlen(Tab); i++)
{
for (j=0; j<strlen(Tab); j++){
if (Tab[i]==Tab[j])
x++;
 
// je ne voi pas en quoi sert cette partie du code donc je le met en commentaire
// for (k=0; k<i; k++){
// TAB[k]=Tab[k];
// }
// for (k=i; k<strlen(Tab); k++){
// TAB[k]=Tab[i+1];
// }
} 
 
// pour éviter la répetition dans l'affichage du résultats 
for(int x = 0; x<i;x++){
if(Tab[i]==Tab[x])
repete++;}
// pour ne pas compter les éspaces 
if(repete == 0 && Tab[i] != ' ') printf("%d fois la lettre %c\n", x, Tab[i]);
x=0; repete=0;
}
 
getchar();	// evite de fermer le programme avant le retour chariot 
}