继承类的泥潭，如何让这段代码可维护

Question

我想创建可维护的代码，但是这种继承情况给我带来了问题。

问题在于我的第二个名为 InitUserExtension 的数据库帮助程序类。

由于 UserExtension 继承自 User，我必须确保将 InitUser 助手中的任何更改镜像到 InitUserExtension。

我真的不喜欢这个，因为它容易出错，有什么解决办法？

我的班级定义：

public class User
{
    public string Name {get; set; }
    public string Age { get; set; }
}

public class UserExtension : User
{
    public string Lastname {get; set; }
        public string Password {get; set; }

}

我的数据库助手：

public static SqlDataReader InitUser(SqlDataReader dr)
{
      User user = new User();

      user.Name = Convert.ToString(dr["name"]);
      user.Age ...
}


public static SqlDataReader InitUserExtension(SqlDataReader dr)
{
      UserExtension user = new UserExtension();


      // note: mirror attributes from User
      user.Name = Convert.ToString(dr["name"]);
      user.Age ...



      user.Lastname = Convert.ToString(dr["lastname"]);
      user.Password = ....;
}

Answer 1

如何将 Name 等的处理转移到一个方法中（接受一个 User 或一个 T : User），然后从两者中调用它？

private static void InitUser(User user, SqlDataReader dr)
{ // could also use an interface here, or generics with T : User
  user.Name = Convert.ToString(dr["name"]);
  user.Age ...
}


public static User InitUser(SqlDataReader dr)
{
    User user = new User();
    InitUser(user, dr);
    return user;
}

public static UserExtension InitUserExtension(SqlDataReader dr)
{
  UserExtension user = new UserExtension();
  InitUser(user, dr);
  user.Lastname = Convert.ToString(dr["lastname"]);
  user.Password = ....;
  return user;
}

作为替代方案，您可以使用泛型减少行数，但增加复杂性：

private static T InitUserCore<T>(SqlDataReader dr) where T : User, new()
{
    T user = new T();
    // ...
    return user;
}
public static User InitUser(SqlDataReader dr)
{
    return InitUserCore<User>(dr);
}
public static UserExtension InitUserExtension(SqlDataReader dr)
{
    UserExtension user = InitUserCore<UserExtension>(dr);
    // ...
    return user;
}

Answer 2

为什么不从InitUserExtension 方法调用InitUser。让基初始化处理基类属性，让扩展初始化器处理扩展类属性。

Answer 3

如果你更喜欢助手，对助手也一样

public class InitUser
{
    public InitUser(SqlDataReader dr, User u) { }
}

public class InitUserExtension : InitUser
{
    public InitUserExtension(SqlDataReader dr , UserExtension u) : base(dr, u) { }
}

Answer 4

我不确定您为什么没有将分配代码放在类本身中，但这可能与您获取信息以存储它的方式有关...

我的解决方案是重写如下：

public class User
{
    public string Name {get; set; }
    public string Age { get; set; }

    public User(DataReader dr)
    {
       user.Name = Convert.ToString(dr["name"]);
       user.Age ...
    }
}

public class UserExtension : User
{
    public string Lastname {get; set; }
    public string Password {get; set; }

    public UserExtension(DataReader dr):base(dr)
    {
        user.Lastname = Convert.ToString(dr["lastname"]);
        user.Password = ....;

    }
}

里程可能会因您的情况而异

这样，调用类似

var MyExtension = new UserExtension(dr); 

将适当地填充所有字段。