Discussion :

[ImplicitNone]

Bonjour

je suis un gros débutant

J'ai un petit probeme avec une boucle "for" , je veux creer une boucle toute simple qui me revoie le nombre de mots d'une chaine de caractéres , mais la valeur "mots" me renvoie toujours 1

Code :

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

1
2
3
4
5
6
7
8
9
10
11
char* phrase;
    printf("Veuillez enter une phrase \n");
    scanf("%s", phrase);
 
 
    for (i=0 ; i<strlen(phrase) ; i++) {
        if (*(phrase+i)== ' ' ) {
           mots++;
           }
        }
    printf("%ld \n", mots+1);

Merci

[souviron34]

comment mots est-il défini et initalisé ??

[Tusbar]

Bonsoir,

Alors déjà, la fonction scanf n'alloue pas de mémoire. Il convient donc de lui passer, soit une chaîne déjà allouée (man malloc(3)), soit un tableau de char déclare comme ceci :

Code :

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

char phrase[1024];

Après, utiliser scanf n'est pas vraiment une bonne idée, il vaut mieux utiliser fgets(3) qui est plus sure.

La variable i n'est pas déclarée.

Ensuite, il faut éviter le strlen dans le for, a chaque passage de ta boucle, la longueur de la chaîne est recalculée.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
...
 
int    size;
 
size = strlen(tachaine);
for (i  = 0; i < size; i++)
{
   ...
}
 
...

Puis il faudrait déclarer, et initialiser mots (un int et a 0 je présume).

Et hop tout ça dans une fonction.

Ensuite niveau algorithme tu peux mieux faire :-)