AndroidのRecyclerviewにおいてリストのスワイプ削除を行っても、リストに表示されるデータが更新されません。

Question

＜実現したいこと＞
Recyclerview（入力保存のあるSQLiteデータベース）を使ったリストのデータをスワイプ削除する。

＜問題点＞
ItemTouchHelper（onSwiped）によって、Recyclerviewで作ったリストのデータをスワイプし、画面上から削除する動き自体はできています。
しかし、その状態で新しいデータを追加保存したり、他の画面から遷移したりすると、削除したはずのデータがそのままリストに表示されてしまいます。
notifyDataSetChanged();を入れても、リストに表示されるデータを更新することができない状態です。

どのようにすれば解決できるのでしょうか。
アドバイスなど頂けたら幸いです。よろしくお願いします。

１（MaintenanceDataShow.java）・・・データを読み込み、Recyclerviewのリストに表示する

package torach.java_conf.gr.jp.carmaintenance;

import android.content.Intent;
import android.database.Cursor;
import android.database.sqlite.SQLiteDatabase;
import android.support.design.widget.FloatingActionButton;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.support.v7.widget.DividerItemDecoration;
import android.support.v7.widget.LinearLayoutManager;
import android.support.v7.widget.RecyclerView;
import android.support.v7.widget.helper.ItemTouchHelper;
import android.view.KeyEvent;
import android.view.Menu;
import android.view.MenuItem;
import android.view.View;

import java.util.ArrayList;

public class MaintenanceDataShow extends AppCompatActivity {

    private MaintenanceDataHelper helper;
    private Cursor cursor;
    private RecyclerView recyclerView;
    private LinearLayoutManager manager;
    private RecyclerView.Adapter adapter;
    private ArrayList<ListItem> data;
    private SQLiteDatabase db;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_maintenance_data_show);

        //FloatingActionButtonの設置
        FloatingActionButton fab = (FloatingActionButton) findViewById(R.id.fab);
        fab.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                Intent intent = new Intent(getApplication(), InputMaintenanceData.class);
                startActivity(intent);
            }
        });


        //SQLiteデータベース読み込み
        readData();

        //区切り線
        RecyclerView.ItemDecoration itemDecoration =
                new DividerItemDecoration(this, DividerItemDecoration.VERTICAL);
        recyclerView.addItemDecoration(itemDecoration);


        //スワイプ削除読み出し
        listsSwipe();

    }

    //SQLite保存データ呼び込み
    public void readData() {

            try {

                helper = new MaintenanceDataHelper(MaintenanceDataShow.this);
                db = helper.getReadableDatabase();

                //テーブル内のデータを配列に渡す
                String[] columns = {"date", "category", "price", "notes"};

                cursor = db.query("maintenanceDB", columns, null, null, null, null, null);

                if(cursor != null && cursor.getCount() > 0) {

                    data = new ArrayList<>();

                    while (cursor.moveToNext()) {

                        ListItem dataItem = new ListItem();

                        //dataItem.setId(cursor.getInt(cursor.getColumnIndex(MaintenanceDataHelper.ROW_ID)));
                        //dataItem.setId((new Random()).nextLong());

                        dataItem.setCategory(cursor.getString(cursor.getColumnIndex(MaintenanceDataHelper.ROW_CATEGORY)));
                        dataItem.setDate(cursor.getString(cursor.getColumnIndex(MaintenanceDataHelper.ROW_NAME)));
                        dataItem.setPrice(cursor.getString(cursor.getColumnIndex(MaintenanceDataHelper.ROW_PRICE)));
                        dataItem.setNotes(cursor.getString(cursor.getColumnIndex(MaintenanceDataHelper.ROW_NOTES)));
                        data.add(dataItem);
                    }
                }

            } finally {
                if (cursor != null) {
                    cursor.close();
                }
                db.close();
            }

        //リサイクルビューのレイアウト
        recyclerView = (RecyclerView) findViewById(R.id.Recycle_View_Layout);
        recyclerView.setHasFixedSize(true);

        //レイアウトマネージャー作成
        manager  = new LinearLayoutManager(this);
        manager.setOrientation(LinearLayoutManager.VERTICAL);
        recyclerView.setLayoutManager(manager);

        //アダプターをRecyclerManagerに設定
        adapter = new ViewAdapter(data);
        recyclerView.setAdapter(adapter);

    }

    //ItemTouchHelper.Simpleを使ったリストのスワイプ削除
    public void listsSwipe() {
        ItemTouchHelper.SimpleCallback mIth = new ItemTouchHelper.SimpleCallback(0, ItemTouchHelper.LEFT) {

                    @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();
                        //((ViewAdapter)recyclerView.getAdapter()).removeItem(position);
                        data.remove(position);
                        adapter.notifyDataSetChanged();

                    }

                };
        new ItemTouchHelper(mIth).attachToRecyclerView(recyclerView);

    }


    //オプションメニューの制御
    //表示
    @Override
    public boolean onCreateOptionsMenu(Menu menu) {
        // オプションメニューを作成する
        getMenuInflater().inflate(R.menu.option_menu_maintenancedatashow,menu);
        return super.onCreateOptionsMenu(menu);
    }

    //動作処理
    @Override
    public boolean onOptionsItemSelected(MenuItem item) {

        // オプションメニュー
        switch (item.getItemId()){
            case R.id.menu_Item5:
                moveToCarDataShow();
                break;
        }

        return super.onOptionsItemSelected(item);
    }

    //画面遷移
    //車種データ画面に遷移
    public void moveToCarDataShow() {
        Intent intent = new Intent(getApplication(), CarDataShow.class);
        startActivity(intent);
    }


    //バックボタン無効化（スプラッシュ画面に戻らない）
    @Override
    public boolean onKeyDown(int keyCode, KeyEvent event) {
        if(keyCode == KeyEvent.KEYCODE_BACK) {
            return false;
        }
        return super.onKeyDown(keyCode, event);
    }


}

