Probleme mit viel Code-Duplizierung

Question

Ich habe viele Methoden, die in großen Teilen den gleichen Algorithmus folgen, und ich möchte idealerweise in der Lage sein, Aufrufe an eine generische Methode, die eine Menge Code-Duplikation beseitigt.

Ich habe Tonnen von Methoden wie die unten, würde ich optimal nur in der Lage sein möge, rufen Save<SQLiteLocation>(itemToSave);, aber ich habe sehr viel Mühe, da, dass die Methoden SQLiteConnection.Find<T> wird einen abstrakten Datentyp wie T nicht akzeptieren in Generika.

Gibt es eine Möglichkeit, dies zu umgehen, wenn ich es ich so viele wie 150 Zeilen Code

meinen Code Hier würde repariert könnte sparen:

public bool SaveLocation(ILocation location, ref int primaryKey) 
    { 
     var dbConn = new SQLiteConnection (dbPath); 

     SQLiteLocation itemToSave = new SQLiteLocation(); 
     itemToSave.LocationName = location.LocationName; 
     itemToSave.Latitude = location.Latitude; 
     itemToSave.Longitude = location.Longitude; 
     itemToSave.PrimaryKey = location.PrimaryKey; 

    ---------------------------------------------------------------------------------------- 

     SQLiteLocation storedLocation = dbConn.Find<SQLiteLocation> 
             (x => x.PrimaryKey == location.PrimaryKey); 

     if (storedLocation != null) 
     { 
      dbConn.Update(itemToSave); 
      return true; 
     } 

     else if (storedLocation == null) 
     { 
      dbConn.Insert(itemToSave); 
      primaryKey = itemToSave.PrimaryKey; 
      return true; 
     } 
     return false; 
    } 

Und hier eine andere Methode sehen, wie der Code in beiden Verfahren unter meiner gestrichelte Linie ist im Grunde das gleiche

public bool SaveInvitation(IInvitation invitation, ref int primaryKey) 
    { 
     var dbConn = new SQLiteConnection(dbPath); 

     SQLiteInvitation itemToSave = new SQLiteInvitation(); 
     itemToSave.GroupName = invitation.GroupName; 
     itemToSave.InviterName = invitation.InviterName; 
     itemToSave.ParseID = invitation.ParseID; 
     itemToSave.GroupParseID = invitation.GroupParseID; 
     itemToSave.PrimaryKey = invitation.PrimaryKey; 

--------------------------------------------------------------------------------------- 

     SQLiteInvitation storedInvitation = dbConn.Find<SQLiteInvitation> 
              (x => x.PrimaryKey == invitation.PrimaryKey); 

     if (storedInvitation != null) 
     { 
      dbConn.Update(itemToSave); 
      return true; 
     } 
     else if (storedInvitation == null) 
     { 
      dbConn.Insert(itemToSave); 
      primaryKey = itemToSave.PrimaryKey; 
      return true; 
     } 
     return false; 
    } 

Answer 1

Aus irgendeinem Grund warf eine nicht unterstützte Ausnahme wenn ich verwendet:

T storedItem = dbConn.Find<T>(x => x.PrimaryKey == item.PrimaryKey); 

Der Code unten, obwohl mein Leiden festgelegt, danke für die Hilfe jeder, ich liebe diesen Ort! Auch hinzugefügt ich eine andere Einschränkung auf T da T eine nicht abstrakte Art sein und eine Schnittstelle ist nur, dass ich

private void SaveItem<T>(T item, ref int primaryKey) 
     where T : ISQLiteClass, new() 
    { 
     var dbConn = new SQLiteConnection(dbPath); 

     T storedItem = dbConn.Find<T>(primaryKey); 

     if (storedItem != null) 
     { 
      dbConn.Update(item); 
     } 
     else if (storedItem == null) 
     { 
      dbConn.Insert(item); 
      primaryKey = item.PrimaryKey; 
     } 
    } 

Answer 2

Sollten Sie so etwas zu tun, nicht in der Lage: Hinweis: ICoMM onInterface ist alles, was zwischen allen zulässigen Klassen, die Sie als T verwenden möchten, üblich ist. Vorzugsweise betrachten Sie Ihren Code, eine Schnittstelle oder Klasse, die die PrimaryKey-Eigenschaft verfügbar macht.

public bool SaveItem<T>(T item, ref int primaryKey) where T : ICommonInterface, new() 
{ 
    var dbConn = new SQLiteConnection(dbPath); 


    T storedItem = dbConn.Find<T>(x => x.PrimaryKey == item.PrimaryKey); 

    if (storedItem != null) 
    { 
     dbConn.Update(item); 
     return true; 
    } 
    else if (storedItem == null) 
    { 
     dbConn.Insert(item); 
     primaryKey = item.PrimaryKey; 
     return true; 
    } 
    return false; 
} 

EDIT: Die neue() Einschränkung der Methode hinzugefügt.