Refactor & commentaires

src/lecteur_pdf/menu/SelectionnerFichier.java

@@ -9,41 +9,49 @@ import javax.swing.*;
 import javax.swing.filechooser.FileNameExtensionFilter;
 import javax.swing.filechooser.FileSystemView;
 import java.io.File;
+
 /**
  * Classe qui permet d'ouvrir une fenêtre pour sélectionner le fichier pdf à
  * ouvrir
+ *
  * @author Léo FRANCH
  * @author Tristan NOGARET
  * @author Lucàs VABRE
  * @author Noé VILLENEUVE
- * @version  1.0
+ * @version 1.0
  */
 public class SelectionnerFichier {
 
     /**
      * Méthode qui créée une fenêtre pour que l'utilisateur choisisse un
      * fichier PDF
+     *
      * @return file le fichier choisi par l'utilisateur
      */
     public static File ouvrirFichier() {
 
-        JFileChooser fileChooser = new JFileChooser
-                     (FileSystemView.getFileSystemView().getHomeDirectory());
-        fileChooser.setDialogTitle("Sélectionnez un PDF");
+        final String TITRE = "Sélectionnez un PDF";
+        final String DESCRIPTION = "PDF files (*.pdf)";
+        final String EXTENSION = "pdf";
+
+        /* Crée une fenêtre de sélection de fichier */
+        JFileChooser fileChooser = new JFileChooser(
+            FileSystemView.getFileSystemView().getHomeDirectory());
+        fileChooser.setDialogTitle(TITRE);
         fileChooser.setMultiSelectionEnabled(false);
         fileChooser.setFileSelectionMode(JFileChooser.FILES_ONLY);
         fileChooser.setAcceptAllFileFilterUsed(false);
-        FileNameExtensionFilter
-            filter = new FileNameExtensionFilter("PDF files (*.pdf)", "pdf");
+
+        /* Filtre uniquement les fichiers PDF */
+        FileNameExtensionFilter filter = new FileNameExtensionFilter(
+            DESCRIPTION, EXTENSION);
         fileChooser.addChoosableFileFilter(filter);
 
-        File file;
+
         int returnValue = fileChooser.showOpenDialog(null);
-        if (returnValue == JFileChooser.APPROVE_OPTION) {
-            file = new File(fileChooser.getSelectedFile().getAbsolutePath());
-        } else {
-            file = null;
-        }
+        File file = returnValue == 0 ? new File(
+            fileChooser.getSelectedFile().getAbsolutePath()) : null;
+
         return file;
     }
 }