Resharper的建议是让我的私有方法静态化吗?

我认为这绝对是静态方法的主要候选者。它不会更改类的任何属性、字段等。

下面是一个例子:

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 */
   }
}

冒着听起来像逆向的风险,我不得不承认,我不喜欢将静态方法与实例方法混合在一起;而且我一般不喜欢静态方法。静态方法很难测试,很难重写,也很难维护。我更喜欢将处理foo对象的所有静态方法都放在一个footils类中——或者更好的是,放在一个foosomethingdoer类的单例实例中。

当然,静态方法在某些情况下是完全有意义的——例如,在创建前面提到的单例或工厂等时。我并不是说所有静态方法都是由纯粹的邪恶构成的;我只是更喜欢在可能的情况下避免它们。

我想是的;看到一个方法是静态的,这就清楚地表明该方法应该 不 与任何实例成员交互。

想象一下,调试一个非静态的方法,并意识到实例没有被触动。即时的气味,如果没有一条评论来解释这个功能的作用,你可能会从真正的问题上分心。

我通常同意R的建议。这是一个私有方法,所以(希望)您没有针对它编写单元测试。使其静态显式地声明它不使用任何实例成员,这使得更容易检查副作用。