Forum |  HardWare.fr | News | Articles | PC | S'identifier | S'inscrire | Shop Recherche
1355 connectés 

  FORUM HardWare.fr
  Programmation
  C

  Un petit programme pour compter les voyelles.

 
 


 Mot :   Pseudo :  
 
Bas de page
Auteur Sujet :

Un petit programme pour compter les voyelles.

n°577844
Master_Jul
Posté le 27-11-2003 à 21:23:05  profilanswer
 

Je cherche à compter les voyelles d'une chaine, le "cout" qui me sert à voir ce qui se passe me montre bien que le programme trouve ce qu'il faut mais il plante avant le retour de la fonction, quel est le problème ?
 

Code :
  1. int NbVoyelles (char * szPhrase)
  2. {
  4. int xvoy=0;
  5. char voyelles[]="aeiouyAEIOUY";
  7.     char *p;
  9. p=strpbrk(szPhrase,voyelles);
  11. while (*p!='\0')
  12. {
  13.  p=strpbrk(p,voyelles);
  14.  cout <<p<<endl;
  15.  p++;
  16.  xvoy++;
  17. }
  19. return(xvoy);
  20. }
  24. void main(void)
  25. {
  26. char chaine[] = "Il me semble qu il pleut dehors.";
  27.     int Voyelles;
  29. Voyelles=NbVoyelles(chaine);
  31. cout <<"La chaine est : "<<chaine<<endl;
  32. cout <<"Le nombre de voyelles est : "<<Voyelles<<endl;
  34. }


Message édité par Master_Jul le 28-11-2003 à 00:07:26
mood
Publicité
Posté le 27-11-2003 à 21:23:05  profilanswer
 

n°577858
SquiZZ
Posté le 27-11-2003 à 21:37:31  profilanswer
 

regarde la doc de strpbrk() (la partie sur la valeur de retour)

n°577865
gilou
Modérateur
Modzilla
Posté le 27-11-2003 à 21:42:52  profilanswer
 

Et si la phrase test n'a pas de voyelle, il va y avoir pb avec ce code...
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
n°577867
Master_Jul
Posté le 27-11-2003 à 21:48:52  profilanswer
 

ça retourne un pointeur sur la première occurence, j'ai essayé de me baser sur l'exemple qu'ils donnent :
 
Return Value
Returns a pointer to the first occurrence of any character from strCharSet in string, or a NULL pointer if the two string arguments have no characters in common.

Code :
  1. int main( void )
  2. {
  3.    char string[100] = "The 3 men and 2 boys ate 5 pigs\n";
  4.    char *result;
  5.    /* Return pointer to first 'a' or 'b' in "string" */
  6.    printf( "1: %s\n", string );
  7.    result = strpbrk( string, "0123456789" );
  8.    printf( "2: %s\n", result++ );
  9.    result = strpbrk( result, "0123456789" );
  10.    printf( "3: %s\n", result++ );
  11.    result = strpbrk( result, "0123456789" );
  12.    printf( "4: %s\n", result );
  13. }


 
Output
1: The 3 men and 2 boys ate 5 pigs
 
2: 3 men and 2 boys ate 5 pigs
 
3: 2 boys ate 5 pigs
 
4: 5 pigs
 
Edit : Pour le problème "d'aucune correspondance", je peux vérifier si un NULL est présent avant le while et retourner 0.


Message édité par Master_Jul le 27-11-2003 à 21:49:47
n°577872
SquiZZ
Posté le 27-11-2003 à 21:57:18  profilanswer
 

Master_Jul a écrit :

ça retourne un pointeur sur la première occurence, j'ai essayé de me baser sur l'exemple qu'ils donnent :
 

Code :
  1. int main( void )
  2. {
  3. // snip
  4. }


 


 
et que ce passe-t-il si tu rajoutes  

Code :
  1. result = strpbrk( result, "0123456789" );
  2.        printf( "5: %s\n", result );


 
à la fin du programme d'exemple ?
 
[edit] damned le printf de crosoft est protégé contre les pointeurs nuls pour les chaines de caractères, c'est même pas drôle ca plante pas.


Message édité par SquiZZ le 27-11-2003 à 22:12:08
n°577873
chrisbk
-
Posté le 27-11-2003 à 21:57:35  profilanswer
 

c celui pour ce soir minuit ?

n°577882
Master_Jul
Posté le 27-11-2003 à 22:15:32  profilanswer
 

SquiZz : ça renvoit la même chose que 4
 
chrisbk : si seulement...
 
