关于实现Runnable接口的类中有公共属性问题

Posted 2021-01-09 huangtao1927

背景：多线程开发的时候，一般都是继承Runnable接口，但是有可能类中有一个公共变量，那么这个变量是不是线程安全的呢？

代码如下：

public class TestThread implements Runnable{

    private int number = 0;

    public TestThread(int num) {
        number = num;
    }

    @Override
    public void run() {
        for(int i = 0;i<10;i++) {
            System.out.println("当前线程为："+Thread.currentThread().getName()+",num="+(number++));
        }
    }

}

当使用线程池进行操作时，可以分为2种情况。

1，共享某一种元素

2，不想共享某一个元素

代码如下：

1，

public class Test {
    public static void main(String[] args) {
        TestThread t = new TestThread(0);
        for(int i = 0;i<10;i++) {
            CommonThreadPool.getInstance().getFixedThreadPool().execute(t);
        }
    }
}

此种写法会有多线程问题，因为共享了一个元素

2，

public class Test {
    public static void main(String[] args) {
//        TestThread t = new TestThread(0);
        for(int i = 0;i<10;i++) {
            CommonThreadPool.getInstance().getFixedThreadPool().execute(new TestThread(0));
        }
    }
}

此种写法某种程度来说不会有多线程问题，因为new对象之后并没有共享同一个元素

如果想避免某一些问题，那么可以加锁看来控制，如：

public class TestThread implements Runnable{

    private int number = 0;

    public TestThread(int num) {
        number = num;
    }

    @Override
    public synchronized void run() {
        for(int i = 0;i<10;i++) {
            System.out.println("当前线程为："+Thread.currentThread().getName()+",num="+(number++));
        }
    }

}

以上只是示例一下，因为加锁的地方是要看具体的情况的。

所以，最初的这段代码：

public class TestThread implements Runnable{

    private int number = 0;

    public TestThread(int num) {
        number = num;
    }

    @Override
    public void run() {
        for(int i = 0;i<10;i++) {
            System.out.println("当前线程为："+Thread.currentThread().getName()+",num="+(number++));
        }
    }

}

其实某种程度上来说并不安全，因为依赖于外部的调用方式。