Discussion :

[Mister Nono]

Salut à tous,

Une appli doit restaurer un fichier(destination) à partir d'une copie d'un autre fichier(source).

Pendant ce traitement de copie de fichier je désire verrouiller l'accès aux 2 fichiers (fichier source et fichier destination).

Comment faire cela ?

Un équivalent sur les objets est : synchronized(monObjet) { ... traitement ... }

Merci.

[thibaut]

Salut,

Si tu es sous java5 tu as FileLock, sinon tu devras programmer le lock toi même, ou créer un fichier .lock pour indiquer que le fichier est vérouillé.

[Mister Nono]

Merci, je vais essayer cela.

[yasinfo]

Salut,

Si tu es sous java5 tu as FileLock, sinon tu devras programmer le lock toi même, ou créer un fichier .lock pour indiquer que le fichier est vérouillé.

Salut,
Ce sujet m'intéresse et je voudrais savoir comment utiliser FileLock.
Merci.

[Kikito]

Salut,
Ce sujet m'intéresse et je voudrais savoir comment utiliser FileLock.
Merci.

Tu récupères une instance de FileChannel via getChannel() de FileInputStream/FileOutputStream.
Tu fais un tryLock() dessus, si y a déja un lock dessus ça renvoie null sinon ca te renvoie une instance de FileLock, à ce moment tu as mis un Lock dessus que tu pourras rendre par release().

[®om]

Le tuto n'est pas fait pour ça, mais il répond à ta question :
http://rom.developpez.com/java-instance-unique/#LII-A-2

[yasinfo]

Salut,
Merci à vous, je vais y jeter un coup d'oeil

[Mister Nono]

Le tuto n'est pas fait pour ça, mais il répond à ta question :
http://rom.developpez.com/java-instance-unique/#LII-A-2

Et oui, je l'avais trouvé aussi, et j'attendais d'être au travail pour essayer cette solution.

Merci encore.

[adiGuba]

Salut,

Si tu es sous java5 tu as FileLock

Juste pour préciser : FileLock est apparut dès Java 1.4 , et son comportement peut varier selon le système (voir la doc.)

a++

[Mister Nono]

Salut,

Si tu es sous java5 tu as FileLock, sinon tu devras programmer le lock toi même, ou créer un fichier .lock pour indiquer que le fichier est vérouillé.

J'utilise Java 5.

[Kikito]

Je n'ai pas testé ton code mais qq remarques comme ça :
- Pour faire de la copie de fichier vers fichier tu as FileChannel.transferTo(...) et FileChannel.transferFrom(...) qui sont extrement rapide.
- Si tu es dans un java.nio utiliser des ByteBuffer va plus vite que des tableaux de bytes.

[Kikito]

Voici un code qui marche sans utiliser de ByteBuffer ou les transfer, juste en corrigeant ce qui ne fonctionnait pas :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
public static synchronized void copieFichier(File source, File destination) throws IOException
    {
	RandomAccessFile sourceFile = new RandomAccessFile(source, "rw");
	FileChannel fcSource = sourceFile.getChannel();
	FileLock verrouSource = fcSource.lock();
 
	FileOutputStream destinationFile = new FileOutputStream(destination);
	FileChannel fcDestination = destinationFile.getChannel();
	FileLock verrouDestination = fcDestination.lock();
 
        // lecture par segment de 0.5 Mo
        byte buffer[] = new byte[512 * 1024];
 
        int nbLecture = -1;
 
        while ((nbLecture = sourceFile.read(buffer)) != -1)
        {
            destinationFile.write(buffer, 0, nbLecture);
        }
 
        verrouSource.release();
        sourceFile.close();
 
        verrouDestination.release();
        destinationFile.close();
 
    }

Dans ton code, le problème est que tu ouvrais deux fois des flux, si tu ouvres un RandomAcessFile il ne faut pas rouvrir un FileInputStream sur le File mais utiliser le RandomAcessFile pour écrire, c'est lui qui est "propriétaire" du lock pas le thread entier.

[Kikito]

Avec un transfer, plus rapide dans la majorité des cas* et plus simple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public static synchronized void copieFichier(File source, File destination) throws IOException
    {
	RandomAccessFile sourceFile = new RandomAccessFile(source, "rw");
	FileChannel fcSource = sourceFile.getChannel();
	FileLock verrouSource = fcSource.lock();
	FileOutputStream destinationFile = new FileOutputStream(destination);
	FileChannel fcDestination = destinationFile.getChannel();
	FileLock verrouDestination = fcDestination.lock();
 
	fcSource.transferTo(0, fcSource.size(), fcDestination);
 
        verrouSource.release();
        sourceFile.close();
        verrouDestination.release();
        destinationFile.close();
 
    }

*sur des petits fichiers un write avec un petit buffer est légèrement plus rapide