Discussion :

[sandra771]

Bonjour,

J'ai une chaîne de caratère que j'ai déclarée comme suit:
char toto[50].
Dans mon programme, je lis un fichier et affecte une valeur à toto, par exemple ' "valeur" '.
Je souhaiterai maintenant supprimer les deux " " et ne récupérer dans la variable toto que le mot 'valeur' (sans les doubles ")

J'ai essayé la méthode donnée par Médinoc dans la discussion intitulée "supprimer un caractère" mais ça plante à l'exécution sur la ligne
if(strchr(chars, c)==NULL) de la fonction str_remove_chars(...)....

Quelqu'un aurait-t-il une solution simple à mon problème?
Merci!

Sandra

[Médinoc]

On peut voir ton code?
Entre autres, comment utilises-tu ma fonction?

[Franck.H]

Je sais pas si j'ai bien saisie le problème mais si tu donne à ton tableau toto une chaîne par exemple comme ceci:

Code :

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

strcpy (toto, "valeur");

Bin les " " ne sont pas ajouté, ce sont des symboles qui indiquent une chaîne de caractères mais ne sont en aucun cas insérés dans la chaîne sauf si elle est sous la forme

Code :

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

"\"valeur\""

Sinon, si je me trompe, montre ton code ... le plus simple serait d'affecter directement valeur à ton tableau car là tu fait deux opération alors qu'on peut le faire en une seule !

[Franck.H]

Voici un exemple simple, comme j'en avais posté un mais que j'ai adapté à la suppression d'un même caractère présent plusieures fois dans la même chaîne:

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
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
 
int main (void)
{
   char toto [50] = { "\"valeur1\"" };
 
 
   {
      char * pr = NULL; /* Pointeur de lecture. */
      char * pw = NULL; /* Pointeur d'ecriture. */
 
 
      pr = toto;
      pw = pr;
 
      printf ("Avant : %s\n", toto);
 
      while (*pr != 0)
      {
         if (*pr == '\"')
         {
            /*
             * On decale le pointeur de lecture pour qu'a la prochaine
             * expression on copie le caractere suivant a la place du
             * double quote.
             */
            pr++;
         }
 
         *pw = *pr;
         pw++;
         pr++;
      }
      *pw = 0; /* Ajout du zero de fin de chaine. */
 
      printf ("Apres : %s\n", toto);
   }
 
 
   return EXIT_SUCCESS;
}

Ce qui nous donne en sortie:

Avant : "valeur1"
Apres : valeur1

J'espère que ca t'ira

[Emmanuel Delahaye]

J'ai une chaîne de caratère que j'ai déclarée comme suit:
char toto[50].

Jusque là, on a pas de chaine de caractères, mais un simple tableau de 50 char non initialisé...

Dans mon programme, je lis un fichier et affecte une valeur à toto, par exemple ' "valeur" '.
Je souhaiterai maintenant supprimer les deux " " et ne récupérer dans la variable toto que le mot 'valeur' (sans les doubles ")
Une simple boucle qui fait une copie caractère par caractère dans le même tableau de char en 'oubliant' les '"' mais pas le 0 final.

Rien de sorcier... Le problème a été évoqué hier et a été l'occasion d'un concours de g33k code mémorable...

Fait de ton mieux et montre le code qui ne fonctionne pas, on est pas devins...

[sandra771]

Non, je suis bien dans le cas "\"valeur\""

J'ai essayé la fonction comme suit (je mets juste une partie du code)

printf("toto : %s\n", toto);
char dest[sizeof(toto)/sizeof(toto[0])];
size_t res1 = str_remove_chars(dest, toto, '"');
printf("dest = \"%s\" (%u chars).\n", dest, (unsigned int)res1);

L'exécution donne:

toto : "valeur"
signal SEGV (no mapping at the fault address) in strchr at 0xfea4e5b0
0xfea4e5b0: strchr+0x0004: ldsb [%o0], %o4
Current function is str_remove_chars
293 if(strchr(chars, c)==NULL)

Voilà

[Emmanuel Delahaye]

Voici un exemple simple, comme j'en avais posté un mais que j'ai adapté à la suppression d'un même caractère présent plusieures fois dans la même chaîne:

Le ++ de

Code :

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

*(pw++) = 0;

est inutile

Code :

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

*p = 0;

suffit.
Quelques simplifications :

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
#include <stdio.h>
#include <string.h>
 
int main (void)
{
   char toto[50] = "\"valeur1\"";
   printf ("Avant : '%s'\n", toto);
 
   {
      char *pr = toto;          /* Pointeur de lecture. */
      char *pw = pr;            /* Pointeur d'ecriture. */
      while (*pr != 0)
      {
         if (*pr != '"') /* on 'oublie' de copier le '"' */
         {
           *pw = *pr;
            pw++;
         }
         pr++;
      }
      *pw = 0;                  /* Ajout du zero de fin de chaine. */
   }
   printf ("Apres : '%s'\n", toto);
   return 0;
}

[Franck.H]

Le ++ de

Code :

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

*(pw++) = 0;

est inutile

Code :

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

*p = 0;

suffit.

Oui exacte, je suis aller un peu trop vite en besognes

[Franck.H]

Quelques simplifications :

[sandra771]

Les deux codes marchent très bien. Merci pour votre aide!
Sandra

[Emmanuel Delahaye]

J'ai essayé la fonction comme suit (je mets juste une partie du code)

Code :

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

1
2
3
4
5
 
printf("toto : %s\n", toto);
char dest[sizeof(toto)/sizeof(toto[0])];
size_t res1 = str_remove_chars(dest, toto, '"');
printf("dest = \"%s\" (%u chars).\n", dest, (unsigned int)res1);

L'exécution donne:

toto : "valeur"
signal SEGV (no mapping at the fault address) in strchr at 0xfea4e5b0
0xfea4e5b0: strchr+0x0004: ldsb [%o0], %o4
Current function is str_remove_chars
293 if(strchr(chars, c)==NULL)

Ben oui, quand le compilateur fait un warning, il faut vérifier son code. :

Code :

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

size_t res1 = str_remove_chars(dest, toto, "\"");

Rappel :

Code :

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

1
2
3
4
5
size_t str_remove_chars(
 char *dest,       //[out] Destination buffer. MUST be at least as long as src.
 char const *src,  //[in] Source string. Can be equal to dest.
 char const *chars //[in] Chars to remove.
 )

Mais des solutions plus simples ont été publiées par la suite...