il faut que je donne tableau des valeurs et arguments, plus définition, plus faire un programme d'exemple pour strcpy, strcat, strncat, strcmp, strncmp, strrev, strset, strstt...  
 
Je n'ai d'exemple imposé que pour strpbrk, pour les autres, je peux inventer ce qui me branche alors ça va.

n°577892
SquiZZ
Posté le 27-11-2003 à 22:32:03  profilanswer
 

pourquoi ce programme pourrait-il planter ?
(hint : pointeur, NULL)
 
est ce que  

Code :
  1. p=strpbrk(p,voyelles);


 
peut te renvoyer un pointeur sur le caractère de fin de chaîne '\0' alors qu'il ne fait pas partie des caractères que tu recherches ?
 
 
Quand est ce que  

Code :
  1. p=strpbrk(p,voyelles);


te retourne NULL ?

n°577897
Taz
bisounours-codeur
Posté le 27-11-2003 à 22:47:24  profilanswer
 

putain je me barre une soirée et c'est le bordel ! et les std::string, et std::string::find_first_of, etc :o

n°577899
SquiZZ
Posté le 27-11-2003 à 22:48:19  profilanswer
 

Taz a écrit :

putain je me barre une soirée et c'est le bordel ! et les std::string, et std::string::find_first_of, etc :o


 
void main(void) :ange:  
 

mood
Publicité
Posté le 27-11-2003 à 22:48:19  profilanswer
 

n°577901
Master_Jul
Posté le 27-11-2003 à 22:54:05  profilanswer
 

Ah ouais OK, s'il ne regarde pas fin de chaine, je fais un  
while (i<strlen(chaine))  
{
i++;
...
}
 
Je peux rencontrer un NULL quand il n'y a pas/plus de correspondance, mais est qu'un NULL peut faire planter le pg ?

n°577907
Master_Jul
Posté le 27-11-2003 à 22:58:47  profilanswer
 

Taz a écrit :

putain je me barre une soirée et c'est le bordel ! et les std::string, et std::string::find_first_of, etc :o


 
 :heink:  
 
J'ai pas encore appris à me servir de ça. :/  
 
Je pense pas que ça soit vital pour mon programme.

n°577908
SquiZZ
Posté le 27-11-2003 à 22:59:42  profilanswer
 

Master_Jul a écrit :

Ah ouais OK, s'il ne regarde pas fin de chaine, je fais un  
while (i<strlen(chaine))  
{
i++;
...
}


ce qui marchera uniquement si toutes les lettes de ta chaine sont des voyelles.
 
 

Master_Jul a écrit :


Je peux rencontrer un NULL quand il n'y a pas/plus de correspondance, mais est qu'un NULL peut faire planter le pg ?  


un NULL ne fait pas planter un programme, déréférencer un pointeur nul, si.
 
et si il n'y a plus de correspondance, ca ne voudrait pas dire par un hazard improbable qu'il ne reste plus de voyelles dans ta chaine ?

n°577910
Master_Jul
Posté le 27-11-2003 à 23:07:06  profilanswer
 

Je ne comprend pas le "déférencement de pointeur"
 
On écrit ça comme ça ?
 
 while (*p!='NULL')
 {
  p=strpbrk(p,voyelles);
  cout <<p<<endl;
  p++;
  xvoy++;
 }
 
ou
 
 while (p!='NULL')

n°577916
SquiZZ
Posté le 27-11-2003 à 23:13:28  profilanswer
 

Master_Jul a écrit :

Je ne comprend pas le "déférencement de pointeur"
 
On écrit ça comme ça ?
 
 while (*p!='NULL')
 {
  p=strpbrk(p,voyelles);
  cout <<p<<endl;
  p++;
  xvoy++;
 }
 
ou
 
 while (p!='NULL')


 
 [:albator7k]  :pfff:  
 
déréférencement de pointeur =>  

Code :
  1. char *p;
  2. *p; /* ca c'est un déréférencement de pointeur


 
Y a de l'idée dans la première solution sauf qu'elle compile pas, c'est pas comme ca qu'on teste un pointeur nul, mais alors pas du tout.
et en plus ca marcherait pas même si ca compilait.
 
test de pointeur :

Code :
  1. char *p;
  3. p = NULL;
  5. if(p == NULL)
  6. {
  7.   /* le pointeur est nul */
  8. }
  10. if(!p)
  11. {
  12.   /* le pointeur est nul */
  13. }


 
