Как обновить ссылку внутри определения вложенного класса?

Question

[Сергей Шиловский]

Немного странная формулировка вопроса, но лучше не придумал. Есть код(упрощен, что бы увидеть главное):

@Override
protected void onCreate(){
	Callback call = new Callback((answer)->{
		MyAdapter adapter = new MyAdapter(answer);
		
		ItemTouchHelper.SimpleCallback itemTouchHelperCallback = new ItemTouchHelper.SimpleCallback(0, ItemTouchHelper.LEFT) {
			@Override
			public void onSwiped(RecyclerView.ViewHolder viewHolder, int direction) {
				adapter.deleteAt(viewHolder.getAdapterPosition());				
		}
	};
	
	btn.setOnClickListener(()->{
		API.response().enqueue(call);
	});
	
	API.response().enqueue(call);
}

После onCreate всё ОК, adapter созданный в call == adapter в onSwiped, но в последующих запросах выясняется что adapter внутри ItemTouchHelper.SimpleCallback не обновляется. Может это и должно быть так, но меня это вводит в заблуждение, ведь в колбеке мы ясно даем понять что нам нужен именно new ItemTouchHelper.SimpleCallback

Comments

[Сергей Горностаев]

Что значит "adapter внутри ItemTouchHelper.SimpleCallback не обновляется"? В приведённом коде экземпляр ItemTouchHelper.SimpleCallback никак не применяется.

[Сергей Шиловский]

Сергей Горностаев, код упрощен, я убрал его применение, но суть не меняется, ссылка adapter внутри itemTouchHelperCallback не меняется, т.е. указывает на первый созданный adapter.

[Сергей Горностаев]

Сергей Шиловский, так вы же её и не меняете.

[Сергей Шиловский]

Сергей Горностаев, подозреваю что нет, раз она не меняется, но не могу понять почему. Разве

ItemTouchHelper.SimpleCallback itemTouchHelperCallback = new ItemTouchHelper.SimpleCallback(0, ItemTouchHelper.LEFT) {
      @Override
      public void onSwiped(RecyclerView.ViewHolder viewHolder, int direction) {
        adapter.deleteAt(viewHolder.getAdapterPosition());				
    }

не вызывается снова после
MyAdapter adapter = new MyAdapter(answer);
С новой ссылкой на adapter?

[Сергей Горностаев]

Сергей Шиловский, аааа, вот теперь я понял о чём вы. Таки покажите место, где вы используете itemTouchHelperCallback.

[Сергей Шиловский]

Сергей Горностаев, он привязывается к recyclerView

new ItemTouchHelper(itemTouchHelperCallback).attachToRecyclerView(recyclerView);

Но для отладки я просто вывожу адреса объектов в лог, в них видно что адрес itemTouchHelperCallback меняется, adapter'а снаружи меняется, а adaptera внутри не меняется.

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

[Сергей Шиловский]

Сергей Шиловский, и правда, вынес ItemTouchHelper в отдельную переменную и добавил touchHelper.attachToRecyclerView(null)
Перед

touchHelper = new ItemTouchHelper(itemTouchHelperCallback);
                        touchHelper.attachToRecyclerView(recyclerView);

Теперь всё норм

Answer 1

[Сергей Шиловский]

Проблема оказалась за пределами предоставленного кода, а именно в строке чуть ниже:

new ItemTouchHelper(itemTouchHelperCallback).attachToRecyclerView(recyclerView);

Новый itemTouchHelperCallback тупо не привязывался, если вручную не отвязать старый:

if(touchHelper != null)
    touchHelper.attachToRecyclerView(null);
touchHelper = new ItemTouchHelper(itemTouchHelperCallback);
touchHelper.attachToRecyclerView(recyclerView);