ReentrantLock没有成功

时间:2015-03-25 15:00:44

标签: java concurrency reentrantlock

我无法弄清楚为什么代码无法正常运行。问题是ReentrantLock没有在ThreadClass.run()

中锁定方法调用

假设哪些方法在ThreadClass中被锁定的资源类

public class ResourceClass {

    private int i; 

    public void setIncrement() {
        i++;
    }

    public int getIncrement() {
        return i;
    }

}

线程级

public class ThreadClass implements Runnable {

    private ResourceClass resource;
    private ReentrantLock reentrantLock = new ReentrantLock();

    ThreadClass(ResourceClass r) {
        resource = r;
    }

    public void run() {
        reentrantLock.lock();
        try {
            resource.setIncrement();
            System.out.println(resource.getIncrement());
        } finally {
            reentrantLock.unlock();
        }
    }

}

主要级

public class MainClass {

    public static void main(String[] args) {

        ResourceClass resource = new ResourceClass();

        Thread thread = new Thread(new ThreadClass(resource));
        thread.start();
        Thread thread2 = new Thread(new ThreadClass(resource));
        thread2.start();
        Thread thread3 = new Thread(new ThreadClass(resource));
        thread3.start();
    }

} 

假设run()中包含锁定的代码必须"同步"因此,只有一个线程可以访问Resourse-object的方法。在实践中它没有。代码导致重复的数字,这意味着两个线程可以同时访问这些方法。我知道这是一个非常简单的问题,但我无法理解如何解决它。感谢您的帮助。

更新

我知道了。那段代码工作正常(我删除了setIncrement()并将所有相关逻辑放入getIncrement()):

public int getIncrement() {
        reentrantLock.lock();
        int incrementResult = i++;
        reentrantLock.unlock();
        return incrementResult;
} 

2 个答案:

答案 0 :(得分:5)

您正在为每个runnable创建一个新的ReentrantLock,这意味着没有同步。您需要与每个Runnable实例共享一个锁。

答案 1 :(得分:0)

所以你的意思是在 ReentrantLock 类中有一个静态的 Thread