Как остановить timer при закрытии Activity?

Question

[Kitoved]

В onCreate создается таймер, который каждые 15 секунд запускает AsyncTask поток, после того как активити или все приложение закрывается, таймер все равно продолжает запускать этот поток. Если в onDestroy или по нажатию кнопки выполняется команда timer.cancel();, то активити выдает ошибку.
вот код таймера
в onCreate

Timer timer = new Timer();
	        MyTimerTask task = new MyTimerTask();
	        
	        timer.schedule(task,0, 15 * 1000);
	        }

TimerTask

class MyTimerTask extends TimerTask {
    	
        public void run() {
        	
        	 String streamUrl = "ссылка";
        	metadataTask =new MetadataTask();
	        try {
	            metadataTask.execute(new URL(streamUrl));
	        } catch (MalformedURLException e) {
	            e.printStackTrace();
	        }
                           	
             String title_artist;
			try {
				streamMeta.refreshMeta();
				title_artist = streamMeta.getStreamTitle();
				Log.i("ARTIST TITLE", title_artist);
			} catch (IOException e) {
				// TODO Auto-generated catch block
				e.printStackTrace();
			}
				
                       
        }
    }

в onDestroy делаю timer.cancel();
и при закрытии

12-31 19:09:04.220: E/AndroidRuntime(1816): FATAL EXCEPTION: main
12-31 19:09:04.220: E/AndroidRuntime(1816): java.lang.RuntimeException: Unable to destroy activity {com.example.kgw/com.example.kgw.TimerActivity}: java.lang.NullPointerException
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.ActivityThread.performDestroyActivity(ActivityThread.java:3499)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.ActivityThread.handleDestroyActivity(ActivityThread.java:3517)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.ActivityThread.access$1200(ActivityThread.java:141)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1309)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.os.Handler.dispatchMessage(Handler.java:99)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.os.Looper.loop(Looper.java:137)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.ActivityThread.main(ActivityThread.java:5103)
12-31 19:09:04.220: E/AndroidRuntime(1816): at java.lang.reflect.Method.invokeNative(Native Method)
12-31 19:09:04.220: E/AndroidRuntime(1816): at java.lang.reflect.Method.invoke(Method.java:525)
12-31 19:09:04.220: E/AndroidRuntime(1816): at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:737)
12-31 19:09:04.220: E/AndroidRuntime(1816): at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:553)
12-31 19:09:04.220: E/AndroidRuntime(1816): at dalvik.system.NativeStart.main(Native Method)
12-31 19:09:04.220: E/AndroidRuntime(1816): Caused by: java.lang.NullPointerException
12-31 19:09:04.220: E/AndroidRuntime(1816): at com.example.kgw.TimerActivity.onDestroy(TimerActivity.java:313)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.Activity.performDestroy(Activity.java:5302)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.Instrumentation.callActivityOnDestroy(Instrumentation.java:1117)
12-31 19:09:04.220: E/AndroidRuntime(1816): at android.app.ActivityThread.performDestroyActivity(ActivityThread.java:3486)
12-31 19:09:04.220: E/AndroidRuntime(1816): ... 11 more

Answer 1

[GavriKos]

У вас скорее всего переменная timer == null на момент уничтожения окна. Вы ее точно задаете глобально? Потому что вот в onCreate у вас похоже локальная переменная.

Comments

[Kitoved]

вы имеете ввиду? что надо определить timer перед oncreate и присвоить значение null ?

[GavriKos]

Kitoved Покажите весь код. Но вообще - вроде вы правильно поняли. timer должен быть глобален в контесте активити.

[Kitoved]

public class MyActivity extends Activity implements OnClickListener {

IcyStreamMeta streamMeta;
MetadataTask metadataTask;
String title_artist;
ImageButton play,stop;
URL url;
TextView track;
Timer timer;

@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.gwfm);
setTitle("FM");
ActionBar bar = getActionBar();
bar.setDisplayHomeAsUpEnabled(true);

track = (TextView) findViewById(R.id.title_artist);

