C# Coding Guidelines

ときどきの雑記帖

てことでちょっと見てみた。
少なくとも、ここのやつよりも遥かにいい。
ただ、

String comparison involves unnecessary overhead. If all you need is to check whether the string is empty, use the Length property.
Code snippets:

//NO
if ( str != "")//comparison of two strings
{
    ...
}
//YES
if ( str.Length > 0)
{
    ...
}
C# Coding Guidelines and Best Practices - 4.1 PERFORMANCE

なんていう同意しかねるようなものもちらほらと。
でも、ベースにするならかなりいいんじゃないかなぁ。