Ou est l'errreur

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Ou est l'errreur

samos38

A soif d'apprendre :P

bonjour tout le monde et bonne année !

voila le programme que je réalise :

Code :

/* TP8 Chaine de caracteres */
#include <stdio.h>
#include <string.h>
#define MAX 50
int control(char c[])
{
int i,con=0,l;
l=strlen(c);
for(i=0;i<l;i++)
{
if((c[i]>='A'&&'Z'<=c[i])||(c[i]>='a'&&'z'<=c[i])||c[i]=='-')
{
con=1;
}
}
return con;
}
void saisie(char n[],char p[])
{
do
{
printf("entrez votre nom : " );
scanf("%s",n);
}while(control(p)==0);
do
{
printf("entrez votre prenom : " );
scanf("%s",p);
}while(control(n)==0);
}
void minus(char c[])
{
int i,max;
max=strlen(c);
for(i=0;i<max;i++)
{
if((c[i]>='A')&&('Z'<=c[i]))
c[i]=c[i]+32;
}
}
int main(void)
{
//int cont;
//char c[MAX];
//scanf("%s",&c);
//cont=control(c);
//printf("\n%d\n",cont);
char n[MAX],p[MAX];
saisie(n,p);
minus(n);
minus(p);
printf("\n%s\n",n);
printf("\n%s\n",p);
system("PAUSE" );
return 0;
}

Ce programme est sencé mettre nom et prénom en minuscule , même si on les a donné avec des majuscules .
Mais ca ne marche pas ca me met n'importe quoi pour les minuscules et je n'arrive pas à retrouver l'erreur.

Merci d'avance pour votre aide.

Message cité 1 fois
Message édité par samos38 le 03-01-2007 à 19:14:23

---------------
débute en C

Publicité

olivthill

Une petite erreur qui saute aux yeux :

if((c[i]>='A')&&('Z'<=c[i]))

est équivalent à

if((c[i]>='A')&&(c[i]>='Z'))

alors que cela devrait être

if((c[i]>='A')&&(c[i]<='Z'))

Ange_blond

Alors voila faut pas chercher loin :

L'idée est bonne mais pas approfondie :

Si au lieu de faire :

Code :

c[i]=c[i]+32;

Tu fesais :

Code :

c[i]=c[i]-32;

ca marcherai...

Autre solution : utilise les fonctions de String.h pour mettre en minuscule... elles existent !

Edit : olivthill tu as raison, il y avait un autre probleme ici aussi mais ca marche quand meme our les lettres

Message cité 2 fois
Message édité par Ange_blond le 03-01-2007 à 20:32:58

Sve@r

Ange_blond a écrit :

Si au lieu de faire :

Code :

c[i]=c[i]+32;

Tu fesais :

Code :

c[i]=c[i]-32;

ca marcherait...

Si on omet les fonctions de <string.h>, c'est encore plus parlant d'écrire :

Code :

c[i]=c[i] - 'A' + 'a'

samos38 a écrit :

Code :

int control(char c[])
{
int i,con=0,l;
l=strlen(c);
for(i=0;i<l;i++)
{
if((c[i]>='A'&&c[i]<='Z')||(c[i]>='a'&&c[i]<='Z')||c[i]=='-')
{
con=1;
}
}
return con;
}

Personnellement je quitte une fonction dès que sa raison d'être n'existe plus.

Code :

int control(char c[])
{
int i, l;
l=strlen(c);
for(i=0;i<l;i++)
{
// J'essaye aussi d'aérer un peu les syntaxes
if ((c[i] >= 'A' && c[i] <= 'Z') || (c[i] >= 'a' && c[i] <= 'Z') || c[i] == '-')
return 1;
}
return 0;
}

Mais cette façon d'écrire est contraire aux principes de la prog structurée et certains (profs ???) peuvent ne pas l'apprécier...

Message édité par Sve@r le 03-01-2007 à 21:37:27

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

Ange_blond

Ben plusieurs return dans une même fonction peut etre considéré comme une erreur de programmation voire algorithmique même... mais bon, chacun ses preferences...

Message cité 2 fois

Sve@r

Ange_blond a écrit :

Ben plusieurs return dans une même fonction peut etre considéré comme une erreur de programmation voire algorithmique même...

C'est là qu'il faut se montrer très prudent et ne pas tomber dans le... fanatisme.

Pour moi, les règles c'est bien "en général". Ensuite, tout en connaissant les règles et le danger associé, on peut, parfois, s'en affranchir (comme dans mon exemple concernant "control()" )...

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

samos38

A soif d'apprendre :P

merci à tous pour votre aide et vos conseils

Emmanuel Delahaye

C is a sharp tool

Ange_blond a écrit :

Autre solution : utilise les fonctions de String.h pour mettre en minuscule... elles existent !

Montre les !

Par contre, je peux te montrer celles de <ctype.h>...

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

rufo

Pas me confondre avec Lycos!

Ange_blond a écrit :

Ben plusieurs return dans une même fonction peut etre considéré comme une erreur de programmation voire algorithmique même... mais bon, chacun ses preferences...

ben suffit de remplacer le for par un while(con == 0) {} et comme ça, on a plus qu'un seul return...

par ailleurs, moi, j'ai toujours appris qu'il fallait mettre des () autour de chaque condition (je fais référence à la ligne 14 du code initial).

Message cité 1 fois
Message édité par rufo le 04-01-2007 à 13:01:38

Sve@r

rufo a écrit :

ben suffit de remplacer le for par un while(con == 0) {} et comme ça, on plus qu'un seul return...

Joli !!! :bounce:

rufo a écrit :

par ailleurs, moi, j'ai toujours appris qu'il fallait mettre des () autour de chaque condition (je fais référence à la ligne 14 du code initial).

C'est le problème de l'apprentissage. Tu subis obligatoirement les habitudes de ton mentor. Moi, le mien me disait "mettre des parenthèses ne fait que montrer que tu ne connais pas les priorités des opérateurs"

Ensuite, entre le "toujours" et le "jamais" on peut arriver à trouver un juste milieu qu'on finalise avec ses propres expèriences du C...

Message cité 2 fois
Message édité par Sve@r le 04-01-2007 à 12:58:16

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

Publicité

rufo

Pas me confondre avec Lycos!

Sve@r a écrit :

Joli !!! :bounce:

C'est le problème de l'apprentissage. Tu subis obligatoirement les habitudes de ton mentor. Moi, le mien me disait "mettre des parenthèses ne fait que montrer que tu ne connais pas les priorités des opérateurs"

Ensuite, entre le "toujours" et le "jamais" on peut arriver à trouver un juste milieu qu'on finalise avec ses propres expèriences du C...

quand je parlais du while, on est bien d'accord que la condition n'était pas complète, hein. C'était pour l'idée. Il est bien évidement qu'il faut tenir compte de la longueur de la chaîne Sinon, on va avoir un petit pb si "con" ne passe jamais à 1...

Message cité 1 fois

Sve@r

rufo a écrit :

Aucun sous-entendu - J'ai réellement trouvé ton idée jolie. Tu as réussi à trouver un juste milieu entre la fonction originelle qui faisait du traitement inutile et la mienne qui ne respectait pas les standards conventionnels d'une fonction...
Pour le pb, c'est du détail...

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.

Emmanuel Delahaye

C is a sharp tool

Sve@r a écrit :

Citation :

Ensuite, entre le "toujours" et le "jamais" on peut arriver à trouver un juste milieu qu'on finalise avec ses propres expèriences du C...

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/

FORUM HardWare.fr

Programmation

Ou est l'errreur

Sujets relatifs
Plus de sujets relatifs à : Ou est l'errreur

Page générée en 0.091 secondes