(PS : c'est un troll ou quoi ?)

n°577922
Master_Jul
Posté le 27-11-2003 à 23:20:40  profilanswer
 

Je suis conscient d'être un gros boulet mais j'ai vraiment du mal sur ce coup. :/
 
Je change d'idée, mais je sens que ça devient de plus en plus con, je fais une boucle infinie que je casse quand je rencontre un NULL suivant la syntaxe que tu m'as donné :
 

Code :
  1. char *p;
  2. p=strpbrk(szPhrase,voyelles);
  4. while (1)
  5. {
  6.  p=strpbrk(p,voyelles);
  7.  cout <<p<<endl;
  9.  if (p == NULL) return(xvoy);
  10.  p++;
  11.  xvoy++;
  13. }


 
Maintenant je vois pas le problème avec le pointeur, quand je fais un :
 
p=strpbrk(p,voyelles);
Ca veut dire que je stocke un pointeur dans p qui en est déjà un... ah c'est peut être ça le hic.

n°577928
Master_Jul
Posté le 27-11-2003 à 23:25:49  profilanswer
 

Je trouve pas, faut que je rende ce truc avant minuit :'(.

n°577929
SquiZZ
Posté le 27-11-2003 à 23:27:23  profilanswer
 

bon si c'est un devoir => pas de commentaires => 0/20.
                       => return dans un while(1) => 0/20.
 
pourquoi ta première solution n'aurait pas marché :
 

Code :
  1. p=strpbrk(szPhrase,voyelles);
  2. while (p != NULL) /*c*/
  3. {
  4.   p=strpbrk(p,voyelles); /*a*/
  5.   cout <<p<<endl;
  6.   p++; /*b*/
  7.   xvoy++;
  8. }


 
a la fin de la recherche, en /*a*/ on se retrouve avec un pointeur nul, super.
en /*b*/, pas de bol on ajoute 1 au pointeur.
donc en /*c*/ le test foire parce que que le pointeur ne vaut pas nul.
 
une solution qui marche [edit ou qui devrait marcher ;)]:

Code :
  1. p=strpbrk(szPhrase,voyelles);
  2. while (p != NULL) /*c*/
  3. {
  4.   cout <<p<<endl;
  5.   p++; /*b*/
  7.   p=strpbrk(p,voyelles); /*a*/
  8.   xvoy++;
  9. }


 
[edit]viré le commentaire, dis moi pourquoi ca devrait marcher.


Message édité par SquiZZ le 27-11-2003 à 23:32:01
n°577931
Master_Jul
Posté le 27-11-2003 à 23:35:02  profilanswer
 

