Ou est l'errreur

Marsh Posté le 03-01-2007 à 19:13:45

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

Reply

Marsh Posté le 03-01-2007 à 19:13:45

Reply

Marsh Posté le 03-01-2007 à 20:25:50

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'))

Reply

Marsh Posté le 03-01-2007 à 20:31:28

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

Reply

Marsh Posté le 03-01-2007 à 21:29:16

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.

Reply

Marsh Posté le 03-01-2007 à 22:02:56

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

Reply

Marsh Posté le 03-01-2007 à 22:59:47

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.

Reply

Marsh Posté le 04-01-2007 à 01:16:17

merci à tous pour votre aide et vos conseils

Reply

Marsh Posté le 04-01-2007 à 01:25:59

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/

Reply

Marsh Posté le 04-01-2007 à 10:53:18

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

Reply

Marsh Posté le 04-01-2007 à 12:54:28

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.

Reply

Marsh Posté le 04-01-2007 à 12:54:28

Reply

Marsh Posté le 04-01-2007 à 13:01:06

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

Reply

Marsh Posté le 04-01-2007 à 13:13:45

rufo a écrit :

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...

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.

Reply

Marsh Posté le 04-01-2007 à 14:17:45

Sve@r a écrit :

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"

+1

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...

+1

---------------
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/

Reply

Ou est l'errreur

Sujets relatifs:

Leave a Replay