我最近注意到，当我创建私有方法并在传递给它们的对象中设置一些字段时，Resharper 提出了一个提示，说明该方法可以设为静态。
这是我可能使用的那种方法的一个大大简化的示例。
private void MakeStatusTheSame(MyClass mc, MySecondClass msc)
{
mc.Status = msc.Status;
}
当我有这样的方法时，Resharper 会建议可以将该方法设为静态。
我尽量避免将公共方法设为静态，因为它们会对单元测试造成严重破坏……但我不确定私有方法是否同样适用。
Resharper 的建议是有效的最佳做法，还是我应该将其关闭？
【问题讨论】：
标签： c# function static resharper
冒着听起来像个逆势而为的风险，我不得不承认我不喜欢将静态方法与实例方法混为一谈。我一般不喜欢静态方法。静态方法难以测试、难以覆盖和难以维护。我更喜欢将处理 Foo 对象的所有静态方法粘贴到单个 FooUtils 类中——或者更好的是，粘贴到 FooSomethingDoer 类的单例实例中。
当然，静态方法在某些情况下非常有意义——例如，在创建上述单例或工厂等时。我并不是说所有静态方法都是由纯粹的邪恶组成的；我只是宁愿在可能的情况下避免它们。
【讨论】：
我认为这绝对是静态方法的主要候选者。它不会改变任何类的属性、字段等。
这是一个例子：
class MyClass
{
public static void MakeStatusTheSame(MyClass mc, MySecondClass msc)
{
mc.status = msc.status;
}
private void MakeStatusTheSame(MySecondClass msc)
{
this.status = msc.status;
}
private int status;
}
此外，您可以将其设为扩展方法（也可以是静态的）：
public static class Extensions
{
public static MyClass MakeStatusTheSame(this MyClass mc, MySecondClass msc)
{
mc.status = msc.status
return mc; /* make the method chainable */
}
}
【讨论】：
我通常会接受 R# 的建议。这是一种私有方法，因此（希望）您不会针对它编写单元测试。将其设为静态明确表示它不使用任何实例成员，这样更容易检查副作用。
【讨论】：
我想是的；看到一个方法是静态的，这清楚地表明该方法不应该与任何实例成员交互。
想象一下调试一个非静态方法并意识到实例没有被触及。立即闻到气味，如果没有注释解释该功能的作用，您可能会从真正的问题上分心。
【讨论】：
String.IsNullOrEmpty() 将其使用的数据作为参数，而不是使用实例成员值。