<p>I've been doing a massive code review and one pattern I notice all over the place is this:</p>

<pre><code>public bool MethodName()
{
    bool returnValue = false;
    if (expression)
    {
        // do something
        returnValue = MethodCall();
    }
    else
    {
        // do something else
        returnValue = Expression;
    }

    return returnValue;
}
</code></pre>

<p>This is not how I would have done this I would have just returned the value when I knew what it was. which of these two patterns is more correct?</p>

<p>I stress that the logic always seems to be structured such that the return value is assigned in one plave only and no code is executed after it's assigned.</p>

I would have used ternary, to reduce control structures...
```
```
return expression ? MethodCall() : Expression;
```
```