Fonction renvoyant une string

Bonjour, ce message fait suite au poste http://www.developpez.net/forums/sho...d.php?t=140630 où j'ai une fonction qui transforme ma chaine de caractères en majuscule :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

#include <ctype.h>   void strtoupper(char *s) { if (s) { while(*s) { *s = toupper(*s); s++; } } }

---

maintenant, je désire que ma string s ne soit pas modifiée donc voici ce que j'ai fait :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31

#include<stdio.h> #include<stdlib.h> #include<string.h> #include<ctype.h>   char * strtoupper(char * s) { /* converts the string s with all these chars in capital letter */   char * str=strdup(s); printf("str = %s\n",str);   while(*str!='\0') { printf("*str = %c\n",*str); *str=toupper(*str); ++str; } printf("str = %s\n",str); return str; }   int main() { char s1[]="c7H15-2abc"; char * s2=strtoupper(s1); printf("s2 = %s\n",s2); free(s2); s2=NULL; return 0; }

---

mais là il y a un bug lors de mon free(s2). Pourtant après un strdup il faut faire un free non ?

Merci

Le free ne fonctionne qu'avec une adrese allouée par malloc, ce qui n'est pas le cas pour s2.

En effet, il faut faire un free() apres strdup(), mais il faut envoyer a free() le pointeur non-modifie, tel qu'attribue par strdup(). Dans ta fonction, tu incrementes str...
Il faut que tu utilises une variable temporaire:

Code:

---

1 2 3 4 5 6 7 8 9 10

str = strdup(s); if (str) { char *p = str; ... p++; ... } return str;

---

Tu dois faire un free du pointeur renvoyé par strdup. Or dans ta fonction tu incrémentes ton pointeur. Ca ne risque pas de marcher.

Citation:

---

Envoyé par salseropom

Code:

---

1 2 3 4

#include <ctype.h>   void strtoupper(char *s)

---

---

Et j'ajoute que 'strtoupper' est un identificateur réservé.

http://emmanuel-delahaye.developpez....htm#id_reserve

'str_toupper' est correct.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37

#include<stdio.h> #include<stdlib.h> #include<string.h> #include<ctype.h>   char * str_toupper(char const * s) { /* converts the string s with all these chars in capital letter */   char * str = strdup(s); if (str != NULL) { char *p = str;   while (*p != '\0') { *p = toupper(*p); p++; } } return str; }   int main() { char s1[] = "c7H15-2abc"; printf("s1 = '%s'\n", s1); char * s2 = str_toupper(s1);   if (s2 != NULL) { printf("s2 = '%s'\n", s2); free(s2), s2 = NULL; } return 0; }

---

Oups, desole d'avoir utilise un idenficateur reserve. Ils sont chiant a l'ISO de choper tous les noms simples!

Bonjour, merci de vos explications. Voici donc ma nouvelle fonction qui marche

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28

#include<stdio.h> #include<stdlib.h> #include<string.h> #include<ctype.h>   char * str_toupper(char * s) { /* converts the string s with all these chars in capital letter */   char * str_=strdup(s); unsigned short length=strlen(str_),i; char str[length]; strcpy(str,str_);   for(i=0;i<length;++i) str[i]=toupper(str[i]); strcpy(str_,str); return str_; }   int main() { char s1[]="c7H15-2abc"; char * s2=str_toupper(s1); printf("s2 = %s\n",s2); free(s2); s2=NULL; return 0; }

---

ma fonction n'est sûrement pas optimisée (car il y a deux copies) mais au moins elle marche...
Merci.

oups, désolé, je n'avais pas vu la fin de ton post Emmanuel. Ta fonction est bcp mieux ! Merci.

Citation:

---

Envoyé par salseropom

Code:

---

1 2 3 4 5

char * str_=strdup(s); unsigned short length=strlen(str_),i; char str[length]; strcpy(str,str_);

---

Merci.

---

Tu n'as absolument pas besoin de ce tableau intermédiaire (VLA, en plus : status 'broken' chez gcc !)

2 corrections (identiques) ont été publiées.