不良代码展示-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();
    }
}

<span style="background-color: rgb(255, 255, 255);">这个例子呢，显然运行的时候会报错的。请见我的另一篇文章：<a target="_blank" href="http://blog.csdn.net/yihui823/article/details/6722784">Android画面UI中的线程约束</a>。我们在非UI线程里去控制UI界面，就必须使用Handler来发送消息。修改代码如下：</span>

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();
	
	<span style="color: #ff6666;">//外线程访问UI线程的Handle
	private Handler mhandle = new Handler(){
		@Override
		public void handleMessage(Message msg) {
			Toast.makeText(BadCodeActivity.this, "登录成功", Toast.LENGTH_LONG).show();
		}
	};</span>
	
	
    /** 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();
					}
				}
				<span style="color: #ff0000;">mhandle.sendEmptyMessage(0);</span>
			}
		}).start();
    }
}

<p><span style="background-color: rgb(255, 255, 255);">红色部分代码，就是修改的地方。现在，这段代码可以运行了，而且还貌似不错，是吧。</span></p>
<p><span style="background-color: rgb(255, 255, 255);">但是，一个好的程序，不能只是应付正常情况，还要应付错误情况，是吧。如果登录总是出错怎么样呢？我们把LoginService类略微改动，如下：</span></p>

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);
    }
}

<pre name="code" class="java">
</pre>

别忘了修改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);
        
 <span style="color: #ff0000;">       Button btn = (Button)findViewById(R.id.btn);
        btn.setOnClickListener(new OnClickListener() {
			@Override
			public void onClick(View arg0) {
		    	startActivity(new Intent(BadCodeActivity.this,TempActivity.class));
			}
        });</span>
        
        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());
        <span style="color: #ff0000;">loginThread = new LoginThread();</span>
        loginThread.start();
    }
    
    public void onPause() {
    	super.onPause();
    	loginThread.stopLogin();
    }
}

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

我们总结一下：

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

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

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

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