`if (IsSomething() == true) { }` は有り派。

なぜならばコード1はコード2よりも読む時の驚きが小さいから。

// コード1
if (IsSomething1() == true ) { }
if (IsSomething2() == false) { }

// コード2
if (IsSomething1()         ) { }
if (IsSomething2() == false) { }

ちなみに、必要なところのみに`== true`を書くというのはコーディングルールの一貫性が無くなるのでもっと駄目だと思う。
ちなみに、「!」演算子は見逃しやすいので使っては駄目だと思う。