复杂的共享/静态成员的最佳实践

Question

我接管了ASP.NET应用程序，并在应用程序的多个类中找到了它。 程序员之前定义了几个共享/静态变量，它们在整个应用程序中充当“复杂枚举”。 作为一个相当新的程序员，这似乎不是最佳实践。

这是一个例子：

Public Shared SecureCommentsWrite As New Task("Secure Comments Write")
Public Shared SecureCommentsRead As New Task("Secure Comments Read")
Public Shared EditEmergencyContact As New Task("Edit Emergency Contact")
Public Shared DisplayPersonalReferences As New Task("Display Personal References")
Public Shared EditPersonalReferences As New Task("Edit Personal References")

构造函数进行描述，然后使用存储过程（数据库为SQL Server）从数据库中加载ID密钥。这似乎是一个好主意，因为我们将此应用程序部署到多个数据库并希望确保加载ID密钥它在该数据库中，以防它发生更改。 但是，由于在应用程序中实际上有数百个，因此第一次加载要花一些时间。

这是否被认为是最佳实践，如果不是，那么在这种情况下什么被认为是最佳实践？

Answer 1

对我来说，这是一种可怕的做法，如果您要告诉我，对于上面这些行和Task构造函数中的每一行，都将调用一个单独的存储过程（即使存储了相同的存储过程）。

在这种情况下，最佳实践是重构，对该存储过程进行一次调用并对其进行修改，以返回所有ID和名称，然后有一个小的TaskManager或TaskLoader（无论如何）类，该类映射存储的结果并创建所有不带任何元素的元素进一步的数据库参与。

在这些情况下，请注意，SqlDataReader可能比DataSet更好，因为您只需要转发，只读取对数据的快速访问。

现在就从我这边开始;-)

Answer 2

您可能在这里想要一个工厂类（ TaskFactory ？）。

它应该加载（一次）代表Task项整个列表的数据表或数据集。 这可以由构造函数完成，也可以[懒惰地]在执行实例的第一个请求时完成。

然后，而不是每次都访问数据库，工厂的CreateInstance()方法应查询预加载的数据以获取所需的信息。

Answer 3

在对象构造函数中或任何其他可能耗时的操作中运行数据库查询/存储过程不是一个好习惯。 如果在Task对象的初始化期间出了点问题，由于它被声明为静态/共享成员，则可能会抛出TypeInitializationException，并且您的应用程序将变得无法使用。

我会将这些成员视为某种应用程序配置。 在Application_Start期间执行一个存储过程，该过程将一次（而不是一个接一个地）带走所有数据，并创建配置对象。

Answer 4

拥有代表常数值的静态字段列表并没有什么天生的错误。 正如您所指出的，它基本上与枚举相同，并且Microsoft自己在自己的某些库中完成了此操作。

就是说，如果初始化这些字段会导致明显的速度下降（并且由于它们每个都在访问数据库，这并不令人感到意外），则可以使用一些技术来缩短加载时间。 一个明显的解决方案是采用延迟加载-换句话说，除非绝对必要，否则请不要访问数据库！ 这从根本上摊销了在程序的整个生命周期中命中数据库以初始化这些字段的成本，从而使您可以更快地启动，以换取其他地方性能的降低。

当然，如果必须一次延迟加载100或1000，则这也不是理想的解决方案。 在这种情况下，您只是解决了巨大的延迟，而不是将其分解。

另一个想法是提高负责检索这些ID的SQL的效率。 您可能编写了一个Initialize()方法，该方法执行一个查询以一次提取您需要的所有ID，而不是在100个不同的查询中执行。 这几乎肯定会更快。