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

Question

我试图实现一个使用 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 child 的实例。

您可能已经注意到问题出现在catch 段中，我需要返回一个T 对象，但我无法实例化从CustomDBREsult 派生的适当错误对象。

我可以将SendSelectQuery<T> 的返回类型更改为IDBResult 并在catch 段中返回，如下所示：

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

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

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

}
else if (res is DefaultDBError ddberror)
{

}

另一种选择是“提升”trycatch 块并在其中使用SendSelectQuery<T>，但我需要在使用SendSelectQuery<T> 的任何地方复制此块，并且仍将IDBResult 转换为@987654341 @。

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

Answer 1

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

首先，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;
    }
  }

同样，除非这是一项您正在挑战自己的练习，以提高您的设计技能，或者类似的，有很多好的库已经为您完成了大部分这些工作并且可以节省重新发明轮子。