2017-03-17 13 views
0

boost :: signalsのようなシグナルライブラリの単純な実装を構築するためのアイディアを思いついただけです。C++バリデーションテンプレートのコンパイラエラー

私はこのためのシンプルなテンプレートを書きましたが、すべてのコールバックを発射する)(oparatorを起動しようとしたとき、私はエラーを取得しています:

template <typename funcDef> 
struct Connection 
{ 
    typename std::vector<std::function<funcDef>>::iterator connectionItem; 
}; 

template <typename funcDef> 
class Signal; 

template <typename retType, typename... args> 
class Signal<retType(args...)> 
{ 
    using return_type = retType; 
    using argument_type = std::tuple<args...>; 
public: 
    Signal() {}; 
    ~Signal() {}; 

    Connection<retType(args...)> connect(std::function<retType(args...)> callback) 
    { 
     Connection<retType(args...)> connection; 
     m_callbacks.push_back(callback); 
     connection.connectionItem = std::prev(m_callbacks.end()); 

     return connection; 
    } 

    void disconnect(Connection<retType(args...)> connection) { m_callbacks.erase(connection.connectionItem); } 

    void operator() (args...) 
    { 
     for (const auto & cb : m_callbacks) 
      cb(args...); // this generates compiler error: Error C2062 type 'int' unexpected  

    } 

private: 
    std::vector<std::function<retType(args...)>> m_callbacks; 
}; 


int sum(int a, int b){ return a + b; } 
int sub(int a, int b){ return a - b; } 

int main() 
{ 

    Signal<int(int, int)> sig; 
    auto c1 = sig.connect(sum); 
    auto c2 = sig.connect(sub); 

    sig(2, 2); 

    return 0; 

} 

任意のアイデアこのコンパイラエラー(エラーC2062タイプの理由何上記のスニペットで)予想外の「int型」

答えて

2
void operator() (args...) 
{ 
    for (const auto & cb : m_callbacks) 
     cb(args...); 
} 

argsタイプパックです - あなたはcbの内側にそれを拡大する前に、それに名前を付ける必要があります。

void operator() (args... xs) 
{//    ^^^^ ^^ 
//    type name 
    for (const auto & cb : m_callbacks) 
     cb(xs...); 
} 
関連する問題