２（MaintenanceDataHelper.java）・・・データベースヘルパー

package torach.java_conf.gr.jp.carmaintenance;

import android.content.Context;
import android.database.sqlite.SQLiteDatabase;
import android.database.sqlite.SQLiteOpenHelper;


public class MaintenanceDataHelper extends SQLiteOpenHelper {

    //テーブル情報定義
    public static final String TABLE_NAME = "maintenanceDB";
    public static final String ROW_ID = "_id";
    public static final String ROW_NAME = "date";
    public static final String ROW_CATEGORY = "category";
    public static final String ROW_PRICE = "price";
    public static final String ROW_NOTES = "notes";

    //データベースファイル名とバージョン情報
    public static final String DB_NAME = "maintenance.db";
    private static final int DB_VERSION = 1;

    //コンストラクタ
    public MaintenanceDataHelper(Context context) {

        super(context, DB_NAME, null, DB_VERSION);
    }

    //テーブル作成
    private static final String SQL_CREATE =

        "CREATE TABLE IF NOT EXISTS "
                + TABLE_NAME
                + " ("
                + ROW_ID
                + " INTEGER PRIMARY KEY AUTOINCREMENT,"
                + ROW_NAME
                + " TEXT,"
                + ROW_CATEGORY
                + " TEXT,"
                + ROW_PRICE
                + " TEXT,"
                + ROW_NOTES
                + " TEXT"
                + ");";


    //データベースのバージョンアップ
    @Override
    public void onCreate(SQLiteDatabase db) {
        db.execSQL(SQL_CREATE);
    }

    //データベースのバージョンアップ後のテーブル再作成
    @Override
    public void onUpgrade(SQLiteDatabase db, int oldVersion, int newVersion) {
        db.execSQL("DROP TABLE IF EXISTS maintenanceDB");
        onCreate(db);
    }
    
}

３（ViewAdapter.java）・・・アダプター

package torach.java_conf.gr.jp.carmaintenance;

import android.support.v7.widget.RecyclerView;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;

import java.util.ArrayList;

public class ViewAdapter extends RecyclerView.Adapter<ViewHolder> {


    private ArrayList<ListItem> data;

    //コンストラクター
    public ViewAdapter(ArrayList<ListItem> data) {
        this.data = data;

    }


    //ビューホルだ生成
    @Override
    public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View view = LayoutInflater.from(parent.getContext()).inflate(R.layout.data_field, parent,false);
        return new ViewHolder(view);
    }

    //ビューにデータを割り当てる・リストの項目の生成
    @Override
    public void onBindViewHolder(final ViewHolder holder, int position) {

        ListItem dataSet = data.get(position);

        holder.category.setText(dataSet.getCategory());
        holder.date.setText(dataSet.getDate());
        holder.price.setText(dataSet.getPrice());
        holder.notes.setText(dataSet.getNotes());
    }

    //データ項目数を取得
    @Override
    public int getItemCount()
    {
        return data == null ? 0 : data.size();
    }
}

