Discussion :

[salseropom]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Trap D]

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

[DaZumba]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
 
str = strdup(s);
if (str)
{
   char *p = str;
   ...
      p++;
   ...
}
return str;

[crocodilex]

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.

[Emmanuel Delahaye]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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;
}

[DaZumba]

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

[salseropom]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[salseropom]

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

[Emmanuel Delahaye]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.