Count.java:
- public class Count {
- private int num;
- public void count() {
- for(int i = 1; i <= 10; i++) {
- num += i;
- }
- System.out.println(Thread.currentThread().getName() + "-" + num);
- }
- }
ThreadTest.java:
- public class ThreadTest {
- public static void main(String[] args) {
- Runnable runnable = new Runnable() {
- Count count = new Count();
- public void run() {
- count.count();
- }
- };
- for(int i = 0; i < 10; i++) {
- new Thread(runnable).start();
- }
- }
- }
- Thread-0-55
- Thread-1-110
- Thread-2-165
- Thread-4-220
- Thread-5-275
- Thread-6-330
- Thread-3-385
- Thread-7-440
- Thread-8-495
- Thread-9-550
楼主这个例子至少两点是不正确的: 1、这个也不是共享变量的问题。num是实例变量,每个对象中都会有自己单独的num属性,所以不存在共享的问题。之所以num会被累加是因为此处只有一个对象,楼主的例子只是证明了一点:多个线程共享了一个对象! 2、for循环的次数太少,同一个for循环没有被其他线程切断,具体说是num+没被切断,因为num+是非原子性操作,可以分为读和写两步操作,如果第一个线程再读num后被切断(假如此时num=1),这时第二个线程读到的num还是1,如果第二个线程读完以后没被其他线程切断,就会将num改为2,然后假如第一个线程这时候又抢到了执行机会,他会接着进行之前被切断的操作在num加1,由于线程一之前读到的num=1,他并不知道此刻别的线程已将num变成了2,所以他会在1的基础上对num加1,此时num还会是2,这就导致了线程的不安全…… 所以说此处的550是个偶然,原理上讲总结果会小于这个数!
只有Thread-0线程输出的结果是我们期望的,而输出的是每次都累加的,这里累加的原因以后的博文会说明,那么要想得到我们期望的结果,有几种解决方案:
1. 将Count中num变成count方法的局部变量;
- public class Count {
- public void count() {
- int num = 0;
- for(int i = 1; i <= 10; i++) {
- num += i;
- }
- System.out.println(Thread.currentThread().getName() + "-" + num);
- }
- }
- 而楼主将num从全局变量改为count的局部变量,每次调用count的方法,本身就会重新分配局部变量
2. 将线程类成员变量拿到run方法中,这时count引用是线程内的局部变量;
- public class ThreadTest4 {
- public static void main(String[] args) {
- Runnable runnable = new Runnable() {
- public void run() {
- Count count = new Count();
- count.count();
- }
- };
- for(int i = 0; i < 10; i++) {
- new Thread(runnable).start();
- }
- }
- }
再作者关于运行结果的描述等部分内容是错误的,导致文章质量大打折扣
3. 每次启动一个线程使用不同的线程类,不推荐。
上述测试,我们发现,存在成员变量的类用于多线程时是不安全的,不安全体现在这个成员变量可能发生非原子性的操作,而变量定义在方法内也就是局部变量是线程安全的。想想在使用struts1时,不推荐创建成员变量,因为action是单例的,如果创建了成员变量,就会存在线程不安全的隐患,而struts2是每一次请求都会创建一个action,就不用考虑线程安全的问题。所以,日常开发中,通常需要考虑成员变量或者说全局变量在多线程环境下,是否会引发一些问题。