Fonction majuscule

Résolu/Fermé
hcp7kuz Messages postés 238 Date d'inscription mardi 18 août 2015 Statut Membre Dernière intervention 23 mars 2019 - 6 juil. 2017 à 00:16
hcp7kuz Messages postés 238 Date d'inscription mardi 18 août 2015 Statut Membre Dernière intervention 23 mars 2019 - 6 juil. 2017 à 12:03
Bonjour,

J'ai écris une fonction qui retourne une chaine mise en majuscule.
Problème, le fonction retourne effectivement la chaine en majuscule, mais avec également quelques caractères en plus, et je ne vois pas pourquoi.
Quelqu'un a un idée ?

Code :

char* strToUpper(char* chaine) {
    char *result = NULL;
    result = malloc((strlen(chaine) - 1) * sizeof(char));
    int i;
    for (i = 0; i < strlen(chaine); i++) {
        result[i] = toupper(chaine[i]);
    }
    return result;
}


Merci,

hcp7kuz

1 réponse

yg_be Messages postés 21304 Date d'inscription lundi 9 juin 2008 Statut Contributeur Dernière intervention 18 mars 2023 1 326
6 juil. 2017 à 09:11
bonjour, la chaîne de départ se termine par un caractère NULL, indiquant la fin de la chaîne.
strlen() te donne la longueur de la chaîne, sans ce caractère NULL.
donc, ta fonction retourne une chaîne non terminée par un caractère NULL, ce qui explique ce que tu observes.
0
[Dal] Messages postés 6057 Date d'inscription mercredi 15 septembre 2004 Statut Contributeur Dernière intervention 17 mars 2023 1 043
Modifié le 6 juil. 2017 à 10:48
oui, il doit terminer
result
par le caractère
'\0'
, mais il a aussi un dépassement de tampon mémoire, car il alloue
result = malloc((strlen(chaine) - 1) * sizeof(char));
(soit une taille inférieure d'un char à la longueur de la chaîne sans compter le caractère de fin ... donc trop petite de 2 char), alors qu'il devrait allouer
result = malloc((strlen(chaine) + 1));
(la taille de la chaîne avec un char additionnel pour le caractère de fin).

A noter, qu'en C
sizeof(char)
est garantit de valoir 1, il est donc inutile.

char * strToUpper(char * chaine) {
    char * result = NULL;
    int i;

    result = malloc((strlen(chaine) + 1));
    for (i = 0; i < strlen(chaine); i++) {
        result[i] = toupper(chaine[i]);
    }
    result[i] = '\0';

    return result;
}

On peut optimiser le code, aussi, en évitant de mettre
strlen(chaine)
dans la boucle for et en ne l'exécutant qu'une fois au début de la fonction.

Enfin, cela serait bien de tester que malloc a bien alloué la mémoire nécessaire et de décider de ce fait la fonction si ce n'est pas le cas.

Dal
0
hcp7kuz Messages postés 238 Date d'inscription mardi 18 août 2015 Statut Membre Dernière intervention 23 mars 2019 24
6 juil. 2017 à 12:03
Effectivement, j'avais complétement oublié le
\0
.
Merci !
0