麻烦与大量的代码重复

时间:2014-12-04 19:49:48

标签: c# generics android-sqlite maintainability

我有很多方法在大部分方法中遵循相同的算法,理想情况下我希望能够调用一个消除大量代码重复的泛型方法。

我有很多类似下面的方法,我最好希望能够打电话 Save<SQLiteLocation>(itemToSave);但是自从这些方法以来我遇到了很多麻烦 SQLiteConnection.Find<T>在泛型中不接受像T这样的抽象数据类型。

有没有办法解决这个问题,如果我能解决这个问题,我会节省多达150行代码

这是我的代码:

    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;
    }

这里有另一种方法,看看我的虚线下方的两种方法中的代码基本上是一样的

    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;
    }

2 个答案:

答案 0 :(得分:1)

你不应该做这样的事情: 注意:ICommonInterface是您希望用作T的任何允许类之间通用的任何内容。最好查看代码,公开PrimaryKey属性的接口或类。

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;
}

编辑:在方法中添加了new()约束。

答案 1 :(得分:0)

由于某种原因,它抛出了一个不受支持的异常 我用的时候:

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

下面的代码解决了我的困境,感谢大家的帮助,我喜欢这个网站! 另外我在T上添加了另一个约束,因为T必须是非抽象类型,而界面就是我想的

    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;
        }
    }