Java线程不应使用stop方法粗暴地停止线程运行,使用状态标识解决
public class StopThreadUnsafe {
public static User u = new User();
public static class User {
private int id;
private String name;
public User() {
id = 0;
name = "0";
}
public int getId() {
return id;
}
public void setId(int id) {
this.id = id;
}
public String getName() {
return name;
}
public void setName(String name) {
this.name = name;
}
public String toString() {
return "User [id=" + id + ", name=" + name + "]";
}
}
public static class ChangeObjectThread extends Thread {
/**
* modify method to aoid Thread stop's effect
*
* @param args
*/
volatile boolean stopMe = false;
public void stopMe() {
stopMe = true;
}
@Override
public void run() {
while (true) {
// test
if (stopMe) {
System.out.println("exit by me");
break;
}
synchronized (u) {
int v = (int) (System.currentTimeMillis() / 1000);
u.setId(v);
try {
Thread.sleep(100);
} catch (InterruptedException e) {
e.printStackTrace();
}
u.setName(String.valueOf(v));
}
Thread.yield();
}
}
}
public static class ReadObjectThread extends Thread {
@Override
public void run() {
while (true) {
synchronized (u) {
if (u.getId() != Integer.parseInt(u.getName())) {
System.out.println(u.toString());
}
}
Thread.yield();
}
}
}
public static void main(String[] args) {
new ReadObjectThread().start();
while (true) {
ChangeObjectThread t = new ChangeObjectThread();
t.start();
try {
Thread.sleep(150);
} catch (InterruptedException e) {
e.printStackTrace();
}
if (true) {
t.stop();//thread stop lead to data consistency for it is too rude
} else {
t.stopMe();//thread stop should use stopMe method for it is graceful
}
}
}
}