При компиляции программы ниже я получаю сообщение об ошибке:
Ошибка 1, ошибка C2228: слева от «.get_future» должен быть класс / структура / объединение c: \ users \ haliaga \ documents \ visual studio 2010 \ projects \ test \ test \ аккумулятораteexceptionsafe.cpp 62 1 Тест
что на самом деле не настоящая проблема.
если вы прокомментируете строки:
//futures[i]=task.get_future();
//threads[i]=std::thread(std::move(task),block_start,block_end);
//block_start=block_end;
Вы получите предупреждение ниже, в котором говорится, что «tasK» не был вызван:
* предупреждение C4930: ‘std :: packaged_task<> задача (накапливать_блок (__cdecl ) (void)) ‘: функция-прототип не вызвана (было ли предназначено определение переменной?)
1> с
1> [
1> = int (std :: _ List_iterator >>, std :: _ List_iterator >>),
1> Iterator = std :: _ List_iterator >>,
1> T = int
1>]
что бы правильно указать:
std::packaged_task<T(Iterator,Iterator)> task(accumulate_block<Iterator,T>());
?
Спасибо
PS: найдите под кодом:
#include <list>
#include <numeric>
#include <vector>
#include <thread>
#include <future>
using namespace std;
template<typename Iterator,typename T>
struct accumulate_block
{
T operator()(Iterator first, Iterator last)
{
std::thread::id id = std::this_thread::get_id();
return std::accumulate(first, last, T());
}
};
class join_threads
{
std::vector<std::thread>& threads;
public:
explicit join_threads(std::vector<std::thread>& threads_):
threads(threads_)
{
std::thread::id id = std::this_thread::get_id();
}
~join_threads()
{
std::thread::id id = std::this_thread::get_id();
for(unsigned long i=0;i<threads.size();++i)
{
if(threads[i].joinable())
threads[i].join();
}
}
};
template<typename Iterator,typename T>
T parallel_accumulate(Iterator first,Iterator last,T init)
{
std::thread::id id = std::this_thread::get_id();
unsigned long const length=std::distance(first,last);
if(!length)
return init;
unsigned long const min_per_thread=25;
unsigned long const max_threads=(length+min_per_thread-1)/min_per_thread;
unsigned long const hardware_threads=std::thread::hardware_concurrency();
unsigned long const num_threads=std::min(hardware_threads!=0?hardware_threads:2,max_threads);
unsigned long const block_size=length/num_threads;
std::vector<std::future<T> > futures(num_threads-1);
std::vector<std::thread> threads(num_threads-1);
join_threads joiner(threads);
Iterator block_start=first;
for(unsigned long i=0;i<(num_threads-1);++i)
{
Iterator block_end=block_start;
std::advance(block_end,block_size);
std::packaged_task<T(Iterator,Iterator)> task(accumulate_block<Iterator,T>());
futures[i]=task.get_future();
threads[i]=std::thread(std::move(task),block_start,block_end);
block_start=block_end;
}
T last_result=accumulate_block<Iterator, T>()(block_start,last);
T result=init;
for(unsigned long i=0;i<(num_threads-1);++i)
{
result+=futures[i].get();
}
result += last_result;
return result;
};
int main()
{
list<int> l;
for(int i=0; i<26; ++i)
l.push_back(i);
std::thread::id id = std::this_thread::get_id();
int res = ::parallel_accumulate(l.begin(), l.end(), 0);
return 0;
}
Самый неприятный разбор.
std::packaged_task<T(Iterator,Iterator)> task(accumulate_block<Iterator,T>());
объявляет функцию с именем task
который принимает параметр типа указатель на функцию, не принимая аргументов и возвращая accumulate_block<Iterator,T>
и возвращает std::packaged_task<T(Iterator,Iterator)>
,
Устранить неоднозначность, используя единый синтаксис инициализации:
std::packaged_task<T(Iterator,Iterator)> task(accumulate_block<Iterator,T>{});
Или с дополнительной парой скобок для древних компиляторов, которые не поддерживают равномерную инициализацию:
std::packaged_task<T(Iterator,Iterator)> task((accumulate_block<Iterator,T>()));
// ^ ^