如何制作此线程安全

Question

我一直在阅读很多关于线程的知识，但我不确定处理这种情况的正确方法。可以说我有一个引用对象的实用程序类。该实用程序类并在对象上工作，它可以从多个线程

class Utility{ 
    Foo xx; 
    public Utility(Foo xx){ 
     this.xx = xx; 
    } 

    public void doWork(){ 
     x.action(); 
    } 
    } 

现在，如果我有两个线程将要访问的实用工具类和调用的doWork（），其中xx总是指同一被称为对象，我怎样才能使它线程安全？

编辑：有2个实例类（每个线程一个）的实例，但相同的引用xx对象。请参阅编辑：

public class ThreadSafety { 

    public static void main(String[] args) throws InterruptedException { 
     Foo xx = new Foo(); 
     Utility util1 = new Utility(xx); 
     Utility util2 = new Utility(xx); 
     Thread t1 = new Thread(new MyRunnable(util1) , "t1"); 
     Thread t2 = new Thread(new MyRunnable(util2) , "t2"); 

     t1.start(); 
     t2.start(); 
     t1.join(); 
     t2.join(); 

    } 

} 

class MyRunnable implements Runnable{ 
    Utility util; 

    @Override 
    public void run() { 
     util.doWork(); 
    } 


    private void MyRunnable (Utility util) { 
     this.util = util; 
    } 

} 

Answer 1

public class Utility{ 
    private final Foo xx; 

    public Utility(Foo xx){ 
    this.xx = xx; 
    } 

    public void doWork() { 
    // ensures only one thread can be calling action 
    synchronized(xx) { 
     xx.action(); 
    } 
    } 
} 

一些变化如下：

1. 让xxfinal和private。使它private强制执行对如何访问xx的控制。
2. synchronize on xx致电doWork。您可以将​​放在方法声明上，但我更愿意明确说明您锁定的是哪个对象。请注意，这不会“同步xx”。它只是使用xx作为锁。在访问有潜在危险的代码块的任何地方，都必须使用同步的块与同一监视器对象。

Answer 2

您可以使用​​关键字。

public synchronized void doWork(){ 
     x.action(); 
    } 

这将使该方法线程安全。更多关于Oracle documentation的同步方法。

随着更新的问题： 使用静态方法作为同步，然后锁将在类不是个别对象。

Answer 3

Java有一个​​关键字，您可以在这种情况下使用。

public synchronized void doWork() 

阅读

• 有一个在Java Docs关键字的完整说明。

Answer 4

这里是另一种提醒：不要不使用​​。使用同步的代码往往是复杂的，错误的，并且无法测试。你会遇到问题，你永远不会知道什么时候会炸毁，甚至当它运行时，你可能会遭受糟糕的表现。做到这一点，而不是：

创建不变类，你会打电话给你Message线程之间进行通信，并使用Message在预期其他线程存储邮件到你的主线程中处理的BlockingQueue。然后，在你的主线程中，保持从队列中弹出消息并处理它们。简单（* 1），简单，可测试，万无一失。

BlockingQueue使用​​在内部实现，它是由知道自己在做什么的人编写的，并且已经过全面测试，因此它可以工作。这是唯一应该包含​​的地方。 （* 1）起初它似乎并不像在几个地方粘贴​​关键字那样容易，但事实上，从长远来看，​​总是变得更加复杂和困难得到正确的比最初似乎。

Answer 5

如果您有更改数据，请考虑使用不可变对象，而不仅仅是简单地使用同步关键字。

阅读

• This stackoverflow question on immutable data and thread safety
• Oracle's documentation on immutability