c++ les espaces ?

Version imprimable

19/10/2010, 04h02
panthere noire

c++ les espaces ?
hello

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

j'ai bien tenter un:
Code:

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

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:

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

19/10/2010, 08h58
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é.
19/10/2010, 12h13
Luc Hermitte

"idiome erase-remove", ce n'est pas dans la FAQ ?
19/10/2010, 18h12
Mat007

Salut,

Citation:

Envoyé par Luc Hermitte

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

Si, là.

MAT.
19/10/2010, 19h21
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).

Ah oui, on dirait bien qu'il cherche à faire un :%s/\s\+/ /g
(parfois un bon éditeur de texte fera très bien le boulot)

Sinon, cela ressemble aussi à un copy_if avec état dont le prédicat ressemblerait à

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
// non testé
struct MergeSpaces {
    MergeSpaces() : m_started(false), m_prev(0) {}
    bool operator()(char c) {
        if (c==' ') // ou tab ... 
        {
            if (m_prev != ' ') {
                m_prev = c;
                m_started = true;
                return true;
            }
            else return false;
        }
        else return m_started;
    }
private:
    bool m_started;
    char m_prev;
};

19/10/2010, 20h12
panthere noire

Merci pour vos réponse, pour ce qui est de la faq je vai me pencher dessus car sa correspond a ce que je veux faire. mai j'aime pas le paramètre char en argument....

Cependant je vais m'en tenir a ce que Saintux a proposer mai pour ce qui est de size_t je vai le garder insi , car int a une limite trop basse :?

Merci pour votre coup de pouce :ccool:
20/10/2010, 09h42
dragonjoker59

int ? limite trop basse ? euh, elle fait quelle taille ta chaîne de caractères ? parce que 2^31 -1 ça me parait correct, comme limite ...
21/10/2010, 08h58
Saintux

Citation:

Je te conseillerai d'utiliser un int à la place de size_t

Je retire ça, garde le size_t en effet...

Allez savoir pourquoi j'ai mis ça :aie: