Class using only common functions. Why is that bad?

Question

I have a class called MemberhipHelper that I use in my ASP.NET project. It looks like this:

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

I read somewhere that it’s nice to have a class with only shared functions, but I don’t remember where.

Why is it bad and how can I improve it?

thanks

Answer 1

From the name you used for your functions, it seems that all functions describe the user's properties (for example, whether the user is an administrator).

**, IRole.

** , . - .

Answer 2

, , , .

, , , , , Helper/Extension.