方法可以做成静态，但应该吗？

Question

Resharper喜欢指出每个asp.net页面可以变为静态的多个函数。如果我让它们静止，对我有帮助吗？我应该将它们设置为静态并将它们移动到实用程序类吗？

Answer 1

静态方法与实例方法 点击 C＃的 10.2.5静态和实例成员语言规范解释了差异。通常，静态方法可以提供比实例方法更小的性能增强，但仅限于极端情况下（请参阅此答案关于那个的更多细节。

FxCop或代码分析中的规则CA1822声明：

  

＆quot;在[将成员标记为静态]之后，编译器将向这些成员发出非虚拟呼叫站点，这将阻止对   确保当前对象指针的每个调用的运行时   非空。这可以带来可衡量的性能提升   性能敏感的代码。在某些情况下，无法访问   当前对象实例表示正确性问题。“

公用事业类
除非在您的设计中有意义，否则不应将它们移动到实用程序类。如果静态方法与特定类型相关，如 ToRadians（双度）方法与表示角度的类有关，则该方法作为该类型的静态成员存在是有意义的（注意，这是一个令人费解的例子，用于示范）。

Answer 2

在我看来，性能，命名空间污染等都是次要的。问自己什么是合乎逻辑的。该方法是在逻辑上对该类型的实例进行操作，还是与类型本身相关？如果是后者，请将其设为静态方法。如果它与不受您控制的类型相关，则只将其移动到实用程序类中。

有时会有一些逻辑上对某个实例起作用的方法，但是没有碰巧使用任何实例的状态还是。例如，如果你正在构建一个文件系统并且你已经获得了一个目录的概念，但是你还没有实现它，你可以编写一个返回文件系统对象类型的属性，它总是只是＆QUOT;文件＆QUOT; - 但它在逻辑上与实例相关，因此应该是实例方法。如果要将方法设置为虚拟，这也很重要 - 您的特定实现可能不需要状态，但派生类可能不需要。 （例如，询问集合是否为只读集合 - 您可能尚未实现该集合的只读形式，但它显然是集合本身的属性，而不是类型。）

Answer 3

将一个方法标记为类中的 static ，很明显它不使用任何实例成员，这有助于了解何时浏览代码。

你不一定要将它移到另一个类，除非它是由另一个与概念相关联的类共享的。

Answer 4

我确信这种情况不会发生在你的情况下，而是一种“难闻的气味”。我已经在一些代码中看到过，我不得不通过维护大量的静态方法而受到影响。

不幸的是，它们是假定特定应用程序状态的静态方法。 （为什么我们每个应用程序只有一个用户！为什么不让User类在静态变量中跟踪它？）它们是访问全局变量的美化方式。他们也有静态构造函数（！），这几乎总是一个坏主意。 （我知道有几个合理的例外）。

但是，静态方法在分解出实际上并不依赖于对象实例状态的域逻辑时非常有用。它们可以使您的代码更具可读性。

请确保你把它们放在正确的位置。静态方法是否会侵入性地操纵其他对象的内部状态？可以说一个好的案例，他们的行为属于这些类之一吗？如果你没有正确地分离问题，你可能会在以后遇到麻烦。

Answer 5

这是有趣的读物：

http://thecuttingledge.com/?p=57

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

但这是resharper documentaion所说的： http://confluence.jetbrains.net/display/ReSharper/Member + +可以被制成+ +静态

Answer 6

添加到@Jason True的回答，重要的是要意识到只在方法上加上'静态'并不能保证方法是“纯粹的”。对于声明它的类，它将是无状态的，但它可能会访问具有状态（应用程序配置等）的其他“静态”对象，这可能并不总是坏事，但其中一个原因是我个人倾向于选择静态方法，如果它们是纯粹的，你可以单独测试和推理它们，而不必担心周围的状态。

Answer 7

在特定场景中，您应该做一些最具可读性和直观性的内容。

性能参数不是很好的，除非在最极端的情况下，因为实际发生的唯一事情就是将一个额外的参数（ this ）推送到堆栈上，例如方法。

Answer 8

对于类中的复杂逻辑，我发现私有静态方法在创建隔离逻辑时非常有用，其中实例输入在方法签名中明确定义，并且不会发生实例副作用。所有输出必须通过返回值或out / ref参数。将复杂的逻辑分解为无副作用的代码块可以提高代码的可读性和开发团队对它的信心。

另一方面，它可能会导致一个被实用方法激增污染的类。像往常一样，团队编码约定的逻辑命名，文档和一致应用可以缓解这种情况。

Answer 9

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

Answer 10

如果函数在许多页面上共享，您也可以将它们放在基页类中，然后让所有使用该功能的asp.net页面继承它（并且函数仍然可以是静态的）。 / p>

Answer 11

使方法静态意味着您可以从类外部调用该方法，而无需先创建该类的实例。在使用第三方供应商对象或加载项时，这很有用。想象一下，如果你必须首先创建一个Console对象“con”。在调用con.Writeline（）;

之前

Answer 12

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

Answer 13

只是我的tuppence：将所有共享静态方法添加到实用程序类允许您添加

using static className; 

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

我不知道这是不是很好的做法。我有很多东西可以学习C＃4/5以及如此多的遗留代码来重构我只是想让Roselyn技巧引导我。

乔伊