This really bugs me, a boolean logic statement with no code block...
if(variable1 == variable2) DoSomething();
I know it is done all the time and it's OK. However, it lends itself to FatFingers which produces buggy code. Better solution?
if(variable1 == variable2) { DoSomething(); }
A pain to add the code block, but much safer. This way if something is added at a later time, like...
if(variable1 == variable2) { DoSomething(); DoSomethingElse(); }
...the code block is already there. Besides, the compiler has to add that code block anyway!