1

我有一个Administrator具有其属性的模型,但它也包含许多静态方法,这些方法并没有真正与当前对象本身有任何联系,例如GetByCredentials(string username, string password);. 是否有可能在其他地方划分静态方法并使对象尽可能纯净?

例子

public class Administrator : Entity
{
    // OBJECT START
    public int Id { get; set; }
    public DateTime CreatedDateTime { get; set; }
    public DateTime UpdatedDateTime { get; set; }
    public string Username { get; set; }
    public string Password { get; set; }
    public string PasswordSalt { get; set; }

    public void SetNewPassword(string password)
    {
        var cryptoService = new PBKDF2();
        this.Password = cryptoService.Compute(password);
        this.PasswordSalt = cryptoService.Salt;
    }

    public override void OnBeforeInsert()
    {
        this.CreatedDateTime = DateTime.Now;
        this.UpdatedDateTime = DateTime.Now;

        this.SetNewPassword(this.Password);
    }

    public override void OnBeforeUpdate()
    {
        this.UpdatedDateTime = DateTime.Now;
    }
    // OBJECT END

    // Now I have multiple static methods that do not really
    // have anything to do with current object
    public static Administrator GetByCredentials(string username, string password)
    {
        var db = new MainDataContext();
        var admin = db.Administrators.SingleOrDefault(x => x.Username == username);
        if (admin == null) return null;

        ICryptoService cryptoService = new PBKDF2();
        var hash = cryptoService.Compute(password, admin.PasswordSalt);

        if (hash == admin.Password) return admin;
        return null;
    }

    public static bool IsCurrentIpBanned
    {
        get
        {
            const int minutesBlocked = 5;
            const int maxLoginCount = 5;

            var db = new MainDataContext();
            var loginCount = db.AdministratorAuthorizationLogs.AsEnumerable().Count(x => x.Ip == HttpContext.Current.Request.UserHostAddress && x.CreatedDateTime.AddMinutes(minutesBlocked) > DateTime.Now && x.IsSuccess == false);

            return loginCount > maxLoginCount;
        }
    }

    public static void LogSuccess(Administrator admin)
    {
        Administrator.Log(admin, true);
    }

    public static void LogFailure(Administrator admin)
    {
        Administrator.Log(admin, false);
    }

    private static void Log(Administrator admin, bool success)
    {
        var db = new MainDataContext();
        db.AdministratorAuthorizationLogs.Add(new AdministratorAuthorizationLog
        {
            Username = admin.Username,
            Password = admin.Password,
            Ip = HttpContext.Current.Request.UserHostAddress,
            IsSuccess = success
        });

        db.SaveChanges();
    } 
}
4

2 回答 2

1

这里有几个选项,但主要是 C# 类是分离关注点工具。

最明显的是在它们自己的抽象中捕获这些东西。例如,作为不同类或类似GetByCredentials类的(非静态)成员可能会更好。Authority该类只需要能够创建一个Administrator类型。

您还可以使用扩展方法。一个可能的候选者是Log,它将 aAdministrator作为参数并仅使用其上的公共设施。扩展方法是在一个单独的类中定义的,但允许您“就好像”它们是扩展类的成员一样使用它们,例如:

public static class AdministratorExtensions
{
    public static void log( this Administrator admin, bool success ) { ... }
}

var admin = new Administrator();
admin.Log( true );

关键是识别真正的抽象,并通过以合理的方式组合它们来构建您的系统。分离关注点是该过程的一部分。

于 2013-08-01T15:03:37.943 回答
0

这是您的班级“知道太多”的暗示。管理员类应该只知道管理员关心的事情。他不应该能够查询数据库和检索实体。

您应该查看存储库模式。尝试将您的应用程序分解为多个层。例如,您可能有一个 DataRepository 类,其主要关注点是查询和更新数据库实体。

于 2016-09-05T18:07:50.893 回答