Discussion :

[panthere noire]

hello

je cherche a supprimer les espace d' une std::string

j'ai bien tenter un:

Code :

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

1
2
3
4
5
6
7
8
 
std::string buff_space=" mastring  a la  c**  suite"
size_t pos_space;
pos_space=buff_space.find("  ");
if (pos_space != string::npos)
{
     buff_space.replace (pos_space,"  "," ");
}

sa plante avec un message qui est ilisible...
j'ai aussi tester avec erase()
mai visiblement je me rate quelque part
merci d'avance

[Saintux]

Salut,

C'est à l'étape de compilation que ton code ne passe pas :

• Oublie d'un ; à la première ligne
• Je te conseillerai d'utiliser un int à la place de size_t
• Dans le if, tu as oublié le std:: devant string
• Mauvais appel de la fonction replace() ( voir là : http://www.cplusplus.com/reference/s...tring/replace/ )

De plus, avec ton code, tu ne remplacera qu'une seule occurrence. Pour tout remplacer, utilise une boucle.

Voilà le code corrigé :

Code :

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

1
2
3
4
5
6
7
8
9
 
    std::string buff_space = " mastring  a la  c**  suite"; // Rajout d'un ;
    int pos_space;    // Utilisation d'un int à la place de size_t
    pos_space = buff_space.find("  ");
    while (pos_space != std::string::npos)     // Boucle + std::string::npos à la place de string::npos
    {
        buff_space.replace(pos_space, 2,"_");     // Utilisation correcte de la fonction replace()
        pos_space = buff_space.find("  ", pos_space);    // Pour l'occurrence suivante
    }

PS: La prochaine fois met l'erreur même si elle est illisible

[dragonjoker59]

Je suis d'accord avec la majeure partie de ce que Saintux dit, sauf pour la partie 'int à la place de size_t'. Sachant que la fonction 'find' de std::string retourne un size_t et que 'npos' est défini en tant que 'static const size_t', je ne comprends pas pourquoi il faudrait utiliser un int...
Pour tout le reste, je suis d'accord sur ce qu'il t'a préconisé.

[Luc Hermitte]

"idiome erase-remove", ce n'est pas dans la FAQ ?

[Mat007]

Salut,

"idiome erase-remove", ce n'est pas dans la FAQ ?

Si, là.

MAT.

[JolyLoic]

Tel que j'ai compris les besoins, l'idiome n'est pas directement aisément applicable ici, puisque le prédicat dépend non seulement du caractère courant, mais du précédent.

J'ai tendance à bien aimer les algo de string de boost pour ce genre de chose (par exemple http://www.boost.org/doc/libs/1_44_0...place_all.html) mais je n'ai jamais testé ce que ça donnait en perfs. Mais ça ne peut que difficilement être plus mauvais que ce qui est proposé dans les solutions précédentes (indice : Une chaîne de taille N avec M occurrences de " " doit faire avec l'algo proposé un nombre d'opération de l'ordre de grandeur de N*M, alors qu'un nombre d'opérations de l'ordre de N existe).