ReSharper不检查逻辑。它只检查方法是否使用实例成员。 如果方法是私有的，并且只被(可能只有一个)实例方法调用，这是一个让它成为实例方法的标志。

读起来很有趣: http://thecuttingledge.com/?p=57

ReSharper实际上并不是建议您将方法设置为静态的。 你应该问问自己，为什么这个方法在那个类中，而不是在它的签名中出现的类之一…

但ReSharper documentaion是这么说的: http://confluence.jetbrains.net/display/ReSharper/Member+can+be+made+static

它有助于控制名称空间污染。

您应该在给定的场景中使用最易读和最直观的方法。

除了在最极端的情况下，性能参数并不是一个好的参数，因为实际发生的唯一事情是将一个额外的参数(this)推入到实例方法的堆栈中。

在类中将方法标记为静态可以明显地表明它不使用任何实例成员，这有助于在浏览代码时了解这一点。

你不一定要把它移到另一个类中，除非它被另一个在概念上紧密关联的类所共享。

只是我的一点钱:将所有共享的静态方法添加到实用程序类中，您就可以添加

using static className; 

到您的using语句，这使得代码更快地输入和更容易阅读。例如，我在继承的一些代码中有大量所谓的“全局变量”。我没有在一个属于实例类的类中创建全局变量，而是将它们全部设置为全局类的静态属性。它完成了这项工作，虽然有些混乱，但我可以按名称引用属性，因为我已经引用了静态名称空间。

我不知道这是不是一个好的练习。关于c# 4/5，我有很多东西要学，还有很多遗留代码要重构，所以我只是想让Roselyn的技巧来指导我。

Joey