asp.net - 仅具有共享函数的类 - 为什么它不好？

Question

我有一个名为 MembershipHelper 的类，我在 ASP.NET 项目中使用它。它看起来像这样:

Public Class MembershipHelper

    Public Shared Function IsMultiStoreUser() As Boolean
     return Roles.IsUserInRole(....)
    End Function

    Public Shared Function IsAdmin() As Boolean
     return Roles.IsUserInRole(....)
    End Function

    Public Shared Function IsReaderOnly() As Boolean
     return Roles.IsUserInRole(....)
    End Function

End Class

我在某处读到，拥有一个仅具有共享函数的类不是一个好主意 - 但我不记得在哪里。

为什么这样不好？我该如何改进？

谢谢

Answer 1

从您用于函数的命名来看，似乎所有函数都描述了用户的属性(例如，用户是否是管理员)。

因此，将这些函数替换为用户对象的属性或让用户实现 IRole 接口(interface)似乎更自然**。

** 我并不是说你的设计好或坏。根据上下文，这样的帮助器类很可能是合理的。