时间戳

首页 > 解决方案 > 处理泛型方法时无法进行适当的抽象（C#）

c# - 处理泛型方法时无法进行适当的抽象（C#）

问题描述

我试图实现一个使用 System.Data.SQLite 功能的包装器，以便以良好的 OOP 方式摆脱重复代码。所以，我有以下通用方法：

public T SendSelectQuery<T>(string sql, Func<SQLiteDataReader, T> processResult) where T : IDBResult
{
    try
    {
        using (var dbConnection = new SQLiteConnection("path"))
        using (var cmd = new SQLiteCommand(sql, dbConnection))
        {
            dbConnection.Open();

            cmd.CommandType = CommandType.Text;

            using (SQLiteDataReader rdr = cmd.ExecuteReader(CommandBehavior.CloseConnection))
            {
                return processResult(rdr);
            }
        }
    }
    catch (Exception ex)
    {
        return T ??????
    }
}

T 是一个结果对象，即：

public interface IDBResult
{
    bool Completed { get; set; }
    string Exception { get; set; }
}

public abstract class CustomDBREsult : IDBResult
{
    public bool Completed { get; set; }
    public string Exception { get; set; }

    public string Payload { get; set; }
    public CustomDBREsult(bool Completed, string exception, string Payload)
    {
        this.Completed = Completed;
        this.Exception = exception;
        this.Payload = Payload;
    }
}

public class SuccessCustomDBResult : CustomDBREsult
{
    public SuccessCustomDBResult(string Payload) : base(true, string.Empty, Payload)
    {
    }
}

public class ErrorCustomDBResult : CustomDBREsult
{
    public ErrorCustomDBResult() : base(false, "exception", string.Empty)
    {
    }
}

我想打电话SendSelectQuery<CustomDBREsult>(...)并获取一个CustomDBREsult孩子的实例。

您可能已经注意到问题发生在catch我需要返回T对象的段中，但我无法实例化从CustomDBREsult.

我可以像这样更改SendSelectQuery<T>to的返回类型IDBResult并在catch段 smth 中返回：

public class DefaultDBError : IDBResult
{
    public bool Completed { get; set; } = false;
    public string Exception { get; set; } = "db exception";
}

但在这种情况下，我需要将 from 的结果SendSelectQuery<T>转换为IDBResultto T。而且这似乎不是一个很好的做法。

IDBResult res = DBMethods.SendSelectQuery<CustomDBREsult>("sql query", processResult);
if (res is CustomDBREsult cdbres)
{

}
else if (res is DefaultDBError ddberror)
{

}

另一种选择是“抬起”try catch块并SendSelectQuery<T>在其中使用，但我需要在我使用的任何地方复制这个块SendSelectQuery<T>并仍然转换IDBResult为T.

如果 smn 设法理解我，将不胜感激您的意见。我想我的问题是对逻辑进行良好的抽象。

标签： c#design-patternsabstraction

解决方案

我认为您在这里过度设计了问题。

首先，IDBResult 接口并没有真正起到任何作用，所以你可以取消它。

拥有一个抽象类，然后您派生该类的成功和失败版本，它们本身似乎没有做任何事情，除了表明它是失败还是成功，也没有真正起到很多作用。

我会将您的自定义数据库结果更改为通用数据库，就像这样，因为您正在谈论在原始 Select 中使用 T 作为有效负载。

   public class CustomDBREsult<T>
   {
     public bool Completed { get; set; }
     public string Exception { get; set; }

    public T Payload { get; set; }
    public CustomDBREsult(bool Completed, string exception, T Payload)
    {
        this.Completed = Completed;
        this.Exception = exception;
        this.Payload = Payload;
    }
  }

再说一次，除非这是一个你正在挑战自己的练习，以提高你的设计技能，或者类似的，有很多很好的库已经为你做了大部分这些事情，并且可以节省重新发明轮子.

推荐阅读