这种关闭线程的方法是否安全或者有什么不足

Question

这种关闭线程的方法是否安全或者有什么不足？
代码如下：
[code="java"]
package test;

public class Test extends Thread{

private boolean b = true;

/**
 * 启动线程的方法
 */
synchronized public void run(){
    System.out.println("线程启动");
    while(b){
        //做一些事
    }
}

/**
 * 关闭线程的方法
 */
public void close(){
    b = false;
    System.out.println("关闭线程");
}

}
[/code]

Answer 1

这种做法应该没什么问题，我在开源软件源码里也看到过。不过就是要注意一点，在b被设置为false的时候，run方法可能刚好进入while，这样在你close之后，还会多做一次//做一些事