Answer 1

データベースからの削除がありません.

Comments

[circulay]

ありがとうございます。
該当するリストデータの主キーを取得した上で、その主キーを含む行をテーブルから削除することを試みました。しかし、スワイプするデータの主キーを取得できずにいます。
主キーを取得するにはどのように記述するとよいでしょうか。

@Override
public void onSwiped(RecyclerView.ViewHolder viewHolder, int direction) {
int position = viewHolder.getAdapterPosition();
data.remove(position);
adapter.notifyDataSetChanged();
try {
db = helper.getWritableDatabase();
db.delete(MaintenanceDataHelper.TABLE_NAME, MaintenanceDataHelper.ROW_ID + " = ?", new String[]{String.valueOf(position)});
} finally {db.close();
}

[jimbe]

このような場合に使用することを想定して, [getItemId]( https://developer.android.com/reference/android/support/v7/widget/RecyclerView.Adapter.html#getitemid ) というメソッドがあります.
先ず, ListItem に _ID 列も保持するようにします.
アダプタの [hasStableIds]( https://developer.android.com/reference/android/support/v7/widget/RecyclerView.Adapter.html#hasstableids ) を true を返すようにします.
getItemId を実装し, position から _ID 列の値を返すようにします.
onSwiped で Adapter.getItemId(position) として _ID を得ます.

蛇足ですが, data の操作やデータベースへのアクセスなど様々な処理の詳細を直接メソッド内に並べるのは, オブジェクト指向的に好ましくないと思います.
例えば data の操作はアダプタが担当し, データベースへのアクセスは別の専用クラスが担当するようにし, リスナではそれらを呼び出すだけとするように, 担当クラスを決めて詳細はそれらに封じ込める形にすることで, 役割や責任を明確にして全体を整理することがオブジェクトを使う理由でもあります.

[jimbe]

「封じ込める」という意味では, MaintenanceDataShow にフィールドとして MaintenanceDataHelper や Cursor を持っていることも, 止めたほうが良いかと思います.
これらは使用時に生成され, 同じメソッド内で役割を終えています. そのようなモノは各メソッド内で宣言するべきで, そうでなければ不要なオブジェクトが残り続けることになります. また, 読む側(後日の作者本人も含みます)においても, フィールドは「メソッド内で完結しないモノ」という認識で読みますので, 不必要な注意を払うことになります.

[circulay]

とても貴重なアドバイスを頂き、本当にありがとうございます。
ご助言を参考にして修正したところ、少しデータベースから削除されるようになってきました。
しかし、リストからスワイプしたデータとは別のデータ（１つ下の行）が削除されてしまいます。
またリストの最終行のデータを削除しようとすると、IndexOutOfBoundsExceptionエラーが発生します。
onSwipedの中のpositionの扱い方に問題があるのでしょうか。

＜修正した部分＞
○ListItemに追記
　private long id;
　long getId() {return id;}
　void setId(long id) {this.id = id;}
○アダプター
　@Override
public long getItemId(int position) {return data.get(position).getId();
}
＊以下hasStableIdsはエラーになります(cannot verride）
@Override
public boolean hasStableIds () {
return true; }
○onSwiped内
final int position = viewHolder.getAdapterPosition();
data.remove(position);
adapter.notifyDataSetChanged();
long id = adapter.getItemId(position);
---省略---
db.delete("maintenanceDB", "_id=?", new String[]{String.valueOf(id)});

[jun74]

「data.remove(position);」を「long id = adapter.getItemId(position);」の先にやっているからでは？
db.deleteが1で1件削除を確認してから、
data.remove(position);
adapter.notifyDataSetChanged();
を実施すべきと考えます。
データが消えてないのに行消すのは変なので。

[circulay]

ご指摘いただいたところを直したら解決しました。
ありがとうございました。
とても助かりました。

[jimbe]

jun74 さん, 説明不足の所ご指摘ありがとうございます.

[jun74]

>circulayさん
直って良かったです。
>jimbeさん
いえいえ。これからもよろしくお願いします。