ifとマクロがぶら下がっています

Question

私は警告を取り除く必要があります。その1つは-Wdangling-elseです。

#define FOR_EACH_REF(var, container) \ 
    if(bool _cont = true) \ 
     for(our::remove_reference<decltype(container)>::type::iterator _it = (container).begin(); _it != (container).end() && _cont; ++_it) \ 
     if((_cont = false)) {} else \ 
      for(our::remove_reference<decltype(container)>::type::value_type& var = *_it; !_cont; _cont = true) 

私は認めなければならない、私は本当にこのマクロに得ることはありませんので、この警告を解決する方法を、知らない：それは、次のマクロ定義内の場合はダングリングノート。

誰かが抱擁を回避する方法について私に助けてくれますか？

Answer 1

：

#define FOR_EACH_REF(var, container) \ 
    if (bool _cont = true) \ 
     for(our::remove_reference<decltype(container)>::type::iterator _it = (container).begin(); _it != (container).end() && _cont; ++_it) \ 
     if(!(_cont = false)) \ 
      for(our::remove_reference<decltype(container)>::type::value_type& var = *_it; !_cont; _cont = true) 

C++ 11ので、それも書きやすいだろう：

#define FOR_EACH_REF(var, container) for (auto& var : container) 

Answer 2

他の方法を避けるにはどうすればよいですか？

最も簡単な（推奨しない）オプションは、-Wno-dangling-elseというコンパイラで警告を無効にすることです。

真の解決策は、if/else枝の周りにカッコを追加することです：

#define FOR_EACH_REF(var, container) \ 
    if(bool _cont = true) { \ 
    for(/* ... */) \ 
     if((_cont = false)) {} else { \ 
     for(/* ... */) 

#define FOR_EACH_REF_END }} 

使用法：

FOR_EACH_REF(foo, bar) 
{ 

} 
FOR_EACH_REF_END 

---

あなたは範囲を反復処理したい場合は、C++ 11その言語の構文が提供されます。このマクロは、この忌み嫌いのあるマクロの代わりに使用する必要があります。

あなたがブロック-ifと elseので、問題は空持つことを避けるために条件を逆かもしれない