Convertir une chaîne en minuscules

Bonjour, ça plante et je ne comprend pas pourquoi.8O

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16

#include <stdio.h> #include <stdlib.h>   void maj(char *s){ int i; while(*s){ s[i]&=0xFD; /*En effet, 2^5 = 32 et dans la table ASCII les minuscules et les majuscules ont 32 bits de différences. */ } printf("%s",s); }   int main() { maj("Bonjour"); return 0; }

---

Essaies d'exécuter la fonction maj à la place de l'ordinateur sur une feuille de papier, ligne par ligne, en gardant trace des valeurs de chaque variable. Tu devrais vite mettre le doigt sur ce qui cloche.

Allez, un indice quand même: combien vaut "i" ? quand-est ce que la boucle s'arrête ?

Si tu es sur gcc, compile avec le flag -Wwrite-strings et tu verras l'autre problème.

Je rajouterais que ta fonction part du principe que la chaîne en entrée est intégralement en minuscule. Donc le B en majuscule va donner un accent grave (`), soit le caractère 0x60.

Non, vu qu'un & est utilisé plutôt qu'une soustraction, ça devrait aller.

Salut,

petite liste des trucs qui sautent aux yeux :
- tu fais un while *s sans bouger ton pointeur (à part à lui filer une chaine vide t'es parti pour une boucle infinie),
- dans ta boucle tu utilises un i comme index : il n'est ni initialisé, ni incrémenté,
- appliquer un mask FD doit pas avoir le même résultat que faire -32 ("32 bits de différences" ?)
- tu appelles maj avec une constante, du coup ta fonction va pas trop aimer réécrire dedans.

Et oui, attention à n'appliquer le traitement que pour les char qui représentent une minuscule.

Bon dev.

edit : -32 (mask 0xDF) et pas +32 (flag 0x20), du moins dans le cas des caractères alpha.

En effet, 0xFD n'est pas le bon masque; le bon masque serait plutôt ~0x20, c'est-à-dire 0xDF.