C++

problème avec for_each

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : problème avec for_each

blackgoddess

vive le troll !

Bonjour, j'ai une erreur avec le code suivant :

Code :

class format_stack
{
bool bfirst; // 1ere insertion ?
std::ostringstream oss;
public:
format_stack()
: bfirst(true)
{
}
void operator () (const std::string &s)
{
if(bfirst)
bfirst = false;
else
oss << " -> ";
oss << s;
}
std::string get() const
{
return oss.str();
}
};
std::list<std::string> l;
l.push_back(std::string("1" ));
l.push_back(std::string("2" ));
format_stack fs;
std::for_each(l.rbegin(), l.rend(), fs);

compilé avec visual studio 2003

a cette ligne : std::for_each(l.rbegin(), l.rend(), fs);
error C2664: 'std::for_each' : impossible de convertir le paramètre 3 de 'black::stack_trace::format_stack' en
'black::stack_trace::format_stack'

Je suppose que c'est un problème de prototype de l'opérateur () du foncteur, mais je n'arrive pas à trouver ...
Toute aide est bienvenue, merci

Message édité par blackgoddess le 05-08-2004 à 15:20:01

---------------
-( BlackGoddess )-

Publicité

Taz

bisounours-codeur

moi je dirais qu'il apprécie pas trop la copie de stream (mais j'ai pas testé, j'ai pas de compilo c++ au boulot)

cela dit moi j'aurais plutot fait avec un std::copy + iterator perso

Taz

bisounours-codeur

ou transform en fait !

blackgoddess

vive le troll !

c'était effectivement la copie de stream qu'il n'aimait pas.

j'y suis arrivé comme ca :

Code :

class format_stack
{
bool bfirst; // 1ere insertion ?
std::ostringstream & oss;
public:
format_stack(std::ostringstream & s)
: bfirst(true), oss(s)
{
}
void operator () (const std::string &s)
{
if(bfirst)
bfirst = false;
else
oss << " -> ";
oss << s;
}
};
std::list<std::string> l;
l.push_back(std::string("1" ));
l.push_back(std::string("2" ));
std::ostringstream oss;
std::for_each(l.rbegin(), l.rend(), format_stack(oss));

j'ai essayé avec tranform, mais je n'y suis pas arrivé :
je ne vois pas comment remplir un std::string (ou std::ostreamstring ou autre)
à partir d'un std::list<std::string>

aurais-tu un petit exemple stp ?

Message édité par blackgoddess le 05-08-2004 à 16:19:16

---------------
-( BlackGoddess )-

Taz

bisounours-codeur

ben un stingstream, c'est un stream, tu peux très bien faire

* istream_iterator<string>(in)++ = "hello";

xterminhate

Si vis pacem, para bellum.

Hormis le pb de compilation, il me semble que ton functor etait mal conçu. La valeur de oss appartenant à l'instance fs n'est pas modifiée par le for_each. Je me trompe ?

Code :

#include<iostream>
#include<string>
#include<list>
using namespace std;
class format_stack
{
string& os;
public:
format_stack( string& s ) : os(s) {}
void operator() ( const string& s ) const {
if( os.size() != 0 )
os += " -> ";
os += s;
}
};
int main()
{
list<std::string> l;
l.push_back( "1" );
l.push_back( "2" );
string s;
for_each( l.rbegin(), l.rend(), format_stack(s) );
cout << s << endl;
}

Message édité par xterminhate le 06-08-2004 à 17:34:30

---------------
Cordialement, Xterm-in'Hate...

blackgoddess

vive le troll !

la méthode que j'ai mise ci-dessus fonctionne (edit : a l'exécution aussi), en effet l'objet ostringstream est passé par référence.

au niveau de l'optimisation, il veut mieux remplir un std::ostringstream ou un std::string pour des concaténations comme ca ?

Message édité par blackgoddess le 06-08-2004 à 17:46:15

---------------
-( BlackGoddess )-

xterminhate

Si vis pacem, para bellum.

J'ai préféré le string parce qu'aucune conversion n'etait nécessaire dans ton cas d'utilisation (juste concaténation). Niveau optimisation, je ne sais pas dire exactement.... je pencherais pour string plus rapide que ostringstream ! En attendant Taz, tu paries sur quoi ?

---------------
Cordialement, Xterm-in'Hate...

Taz

bisounours-codeur

ben t'as pas besoin d'un ostringstream pour concaténer des string. ni même de foreach

blackgoddess

vive le troll !

je suppose que le std::string ralloue son buffer a chaque concaténation, ce qui n'est peut-être pas le cas d'un buffer de stream ... donc je voterais ostringstream plus rapide

---------------
-( BlackGoddess )-

Publicité

Taz

bisounours-codeur

tu supposes mal à priori

xterminhate

Si vis pacem, para bellum.

Un gage, un gage, un gage !

---------------
Cordialement, Xterm-in'Hate...

HelloWorld

Salut tout le monde!

Et pourquoi pas une simple petite fonction templatee qui utilise un ostream_iterator :

Code :

template<typename Iter>
std::string format_stack( Iter First, Iter Last )
{
std::ostringstream oss;
std::copy(
First,
--Iter( Last ),
std::ostream_iterator<std::string>( oss, "->" ) );
oss << *(--Last);
return oss.str();
}
int main()
{
std::list<std::string> l;
l.push_back( "1" );
l.push_back( "2" );
l.push_back( "3" );
l.push_back( "4" );
std::cout << format_stack( l.begin(), l.end() ) << '\n';
std::cout << format_stack( l.rbegin(), l.rend() ) << '\n';
}

---------------
FAQ fclc++ - FAQ C++ - C++ FAQ Lite

Taz

bisounours-codeur

je penser à un truc comem effectivement

blackgoddess

vive le troll !

ah voui en effet mci pour cette solution

xterminate => jdois courir en slip a travers le forum ?

---------------
-( BlackGoddess )-

xterminhate

Si vis pacem, para bellum.

HelloWorld > solution séduisante ! :jap:

BlackGoddess > En cette période estivale, le string est de rigueur !

---------------
Cordialement, Xterm-in'Hate...

FORUM HardWare.fr

Programmation

C++

problème avec for_each

Sujets relatifs
probleme configurartion serveur apache pour php	[VBS/VBA] Probleme avec after et before ds un vbs
Probleme pour l'exécution	problème de menu flash avec url +arguments en PHP
[PHP] - Probleme pour ajouter des jours à une date	probleme de javascript avec certains browser
Problème de jointure ORACLE avec comparaison de champs null	Problème création miniatures
Problème de joker * et selection for-each	Problème avec for_each
Plus de sujets relatifs à : problème avec for_each

Page générée en 0.058 secondes