我遇到上一个问题的问题,我不确定我的代码是否有内存泄漏。一些答案涉及它在UI线程上运行,因此阻塞。确实,它在UI线程上运行并且不会产生新的线程。
因此,为了解决这个问题,我使用Thread
而不是Handler
来生成UI之外的新线程。现在的问题是,我无法像在UI线程中运行的那样延迟它。
这是我之前的问题,我的原始UI线程代码是:Is this Runnable safe from memory leak?,以下是产生新线程的更新代码:
package com.example.helloworld;
import android.app.Activity;
import android.os.Bundle;
import android.os.Handler;
import android.widget.TextView;
import android.util.Log;
import java.lang.ref.WeakReference;
public class HelloWorldActivity extends Activity
{
private static TextView txtview;
/** Called when the activity is first created. */
@Override
public void onCreate(Bundle savedInstanceState)
{
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
txtview = (TextView) findViewById(R.id.mainview);
Thread t = new Thread(new WeakRunnable(txtview));
t.start();
}
private static final class WeakRunnable implements Runnable {
private final WeakReference<TextView> mtextview;
protected WeakRunnable(TextView textview){
mtextview = new WeakReference<TextView>(textview);
}
@Override
public void run() {
TextView textview = mtextview.get();
if (textview != null) {
try {
Thread.sleep(1500);
} catch (InterruptedException e) {
e.printStackTrace();
}
int test = 5*5;
txtview.setText("Hola Mundo"+test);
}
Log.d("com.example.helloworld", "" + Thread.currentThread().getName()); // Outputs "Thread-<num>" if not running on UI thread
}
}
}
它只是设置视图文本并附加5*5
。
一旦我启动应用程序,它就会自行退出,我不明白为什么。有些东西告诉我,我以错误的方式推迟或使用runOnUiThread
错误。即使将txtview.setText("Hola Mundo"+test);
更改为runOnUiThread( txtview.setText("Hola Mundo"+test) );
也不会编译错误:'void' type not allowed here
。
简而言之:计算(在这种情况下为5*5
)应该在一个单独的线程上完成,以避免阻塞主(UI)线程,并且应该在UI上设置文本,从中获取计算项目单独的线程。你自己的一个简单例子也可以。
更新
我已经在实施AsyncTask
的问题上发布了答案。
答案 0 :(得分:2)
改变,
Thread t = new Thread(new WeakRunnable(txtview));
t.start();
到:
txtview.postDelayed(new WeakRunnable(txtview), 1500);
然后在runnable中删除以下sleep子句,因为它不再需要
try {
Thread.sleep(1500);
} catch (InterruptedException e) {
e.printStackTrace();
}
答案 1 :(得分:1)
您可以在满足条件之前使用条件,而不是使用thread.sleep();
Condition.wait(new Callable<Boolean>() {
@Override
public Boolean call() throws Exception {
return textview != null;
}
});
答案 2 :(得分:1)
正如@Axxiss所说,这种情况更适合AsyncTask
。我更新了我的代码,使用AsyncTask
和WeakReference
来避免内存泄漏:
package com.example.helloworld;
import android.app.Activity;
import android.os.Bundle;
import android.os.Handler;
import android.os.AsyncTask;
import android.widget.TextView;
import android.util.Log;
import java.lang.ref.WeakReference;
public class HelloWorldActivity extends Activity
{
private static TextView txtview;
/** Called when the activity is first created. */
@Override
public void onCreate(Bundle savedInstanceState)
{
super.onCreate(savedInstanceState);
setContentView(R.layout.main);
txtview = (TextView) findViewById(R.id.mainview);
SimpleTask objSimpleTask=new SimpleTask();
objSimpleTask.execute();
}
private static class SimpleTask extends AsyncTask<Void,Void,String> {
private WeakReference<TextView> mtextview = new WeakReference<TextView>(txtview);
@Override
protected String doInBackground(Void... params) {
Log.d("com.example.helloworld", "" + Thread.currentThread().getName());
try{
Thread.sleep(1500);
} catch(InterruptedException ex) {
Thread.currentThread().interrupt();
}
return "Hola Mundo";
}
@Override
protected void onPostExecute(String result) {
TextView mtxtview = mtextview.get();
mtxtview.setText(result);
}
}
}
doInBackground
在一个单独的线程中执行代码,而稍后您可以使用onPostExecute
捕获结果并在主(UI)线程中执行它而没有任何装饰。