suppression caratère " dans une chaîne

Version imprimable

16/11/2006, 14h00
sandra771

suppression caratère " dans une chaîne

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
16/11/2006, 14h07
Médinoc

On peut voir ton code?
Entre autres, comment utilises-tu ma fonction?
16/11/2006, 14h11
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:

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:

"\"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 !

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:

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:

Citation:

Avant : "valeur1"
Apres : valeur1

J'espère que ca t'ira ;)

16/11/2006, 14h32
Emmanuel Delahaye

Citation:

Envoyé par sandra771

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é...

Citation:

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...
16/11/2006, 14h33
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à

Citation:

Envoyé par 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:

Le ++ de

Code:

*(pw++) = 0;

est inutile

Code:

*p = 0;

suffit.
Quelques simplifications :

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

16/11/2006, 14h38
Franck.H

Citation:

Envoyé par Emmanuel Delahaye

Le ++ de

Code:

*(pw++) = 0;

est inutile

Code:

*p = 0;

suffit.

Oui exacte, je suis aller un peu trop vite en besognes :oops: :lol:
16/11/2006, 14h44
Franck.H

Citation:

Envoyé par Emmanuel Delahaye

Quelques simplifications :

:king:
16/11/2006, 14h47
sandra771

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

Citation:
Envoyé par sandra771
J'ai essayé la fonction comme suit (je mets juste une partie du code)
Code:

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:

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

Rappel :
Code:

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...