處理泛型方法時無法進行適當的抽象 (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 是結果 object，即：

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子實例。

正如您可能已經注意到的那樣，問題出現在我需要返回T object 的catch段中，但我無法實例化從CustomDBREsult派生的適當錯誤 object。

我可以將SendSelectQuery<T>的返回類型更改為IDBResult並返回catch段 smth，如下所示：

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> ，但我需要在使用SendSelectQuery<T>的任何地方復制此塊並且仍然將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;
    }
  }

不過，再次重申，除非這是您要挑戰自己的具體練習，以提高您的設計技能等，否則有很多不錯的庫已經為您完成了大部分這些工作，並且可以節省重新發明輪子的時間.