Fonction majuscule

Résolu

hcp7kuz Messages postés 243 Date d'inscription Statut Membre Dernière intervention -
hcp7kuz Messages postés 243 Date d'inscription Statut Membre 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

A voir également:

Fonction majuscule
Fonction si et - Guide
É majuscule - Guide
Majuscule - Guide
Code ascii de a majuscule - Guide
Mettre en majuscule - Guide

1 réponse

Réponse 1 / 1

yg_be Messages postés 23541 Date d'inscription Statut Contributeur Dernière intervention Ambassadeur 1 584

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] Messages postés 6205 Date d'inscription Statut Contributeur Dernière intervention 1 105

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 Messages postés 243 Date d'inscription Statut Membre Dernière intervention 23

Effectivement, j'avais complétement oublié le

\0

.
Merci !

Discussions similaires

Touches majuscule et minuscule inversées sur mon clavier

Une majuscule dans une adresse mail la modifie-t-elle ?

Dimensions L x l x H : comment lire ces infos ?

Votre réponse

Discussions similaires