Probleme avec une fonction - Page 1

Voici ma fonction

char *coule(short num)
{
char coul=malloc(7);
switch (num)
{
case 0:
strcpy(coul,"Noir"); break;
case 1:
strcpy(coul,"Marron"); break;
case 2:
strcpy(coul,"Rouge"); break;
case 3:
strcpy(coul,"Orange"); break;
case 4:
strcpy(coul,"Jaune"); break;
case 5:
strcpy(coul,"Vert"); break;
case 6:
strcpy(coul,"bleu"); break;
case 7:
strcpy(coul,"Violet"); break;
case 8:
strcpy(coul,"Gris"); break;
case 9:
strcpy(coul,"Blanc"); break;
}
return coul;
free(coul); }

Le probleme c'est que mon pointeur coul n'est jamais libere, logique puisque qu'il ya le return avant!! Alors mon espace utilise n'est pas libere et je fait appel a cette fonction plusieur fois donc l'espace devient non negligeable. Alors commen faire? Merci

Il faut libérer coul quand tu as fini de l'utiliser.

erf... Ta fonction retourne coul, donc tu peux le free() plus tard, vers la fin de ton programme, ou bien dès que t'as fini de l'utiliser. Dans tous les cas, le free(coul) à la fin de ta fonction n'a aucun sens.

Au fait, t'as vraiment besoin d'allouer de la mémoire pour ça ?

void coule(char *coul, short num)
{
    static char coulTab[][7] = {"Noir", "Marron", "Rouge", "Orange", "Jaune", "Vert", "Bleu", "Violet", "Gris", "Blanc"};
    strcpy(coul,coulTab[num]);
}

[edit] cross

fait le free en dehors de la fonction, ou met en paramettre de ta fonction un pointeur vers ou copier la chaine
ou bien fait retourner a ta fct un double (8 octets) qui pourra contenir la chaine

[edit] cross aussi ^^'

Super verytos j'avai pas penser a ca!!! Merci

char coul=malloc(7);

pustôt: char *coul=malloc(7);

Mais de toute façon il n'a aucune raison de faire un malloc de cette taille non ?

jfg :
Mais de toute façon il n'a aucune raison de faire un malloc de cette taille non ?

Dépendamment du but de la fonction, il pourrait passer un buffer en paramètre et concaténer la couleur directement …

Uther
:

char coul=malloc(7);

pustôt: char *coul=malloc(7);

Tout à fait, et j'ajoute que si tu n'actives pas ou que tu ne lis pas les warnings du compilateur, tu vas passer à côté de plein de bugs...

Mais de toute façon il n'a aucune raison de faire un malloc de cette taille non ?

Si, vu qu'il compte y copier un mot pouvant aller jusqu'à 6 caractères, il faut compter un octet par caractère plus le "\0" final.

Utiliser un espace mémoire avec malloc de seulement 7 octets, c'est un peu stupide. Autant réserver directement dans le programme 7 octets en créant un tableau (unsigned char tab_txt[7].

ou ne rien reserver du tout et retourner l'adresse des chaines ecrites en dur

[edit] euh en fait Quesoft a ptet dit la mm chose mais j'avais pas compris (concatener ?)

J'ai demande a ce que les poste soit locker!! Car mon probleme est resolut Merci

pourquoi locker ?
si certaines personnes ont une question qui correspond à ce sujet, pourquoi ne pas leur laisser la possibilité de continuer la conversation ici, en profitant de ce qui a déjà été dit ?