Menu

Fonction majuscule [Résolu]

hcp7kuz 220 Messages postés mardi 18 août 2015Date d'inscription 1 mars 2018 Dernière intervention - 6 juil. 2017 à 00:16 - Dernière réponse : hcp7kuz 220 Messages postés mardi 18 août 2015Date d'inscription 1 mars 2018 Dernière intervention
- 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
Afficher la suite 

3 réponses

Répondre au sujet
yg_be 5286 Messages postés lundi 9 juin 2008Date d'inscriptionContributeurStatut 23 avril 2018 Dernière intervention - 6 juil. 2017 à 09:11
0
Utile
2
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.
[Dal] 4568 Messages postés mercredi 15 septembre 2004Date d'inscriptionContributeurStatut 20 avril 2018 Dernière intervention - 6 juil. 2017 à 10:40
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
hcp7kuz 220 Messages postés mardi 18 août 2015Date d'inscription 1 mars 2018 Dernière intervention - 6 juil. 2017 à 12:03
Effectivement, j'avais complétement oublié le
\0
.
Merci !
Commenter la réponse de yg_be