Boolean types

Question

During code review I discovered many places of our C# code that looks like this:

if(IsValid()) {
     return true;
}
else {
     return false;
}

<

Answer 1

I think return IsValid(); is perfectly valid and readable code.

BTW, I would certainly slap anyone who writes (IsValid() ? true : false) in the face. It's unnecessarily complicated.

PS. This is what svn blame is designed for.