Discussion :

[AlexMog]

Bonjour à tous,
Etant amateur du développement en java (en réalité, je me suis intéressé à java pour ses fonctions multi-plateforme), je me suis développé des petits utilitaires java empaquetés dans une lib.

N'ayant trouvé aucune lib (qui ne soit pas effroyablement lourde) contenant ces différents utilitaires, j'ai créé la mienne.

J'espère que vous y trouverez une utilité, et j'espère pouvoir récolter vos avis, demandes et retours .

GitHub du projet: https://github.com/AlexMog/JavaMogLib
JavaDoc du projet: http://alexmog.tv/javamoglib/doc/
Téléchargements du projet: http://www.mediafire.com/folder/dl3b...nfd/JavaMogLib

Je vous remercie d'avoir pris le temps de vous intéresser au projet.


Cordialement,
AlexMog.

[Logan Mauzaize]

As tu pensé à regarder du côté de java.nio.file.Files ou Google Guava ou Apache Commons (Lang/IO) ?

Quelques conseils :

org.MogLib.library.utils.DirUtils#readDir(File)

• Renomme la méthode de sorte que la recherche des sub-directories se lise dans le nom (ou bien utilise un paramètre).
• Les répertoires ne sont jamais listés
• Je te conseillerai de trier les éléments
• Je te conseillerai d'utiliser une liste en paramètre pour la récursion plutôt que de créer une nouvelle liste à chaque fois. De plus ca permettrait de choisir l'implémentation de son choix.

Code Java :

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

1
2
3
4
5
if (list != null) {
  for (int i = 0; i < list.length; i++)
    listFiles.addAll(readDir(list[i]));
} else
  throw new Exception("File read error");

• Si tu ne mets pas d'accolade, je te conseille de mettre l'instruction sur la même ligne.
• Dans certains cas, listFiles renvoie null pour les répertoires vides.

org.MogLib.library.utils.DirUtils#copyDir(File, File)

• Crée des variables pour stocker destination.getAbsolutePath() et dir.getAbsolutePath().
• Si tu n'utilisais pas le getAbsolutePath tu n'aurais pas besoin de faire de replace.
• Renomme dir en source
• Tu reconstruits le parent file à chaque fois avec ta méthode.
• Les répertoires vides ne sont pas copiés.

org.MogLib.library.utils.DirUtils#moveDir(File, File)

• Essaye d'abord avec renameTo avant de faire ton bazar.
• Tu laisses les répertoires vides de la source.
• Les répertoires vides de la source ne sont pas créés.

org.MogLib.library.utils.FileUtils

• Fusionne cette classe et la précédente et gère les spécificités par des conditions et des méthodes privées.
• Remplace JProgressBar par BoundedRangeModel.

org.MogLib.library.utils.FileUtils.FileUtils()

• Change la visibilité par "protected" ca permet de "fusionner" des classes utilitaires étendues et rend la classe "abstract".

org.MogLib.library.utils.FileUtils.copyFile(File,File,JProgressBar)

• Utilise un paramètre pour définir la taille du buffer
• Utilise plutôt 4096 comme valeur par défaut pour la taille du buffer
• Ne recopie pas tout le buffer dans le fichier de sortie mais utilise le nombre d'octet lu
• Utilise try-with-resources pour gérer proprement les fermetures de flux.

Pour le reste même remarques que précédemment.

Je regarderai le reste si j'ai le temps.

[AlexMog]

Merci beaucoup pour tes conseils, je vais tenter de les mettre en place .
Pour la fonction renameTo, je ne pensais pas qu'elle prenait aussi en compte le chemin. Merci pour l'info!

Cordialement,
AlexMog.

[Logan Mauzaize]

Attention au renameTo, car il renvoie false (et ne fait rien) si le fichier n'est pas sur le même disque ou la même partition.

[AlexMog]

Attention au renameTo, car il renvoie false (et ne fait rien) si le fichier n'est pas sur le même disque ou la même partition.

Du coup il faudra que je copie à la main si il renvois false.
Merci à toi.