<p>The most egregiously redundant code construct I often see involves using the code sequence</p>

<pre><code>if (condition)
    return true;
else
    return false;
</code></pre>

<p>instead of simply writing</p>

<pre><code>return (condition);
</code></pre>

<p>I've seen this beginner error in all sorts of languages: from Pascal and C to PHP and Java.  What other such constructs would you flag in a code review?</p>

Returning uselessly at the end:
```
```
// stuff
   return;
}
```
```