创建静态方法时

Question

我使用 resharper 并且 resharper 建议我将一种方法声明为静态方法，而另一种方法则不是。但我不明白为什么其他方法不能是静态的？

建议使用静态方法

 private static string Prehod(string retazec)
    {
        var pole = retazec.ToCharArray();
        var output = "";
        char? temp = null;

        for (var i = 0; i < pole.Length; i++)
        {
            if (temp == null)
            {
                temp = pole[i];
                continue;
            }

            output += pole[i].ToString() + temp.ToString();
            temp = null;
        }
        return output;
    }

不建议使用静态方法

 public string HashToString(string hash,int dlzka)
    {
        var hashChar = hash.Substring(0, dlzka*2);
        var retazec = "";

        for (var i = 0; i < dlzka*2; i++)
        {
            if(i%2 != 0)
            {
                retazec += hashChar.Substring(i, 1);
            }
        }
        return retazec;
    }

Answer 1

我看不出为什么第二种方法不能是静态的，至于为什么 Resharper 建议一种而不是另一种......你必须询问 Resharper 开发人员。请记住，它是一种工具，而不是一本规则手册。

Answer 2

Resharper 不会为您的类的公共成员提供建议，因为它们可以被其他类使用。

但如果公共实例方法根本不需要实例，它仍然是一个标志（不是说“气味”）。

Answer 3

我不明白为什么您的第二种方法不能是静态的。据我所知，它不访问任何实例字段。

将其设置为静态不会有问题。这么多你的问题，为什么它不能是静态的。至于 ReSharper，我不知道为什么它不建议将其作为静态方法，而将其用于其他方法。

Answer 4

您的第一个方法是私有的，并且可能在类中静态使用。例如：var ehy = Prehod("testing");.

第二种方法是公开的。可能你已经在某个地方使用它了：

   var okBaby = new MyClass();
   Console.WriteLine(okBaby.HashToString("something", 10));

resharper 可能认为这样做是有原因的，并且不建议进行更改。

不过，这只是一次盲目的尝试。

Answer 5

这两种方法都可以是静态的：它们不使用类的任何实例成员。并且这两种方法都可以是实例方法。但是方法是否是静态的应该由它的可能用途来暗示，而不是由 ReSharper 暗示。您是否希望您的方法作为实例的一部分被调用：

YourClass instance = new YourClass();
string x = instance.Prehod(...);

或者你希望你的方法作为类的一部分被调用：

string x = YourClass.Prehod(...);

在您意识到哪个是最好的之前，您可能会在开发过程的后期进行。如果你想让你的类的用户“感觉”它作用于实例（new ClassName()），你必须选择实例方法。如果您确实需要在没有实例的情况下使用该方法（ClassName.YourMethod()），您必须选择静态。

更新：将私有方法设为静态并不常见，除非您需要其他静态方法来调用私有静态方法。如果只有其他实例方法使用您的私有方法，则没有理由将其设为静态方法。