Singleton avec Lock - blocage possible? en java

Question

J'utilise singleton pour enregistrer tous les événements se produisant simoultanously dans le système.

public class Singleton { 
    private static Lock dbLock; 
    protected Singleton() {} 
    private static class SingletonHolder { 
     private final static Singleton instance = new Singleton(); 
    } 
    public static Singleton getInstance() { 
     return SingletonHolder.instance; 
    } 
    public void recordEvent(Event ev) { 
     dbLock.lock(); 
    try { 
      database.insert(ev); 
     } finally { 
     dbLock.unlock(); 
    } 
    } 
    public File lockAndGetDB() { 
     dbLock.lock(); 
     return database.getFile(); 
    } 
    public void unlockDB() { 
     dbLock.unlock(); 
    } 
} 

Toutes les heures je veux verrouiller l'événement d'enregistrement pour envoyer la base de données sur Internet.

file = Singleton.getInstance().lockAndGetDB(); 
try { 
    sendViaHTTP(file); 
} 
finally { 
    Singleton.getInstance().unlock(); 
} 

Y a-t-il des blocages possibles? Est-ce que le fil est sûr?

EDITED. (essayez/enfin ajouté pour l'envoi viaHTTP) Mais ce n'est pas le problème principal. La question est, s'il y a des threads qui attendent sur dbLock.lock(); à recordEvent (Event ev) sont mon Singleton.getInstance(). unlock(); sera en mesure d'entrer le code getInstance() et de déverrouiller?

Answer 1

Plus probablement que non, oui, il peut y avoir des blocages. Que se passe-t-il si une erreur se produit pendant database.getFile ou sendViaHTTP? Il n'y a pas de motif try/finally pour s'assurer que Lock.unlock est appelée.

Answer 2

Les blocages surviennent lorsque vous acquérez des verrous dans différents ordres. Comme il n'y a qu'un seul verrou ici, la question ne se pose pas. Votre cas ressemble plus à un candidat pour un verrou en lecture-écriture: utilisez des verrous en lecture pour autoriser une simultanéité maximale des opérations SGBD et un verrou en écriture lorsque vous voulez qu'ils arrêtent et envoient le fichier.