0

以下のプログラムをコンパイルすると、次のエラー メッセージが表示されます。

エラー 1 エラー C2228: '.get_future' の左側にはクラス/構造体/共用体が必要です c:\users\haliaga\documents\visual studio 2010\projects\test\test\accumulateexceptionsafe.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<> task(accumulate_block (__cdecl )(void))': プロトタイプ化された関数が呼び出されませんでした (意図した変数定義でしたか?) 1> with 1> [ 1> =int (std::_List_iterator >>,std::_List_iterator>>), 1> イテレータ=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;
}
4

1 に答える 1

4

最も厄介な解析。

std::packaged_task<T(Iterator,Iterator)> task(accumulate_block<Iterator,T>());

task引数を取らず、 a を返す関数へのポインタ型のパラメータを取り、 aaccumulate_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>()));
//                                            ^                              ^
于 2015-02-21T01:26:39.170 に答える