A tout hasard (j'avoue), j'avais essayé de mettre le p++; avant de tester mais ça devait pas être au moment où je testais la bonne chose (cf. le !='\0') :/
 
En tout cas, j'ai compris mes erreurs, merci pour toutes tes explications, je promet de me creuser un peu plus la tête la prochaine fois. Je devrai utiliser un peu plus le debugger en pas à pas à mon avi.
 
Encore merci !
 
Edit : J'ai vu avant l'edit  [:cupra] mais j'aurai compris, j'ai déjà rencontré ce genre de bourde avec les chaines quand on récupèrait la longueur de celle-ci "à la main" en recherchant le terminator.


Message édité par Master_Jul le 27-11-2003 à 23:37:32
n°577936
Taz
bisounours-codeur
Posté le 27-11-2003 à 23:45:00  profilanswer
 

quelqu'un pour déplacer ça dans la cat C ?

n°577938
SquiZZ
Posté le 27-11-2003 à 23:47:46  profilanswer
 

Taz a écrit :

quelqu'un pour déplacer ça dans la cat C ?


tiens j'avais pas fait gaffe à la cat.
Quoi que c'est a moitié du C++, y a des 'cout'  [:ddr555]

n°577939
Taz
bisounours-codeur
Posté le 27-11-2003 à 23:49:30  profilanswer
 

SquiZz a écrit :

Quoi que c'est a moitié du C++, y a des 'cout'  [:ddr555]  

:pfff:

n°577944
gilou
Modérateur
Modzilla
Posté le 28-11-2003 à 00:09:23  profilanswer
 

SquiZz a écrit :


tiens j'avais pas fait gaffe à la cat.
Quoi que c'est a moitié du C++, y a des 'cout'  [:ddr555]  


J'ai deplacé parce que sinon le Taz nous faisait une attaque d'apoplexie :D
Z'etes dur avec lui de poster ca dans la cat C++, alors que c'est du C plusplus-isant.
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
n°577947
gilou
Modérateur
Modzilla
Posté le 28-11-2003 à 00:16:52  profilanswer
 

Bon sinon, pour M_J, j'explicite ma remarque:

Code :
  1. ......
  2.     p=strpbrk(szPhrase,voyelles);
  3.    
  4.      while (*p!='\0')
  5. ..........


Si tu n'as pas de voyelle dans szPhrase p sera mis a NUL. Or ton test se fait sur *p  :non:  :non:  :non:  
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
n°577953
gilou
Modérateur
Modzilla
Posté le 28-11-2003 à 00:45:04  profilanswer
 

Tiens, un exemple de solution en pur C  
Reecris donc ca ensuite en vrai C++ pour faire plaisir a Taz
J'ai modifie certains de tes identificateurs pour les mettre plus conformes aux habitudes du code C
Au fait, main est pas du prototype que tu avait mis, meme si ton compilo l'accepte
 

Code :
  1. #include <string.h>
  2. #include <stdio.h>
  5. int compteVoyelles (char *szPhrase)
  6. {
  8. int compteur = 0;
  10. // si pas de chaine ou chaine vide, on a pas besoin de compter
  11. if (szPhrase && *szPhrase)
  12. {
  13.  char voyelles[] = "aeiouyAEIOUY";
  14.      char *p = szPhrase;
  16.  while (p = strpbrk(p, voyelles))
  17.  {
  18.   // printf("%s\n", p); // mis ton test de debug en commentaires
  19.   compteur++;
  20.   p++;
  21.  }
  22. }
  24. return(compteur);
  25. }
  29. int main(int argc, char *argv[])
  30. {
  31. char chaine[] = "Il me semble qu il pleut dehors.";
  32.     int  nbVoyelles;
  34. nbVoyelles = compteVoyelles(chaine);
  36. printf("La chaine est : %s\n", chaine);
  37. printf("Le nombre de voyelles est : %d\n", nbVoyelles);
  39. return 0;
  41. }


 
A+,


---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
n°577959
Taz
bisounours-codeur
Posté le 28-11-2003 à 01:17:16  profilanswer
 

pour ceux qui veulent je faire en C++, plusieurs solutions simples :
- une boucle avec std::string::find_first_of
- std::count_if + ce_qu'_il_faut

n°577990
Konar
Posté le 28-11-2003 à 04:58:20  profilanswer
 

j'ai bon ?
 

Code :
  1. #include <string>
  2. #include <algorithm>
  4. using namespace std;
  6. string Voyelles = "aeiouyAEIOUY";
  8. bool Bordel(char c)
  9. {
  10. return (Voyelles.find(c) != string::npos);
  11. }
  13. int main()
  14. {
  15. string str = "Dans ton cul, konar !";
  16. size_t count = count_if(str.begin(), str.end(), Bordel);
  18. return 1;
  19. }

n°578014
Taz
bisounours-codeur
Posté le 28-11-2003 à 08:57:38  profilanswer
 

oui sauf la variable globale (de visibilité externe qui plus est) qui est très laide. et quand on peut le faire, on utilise le constructeur avec argument et pas l'initialisation par copie. quelque parenthèse en trop


Message édité par Taz le 28-11-2003 à 08:57:59
n°578224
Konar
Posté le 28-11-2003 à 13:01:44  profilanswer
 

ouais la globale j'en étais pas très fier, et pour les parenthèses c'était une norme de mon ancien bahut de merde, qui disait qu'il fallait en mettre après chaque mot clé (for, sizeof, if, return, etc).
 
sinon ok pour le constructeur, j'y pense jamais mais faudrait.
 
et tout compte fait cette solution me semble mieux :
(enfin plus simple, et y a pas une fonction Bordel() qui traine)
 

Code :
  1. size_t Count(const string &str)
  2. {
  3. size_t res = 0;
  4. string Voyelles("aeiouyAEIOUY" );
  5. for (string::const_iterator i = str.begin(); i != str.end(); i++)
  6.  if (Voyelles.find(*i) != string::npos)
  7.   res++;
  9. return res;
  10. }

n°578255
antp
Super Administrateur
Champion des excuses bidons
Posté le 28-11-2003 à 13:30:38  profilanswer
 

Et les voyelles accentuées, faut les compter aussi ? :o


---------------
mes programmes ·· les voitures dans les films ·· apprenez à écrire
n°578264
chrisbk
-
Posté le 28-11-2003 à 13:35:19  profilanswer
 

antp a écrit :

Et les voyelles accentuées, faut les compter aussi ? :o


 
ben øüì, cä me påräìt evídent

n°578283
antp
Super Administrateur
Champion des excuses bidons
Posté le 28-11-2003 à 13:48:03  profilanswer
 

Y a des cas sympathiques [:figti] par ex en néerlandais "ij" peut s'écrire "ÿ", ça fait une voyelle + une consonne :D


---------------
mes programmes ·· les voitures dans les films ·· apprenez à écrire
n°578619
SquiZZ
Posté le 28-11-2003 à 20:29:39  profilanswer
 

quelqu'un pour déplacer ça dans la cat C++ ? (spéciale dédicace à Taz  [:ddr555]^2

n°1927744
keek007
Posté le 29-09-2009 à 18:12:38  profilanswer
 

Salut, moi aussi je dois faire un compteur de voyelle dans une phrase se terminant par *
Voila deja ce que j'ai fais.
 

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  4. int main()
  5. {
  6.   char phrase[100];
  7.   int c = 0;    // Compteur de voyelle
  8.   int d = 0; 
  9.  
  10.   printf("Rentrez une phrase se terminant par *\n\n" );
  11.   scanf("%s", &phrase);
  12.  
  13.   while ( phrase[d] != '*' )
  14.    {
  15.         if ( phrase[d] == 'a' || phrase[d] == 'e' || phrase[d] == 'i' || phrase[d] == 'o' || phrase[d] == 'u' || phrase[d] == 'y' )
  16.           {
  17.                     c++; // Incrementation du compteur de voyelles
  18.                     d++; // Incrementation du tableu
  19.           }
  20.         else
  21.           { d++; }
  22.    }
  23.  
  24.  
  25.   printf("Il y a %i voyelles dans votre phrase !\n", c);
  26.   system("PAUSE" );
  27.   return 0;
  28. }


 
Voila , sa marche parfaitement si je colle tout mais des que je met des espaces dans ma phrase sa m'affiche un nombre de voyelle erronés pourquoi cela ? Merci de vos explication !

Message cité 1 fois
n°1927774
Trap D
Posté le 29-09-2009 à 21:36:18  profilanswer
 

Renseigne toi sur l'utiliisation du scanf !
La saisie des chaines au clavier se fait avec fgets(buf, sizeof buf, stdin).

n°1928100
Sve@r
Posté le 30-09-2009 à 23:25:32  profilanswer
 

keek007 a écrit :


Voila deja ce que j'ai fais.
 

Code :
  1. ...
  2.   char phrase[100];
  3. ...
  4.   scanf("%s", &phrase);
  5. ...
  6. if (...)
  7. {
  8.      ...
  9.      d++;
  10. }
  11. else
  12. { d++; }


 
Voila , sa marche parfaitement


Supaire !!! :sarcastic:


Message édité par Sve@r le 30-09-2009 à 23:26:17

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
n°1928120
licaprout
Posté le 01-10-2009 à 02:09:16  profilanswer
 

Salut les bizus de tek1
J'ai pas regardé si la réponse a été postée, trop de flood.
 
en C pur et dur sans fonction de la bibli pour compter le nombre de voyelles d'une phrase on a : (a la norme en plus !)
 

Code :
  1. int compteur(char *str)
  2. {
  3.  char *voyelle;
  4.  int compteur;
  5.  int i;
  6.  
  7.  compteur = 0;
  8.  voyelle = "aeiouyAEIOUY";
  9.  while (*str != '\0')
  10.  {
  11.   i = 0;
  12.   while (voyelle[i] != '\0')
  13.    if (voyelle[i++] == *str)
  14.     compteur++;
  15.   str++;
  16.  }
  17.  return (compteur);
  18. }


Voilou


Message édité par licaprout le 01-10-2009 à 02:10:15
mood
Publicité
Posté le   profilanswer
 


Aller à :
Ajouter une réponse
  FORUM HardWare.fr
  Programmation
  C

  Un petit programme pour compter les voyelles.

 

Sujets relatifs
[PHP] petit pb avec une regexp[C++] Découpe d'un programme en plusieurs fichiers
petit soucis avec un preg_replaceCompter les caracteres
Petit Probleme sur les ExceptionsCompteur PHP - Comment ne pas compter plusieurs fois la même connectio
Cherche Programme de Visualistation/Debeuging de code XML/XSLprogramme qui communique avec le port serie ???
récupérer le résultat d'un programme dans un .bat[java] newbye : rentrer des valeurs dans mon programme à l'exécution
Plus de sujets relatifs à : Un petit programme pour compter les voyelles.

Forum MesDiscussions.Net, Version 2010.2
(c) 2000-2011 Doctissimo
Page générée en 0.091 secondes

Copyright © 1997-2022 Hardware.fr SARL (Signaler un contenu illicite / Données personnelles) / Groupe LDLC / Shop HFR