我有一个主线程和一个工作线程。主线程将任务添加到队列中,工作线程将它们用于计算数据。在将对象放入队列之前,我在任务对象内部调用ReentrantLock对象(主线程上的)。当工作线程完成处理队列中的任务时,我调用unlock(工作线程 )。问题是我得到了IllegalMonitorStateException,因为我在不同的线程上调用了lock和unlock。
我正在寻找一种替代锁定系统,我可以在不同的线程上执行此操作。
示例:
public class Worker extends Thread {
public static Queue<Task> tasks = new ConcurrentLinkedQueue<Task>();
@Override
public void run() {
while (true) {
Task task = tasks.poll();
if (task != null) {
task.work();
task.lock.unlock(); // Here is the unlock, Task#i should not change up to now
}
}
}
}
public class Task {
private int i = 0;
public Lock lock;
public void setI(int i) {
lock.lock();
this.i = i;
lock.unlock();
}
public void work() {
System.out.println(i);
}
}
public class Test {
Task task = new Task();
public void addTask() {
task.lock.lock(); // Here is the lock, Task#i should not change
Worker.tasks.add(task);
}
}
答案 0 :(得分:4)
为什么不使用只有一个许可证的Semaphore?您可以获得单一许可证,而不是锁定操作。您应该始终使用release()释放锁。
答案 1 :(得分:0)
根据问题,设计多线程应用程序看起来并不正确。
工作线程应该处理对象的创建,或者你应该将不可变对象传递给工作线程,一旦工作线程完成,它就可以将结果传递回主线程。
我认为在一个线程中获取锁定并在另一个线程中解锁是不可能的。
答案 2 :(得分:0)
您不需要替代锁定系统。 ConcurrentLinkedQueue数据结构已经提供了它自己的锁系统并保证了线程安全。任何额外的锁定都是不必要的。
但是你在这里重新发明轮子。我建议您查看ExecutorService和ThreadPools。虽然自己构建东西是一个很好的学习经历。它也是错误的重要来源。
ExecutorService workerPool = Executors.newFixedThreadPool(10); // 10 worker threads
...
Runnable myTask = ...;
workerPool.submit(myTask); // called from the main thread
...