不良代码展示-Activity中使用线程的例子

一段坏代码如何变成优雅代码

原创文章，如有转载，请注明出处：http://blog.csdn.net/yihui823/article/details/6741411

今天看到一段很糟糕的代码。于是做了一个工程，模拟这段代码，向大家说明一下线程在使用中要注意的几点。这个例子适合给新手，也欢迎各位高手来指点一下。

首先，上代码。

第一个类LoginService，这是一个模拟类，把业务剥离出去了。只是模拟登录操作而已。

package com.study; package com.study; /** * 虚拟的一个登录服务. * @author yihui823 */ public class LoginService { //单例 private static LoginService oneInstance = new LoginService(); /** * 得到唯一的一个单例 * @return 唯一的一个单例 */ public static LoginService getInstance() { return oneInstance; } //登录成功标记 private boolean hadLogin = false; /** * 模拟登录操作 * @return true:登录成功 */ public boolean login() { try { Thread.sleep(2000); } catch (InterruptedException e) { } hadLogin = true; return hadLogin; } /** * 判断是否登录 * @return true:已经登录 */ public boolean isLogin() { return hadLogin; } }
第二个类就是我们的Activity了。

package com.study; import android.app.Activity; import android.os.Bundle; import android.widget.Toast; /** * 一个段不好的代码 * @author yihui823 */ public class BadCodeActivity extends Activity { //登录服务 private LoginService lService = LoginService.getInstance(); /** Called when the activity is first created. */ @Override public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.main); new Thread(new Runnable() { @Override public void run() { while(!lService.isLogin() ) { try { lService.login(); Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } } Toast.makeText(BadCodeActivity.this, "登录成功", Toast.LENGTH_LONG).show(); } }).start(); } }
这个例子呢，显然运行的时候会报错的。请见我的另一篇文章：Android画面UI中的线程约束。我们在非UI线程里去控制UI界面，就必须使用Handler来发送消息。修改代码如下：

package com.study; import android.app.Activity; import android.os.Bundle; import android.os.Handler; import android.os.Message; import android.widget.Toast; /** * 一个段不好的代码 * @author yihui823 */ public class BadCodeActivity extends Activity { //登录服务 private LoginService lService = LoginService.getInstance(); //外线程访问UI线程的Handle private Handler mhandle = new Handler(){ @Override public void handleMessage(Message msg) { Toast.makeText(BadCodeActivity.this, "登录成功", Toast.LENGTH_LONG).show(); } }; /** Called when the activity is first created. */ @Override public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.main); new Thread(new Runnable() { @Override public void run() { while(!lService.isLogin() ) { try { lService.login(); Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } } mhandle.sendEmptyMessage(0); } }).start(); } }

红色部分代码，就是修改的地方。现在，这段代码可以运行了，而且还貌似不错，是吧。

但是，一个好的程序，不能只是应付正常情况，还要应付错误情况，是吧。如果登录总是出错怎么样呢？我们把LoginService类略微改动，如下：

package com.study; import android.util.Log; /** * 虚拟的一个登录服务. * @author yihui823 */ public class LoginService { private static final String TAG = "LoginService"; //单例 private static LoginService oneInstance = new LoginService(); /** * 得到唯一的一个单例 * @return 唯一的一个单例 */ public static LoginService getInstance() { return oneInstance; } //登录成功标记 private boolean hadLogin = false; /** * 模拟登录操作 * @return true:登录成功 */ public boolean login() { try { Thread.sleep(2000); } catch (InterruptedException e) { } Log.d(TAG, "we are login"); // hadLogin = true; return hadLogin; } /** * 判断是否登录 * @return true:已经登录 */ public boolean isLogin() { return hadLogin; } }
增加了Log，以便查看登录情况。模拟业务代码只改了一行，就是登录永远是失败。现在运行一下呢。停在页面上没有动静了，logcat里也不断的打出：
we are login
这个也不会有什么错误，对吧。但是，我们如果按“返回”键退出页面，再看看logcat呢？
we are login的log还在不停的输出，是吗？
我想现在大家应该知道哪里出了问题了。就是说，我们的线程启动之后，就没法停掉了。
这里我要说一下。我一直认为，
new Thread(new Runnable() {…}().start();
这种代码写的非常的不好。你直接构造了一个对象，但是这个对象你没有任何的变量去指向它。这个线程被你启动之后，你已经无法再去跟踪、调用、管理了。这个线程，只能自生自灭，永远游离在你的控制范围之外。你会不会觉得，这个线程跟僵尸一样?对，这就是僵尸进程，如果它没有停止的条件，就永远在你的系统里消耗你的资源。
所以我觉得使用线程的一个基本认识：生成的线程类，一定要有一个变量去指向它，以便在合适的时候销毁。
这里说到销毁，这就是另一个问题了。Thread类已经废弃了stop方法了，因为线程需要自行去释放该释放的资源，不能光依赖于运行框架的控制。我们需要在Thread里面，加上他自己停止的代码。也就是说，不论如何，线程应该会自己去停止掉，而不应该是无限制的运行。
另外，我们在Android里面，还应该注意Activity的各个状态周转。一般来说，线程的启动在onCreate里是不合适的，我们必须考虑到onResume和onPause的情况。
那么，我们总结下，Activity里使用线程有三个注意：
1， 线程对象一定要有变量指向它，以便我们可以控制。
2， 线程类一定要有停止条件，以便外界通知线程自行停止。
3， 在onResume里启动线程，在onPause里停止线程。
我们根据以上三点，重新写一下Activity。

package com.study; import android.app.Activity; import android.os.Bundle; import android.os.Handler; import android.os.Message; import android.util.Log; import android.widget.Toast; /** * 一个段不好的代码 * @author yihui823 */ public class BadCodeActivity extends Activity { private static final String TAG = "BadCodeActivity"; //登录服务 private LoginService lService = LoginService.getInstance(); //外线程访问UI线程的Handle private Handler mhandle = new Handler(){ @Override public void handleMessage(Message msg) { Toast.makeText(BadCodeActivity.this, "登录成功", Toast.LENGTH_LONG).show(); } }; //通知停止线程的标记 private boolean stopFlag = false; //登录成功标记 private boolean loginOk = false; /** * 登录用的线程类 */ private class LoginThread extends Thread { @Override public void run() { while(!stopFlag) { loginOk = lService.isLogin(); if (loginOk) { break; } try { lService.login(); Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } } mhandle.sendEmptyMessage(0); } /** * 通知线程需要停止 */ public void stopLogin() { stopFlag = true; } }; //用来登录的线程 private LoginThread loginThread = new LoginThread(); /** Called when the activity is first created. */ @Override public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.main); Log.d(TAG, "BadCodeActivity instance is called onCreate :" + this.hashCode()); } public void onResume() { super.onResume(); Log.d(TAG, "BadCodeActivity instance is called onResume :" + this.hashCode()); loginThread.start(); } public void onPause() { super.onPause(); loginThread.stopLogin(); } }
现在，我们的线程可以在页面退出的时候正常停止了。

但是这段代码还是有问题的。我们仔细看看，线程在Activity构造的时候就已经创建了，然后在程序进到前台的时候启动，退到后台的时候停止。但是线程有这么一个特性：
一旦线程的run()函数运行结束了，这个线程就销毁了，不能再启动了。
现在我们的程序，在退出后将不可能再次显示，所以系统会马上回收掉Activity。如果我们的页面增加一个按钮，迁移到另一个页面，那么在那个页面返回的时候，就会有异常出现。我们修改一下代码来试试。
增加一个Activity:

package com.study; import android.app.Activity; import android.os.Bundle; /** * 临时页面 * @author yihui823 */ public class TempActivity extends Activity { /** Called when the activity is first created. */ @Override public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.main); } }

别忘了修改AndroidManifest.xml，增加Activity的说明：

<activity android:name=".TempActivity"
android:label="@string/app_name"/>

修改BadCodeActivity：

package com.study; import android.app.Activity; import android.content.Intent; import android.os.Bundle; import android.os.Handler; import android.os.Message; import android.util.Log; import android.view.View; import android.view.View.OnClickListener; import android.widget.Button; import android.widget.Toast; /** * 一个段不好的代码 * @author yihui823 */ public class BadCodeActivity extends Activity { private static final String TAG = "BadCodeActivity"; //登录服务 private LoginService lService = LoginService.getInstance(); //外线程访问UI线程的Handle private Handler mhandle = new Handler(){ @Override public void handleMessage(Message msg) { Toast.makeText(BadCodeActivity.this, "登录成功", Toast.LENGTH_LONG).show(); } }; //通知停止线程的标记 private boolean stopFlag = false; //登录成功标记 private boolean loginOk = false; /** * 登录用的线程类 */ private class LoginThread extends Thread { @Override public void run() { while(!stopFlag) { loginOk = lService.isLogin(); if (loginOk) { break; } try { lService.login(); Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } } mhandle.sendEmptyMessage(0); } /** * 通知线程需要停止 */ public void stopLogin() { stopFlag = true; } }; //用来登录的线程 private LoginThread loginThread = new LoginThread(); /** Called when the activity is first created. */ @Override public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.main); Button btn = (Button)findViewById(R.id.btn); btn.setOnClickListener(new OnClickListener() { @Override public void onClick(View arg0) { startActivity(new Intent(BadCodeActivity.this,TempActivity.class)); } }); Log.d(TAG, "BadCodeActivity instance is called onCreate :" + this.hashCode()); } public void onResume() { super.onResume(); Log.d(TAG, "BadCodeActivity instance is called onResume :" + this.hashCode()); loginThread.start(); } public void onPause() { super.onPause(); loginThread.stopLogin(); } }

其实就是加了一个按钮，做一个页面迁移。别忘了在main.xml里面加上：

<Button android:id="@+id/btn" android:layout_width="fill_parent" android:layout_height="wrap_content" android:text="@string/hello" />

现在我们运行程序。运行之后，点击按钮，画面闪动一下说明是切换了页面。我们偷了个懒，两个Activity共用一个layout，所以页面没有任何变化。但是没关系，我们看log，we are login已经停止输出了。这个时候，我们再按返回键，应该是切换回BadCodeActivity。这个时候系统报错：

java.lang.IllegalThreadStateException: Thread already started.

显然，就是说线程已经启动过了，不能再次被利用。

我们对代码需要做一点点修改。当然，我们也顺手改掉一个BUG：在退出的时候还会报告登录成功。

并且，我们把控制变量都放在内部类里，做到变量最小化生存空间。

修改后如下：

package com.study; import android.app.Activity; import android.content.Intent; import android.os.Bundle; import android.os.Handler; import android.os.Message; import android.util.Log; import android.view.View; import android.view.View.OnClickListener; import android.widget.Button; import android.widget.Toast; /** * 一个段不好的代码 * @author yihui823 */ public class BadCodeActivity extends Activity { private static final String TAG = "BadCodeActivity"; //登录服务 private LoginService lService = LoginService.getInstance(); //外线程访问UI线程的Handle private Handler mhandle = new Handler(){ @Override public void handleMessage(Message msg) { Toast.makeText(BadCodeActivity.this, "登录成功", Toast.LENGTH_LONG).show(); } }; /** * 登录用的线程类 */ private class LoginThread extends Thread { //通知停止线程的标记 private boolean stopFlag = false; //登录成功标记 private boolean loginOk = false; @Override public void run() { while(!stopFlag) { loginOk = lService.isLogin(); if (loginOk) { break; } try { lService.login(); Thread.sleep(1000); } catch (InterruptedException e) { e.printStackTrace(); } } if (loginOk) { mhandle.sendEmptyMessage(0); } } /** * 通知线程需要停止 */ public void stopLogin() { stopFlag = true; } }; //用来登录的线程 private LoginThread loginThread; /** Called when the activity is first created. */ @Override public void onCreate(Bundle savedInstanceState) { super.onCreate(savedInstanceState); setContentView(R.layout.main); Button btn = (Button)findViewById(R.id.btn); btn.setOnClickListener(new OnClickListener() { @Override public void onClick(View arg0) { startActivity(new Intent(BadCodeActivity.this,TempActivity.class)); } }); Log.d(TAG, "BadCodeActivity instance is called onCreate :" + this.hashCode()); } public void onResume() { super.onResume(); Log.d(TAG, "BadCodeActivity instance is called onResume :" + this.hashCode()); loginThread = new LoginThread(); loginThread.start(); } public void onPause() { super.onPause(); loginThread.stopLogin(); } }

现在，我们点击按钮，进入到TempActivity的时候，登录log停止输出；然后按返回键，回到BadCodeActivity的时候，登录log又继续输出。程序基本完成，没有僵尸线程存在了。红色的那行代码是关键！

我们总结一下：

1， 线程对象一定要有变量指向它，以便我们可以控制。

2， 线程类一定要有停止条件，以便外界通知线程自行停止。

3， 线程启动之后，不管是不是已经停止了，都是不能再次利用的。

4， 在onResume里新建和启动线程，在onPause里停止线程。