Discussion :

[dj.motte]

Salut,
Dans les exemples et FAQ du C je trouve le code suivant pour supprimer tous les espaces d'une chaîne de caractères :

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
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
 
char* supprime (const char *src)
{
  char *dest = NULL, *res;
  int n = strlen (src);
  int i, dansunmot, j = 0;
 
  dest = malloc (n+1);
  if(dest == NULL)
  {
    return NULL;
  }
 
  /* Cherche le premier non espace */
  i = 0;
  while ((i < n) && (src[i] == ' '))
  {
    i++;
  }
 
  /* On est dans un mot */
  dansunmot = 1;
 
  /* Jusqu'a la fin de la chaine */
  while (i < n)
  {
    /* Si c'est un espace, on fait remarquer qu'on n'est plus dans un mot */
    if (src[i] == ' ')
    {
      dansunmot = 0;
    }
    else 
    {
      /* Si on n'est pas dans un mot, on met un espace */
      if (dansunmot == 0)
      {
        dest[j]=' ';
        j++;
      }
      /* On est dans un mot */
      dansunmot = 1;
      /* On copie la lettre */
      dest[j] = src[i];
      /* On incremente l'indice de destination */
      j++;
    }
    /* On incremente l'indice de la source */
    i++;
  }
 
  /* Mettre le '\0' a la fin */
  dest[j] = '\0';
  j++;
 
  /* On realloue pour avoir la bonne taille */
  res = realloc (dest, j);
  if (res)
  {
    return res;
  }
  else
  {
    return dest;
  }
}

A vrai dire je demande si un code comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
char * str_pack_char( char * z , short c )
{ char * s = z ;
  char * str = s  ; 
  if ( z != NULL )
  {
    while( *z )
    { if( *z != c ) *s++ = *z ;
      z++;
    }
   *s = '\0' ;
  } 
  return str ; 
}

ne ferait pas mieux ?
Mais bon la chaîne rétrécit mais pas son espace mémoire ...

[Médinoc]

Je n'ai pas vu les détails de ton code donc j'ignore s'il marche, mais je le trouve plus approprié en effet.
L'autre m'a l'air d'une usine à gaz...

Edit: Mais pour faire plus simple, je supprimerais la variable str et je ferais retourner void à la fonction...

[totaliou]

J'ai l'impression que le code de la faq supprime les espaces en trop et alloue une nouvelle chaine.

Code :

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

"Bonjour       tout le    monde" --> "Bonjour tout le monde"

[dj.motte]

Salut,

Il me semblait que l'exemple de la FAQ voulait faire à la place de "Bonjour tout le monde" un truc du genre "Bonjourtoutlemonde" ?

[Emmanuel Delahaye]

A vrai dire je demande si un code comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
char * str_pack_char( char * z , short c )
{ char * s = z ;
  char * str = s  ; 
  if ( z != NULL )
  {
    while( *z )
    { if( *z != c ) *s++ = *z ;
      z++;
    }
   *s = '\0' ;
  } 
  return str ; 
}

ne ferait pas mieux ?
Mais bon la chaîne rétrécit mais pas son espace mémoire ...

C'est vrai que l'exemple de de la FAQ est assez dramatique... J'ai ça dans ma CLIB :

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
 
/* ---------------------------------------------------------------------
   STR_DelBlanks()
   ---------------------------------------------------------------------
   Suppression des blancs dans une chaine
   LA CHAINE EST MODIFIEE !!!
   ---------------------------------------------------------------------
   E : Chaine initiale
   S :
   --------------------------------------------------------------------- */
void STR_DelBlanks (char *const s)
{
   if (s && *s)
   {
      char const *pr = s;
      char *pw = s;
 
      while (*pr)
      {
         if (*pr != ' ')
         {
            *pw = *pr;
            pw++;
         }
         pr++;
      }
 
      *pw = 0;
   }
}

Ca ne doit pas être très éloigné de ta solution...

EDIT : Si l'exemple de la FAQ consiste à supprimer les espaces en trop, c'est autre chose (mais ça me parait bien compliqué quand même...)

[dj.motte]

Salut,

Pour reprendre l'exemple d'Emmanuel j'aurais 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
 
void STR_DelTheChar (char * s, int c )
{
   if (s && *s)
   {
      char *pr = s;
      char *pw = s;
 
      while (*pr)
      {
         if (*pr != c )
         {
            *pw = *pr;
            pw++;
         }
         pr++;
      }
 
      *pw = 0;
   }
}

Stupide de faire une fonction uniquement que pour les espaces... Le "char const * " je ne vois pas l'utilité ?

Mieux encore serait une fonction de prototype :

Code :

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

1
2
 
char * str_remove_set( char * z, const char * s ) ;

où l'on supprime dans "z" tous les caractères faisant partie de "s".

Par exemple :

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
 
void str_remove_set(char * z, const char * s )
{
   if ( z && s  )
   {
      char *pr = z;
      char *pw = z;
 
      while (*pr)
      {
         if( ! strchr( s, *pr  ) ) 
         {
            *pw = *pr;
            pw++;
         }
         pr++;
      }
 
      *pw = 0;
   }
}

Mais bon ...

[Emmanuel Delahaye]

Salut,

Pour reprendre l'exemple d'Emmanuel j'aurais 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
 
void STR_DelTheChar (char * s, int c )
{
   if (s && *s)
   {
      char *pr = s;
      char *pw = s;
 
      while (*pr)
      {
         if (*pr != c )
         {
            *pw = *pr;
            pw++;
         }
         pr++;
      }
 
      *pw = 0;
   }
}

Stupide de faire une fonction uniquement que pour les espaces...

Bah, merci pour le 'stupide'. Tu prends donc la responsabilité de l'ouverture des hostilités, j'en prend bonne note.
J'ai repris du code existant qui correspondait à un besoin précis... J'ai pas cherché à monter que j'avais la plus grosse...

Le "char const * " je ne vois pas l'utilité ?

Parce qu'on est jamais à l'abri d'une erreur de codage et que donner au compilateur le maximum d'info pour s'en préserver n'est pas inutile pour moi.

[dj.motte]

Salut,

Bah, merci pour le 'stupide'. Tu prends donc la responsabilité de l'ouverture des hostilités, j'en prend bonne note.

Désolé j'ai été un peu rapide. Je voulais dire que l'on pouvait dans la foulée paramétrer le caractère à supprimer.

Ceci dit, je n'ai jamais vraiment compris la différence entre "const char*" et "char const *".

Je comprends bien "const char*" mais l'autre alors je ne vois pas bien ?

[Emmanuel Delahaye]

Ceci dit, je n'ai jamais vraiment compris la différence entre "const char*" et "char const *".

Je comprends bien "const char*" mais l'autre alors je ne vois pas bien ?

Qui a dit qu'il avait une différence ? Qu'en pense ton livre de C ?

[Médinoc]

const char * est la même chose que char const *
Par contre, char * const est différent.

Et on peut les combiner: const char * const.