处理泛型方法时无法进行适当的抽象 (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 子实例。

您可能已经注意到问题出现在 catch 段中，我需要 return 一个 T 对象，但我无法实例化从 CustomDBREsult.

我可以将 SendSelectQuery<T> 的 return 类型更改为 IDBResult 和 catch 段中的 return 像这样：

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)
{

}

另一种选择是“提升”try catch 块并在其中使用 SendSelectQuery<T>，但我需要在使用 [=19= 的任何地方复制此块] 并仍然将 IDBResult 转换为 T.

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

Answer 1

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

首先，IDBResult 接口实际上没有任何用途，因此您可以取消它。

拥有抽象 class，然后您可以导出 class 的成功版本和失败版本，它们本身似乎没有做任何事情，除了指示它是失败还是成功, 也没有真正起到很多作用。

我会将您的自定义数据库结果更改为通用结果，就像这样，因为您正在谈论在原始 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;
    }
  }

不过，再说一遍，除非这是您要挑战自己的具体练习，以提高您的设计技能等，否则有很多不错的库已经为您完成了大部分这些工作，并且将节省重新发明轮子。

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

Cant make an appropriate abstraction when dealing with generic method (C#)

c#

abstraction

design-patterns