Android: закрытие приложения по кнопке "назад"

Question

[ertaquo]

Подскажите, пожалуйста, как правильно закрывать приложение по нажатию кнопки «назад»? Сейчас использую вот такой код (в Activity):

	@Override
	public void onBackPressed()
	{
		moveTaskToBack(true);
		finish();
		System.runFinalizersOnExit(true);
        System.exit(0);
	}
	
	public void onDestroy() {
        super.onDestroy();

        System.runFinalizersOnExit(true);
        System.exit(0);
    }

Но что-то мне говорит, что это не совсем верно и избыточно. Приложение — игра с libgdx, используется только одно Activity.

Answer 1

[sheremetat]

Когда Вы закрываете приложение кнопной «Назад», то вызывается метод onDestroy(). Для этого перехватывать нажатие этой кнопки в методе onBackPressed() не имеет смысла. Достаточно сделать так:

public void onDestroy() {
moveTaskToBack(true);

super.onDestroy();

System.runFinalizersOnExit(true);
System.exit(0);
}

Перехватывать нажатие кнопки «Назад» имеет смысл тогда, когда надо реализовать подтверждение выхода из игры или что-то в этом роде.

Answer 2

[YoungSkipper]

Если вы от Application не наследуетесь, и как следсвие не изпользуете другого Context кроме Context-а вашего Activity — то достаточно только finish(). Ну и плюс конечно, если вы сами создаете thread-ы то их лучше явным образом удалить.

Comments

[BorisPlus]

Как правильно завершить свой поток в Actitvity в onDestroy? Не подумайте чего — все облазил — каких методов только не предлагается. Но видимо так?

public void onDestroy() {
if(backgroundThread!=null)backgroundThread.interrupt();
super.onDestroy();
System.runFinalizersOnExit(true);
System.exit(0);
}

А если еще и SurfaceView используется, то нужно его принудительно завершать? удалять калбак из него? finish() при использовании SurfaceView не всегда закрывает приложение корректно — иногда появлzется исключение — видимо не закрывается поток рисования в SurfaceView, хотя в SurfaceView.onDestroy и делается drawingThread.interrupt().

[BorisPlus]

и кстати — нужен ли runFinalizersOnExit если он «deprecated»

[BorisPlus]

Короче у меня работоспособный вариант выглядит так примерно ACTIVITY:

//Это для пункта EXIT собственного меню
public void destroy() {
  backgroundThread.destroy();
  finish();
}

Но при этом у потока backgroundThread есть «чудо» переменная indicatorOfWork

public void destroy() {
  this.indicatorOfWork = false;
  try{
    this.join();
  }catch(){
  }
}
public run(){
  while(this.indicatorOfWork){
    doSomeWork();
  }
}

Тогда как должен выглядеть для кнопки НАЗАД этот метод?

//Это стандартное
public void onDestroy() {
  super.onDestroy();
}

а то все возможные варианты, что приходили на ум давали исключение при нажатии Назад