Isso precisa de sincronização explícita?
Eu tenho dois threads e quero ter certeza de que estou fazendo a sincronização corretamente no LinkedBlockingQueue. Isso está correto? Ou a sincronização explícita em (messageToCommsQueue) não é necessária?
Declaração:
private LinkedBlockingQueue<BaseMessage> messagesToCommsQueue;
Método um:
private void startOperationModeStatusMessageExecutor() {
ScheduledExecutorService operationModeStatusExecutor = Executors.newSingleThreadScheduledExecutor();
operationModeStatusExecutor.scheduleAtFixedRate((new Runnable() {
@Override
public void run() {
MT02OperationModeStatusMessage commsOperateMsg;
commsOperateMsg = MessageFactory.getMT1102OperationModeStatusMessage(status.ordinal());
synchronized (messagesToCommsQueue) {
messagesToCommsQueue.add(commsOperateMsg);
}
}
}), 0, 10, TimeUnit.SECONDS);
}
Método dois:
Executor commsSenderExecutor = Executors.newSingleThreadExecutor();
commsSenderExecutor.execute(new Runnable() {
@Override
public void run() {
while (getStatus().equals(OperationModeStatus.INITIATE) || getStatus().equals(OperationModeStatus.OPERATE)) {
BaseMessage m = null;
try {
synchronized (messagesToCommsQueue) {
m = messagesToCommsQueue.take();
}
} catch (InterruptedException e) {
// TODO Auto-generated catch block
e.printStackTrace();
}
}
}
});