Не понимаю, как в суппорт-либе v7 перехватить нажатие home/up, если у меня свой тулбар с ActionBarDrawerToggle?

Question

[Vermut756]

Дано: AppCompatActivity. Со своим тулбаром с DrawerLayout и кнопкой-гамбургером. И фрагмент в этой активити, Во фрагменте в какой-то момент юзер открывает всплывающий интерфейс.

Требуется: реализовать закрытие интерфейса двумя способами - железный back и кнопка home/up.
То есть при открытии гамбургер меняется на стрелочку назад. А при ее нажатии всплывающий интерфейс прячется, и снова возвращается гамбургер, который снова может открывать DrawerLayout.

С железным backом сразу заметил проблему. onBackPressed работает только в активити. А тут фрагмент, чтобы в него передать - надо городить что-то вроде своего листенера, а это не так просто из-за довольно сложного лайфцикла. Нужно хорошо подумать, чтобы не привело к подводным камням.
Поэтому решил back пока отложить. Занялся стрелочкой, это вроде проще: уже делал через onOptionsItemSelected и android.R.id.home, а это должно работать и во фрагменте. Как же я ошибался(((

Нет, во фрагменте-то оно работает. Но не работает по другой причине - из-за гамбургера с DrawerLayout, которые сделаны через создание ActionBarDrawerToggle, а он все меняет, и onOptionsItemSelected вообще не вызывается при нажатии home/up, даже после того, как я вызвал setDisplayHomeAsUpEnabled(true).
Покопав форумы и исходный код, нашел решение:

toolbar.setNavigationOnClickListener(new OnClickListener() {
			@Override
			public void onClick(View arg0) {
				// TODO Auto-generated method stub
			}
});

Но это опять головная боль, ведь сразу же возникает вопрос - а где ставить обратно старый листенер?
Да и как его ставить? Точнее, как сохранить старый листенер в буфер?
Никакого метода getNavigationOnClickListener() я не нашел. Решил реализовать с помощью toolbar.getChildAt и поиска того Child, который ImageButton, это и есть та кнопка. Но это уже не совсем чистый хак, а еще и оказалось, что ни ImageButton, ни View тоже не имеют метода getOnClickListener! Это так заложено еще в самом Android, и обойти это можно только через - о ужас- рефлексию... Занавес.

Получается, мне нужно сначала toolbar.setNavigationOnClickListener, затем заново создавать ActionBarDrawerToggle - потому что другого способа вернуть старый обработчик просто не существует - и где все это делать?
Логика подсказывает, что в Activity, объявив 2 соответствующих метода, но обращаться к ним из фрагмента, это опять головная боль, ведь getActivity может и null возвращать, может получиться и так, что фрагмент пересоздастся, всплывающий интерфейс пропадет, но все это произойдет не при нажатии стрелочки, а когда Android захочет очистить память, ну или еще как-то, поэтому вместо гамбургера навсегда останется стрелочка...

Нет слов. Одни эмоции. Ненависть. Тонны ненависти к тем, кто писал суппорт либу и мерзопакостный ActionBarDrawerToggle. Почему они сломали столь удобный onItemSelected, доступный и во фрагментах, и не дали ничего адекватного взамен? Почему???

У меня уже голова пухнет, а мне еще железный back делать....
С onBackPressed тоже "хорошо", почему его нет во фрагментах, даже версии суппорт v4, а еще более новой версии базового класса для фрагмента просто не существует??? Что им, сложно было?

Comments

[Денис Загаевский]

Давайте начнём с простого - что такое этот ваш "всплывающий интерфейс", и почему бы не воспользоваться стандарными средствами (фрагмент, диалогфрагмент) для его "всплывания"? А там, глядишь, и с беком мучаться не надо будет, обрабатывая его кастомно.

[Vermut756]

Денис Загаевский: В том фрагменте листвью (да, пока еще листвью), и кнопка, при нажатии на нее должен появиться интерфейс.
А точнее, выполняется 2 задачи - показать панель с несколькими кнопками (это действительно можно бы сделать через еще 1 вложенный фрагмент, получив уже третий уровень вложенности фрагментов) и сказать адаптеру листвью, чтобы добавил чекбоксы ко всем элементам (а вот это через фрагмент не сделать никак - не создавать же новый листвью чекбоксов поверх основного)

[Vermut756]

Денис Загаевский: Короче, при нажатии на кнопку я добавляю чекбоксы к элементам листвью, а затем надо их убрать с помощью back.

Answer 1

[Денис Загаевский]

Курите жизненный цикл фрагментов и взаимодействуте через активити. На дровер навесьте листенер, который будет звать onBackPressed у активити, а она уже прокидывает во фрагменты (через свой базовый класс BaseFragment или типа того, от которого наследуйте все фрагменты).

Comments

[Vermut756]

А как такой вариант?
В активити:

//onBackPressed 
    ArrayList<OnBackPressedListener> mBackPressedListeners;
    
    public void addOnBackPressedListener(OnBackPressedListener listener) {
    	mBackPressedListeners.add(listener);
    }
    
    public void removeOnBackPressedListener(OnBackPressedListener listener) {
    	mBackPressedListeners.remove(listener);
    }
    
    //onHomeUpPressed 

    public void setOnHomeUpPressed(final OnBackPressedListener listener) {
    	getSupportActionBar().setDisplayHomeAsUpEnabled(true);
    	toolbar.setNavigationOnClickListener(new OnClickListener() {

			@Override
			public void onClick(View arg0) {
				// TODO Auto-generated method stub
				listener.onBackPressed();
			}
    	
    	});
    }
    
    public void resetHomeBtn() {
    	getSupportActionBar().setDisplayHomeAsUpEnabled(false);
        ActionBarDrawerToggle toggle = new ActionBarDrawerToggle(
                this, drawer, toolbar,
                R.string.navigation_drawer_open,
                R.string.navigation_drawer_close);
        drawer.setDrawerListener(toggle);
        toggle.syncState();
    }

    //

	
    @Override
    public void onBackPressed() {
        if (drawer.isDrawerOpen(GravityCompat.START)) {
            drawer.closeDrawer(GravityCompat.START);
        } else {
        	boolean doSuperAction = true;
        	
        	for (OnBackPressedListener listener : mBackPressedListeners)
        	{
        		if (!listener.onBackPressed()) {
        			doSuperAction = false;
        		}
        	}
        	
        	if (doSuperAction) {
        		super.onBackPressed();
        	}
        }
    }

А фрагмент реализует этот кастомный листенер, и:

@Override
	public boolean onBackPressed() {
				
		closeMultiselect();
		
		return false;
	}

    void openSelect(boolean selectAll) {
    	...
    	MainActivity activ = (MainActivity) this.getActivity();
    	activ.setOnHomeUpPressed(this);
    	activ.addOnBackPressedListener(this);
    }
    
    void closeMultiselect() {
        ....
    	MainActivity activ = (MainActivity) this.getActivity();
    	activ.resetHomeBtn();
    	activ.removeOnBackPressedListener(this);
    }

И это все. Ну еще в активити в onCreate я вызываю resetHomeBtn для первого создания гамбургера.
С жизненным циклом не заморачивался вообще, никаких onDestroy. Почему? Потому что пока и так не вылетает вроде. Вот так вот.

[Денис Загаевский]

Vermut756: может веселуха начаться, когда фрагмент решите реплейснуть другим.
Плюс лично мне не нравится, что активити рулит кнопкой.

[Денис Загаевский]

Vermut756: А, ну и странно, что оно у вас не крешится. Вы список обходите foreach, при этом изнутри цикла происходит удаление элемента. Может какая заморочка с тем, что элемент всего один, но вообще должно быть ConcurrentModificationException. Надо Copy-on-write семантику использовать.

[Vermut756]

Денис Загаевский: Когда именно реплейснуть? При открытом "всплывающем интерфейсе" такой возможности не будет - все лишнее будет disabled, это уже требование ТЗ.
А что, лучше рулить ActionBarDrawerToggle из фрагмента? Но тогда код resetHomeBtn придется продублировать в onCreate активити.

[Vermut756]

Денис Загаевский: Где? Тут только один for, и в нем нет удаления, вы что-то путаете.

[Денис Загаевский]

Vermut756: в форе зовётся листенер, который делает closeMultiselect, в котором делается removeOnBackPressedListener, в котором из списка, по которому идёт фор удаляется элемент. Классика багов.

[Vermut756]

Денис Загаевский: Спасибо! Проверю потом, хотя вроде не актуально, коль скоро пользователь не может переключить фрагмент, не удалив листенер предыдущего, так что листенер всегда будет один.

[Денис Загаевский]

Vermut756: сегодня нет, завтра - да. Пишите надёжный код.