如何在java中使用枚举的线程安全性?

时间:2010-03-28 04:14:30

标签: java enums singleton

java中枚举的线程安全性如何? 我正在使用枚举实现Singleton(根据Bloch的Effective Java), 我应该担心我的单身枚举的线程安全吗? 有没有办法证明或证明它是线程安全的?

// Enum singleton - the preferred approach
public enum Elvis { 
    INSTANCE;
    public void leaveTheBuilding() { ... }
}

由于

4 个答案:

答案 0 :(得分:33)

正如@Mike所说,enum的创建保证是线程安全的。但是,添加到枚举类的方法不带任何线程安全保证。特别地,方法leaveTheBuilding可以由多个线程同时执行。如果这种方法有副作用(改变某些变量的状态),那么你需要考虑保护它(即使它成为synchronized)或其中的一部分。

答案 1 :(得分:10)

自定义枚举定义可能不是线程安全的。例如,

RoleEnum.java:

package com.threadsafe.bad;

public enum RoleEnum {
       ADMIN(1),
       DEV(2),
       HEAD(3);

       private Integer value;
       private RoleEnum(Integer role){
              this.value=role;           
       }
       public static RoleEnum fromIntegerValue(Integer role){

              for(RoleEnum x : values()){
                     if(x.value == role ){
                           return x;
                     }
              }
              return RoleEnum.HEAD;             
       }

       Class<?> buildFromClass;
       public void setBuildFromClass(Class<?> classType){
              buildFromClass=classType;
       }
       public Class<?> getBuildFromClass(){
              return this.buildFromClass;
       }
}

Main.java:

package com.threadsafe.bad;

public class Main {

       public static void main(String[] args) {
              // TODO Auto-generated method stub

              Thread threadA = new Thread(){
                     public void run(){
                           System.out.println("A started");
                           RoleEnum role;
                           role=RoleEnum.fromIntegerValue(1);
                           System.out.println("A called fromIntegerValue");
                           role.setBuildFromClass(String.class);
                           System.out.println("A called setBuildFromClass and start to sleep");


                           try {
                                  Thread.sleep(10000);
                           } catch (InterruptedException e) {
                                  // TODO Auto-generated catch block
                                  e.printStackTrace();
                           }
                           System.out.println("Thread A: "+role.getBuildFromClass());
                     }
              };

              Thread threadB = new Thread(){
                     public void run(){
                           System.out.println("B started");
                           RoleEnum role;
                           role=RoleEnum.fromIntegerValue(1);
                           role.setBuildFromClass(Integer.class);
                           System.out.println("B called fromIntegerValue&setBuildFromClass and Start to sleep");
                           try {
                                  Thread.sleep(20000);
                           } catch (InterruptedException e) {
                                  // TODO Auto-generated catch block
                                  e.printStackTrace();
                           }
                           System.out.println("B waked up!");

                           System.out.println("Thread B: "+ role.getBuildFromClass());
                     }

              };

              threadA.start();
              threadB.start();


       }

}

有时输出会是:

B开始了

B调用fromIntegerValue&amp; setBuildFromClass并开始睡眠

已开始

一个名为fromIntegerValue的

一个名为setBuildFromClass并开始睡眠的

线程A:类java.lang.String

B醒了!

线程B:类java.lang.String &lt; -We expect java.lang.Integer

有时输出会是:

已开始

一个名为fromIntegerValue的

一个名为setBuildFromClass并开始睡眠的

B开始了

B调用fromIntegerValue&amp; setBuildFromClass并开始睡眠

线程A:类java.lang.Integer &lt; -We expect java.lang.String

B醒了!

线程B:类java.lang.Integer

答案 2 :(得分:9)

这种技术绝对是线程安全的。枚举值保证只能在使用之前由单个线程初始化一次。但是,我不确定是在加载枚举类还是第一次访问枚举值本身时。使用这种技术实际上比其他技术更安全,因为甚至没有一种方法可以使用反射来获得基于枚举的单例的第二个副本。

答案 3 :(得分:2)

添加 synchronized 可以避免与枚举的状态不一致。

下面的代码将运行将很好地锁定打印“一”。但是,当您注释掉 synchronized 时,也会打印其他值。

import java.util.Random;
import java.util.concurrent.atomic.AtomicInteger;

public class TestEnum
{
    public static AtomicInteger count = new AtomicInteger(1);

    public static enum E
    {
        One("One"),
        Two("Two");

        String s;

        E(final String s)
        {
            this.s = s;
        }

        public void set(final String s)
        {
            this.s = s;
        }

        public String get()
        {
            return this.s;
        }
    }

    public static void main(final String[] args)
    {
        doit().start();
        doit().start();
        doit().start();
    }

    static Thread doit()
    {
        return new Thread()
        {
            @Override
            public void run()
            {
                String name = "MyThread_" + count.getAndIncrement();

                System.out.println(name + " started");

                try
                {
                    int i = 100;
                    while (--i >= 0)
                    {

                        synchronized (E.One)
                        {
                            System.out.println(E.One.get());
                            E.One.set("A");
                            Thread.sleep(new Random().nextInt(100));
                            E.One.set("B");
                            Thread.sleep(new Random().nextInt(100));
                            E.One.set("C");
                            Thread.sleep(new Random().nextInt(100));
                            E.One.set("One");
                            System.out.println(E.One.get());
                        }

                    }
                }
                catch (InterruptedException e)
                {
                    // TODO Auto-generated catch block
                    e.printStackTrace();
                }

                System.out.println(name + " ended");
            }
        };
    }
}