可変長の関数ポインタにインライン関数またはC++のMACROSを使う方が良い

Question

メニューツールバーのすべての要素を処理する多くの機能を持つアプリケーションがあります。私はこの実装を変更します、またはメニューは、サブメニューを持つことができ

subMenuDefaultMenuShortcuts(ui->fileMenu); 
subMenuDefaultMenuShortcuts(ui->editMenu); 
subMenuDefaultMenuShortcuts(ui->windowMenu); 
subMenuDefaultMenuShortcuts(ui->helpMenu); 

subMenuUpdateLabels(ui->fileMenu,hierarchy); 
subMenuUpdateLabels(ui->editMenu,hierarchy); 
subMenuUpdateLabels(ui->windowMenu,hierarchy); 
subMenuUpdateLabels(ui->helpMenu,hierarchy); 

ことが可能です：

のコードは次のようなもののように見えます。したがって、コードの検索と置き換えは、醜いだけでなく、読みにくく、エラーを起こしやすくなります。ので、私のコードは次のようになりwhould

OnAllMenus(functionName,params ...) 

：

理想的に私はこのような何かをしたいwhould

OnAllMenus(subMenuUpdateLabels) 
OnAllMenus(subMenuUpdateLabels,hierarchy) 
OnAllMenus(someFunction,hierarchy,argument1,argument2) 

私はマクロを使用していたが、その使用は推奨されません。 関数のポインタでインライン関数を使用するHowerverは、ほとんど読みにくいコードにつながっているようです。 （そして関数の引数の数が変わることを期待している関数ポインタの例は見られませんでした）。

addindを実行しないと、それほど複雑ではない複雑なコードを作成することはできません。

Answer 1

template<typename FuncPointer, typename ... Args> 
void for_all_menus(FuncPointer func, Args ... args) 
{ 
    f(ui->foo,std::forward<Args>(args)...); 
    f(ui->bar,std::forward<Args>(args)...); 
    // etc 
} 

// use 
for_all_menus(&subMenuLabel, hierarchy); 

Pmrをの答えが、どこでも散乱される愚かboost::bind秒を停止する可変長引数テンプレート。

Answer 2

boost::functionとboost::bindを使用できます。あなたは、単にboost::functionとる関数と機能 テンプレートを置き換える、よりダイナミックなものが必要な場合は

template<typename Func> 
void for_all_menus(Func f) { 
    f(ui->foo); 
    f(ui->bar); 
    // etc 
} 

// use 
for_all_menus(boost::bind(subMenuLabel, _1, hierarchy)); 

// with variadic templates 
template<typename Func, typename Args...> 
struct for_all_menus { 
    Func f; 
    void operator()(Args&&... args) { 
    // umh, I always mess up the syntax 
    // you might want to double check this 
    f(ui->foo, std::forward<Args>(args)...); 
    } 
}; 
template<typename F> 
for_all_menus<F> make_for_all_menus(F f) { return for_all_menus<F>{f}; } 

// use 
auto f = make_for_all_menus(subMenuLabel); 
f(hierarchy); 

。もちろん でもC++ 11の同等物とlambdaを使うことができます。

メニューのリストを1か所にまとめて、別の場所でそのリスト を使用する場合は、Boost.Preprocessorをお勧めします。しかし、 はそれに頼る前に2度考えてもいいかもしれません。