并发访问@Lock(LockType.WRITE)方法

时间:2014-03-18 23:26:08

标签: java java-ee concurrency ejb jms

我想要完成的任务:

我有一个JMS队列,我想异步获取该队列中的元素并发送到打印机。这台打印机非常具体和笨拙,所以它不能在它上面排队或缓冲任何东西。我必须在此打印机上按顺序执行5个步骤,并且排队的elemet的步骤不得与其他元素的步骤重叠。对于每个步骤,打印机将在成功时返回1,在失败时返回另一个数字。

当前实现是一个MDB,其中一个 onMessage 方法从Singleton ejb调用@Lock(LockType.WRITE)方法。

MDB:

    @MessageDriven(activationConfig = {
@ActivationConfigProperty(propertyName = "destinationLookup",
        propertyValue = "jms/MyQueue"),
@ActivationConfigProperty(propertyName = "destinationType",
        propertyValue = "javax.jms.Queue")})

public class SimpleMessageBean implements MessageListener {

@Resource
private MessageDrivenContext mdc;
static final Logger logger = Logger.getLogger("SimpleMessageBean");

@EJB
private Print print;

public SimpleMessageBean() {
}

@Override
public void  onMessage(Message inMessage) {
    try {
        if (inMessage instanceof TextMessage) {
            logger.log(Level.INFO, "MESSAGE BEAN: Message received: {0}", inMessage.getBody(String.class));

            String params[] = ((TextMessage) inMessage).getText().split("\\s+");
            print.doStuff(params[0], params[1]);
        } else {
            logger.log(Level.WARNING,"Error")
        }
    } catch (Exception e) {
        logger.log(Level.SEVERE, "Rollback Happening: {0}", e.toString());
        mdc.setRollbackOnly();
    }
}

The Singleton:

@ConcurrencyManagement(ConcurrencyManagementType.CONTAINER)
@Startup
@Singleton
@AccessTimeout(value = 120000)
public class Print {

static final Logger logger = Logger.getLogger("SimpleMessageBean");

public Print() {
}

@Lock(LockType.WRITE)
public void doStuff(String cpf, String value) throws Exception{
        try{
            int return;
            return = Bematech.AbreCupom(cpf);
            System.out.println("Stage0 = " + return);

            return = Bematech.VendeItem("111112", "Parking", "FF", "hours", "1", 2, value, "%", "00,00");
            System.out.println("Stage1 = " + return);

            return = Bematech.IniciaFechamentoCupom("D", "%", "00,00");
            System.out.println("Stage2 = " + return);

            return = Bematech.EfetuaFormaPagamento("Dinheiro", value);
            System.out.println("Stage3 = " + return);

            return = Bematech.TerminaFechamentoCupom("Thankyou ofr your support");
            System.out.println("Stage4 = " + return);


        }catch ( Exception MensagemErro ){
            throw MensagemErro;
        }
}

我希望这个实现从队列接收数据并在doStuff()方法被锁定时一次打印一个元素,AKA:

Stage0=1
Stage1=1
Stage2=1
Stage3=1
Stage4=1
Stage0=1
Stage1=1
Stage2=1
Stage3=1
Stage4=1
Stage0=1
...

但有时候(似乎是随机的,但是当多个消息一次排队时更常发生)我得到两个并发访问print方法:

Stage0=1
Stage1=1
Stage0=1
Stage2=1
Stage1=1
Stage3=1
Stage2=1
...

从正在打印的内容中我可以看到打印机访问中有2个元素重叠。

有用信息:

Java EE 7。 Glassfish 4-0。 不使用JBOSS。

为什么会发生这种情况?

1 个答案:

答案 0 :(得分:1)

同步关键字解决了我的问题。没找到原始代码的错误。