Не могу обновить RecyclerView после изменения item

@disx · Регистрация: 26.02.2014

Студворк — интернет-сервис помощи студентам

данные для списка берутся из Firebase,

у DatabaseReference//Firebase, есть два метода:

onChildAdd и onChildChanged, первый возвращает все айтемы второй только изменяемый айтем.

и проблема том что после изменения item в RecyclerView появляется еще один, т.е. видны оба и измененный и его старая версия.

в вышеприведенных методах везде стоит Adapter.notifyDataSetChanged()

помогите, как побороть эту проблему?

@Spelcrawler · 18.03.2017, 16:34

disx, а можно код этого обновления?

@disx · 18.03.2017, 17:17 **[ТС]**

Spelcrawler,

это из MainActivity onCreate:

Кликните здесь для просмотра всего текста

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
  mFirebaseAuth = FirebaseAuth.getInstance();
        mFirebaseUser = mFirebaseAuth.getCurrentUser();
        mDatabase = FirebaseDatabase.getInstance().getReference();
        if (mFirebaseUser == null) {
            // Not logged in, launch the Log In activity
            loadLogInView();
        } else {
            mUserId = mFirebaseUser.getUid();
 
            mRecyclerView = (RecyclerView) findViewById(R.id.recyclerView);
            mGridLayoutManager = new StaggeredGridLayoutManager(1, StaggeredGridLayoutManager.VERTICAL);
            mRecyclerView.setItemAnimator(new DefaultItemAnimator());
            mRecyclerView.setLayoutManager(mGridLayoutManager);
 
 
            final String key = mDatabase.child("items").push().getKey();
            //do something with input
 
            // Use Firebase to populate the list.
 
            mDatabase.child("users").child(mUserId).addChildEventListener(new ChildEventListener() {
 
                @Override
                public void onChildAdded(DataSnapshot dataSnapshot, String s) {
                    //adapter.add((String) dataSnapshot.child("title").getValue());
 
                    SearchList value = dataSnapshot.getValue(SearchList.class);
                    data.add(value);
                    Log.d("DisLog", dataSnapshot.getValue().toString()+" /"+dataSnapshot.getKey().toString());
 
                   mAdapter.notifyDataSetChanged();
 
                }
 
                @Override
                public void onChildChanged(DataSnapshot dataSnapshot, String s) {
 
                       // stringUpdate = dataSnapshot.getKey().toString();
                    SearchList value = dataSnapshot.getValue(SearchList.class);
                    data.add(value);
 
                        mAdapter.notifyDataSetChanged();
                   }
 
                @Override
                public void onChildRemoved(DataSnapshot dataSnapshot) {
                    // adapter.remove((String) dataSnapshot.child("title").getValue());
                             mAdapter.notifyDataSetChanged();
                }
 
                @Override
                public void onChildMoved(DataSnapshot dataSnapshot, String s) {
                 //                 mAdapter.notifyDataSetChanged();
                                 mAdapter.notifyDataSetChanged();
                }
 
                @Override
                public void onCancelled(DatabaseError databaseError) {
                }
 
            });
//            mAdapter.notifyDataSetChanged();
            // Set up ListView
            mAdapter = new RecyclerViewAdapter(getBaseContext(), data);
            mRecyclerView.setAdapter(mAdapter);
        }

это onClick диалога для изменения:

Кликните здесь для просмотра всего текста

Java
1
2
3
4
5
6
7
8
9
10
11
   title = etName.getText().toString();
                            number = etNum.getText().toString();
                            additions = etAddit.getText().toString();
                            pincode = etPin.getText().toString();
                            comment = etComm.getText().toString();
 
 
                            SearchList searchList = new SearchList(itemKey, title, number, additions, pincode, comment);
                          mDatabase.child("users").child(mUserId).child("items" + itemKey).setValue(searchList);
 
                            Flag=0;

и RecyclerAdapter:

Кликните здесь для просмотра всего текста

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
package com.dis.passkeeper;
 
import android.app.Activity;
import android.content.Context;
import android.support.v7.widget.RecyclerView;
import android.util.Log;
import android.view.LayoutInflater;
import android.view.MotionEvent;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Button;
import android.widget.EditText;
import android.widget.TextView;
import android.widget.Toast;
 
