SYNCHRONISATION AVEC UN FICHIER PROPERTIES

Commentaire de romuluslepunk le 05/12/2006 17:45:42

Effectivement j'avais pas pensé à celà, je viens d'uploader un nouveau zip

Commentaire de Twinuts le 05/12/2006 21:14:45 administrateur CS

Salut,


concernant ton code il y a des choses que je ne comprend pas tu dis que tu fais de la synchronisation ...... mai c'est faux, je m'explique :


tu déclares la variable private boolean synchronise = false; hors il y à un truc qui ne cole pas si tu fais :
private ConfigManager mesConfig1 = ConfigManager.getInstance();
private ConfigManager mesConfig2 = ConfigManager.getInstance();

ici chacun à sa synchro ce qui n'est pas vraiment propre .... donc 2 solutions soit tu mets synchronise en static soit tu mets le constructeur de ConfigManager en private comme cela déjà tu commenceras à avoir un début de synchronisation...

ensuite autre chose tu ne lock pas le fichier ce qui est assez dommage vu qu'une application '1' peut modifier le fichier qu'utilise l'application 'n'... pourquoi ne pas lock le fichier ?

tu peux le faire avec :
private boolean                 locked          = true;
private RandomAccessFile        raLockFile      = null;
....

try {
   File lockedFile = new File(fileName);
   raLockFile = new RandomAccessFile(lockedFile, "rw");
   if(raLockFile.getChannel().tryLock() == null)
       locked = false;
    else
       locked = true;
} catch (Exception e) { }

....
public boolean isLocked(){
    return locked;
}

et pour le déloquer
raLockFile.getChannel().close();


bref c'est pas vraiment une critique mais plus une pitite remarque qui pourrait rendre ta classe réellement synchronisé par une application voir même plusieurs applications

Commentaire de Twinuts le 06/12/2006 19:52:19 administrateur CS

"Mon singleton me semble bon, le constructeur est déjà en private mais son contenue est vide, celà joue sur le résultat ?"
- cela joue si tu utilise le constructeur public sans passer par le getInstance

"j'hesite encore à l'ajouter, en cas d'arret brutal du programme, la methode desynchronise() "
- enfait le lock prendra fin en cas d'arret de l'appli sinon tu peux le faire proprement et utiliser le shuttdownhook de Runtime ref(pour un exemple) : http://www.techmag.biz/java_app_shutdown_hook

"si un programme quelconque utilise et met à jour le fichier properties sans passer par l'api, le lock n'a plus d'interet"
- si par ce qu'il ne pourra ni lire ou modifier le fichier vu que dans mon exemple j'ouvre le fichier en rw soit read & write et le lock le vérouille sur le mode d'ouverture soit read write

Commentaire de Twinuts le 06/12/2006 23:45:39 administrateur CS

Salut,

"Si mon constructeur est en private" <- milles excuses je viens seulement de le voir caché entre deux commentaires :D si le constructeur est private comme c'est le cas et que l'instance interne est static alors pas de risque le singleton fonctionne :)

Commentaire de Twinuts le 19/12/2006 20:08:02 administrateur CS

Salut,

en gros cela veut dire simplement qu'a un instant donné tu es le seul à pouvoir accéder au fichier