関数ポインタへのconst参照

Question

私は関数FooとクラスCDelegateを持っています。 CSpi1::m_Delegate();

：私は私の静的オブジェクトの（）演算子を呼び出すことによって Fooを呼び出したい

CInterruptDelegate CSpi1::m_Delegate(FreeFunction, 0); 

：

typedef void (*typeFctPtr)(void*); 


void Foo(void* dummy) 
{ 
    cout << "Foo\n"; 
} 


class CDelegate 
{ 
    public: 

    CDelegate (const typeFctPtr& f_ref_Wrapper, void* f_pvSubscriber) 
    : m_ref_Wrapper(f_ref_Wrapper), m_pvSubscriber(f_pvSubscriber) 
    { 
    } 

    inline void operator()() const 
    { 
    (*m_ref_Wrapper)(0); 
    } 

    inline void operator=(const CInterruptDelegate& D) 
    { 
    } 

private: 
    void* m_pvSubscriber; 
    const typeFctPtr& m_ref_Wrapper; 
}; 

第二のクラスは、私はこのようなコンストラクタを使用して初期化する静的メンバstatic CDelegate m_Delegate;を持っています

例外が発生しました(*m_ref_Wrapper)(0); 構文に問題がありますか？私がしようとしていることがまったく可能であるかどうかは、私は確信していません。私は、CDelegateのコンストラクタが関数ポインタのconst参照を取っておらず、関数ポインタ自体を取るという実用的な解決法を持っています。私は問題なしで（）演算子で関数を呼び出すことができます。関数のポインタ呼び出しを最適化できず、コンパイル時にすべてを知る必要があるので、const参照を使用して呼び出しを行うことを願っています。

Answer 1

あなたは関数へのポインタへの参照を保持しています（ポインタは、使用するまでに破棄された一時的なものなので、状況はひどく間違っています）。

は、関数型であるためにあなたのtypedefを変更してみてください：

typedef void typeFct(void*); 
... 
const typeFct & m_ref_Wrapper; 

は、その後、あなたの既存のコードを使用すると、関数への参照になってしまいます、あなたは大丈夫です。または、関数へのポインタを格納することができます - const typeFct *。

どちらの場合でも、コールはm_ref_Wrapper(0)になります。

一般的に私は、構文があいまいであるためにのみ、ポインターまたは参照先ではなく関数型をtypedefすることをお勧めします。

Answer 2

私はこの問題は、メンバーの宣言であると信じています：

const typeFctPtr& m_ref_Wrapper; 

代わりに、&を削除しよう：

const typeFctPtr m_ref_Wrapper; 

参照が常に別の既存のオブジェクトを参照する必要があります。この場合、作成された一時オブジェクトを参照して、コンストラクターの呼び出し時に参照を保持します。

私たちはそれに取り組んでいますが、コンストラクタからの参照も削除することをお勧めします。この理由は、スカラーで作業するときに利益がないからです。

コードをより読みやすくするもう1つのことは、関数へのポインタではなく、関数の型をtypedefする場合です。そうすれば、ポインタを渡すことは明らかです。

次の変更をまとめたカットダウン版です私は上記の提案：

typedef void (typeFct)(void*); 

class CDelegate 
{ 
    public: 

    CDelegate (typeFct * f_Wrapper, void* f_pvSubscriber) 
    : m_ref_Wrapper(f_Wrapper), m_pvSubscriber(f_pvSubscriber) 
    { 
    } 

    inline void operator()() const 
    { 
    (*m_ref_Wrapper)(0); 
    } 

private: 
    void* m_pvSubscriber; 
    typeFct * m_ref_Wrapper; 
};