import com.google.firebase.database.DataSnapshot;
import com.google.firebase.database.DatabaseError;
import com.google.firebase.database.ValueEventListener;
 
import java.util.ArrayList;
import java.util.List;
import java.util.Locale;
 
 
 
public class RecyclerViewAdapter extends RecyclerView.Adapter<RecyclerViewAdapter.ViewHolder> {
 
    private List<SearchList> mList;
    private int position;
    Context context;
  // private ArrayList<SearchList> objects;
private static final int IN_EDIT_MODE = 1;
    int flagShowView = 0;
    LayoutInflater inflater;
    ArrayList<SearchList> objects; //массив для фильтра списка
 
    public RecyclerViewAdapter(Context context, List<SearchList> recordsNUM ) {
        this.mList = recordsNUM;
        this.context = context;
  //      this.objects = new ArrayList<SearchList>();
//        this.objects.addAll(mList);
 
 
       this.inflater =  LayoutInflater.from(context);
    }
 
    /**
     * Создание новых View и ViewHolder элемента списка, которые впоследствии могут переиспользоваться.
     */
    @Override
    public ViewHolder onCreateViewHolder(ViewGroup parent, int viewType) {
        View view;
        if (viewType == IN_EDIT_MODE) {
            flagShowView=1;
            view = inflater.inflate(R.layout.item_del, parent, false);
        } else {
            flagShowView=0;
            view = inflater.inflate(R.layout.item, parent, false);
 
        }
        return new ViewHolder(view);
    }
    @Override
    public int getItemViewType(int pos) {
 
      return mList.get(pos).isLongClicked ? IN_EDIT_MODE : -1;
    }
    /**
     * Заполнение виджетов View данными из элемента списка с номером i
     */
    @Override
    public void onBindViewHolder(final ViewHolder viewHolder, final int i) {
 
        SearchList record = mList.get(i);
        viewHolder.textName.setText(record.getName());
        viewHolder.textNum.setText(record.getNumber());
        viewHolder.textAdd.setText(record.getAdditions());
        viewHolder.textPin.setText(record.getPincode());
     //  viewHolder.textComm.setText(record.getComment());
        // viewHolder.textName.setText(record.getTitleName());
 
    }
//public void clickDel(ViewHolder holder){
//    int newPosition = holder.getAdapterPosition();
//    mList.remove(newPosition);
//    notifyItemRemoved(newPosition);
//    notifyItemRangeChanged(newPosition, mList.size());
//}
    @Override
    public int getItemCount() {
        return mList.size();
    }
    public int getPosition() {
        return position;
    }
    public void setPosition(int position) {
        this.position = position;
    }
//    @Override
//    public int getItemCount() {
//        return mList.length;
//    }
    /**
     * Реализация класса ViewHolder, хранящего ссылки на виджеты.
     */
    class ViewHolder extends RecyclerView.ViewHolder{
        private TextView textName;
        private TextView textNum;
        private TextView textAdd;
        private TextView textPin;
        private TextView textComm;
        private Button btDelete;
        TextView textDevider;
 
        public ViewHolder(View itemView) {
            super(itemView);
       textName = (TextView) itemView.findViewById(R.id.textBank);
            textNum = (TextView) itemView.findViewById(R.id.textCardNum);
            textAdd = (TextView) itemView.findViewById(R.id.textAdditionaly);
            textPin = (TextView) itemView.findViewById(R.id.textPincode);
           // textComm = (EditText) itemView.findViewById(R.id.textComm);
//            textName = (TextView) itemView.findViewById(R.id.textReviewItem1);
//            textDevider = (TextView) itemView.findViewById(R.id.textReviewDiv);
            btDelete = (Button) itemView.findViewById(R.id.buttonDelete);
//            btDelete.setOnClickListener(new View.OnClickListener() {
//                @Override
//                public void onClick(View v) {
//                    Log.d("myLogs", "OnClick");
//                }
//            });
            // itemView.setOnCreateContextMenuListener(this);
        }
 
    }
    // Filter Class
    public void filter(String charText) {
        charText = charText.toLowerCase(Locale.getDefault());
        objects = new ArrayList<SearchList>();
        objects.addAll(mList);
//        Log.d("myLogs", charText);
 
        mList.clear();
        if (charText.length() == 0) {
            Log.d("myLogs", "length");
            mList.addAll(objects);
 
        }else {
 
            for (SearchList prod : objects) {
 
                if (prod.getName().toLowerCase(Locale.getDefault()).contains(charText)) {
                    Log.d("myLogs", charText);
                    mList.add(prod);
 
                }
            }
        }
//       notifyDataSetChanged();
        // Set on UI Thread
 
                notifyDataSetChanged();
 
        Log.d("myLogs", "notifyData");
    }
 
}

