Isalpha [Fermé]

Signaler
-
Messages postés
5491
Date d'inscription
mercredi 15 septembre 2004
Statut
Contributeur
Dernière intervention
2 avril 2021
-
Bonjour,

j'ai fais ce programme qui vérifie si une chaine est alphbétique ou non
Le programme est compilable , mais , il accepte toute les chaines saisies .
voici mon code et merci d'avance
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
char k[100];
int i=0,test=0;

do
{printf("donner la phrase \n ");
gets(k);

    for(i=0;i<strlen(k)-1;i++)
    {
        if (!(isalpha(k[i])))
        {
            test=1;

       break; }
         }
}
while(test=1);
    return 0;
}

6 réponses

Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021
834
peut-être :
while(test==1); // mieux avec == au lieu de =
1
Merci

Quelques mots de remerciements seront grandement appréciés. Ajouter un commentaire

CCM 65492 internautes nous ont dit merci ce mois-ci

j'ai corrigé cette faute , mais ça pas marché quand meme
Messages postés
14785
Date d'inscription
vendredi 14 mars 2003
Statut
Modérateur
Dernière intervention
18 avril 2021
284
Essayes de voir ton code en pas à pas.
As-tu le code de la fonction : isalpha ?
malheureusement j'ai pas le code de la fonction isalpha
Messages postés
5491
Date d'inscription
mercredi 15 septembre 2004
Statut
Contributeur
Dernière intervention
2 avril 2021
932
@NHenry : c'est une fonction standard du C :

http://www.cplusplus.com/reference/cctype/isalpha/
Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021
834
Comment détermines-tu que ton programme accepte toute les chaines saisies?
Pourquoi fais-tu une boucle "do"?
Ce serait utile, surtout pour toi, que tu expliques ce que tu veux que ton programme fasse. Pourquoi pas en ajoutant un petit texte en commentaire de ton programme?
Quelle est l'utilisation exacte de ta variable "test"?
Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021
834
Pourquoi fais-tu une boucle "do while"?
Comment sais-tu que ton programme a accepté la chaîne ?
@yg_be
j'ai fais la boucle do while pour qu'il me redemande de lui donner une autre phrase tant que la condition n'est pas vérifiée et puisqu'il ne me redemande pas de lui donner une nouvelle phrase meme qu'elle contienne des chiffres et , *-+ ex... j'ai su qu'il a accepté toute les chaines saisies
@NHenry
isalpha teste si un caractere est alphabetique
Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021
834
Je pense que le programme continue tant que la chaîne est refusée...
Si le programme ne demande plus rien, c'est qu'il a refusé la première chaîne.
De plus, comme tu ne remets pas test à zéro quand tu refais gets() une seconde fois, le programme ne va jamais s’arrêter si la première chaîne est refusée.
Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021
834 >
Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021

Correction : Si le programme ne demande plus rien, c'est qu'il a accepté la première chaîne. (Il ne tient plus vraiment compte des chaines suivantes)
Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021
834
Montre nous de nouveau ton programme avec les deux corrections proposées.
Ajoute des printf dans ton programme, pour suivre ce qu'il fait et par où il passe.
Messages postés
15238
Date d'inscription
lundi 9 juin 2008
Statut
Contributeur
Dernière intervention
18 avril 2021
834
par exemple:
if (!(isalpha(k[i])))
        {
            test=1;
            printf("|%c| is NOT in alphabet\n", k[i] );
       break; }
else
printf("|%c| is in alphabet\n", k[i] );

et
printf("FINI\n");
return 0;
Messages postés
5491
Date d'inscription
mercredi 15 septembre 2004
Statut
Contributeur
Dernière intervention
2 avril 2021
932
Salut siwar,

Je n'ai pas testé ton code, mais quelques observations en plus de la correction du while en
while(test==1);
qui t'a déjà été indiquée par yg_be.

isalpha()
est dans ctype.h, il te faut donc un include sur cet entête standard.

gets()
à à bannir, car la saisie n'est pas sécurisée

fgets() devrait être utilisé

que l'un ou l'autre soit utilisé, si la chaîne saisie est de taille inférieure à l'espace alloué (moins 1),
'\n'
va être copié dans la chaîne capturée (retour à la ligne). Or, il ne s'agit pas d'un caractère alphabétique.

Il faudrait le prendre en compte dans le code, en retirant le
'\n'
ajouté par
fgets()
, et donc en terminant la chaîne à cet endroit si
fgets()
a inséré un retour à la ligne (à tester avec strchr()).

Vois par exemple : https://stackoverflow.com/questions/2693776/removing-trailing-newline-character-from-fgets-input pour du code utilisant
strchr()
pour retirer le retour à la ligne (et d'autres solutions alternatives moins habituelles).

Note que
scanf()
, en revanche, ne va pas capturer le retour à la ligne,.. mais elle n'est pas sans d'autres types de difficultés.


Dal
Messages postés
5491
Date d'inscription
mercredi 15 septembre 2004
Statut
Contributeur
Dernière intervention
2 avril 2021
932
correction, je n'avais pas vu que tu arrêtes ta boucle for à
i<strlen(k)-1
et non à
i<strlen(k)
.. du coup, la plupart du temps, tu vas effectivement t'arrêter avant le retour à la ligne.

cependant ton code ne marchera pas si on saisit 99 caractères et que le dernier est non alphabétique, car il ne sera pas testé,... alors, je pense que la suppression du
'\n'
s'il existe est une meilleure solution (proposée ci-dessus).

Je pense aussi que le texte "donner la phrase" est erroné, car une phrase peut contenir des espaces et de la ponctuation, qui sont des caractères non alphabétiques. En fait "tapez des lettres de l'alphabet" serait mieux.

De plus, les accents ne seront pas pris en compte si ta locale n'est pas réglée.

Dal