Discussion :

[chicabonux]

Bonjour,

voilà l'exercice à résoudre

écrire un programme qui lit une chaîne de caractères CH et qui convertit toutes les majuscules dans des miniscules et vice -versa.
Le résultat sera mémorisé dans la même variable CH et affiché après la conversion

.

On me donne ce corrigé

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#include<stdio.h>
 
main()
{
	char CH[100];
	int I;
 
	printf("ecrire une chaine : ");
	gets(CH);
 
	for(I=0;CH[I];I++)
	{
		if(CH[I]>='A'&&CH[I]<='Z')
		CH[I]=CH[I]-'A'+'a';
 
		else if(CH[I]>='a'&&CH[I]<='z')
			CH[I]=CH[I]-'a'+'A';
	}
	printf("chaine convertie : %s\n",CH);
 
	return 0;
}

Je voudrais savoir pourquoi mon programme ne fonctionne pas et comment utiliser les fonctions tolower et toupper ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
#include<stdio.h>
 
main()
{
    char CH[100];
    int I,J;
 
    printf("ecrire une chaine : ");
    gets(CH);
 
    for(I=0;CH[I];I++)
    {
        I=toupper(CH);
        printf("%s",CH[I]);
    }
 
    for(J=0;CH[J];J++)
    {
        J=tolower(CH);
        printf("%s",CH[J]);
    }
    printf("\n");
 
    return 0;
}

[Lucien63]

Le langage C fait la difference pour les noms de variable avec des majuscules et minuscule.
Autrement dit i et I ne represente pas la même variable.

[souviron34]

parce que tu demandes d'afficher un caractère (CH[i]) avec le format d'une chaîne (%s).

Si tu mettais %c , ça marcherait

plsu la remarque précédente bien entendu

[Emmanuel Delahaye]

On me donne ce corrigé

Code :

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

1
2
3
4
5
6
7
8
9
#include<stdio.h>
 
main()
{
	char CH[100];
	int I;
 
	printf("ecrire une chaine : ");
	gets(CH);

"on" devrait apprendre à coder en C...

gets() est est un bug qu'il ne faut pas utiliser.

http://emmanuel-delahaye.developpez....e=Page6#LXXXII
http://emmanuel-delahaye.developpez....=Page6#LXXXIII

[zehle]

ton programme ne répond pas à la question posé
Exemple : CH = "Un Exemple";
L'exercice veux qu'on est en résultat la chaine "uN eXEMPLE"
Ton code lui ne faire rien de tout ça!!

un lien vers une explication de tolower et toupper

Code :

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

1
2
3
 
int toupper (int c); //convertit la lettre c en majuscule si c'est possible.
int tolower (int c);//convertit la lettre c en minuscule si c'est possible.

Je vais essayer d'imaginer ce que tu voulais faire en commentant ton code (juste la partie qui nous intéresse).

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
    for(I=0;CH[I];I++)
    {
        I=toupper(CH);
        //1ere erreur : CH[I] et non pas CH
        //2eme erreur : tu change I, I qui est TON ITERATEUR, qu'on ne TOUCHE JAMAIS!! Sauf si on sais ce qu'on est en train de faire
        printf("%s",CH[I]);
    }
// l'idée de ce bout de ton code est de changer chaque lettre de ta chaine en minuscule 
//une correction de ton idée (pour changer la chaine en minuscule) serait
    for(I=0;CH[I];I++)
    {
        CH[I]=toupper(CH[I]);
    }
    printf("%s",CH);// On affiche toute la chaine

La suite de ton code fait l'inverse, c'est à dire, mettre toute la chaine en Majuscule
Le résultat d'un tel code, après correction, serait (pour la chaine "Un Exemple")

Chaine d'entrée : Un Exemple
Résultats :
un exemple
UN EXEMPLE

FAUX!!

EDIT : les "I" sont bien en majuscules dans le code, mais vue que [I] en BB veut dire italique ...

[rilou]

Essais de placer des conditions telles que :

Code :

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

1
2
3
if(CH[i]<=z && CH[i]>=a) CH[i]=toupper(CH[i]);
else if(CH[i]<=Z && CH[i]>=A) CH[i]=tolower(CH[i]);
else return EXIT_FAILURE;

[gl]

Essais de placer des conditions telles que :

Code :

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

1
2
3
if(CH[i]<=z && CH[i]>=a) CH[i]=toupper(CH[i]);
else if(CH[i]<=Z && CH[i]>=A) CH[i]=tolower(CH[i]);
else return EXIT_FAILURE;

Tant qu'à faire, autant utiliser isupper() et islower() plutôt que les comparaisons à la main. C'est plus portable et plus cohérent.

[rilou]

Tant qu'à faire, autant utiliser isupper() et islower() plutôt que les comparaisons à la main. C'est plus portable et plus cohérent.

En effet :s Autant pour moi.