@Spelcrawler · 18.03.2017, 18:15

data.add(value); добавляет объект в список для RecyclerView? Он и должен добавляться в таком случае. Нужно удалять старый перед добавлением или просто изменять его.

@disx · 19.03.2017, 14:51 **[ТС]**

Spelcrawler, да добавляет

Нужно удалять старый перед добавлением или просто изменять его.

так не могу понять как это сделать, если удалять, получается удалять надо именно item в адаптере (ну в базе-то нового объекта не создается, просто меняются значения старого), а изменять тоже не получается организовать т.к.

в диалоге по клику отправляю объекту в базе новые значения:

Java
1
2
SearchList searchList = new SearchList(itemKey, title, number, additions, pincode, comment);
                            mDatabase.child("users").child(mUserId).child("items" + itemKey).setValue(searchList);

, где itemKey это key редактируемого объекта.

далее здесь:

Java
1
2
3
4
5
6
7
8
9
10
@Override
                public void onChildChanged(DataSnapshot dataSnapshot, String s) {
 
                       // stringUpdate = dataSnapshot.getKey().toString();
                    SearchList searchList = dataSnapshot.getValue(SearchList.class);
 
                    data.add(searchList);
                    Log.d("DisLog", "itemKey "+itemKey);
                        mAdapter.notifyDataSetChanged();
                   }

получаем "снимок" базы и отправляем в data, и визуально в списке появляется измененный айтем, но старый остается (оcтается только визуально в RecyclerView, не в базе).

т.е. было:
1
2
редактирую второй, и получается:
1
2
22

но в базе айтема со значением 2 уже нету, есть только 1 и 22.

Вот в чем проблема.

Добавлено через 19 часов 48 минут
Spelcrawler,

Нужно удалять старый перед добавлением

не понимаю почему сам до такого не мог неделю додуматься

, просто не подумал что можно добавлять новую запись с сохранением одинакового itemKey.

Все получилось

.

Новые блоги и статьи Все статьи Все блоги /
Ритм жизни kumehtar 27.02.2026 Иногда приходится жить в ритме, где дел становится всё больше, а вовлечения в происходящее — всё меньше. Плотный график не даёт вниманию закрепиться ни на одном событии. Утро начинается с быстрых,. . .	SDL3 для Web (WebAssembly): Сборка библиотек SDL3 и Box2D из исходников с помощью CMake и Emscripten 8Observer8 27.02.2026 Недавно вышла версия SDL 3. 4. 2 библиотеки SDL3. На странице официальной релиза доступны исходники, готовые DLL (для x86, x64, arm64), а также библиотеки для разработки под Android, MinGW и Visual. . .	SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .
Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»

@disx 23 / 23 / 0 Регистрация: 26.02.2014 Сообщений: 684

	Не могу обновить RecyclerView после изменения item 17.03.2017, 16:59. Показов 2175. Ответов 4 Метки нет (Все метки) данные для списка берутся из Firebase, у DatabaseReference//Firebase, есть два метода: onChildAdd и onChildChanged, первый возвращает все айтемы второй только изменяемый айтем. и проблема том что после изменения item в RecyclerView появляется еще один, т.е. видны оба и измененный и его старая версия. в вышеприведенных методах везде стоит Adapter.notifyDataSetChanged() помогите, как побороть эту проблему? 0

@Spelcrawler 535 / 504 / 114 Регистрация: 12.03.2014 Сообщений: 1,671
	18.03.2017, 16:34
	disx, а можно код этого обновления? 0

@Spelcrawler 535 / 504 / 114 Регистрация: 12.03.2014 Сообщений: 1,671
	18.03.2017, 18:15
	Сообщение было отмечено disx как решение Решение data.add(value); добавляет объект в список для RecyclerView? Он и должен добавляться в таком случае. Нужно удалять старый перед добавлением или просто изменять его. 1

Не могу обновить RecyclerView после изменения item

Решение