dangling wenn und Makro

Question

Ich bin in der Notwendigkeit, Warnungen loszuwerden, einer davon ist ein -Wingling-sonst. Er stellt fest, ein unsicherer, wenn in der folgenden Makrodefinition:

#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) 

Ich muss zugeben, ich weiß nicht wirklich in dieses Makro erhalten und somit nicht wissen, wie diese Warnung zu beheben.

Konnte mir jemand mit einer Erklärung helfen, wie man das Dangling sonst vermeidet?

Answer 1

Sie könnten den Zustand umkehren zu leeren, wenn Block und so problematisch else haben zu vermeiden:

#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) 

Da C++ 11, wäre es auch einfach zu schreiben:

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

Answer 2

Wie vermeidet man das Dangling sonst?

Die einfachste (und nicht empfohlene) Option ist das Deaktivieren der Warnung im Compiler mit -Wno-dangling-else.

Die wirkliche Lösung ist Klammern hinzuzufügen, um die if/else Filialen:

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

#define FOR_EACH_REF_END }} 

Verbrauch:

FOR_EACH_REF(foo, bar) 
{ 

} 
FOR_EACH_REF_END 

---

Wenn Sie einen Bereich iterieren wollen, C++ 11 stellt eine Sprachsyntax dafür zur Verfügung, die anstelle dieses abscheulichen Makros verwendet werden sollte:

for(const auto& var : container) { }