Discussion :

[hibou107]

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

Code :

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

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

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

[hibou107]

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

[Joel F]

Code :

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

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

[hibou107]

Code :

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

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!

[camboui]

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

Code :

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

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.