coding-style - 冗余代码结构

Question

我经常看到的最令人震惊的冗余代码结构涉及使用代码序列

if (condition)
    return true;
else
    return false;

而不是简单地写

return (condition);

我在各种语言中都看到过这个初学者错误:从 Pascal 和 C 到 PHP 和 Java。您会在代码审查中标记哪些其他此类构造？

Answer 1

if (foo == true)
{
   do stuff
}

我一直告诉开发人员应该这样做

if ((foo == true) == true)
{
   do stuff
}

但他还没有得到提示。