Discussion :

[bj303931]

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

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
#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;
}

[Matt_Houston]

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.

[nnovic]

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

[Médinoc]

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

[Daïmanu]

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.

[Médinoc]

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

[jopopmk]

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.