play=(ImageButton) findViewById(R.id.btnStart);
stop=(ImageButton) findViewById(R.id.btnStop);
play.setOnClickListener(this);
stop.setOnClickListener(this);
track.setOnClickListener(this);

String streamUrl = "0.0.0.0:8000";

streamMeta = new IcyStreamMeta();

try {
if ( !isOnline() ){

return;
}
else {
streamMeta.setStreamUrl(new URL(streamUrl));
}
} catch (MalformedURLException e) {
e.printStackTrace();
}
if ( !isOnline() ){

return;

}else{
metadataTask =new MetadataTask();
try {
metadataTask.execute(new URL(streamUrl));
} catch (MalformedURLException e) {
e.printStackTrace();
}
}
Timer timer = new Timer();
MyTimerTask task = new MyTimerTask();

timer.schedule(task,0, 15 * 1000);
}
protected boolean isOnline() {
String cs = Context.CONNECTIVITY_SERVICE;
ConnectivityManager cm = (ConnectivityManager)
getSystemService(cs);
if (cm.getActiveNetworkInfo() == null) {
return false;
} else {
return true; }
}
protected class MetadataTask extends AsyncTask
{
@Override
protected IcyStreamMeta doInBackground(URL... urls)
{
try
{
streamMeta.refreshMeta();
Log.e("Retrieving MetaData","Refreshed Metadata");
}
catch (IOException e)
{
Log.e(MetadataTask.class.toString(), e.getMessage());
}
return streamMeta;
}
@Override
protected void onPostExecute(IcyStreamMeta result)
{
try
{
title_artist=streamMeta.getStreamTitle();

Log.e("Retrieved title_artist", title_artist);
if(title_artist.length()>0)
{
track.setText(title_artist);
}

}
catch (IOException e)
{
Log.e(MetadataTask.class.toString(), e.getMessage());
}
}

}

class MyTimerTask extends TimerTask {

public void run() {

String streamUrl = "0.0.0.0:8000";
metadataTask =new MetadataTask();
try {
metadataTask.execute(new URL(streamUrl));
} catch (MalformedURLException e) {
e.printStackTrace();
}

String title_artist;
try {
streamMeta.refreshMeta();
title_artist = streamMeta.getStreamTitle();
Log.i("ARTIST TITLE", title_artist);
} catch (IOException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
}
}
@Override
public void onBackPressed() {
Toast.makeText(this, "Оставайтесь снами", Toast.LENGTH_SHORT).show();
super.onBackPressed();

}

@Override
public void onDestroy() {
timer.cancel();
super.onDestroy();
}
}

[GavriKos]

Kitoved У вас минимум две проблемы в коде. В функции onDestroy вы используете переменную timer, но нигде ее не инициализируете. Потому что в onCreate вы используете локальную переменную. Соответственно в onCreate вам надо вместо Timer timer = new Timer(); написать просто timer = new Timer();. Но это не полное решение - до того участка код может не дойти - выше есть return. Что тут можно сделать - либо перенести инициализацию таймера выше всех ретернов, либо добавить в onDestroy проверку на null.

[Kitoved]

GavriKos: поменял на timer = new Timer(); вроде заработало без ошибок и выполнение таска после закрытия прекратилось. Спасибо большое за развернутый ответ. а на будущее если что проверка на null делается через if else ?

[GavriKos]

Kitoved if( timer!= null) timer.stop(); - как то так. Обратите внимание и на вторую проблему о которой я писал - у вас код не всегда дойдет до инита таймера. В частности если isOnline() вернет false.

[Kitoved]

GavriKos: попробовал оба варианта, оба не выдали ошибок. Думаю лучше оставить проверку на null, что бы не путаться в будущем. Или перенос до ретёрнов будет эффективнее?

[GavriKos]

Kitoved Нет, эффективнее будет проверка на null - н ебудет лишних аллокаций. Но тогда нужно не забывать о ней везде.

[Kitoved]

GavriKos: еще раз спасибо

Answer 2

[Neonoviiwolf]

посмотрите тут