L'état d'un Functor

Bonjour, je suis en train d'écrire un objet fonction template qui retourne la somme d'une range

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

#include <string> #include <iostream> #include <algorithm> #include <vector> using namespace std;   template <typename T>   class Sum {private: T sum; public: Sum(){ sum = 0;}; void operator()( T n ) { sum += n; };   T getResult() { return this->sum; };   };   int main() { vector<int> v; v.push_back( 1 ); v.push_back( 2 ); v.push_back( 3 ); Sum< int > sumFunc; for_each( v.begin(), v.end(), sumFunc ); cout << sumFunc.getResult() << endl; system( "pause" ); return 0; }

---

Par contre, finalement le getResult retourne la valeur 0.
Pouvez vous expliquer pourquoi et comment corriger le problème?
Merci

Les foncteurs sont passés par valeur dans la STL. Si tu dois avoir un foncteur à état utilise un utilitaire du genre boost::ref / std::ref (si t'as un compilo un peu à jour.)

Citation:

---

Envoyé par Goten

Les foncteurs sont passés par valeur dans la STL. Si tu dois avoir un foncteur à état utilise un utilitaire du genre boost::ref / std::ref (si t'as un compilo un peu à jour.)

---

Ou bien je peux utiliser un pointeur pour la variable "sum". Bas merci bien pour ta réponse

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

#include <string> #include <iostream> #include <algorithm> #include <vector> using namespace std;   template <typename T>   class Sum {private: T& sum; public: Sum( T& state ) : sum(state) { sum = 0;}; void operator()( T n ) { sum += n; }; };   int main() { vector<int> v; v.push_back( 1 ); v.push_back( 2 ); v.push_back( 3 );   int s; Sum< int > sumFunc(s); for_each( v.begin(), v.end(), sumFunc ); cout << s << endl; return 0; }

---

Devrait suffire

Citation:

---

Envoyé par Joel F

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

#include <string> #include <iostream> #include <algorithm> #include <vector> using namespace std;   template <typename T>   class Sum {private: T& sum; public: Sum( T& state ) : sum(state) { sum = 0;}; void operator()( T n ) { sum += n; }; };   int main() { vector<int> v; v.push_back( 1 ); v.push_back( 2 ); v.push_back( 3 );   int s; Sum< int > sumFunc(s); for_each( v.begin(), v.end(), sumFunc ); cout << s << endl; return 0; }

---

Devrait suffire

---

Merci, c'est encore meilleur!

Sur base de ton code initial inchangé, sauf l'appel for-each():

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

template<class T> class Sum_wrapper { T *t_ptr; public: typedef T type; explicit Sum_wrapper(T & t): t_ptr(&t) {} operator T& () const { return *t_ptr; } template<typename V> void operator()(V const & r) const { (*t_ptr)(r); } }; template <typename T> class Sum {private: T sum; public: Sum(){ sum = 0;}; void operator()( T n ) { sum += n; }; T getResult() { return this->sum; }; }; int main() { std::vector<int> v; v.push_back( 1 ); v.push_back( 2 ); v.push_back( 3 ); Sum<int> sumFunc; std::for_each( v.begin(), v.end(), Sum_wrapper< Sum<int> >(sumFunc) ); std::cout << sumFunc.getResult() << std::endl; system( "pause" ); return 0; }

---

Ou, comme le suggère goten, voir boost::reference_wrapper (boost::ref, boost::cref) dans <boost/ref.hpp>.
On devrait aussi trouver std::reference_wrapper (std::ref, std::cref) dans <functional> avec les compilos récents.

Bon apres le bon algo pour faire une somme de ce type, c'est plutot std::fold