Question

我是初学者。我的应用程序中有一个SQLite数据库。我已成功添加添加，删除和更新功能。我正在尝试实现刷卡现在删除，但是，我正面临当前的问题：有时我刷卡并删除项目后，它被成功删除（从数据库中），有时它没有，有时会删除上面的项目，有时它上面的项目是2个id。我做了很多试验和错误，但我无法确认为什么会这样。总是会发生这个项目从视图中删除，因为它没有自己的卡片了（我正在使用带有recyclerview的cardview）。

以下是相关代码（如果我没有发布任何重要代码，请告诉我，我试着简化这个代码）：

MainActivity.java（这是recyclerview显示卡片的地方）

RecyclerView rv;
MyAdapter adapter;
ArrayList < Player > players = new ArrayList < > ();
int posToDelete;
DBAdapter db = new DBAdapter(this);

//SWIPE TO DELETE
ItemTouchHelper.SimpleCallback simpleItemTouchCallback =
    new ItemTouchHelper.SimpleCallback(0,
        ItemTouchHelper.RIGHT) {
        @Override
        public boolean onMove(RecyclerView recyclerView,
            RecyclerView.ViewHolder viewHolder,
            RecyclerView.ViewHolder target) {
            return false;
        }

        @Override
        public void onSwiped(RecyclerView.ViewHolder viewHolder, int direction) {

            int position = viewHolder.getAdapterPosition();
            players.remove(position);
            posToDelete = (position);

            delete(position);
            adapter.notifyDataSetChanged();
        }
    };

//recycler
rv = (RecyclerView) findViewById(R.id.myRecycler);


rv.setLayoutManager(new LinearLayoutManager(this));
rv.setItemAnimator(new DefaultItemAnimator());
rv.hasFixedSize();

ItemTouchHelper itemTouchHelper = new
ItemTouchHelper(simpleItemTouchCallback);
itemTouchHelper.attachToRecyclerView(rv);

//DELETE METHOD
public void delete(int id) {
    db.openDB();
    long result = db.Delete(id + 1);

    if (result > 0) {
        //Deleted toast appears but it doesn't get removed from database?
        Toast.makeText(MainActivity.this, "Deleted",
            Toast.LENGTH_SHORT).show();

    } else {
        Toast.makeText(MainActivity.this, "Error",
            Toast.LENGTH_SHORT).show();
    }

    db.close();
}

DBAdapter.java

Context c;
SQLiteDatabase db;
DBHelper helper;

public DBAdapter(Context ctx)
{
    this.c=ctx;
    helper=new DBHelper(c);
}

//OPEN DB
public DBAdapter openDB()
{
    try
    {
        db=helper.getWritableDatabase();
    }catch (SQLException e)
    {
        e.printStackTrace();
    }

    return this;
}

//CLOSE DB
public void close()
{
    try
    {
        helper.close();
    }catch (SQLException e)
    {
        e.printStackTrace();
    }

}

//DELETE FROM DATABASE <---------
public long Delete(int id)
{
    try
    {
        return db.delete(Constants.TB_NAME,Constants.ROW_ID+" =?",new 
String[]{String.valueOf(id)});

    }catch (SQLException e)
    {
        e.printStackTrace();
    }


    return 0;
}

作为一个附带问题，在调试找到问题时，我注意到在成功删除数据库中的对象（使用一直有效的删除功能，而不是刷卡删除）后，数据库就离开了那个id点空。所以例如，如果我有对象1,2,3,4,5和6，当我删除5并创建新对象A和B时，似乎数据库将它们存储为1,2,3,4，（这里没什么），6，A，B。

这是一个问题吗？会有什么不好的后果吗？或者这完全正常吗？

谢谢。

****编辑****你们是对的，传递给delete()方法的位置不是对象的ID，感谢你指出这一点。

我尝试使用ViewHolder.getItemId()方法，但它返回long。我尝试将其转换为integer并将整个方法参数更改为接受long而不是integer，但我总是得到的id值是-1，这无法修复任何东西。

在应用程序的其他位置，我使用onClickListener获取该项目的ID，因为它会向另一项活动发送intent。我想也许我可以在没有任何用户交互的情况下以其他方式获取Id？因为当用户滑动删除时，我无法等待onClickListener触发。

Answer 1

您是否检查了传递给DBAdapter的ID？

看起来您正在根据适配器位置从数据库中删除该项，是否可能是数据库中项目的这些ID与适配器位置不匹配？

之前我已经完成了这个刷卡到删除功能，我很确定我使用了项目ID，例如来自玩家的ArrayList，并根据数据库中的项目删除了项目。

Answer 2

感谢上述评论，我发现错误在于： @覆盖 public void onSwiped（RecyclerView.ViewHolder viewHolder，int direction）{

        int position = viewHolder.getAdapterPosition();
        players.remove(position);
        posToDelete = (position);

        delete(position);
        adapter.notifyDataSetChanged();
    }

我通过创建一个新变量并将该位置的id存储在变量中，然后将新变量传递给delete方法来修复此问题，如下所示：

                        int position = viewHolder.getAdapterPosition();
                        int checkingId;
                        checkingId = players.get(position).getId();
                        players.remove(position);
                        delete(checkingId);
                        adapter.notifyDataSetChanged();
                    }

滑动删除大部分时间都不起作用，它会从View中删除数据，但不会从SQLite数据库

2 个答案: