RecyclerViewの並び替えに応じてArrayListの要素の上書きが最初しかできない。

試したこと

ログに出力して上書きできてるか確認してみたところ、最初しかできていなかった。以降はいくらRecyclerViewの並び替えをしても何も変わらない

知りたいこと

何が原因なのかわからないのでそこを教えてもらいたいです。

Java
1ItemTouchHelper.SimpleCallback simpleItemTouchCallback = new ItemTouchHelper.SimpleCallback(ItemTouchHelper.UP | ItemTouchHelper.DOWN, ItemTouchHelper.LEFT) {
2            @Override
3            public boolean onMove(@NonNull RecyclerView recyclerView, @NonNull RecyclerView.ViewHolder viewHolder, @NonNull RecyclerView.ViewHolder target) {
4                final int fromPos = viewHolder.getAdapterPosition();
5                final int toPos = target.getAdapterPosition();
6                Log.d("fpos",data2.get(0).toString());
7                data.set(toPos,data.get(fromPos).toString());
8                data2.set(toPos,data2.get(fromPos).toString());
9                rAdapter.notifyItemMoved(fromPos, toPos);
10                Log.d("spos",data2.get(0).toString());
11                return true;// 移動した場合はtrue、そうでない場合はfalse
12            }

D/fpos: club （1回目上書き前
D/fpos: school （1回目　上書き後
D/fpos: school （2回目　上書き前
D/fpos: school （2回目　上書き後 ←clubになるはず
・・・

行動規範の内容に同意します

回答1件

ベストアンサー

Javaは苦手なのですが回答させていただきます。

RecyclerViewの並び替えに応じてAdapterにセットしているListを並び替えるという話でしょうか？もしそうであるなら、要素を先頭に上書きするだけではいけません。以下のようにListの要素がどこからどこへ移動したのかを反映させる必要があります。

Java
1ArrayList<String> itemDataList = new ArrayList<>();
2        itemDataList.add("ポメラニアン");
3        itemDataList.add("トイプードル");
4        itemDataList.add("柴犬");
5        itemDataList.add("ブルドック");
6        itemDataList.add("ダックスフント");
7
8
9        ItemTouchHelper.SimpleCallback simpleItemTouchCallback = new ItemTouchHelper.SimpleCallback(ItemTouchHelper.UP | ItemTouchHelper.DOWN, ItemTouchHelper.LEFT) {
10            @Override
11            public boolean onMove(@NonNull RecyclerView recyclerView, @NonNull RecyclerView.ViewHolder viewHolder, @NonNull RecyclerView.ViewHolder target) {
12                final int fromPos = viewHolder.getAdapterPosition();
13                final int toPos = target.getAdapterPosition();
14
15                /*仮にブルドックを上から2番目に移動させたとすると
16                      fromPos:3, toPos:1
17                  という値が得られる。
18                  
19                  つまり、index3の要素（ブルドック）が消えて [itemDataList.remove(fromPos)]
20                  index1の場所に今消した要素（itemDataList.remove(fromPos)の戻り値）がaddされるということ
21                  
22                  この移動をコードに表すと... ↓*/
23                
24                itemDataList.add(toPos, itemDataList.remove(fromPos));
25
26                rAdapter.notifyItemMoved(fromPos, toPos);
27
28                return true;// 移動した場合はtrue、そうでない場合はfalse
29            }
30
31            @Override
32            public void onSwiped(@NonNull RecyclerView.ViewHolder viewHolder, int direction) {
33
34            }
35        };