JAL-3141 Reformatted code in eclipse
[jalview.git] / src / jalview / io / BackupFiles.java
index 0b6b090..e913fbc 100644 (file)
@@ -9,13 +9,14 @@ import java.io.File;
 import java.io.IOException;
 import java.util.Arrays;
 import java.util.HashMap;
+import java.util.Map;
 import java.util.TreeMap;
 
 /*
  * BackupFiles used for manipulating (naming rolling/deleting) backup/version files when an alignment or project file is saved.
  * User configurable options are:
  * BACKUPFILES_ENABLED - boolean flag as to whether to use this mechanism or act as before, including overwriting files as saved.
- * BACKUPFILES_SUFFIX - a template to insert just before the file extension.  Use '%n' to be replaced by a 0-led SUFFIX_DIGITS long integer.
+ * BACKUPFILES_SUFFIX - a template to insert after the file extension.  Use '%n' to be replaced by a 0-led SUFFIX_DIGITS long integer.
  * BACKUPFILES_NO_MAX - flag to turn off setting a maximum number of backup files to keep.
  * BACKUPFILES_ROLL_MAX - the maximum number of backupfiles to keep for any one alignment or project file.
  * BACKUPFILES_SUFFIX_DIGITS - the number of digits to insert replace %n with (e.g. BACKUPFILES_SUFFIX_DIGITS = 3 would make "001", "002", etc)
@@ -27,25 +28,28 @@ public class BackupFiles
 {
 
   // labels for saved params in Cache and .jalview_properties
-  private static String NS = "BACKUPFILES";
+  public static final String NS = "BACKUPFILES";
 
-  public static String ENABLED = NS + "_ENABLED";
+  public static final String ENABLED = NS + "_ENABLED";
 
-  public static String SUFFIX = NS + "_SUFFIX";
+  public static final String SUFFIX = NS + "_SUFFIX";
 
-  public static String NO_MAX = NS + "_NO_MAX";
+  public static final String NO_MAX = NS + "_NO_MAX";
 
-  public static String ROLL_MAX = NS + "_ROLL_MAX";
+  public static final String ROLL_MAX = NS + "_ROLL_MAX";
 
-  public static String SUFFIX_DIGITS = NS + "_SUFFIX_DIGITS";
+  public static final String SUFFIX_DIGITS = NS + "_SUFFIX_DIGITS";
 
-  protected static String NUM_PLACEHOLDER = "%n";
+  public static final String NUM_PLACEHOLDER = "%n";
 
-  public static String REVERSE_ORDER = NS + "_REVERSE_ORDER";
+  public static final String REVERSE_ORDER = NS + "_REVERSE_ORDER";
 
-  public static String CONFIRM_DELETE_OLD = NS + "_CONFIRM_DELETE_OLD";
+  public static final String CONFIRM_DELETE_OLD = NS
+          + "_CONFIRM_DELETE_OLD";
 
-  private static String DEFAULT_TEMP_FILE = "jalview_temp_file_" + NS;
+  private static final String DEFAULT_TEMP_FILE = "jalview_temp_file_" + NS;
+
+  private static final String TEMP_FILE_EXT = ".tmp";
 
   // file - File object to be backed up and then updated (written over)
   private File file;
@@ -89,13 +93,11 @@ public class BackupFiles
   // REVERSE_ORDER
   public BackupFiles(File file)
   {
-    this(file, "-v" + NUM_PLACEHOLDER, false, 4, 3, false);
+    this(file, ".v" + NUM_PLACEHOLDER, false, 4, 3, false);
   }
 
-  public BackupFiles(File file,
-          String defaultSuffix, boolean defaultNoMax, int defaultMax,
-          int defaultDigits,
-          boolean defaultReverseOrder)
+  public BackupFiles(File file, String defaultSuffix, boolean defaultNoMax,
+          int defaultMax, int defaultDigits, boolean defaultReverseOrder)
   {
     classInit();
     this.file = file;
@@ -114,11 +116,11 @@ public class BackupFiles
       {
         String tempfilename = file.getName();
         File tempdir = file.getParentFile();
-        temp = File.createTempFile(tempfilename, ".tmp", tempdir);
+        temp = File.createTempFile(tempfilename, TEMP_FILE_EXT, tempdir);
       }
       else
       {
-        temp = File.createTempFile(DEFAULT_TEMP_FILE, ".tmp");
+        temp = File.createTempFile(DEFAULT_TEMP_FILE, TEMP_FILE_EXT);
       }
     } catch (IOException e)
     {
@@ -185,11 +187,6 @@ public class BackupFiles
     return path;
   }
 
-  public static String getNumPlaceHolder()
-  {
-    return NUM_PLACEHOLDER;
-  }
-
   public boolean setWriteSuccess(boolean flag)
   {
     boolean old = this.tempFileWriteSuccess;
@@ -207,7 +204,6 @@ public class BackupFiles
     return tempFile.renameTo(file);
   }
 
-
   // roll the backupfiles
   public boolean rollBackupFiles()
   {
@@ -235,29 +231,19 @@ public class BackupFiles
     }
     String filename = file.getName();
     String basename = filename;
-    String extension = "";
-    int dotcharpos = filename.lastIndexOf('.');
-    // don't split of filenames with the last '.' at the very beginning or
-    // very end of the filename
-    if ((dotcharpos > 0) && (dotcharpos < filename.length() - 1))
-    {
-      basename = filename.substring(0, dotcharpos);
-      extension = filename.substring(dotcharpos); // NOTE this includes the '.'
-    }
 
     boolean ret = true;
     // Create/move backups up one
 
     File[] oldFilesToDelete = null;
-    
+
     // find existing backup files
     BackupFilenameFilter bff = new BackupFilenameFilter(basename, suffix,
-            digits,
-            extension);
+            digits);
     File[] backupFiles = dirFile.listFiles(bff);
     int nextIndexNum = 0;
     String confirmDeleteExtraInfo = null;
-    
+
     if (backupFiles.length == 0)
     {
       // No other backup files. Just need to move existing file to backupfile_1
@@ -265,18 +251,8 @@ public class BackupFiles
     }
     else
     {
-
-      // sort the backup files (based on integer found in the suffix) using a
-      // precomputed Hashmap for speed
-      HashMap<Integer, File> bfHashMap = new HashMap<>();
-      for (int i = 0; i < backupFiles.length; i++)
-      {
-        File f = backupFiles[i];
-        BackupFilenameParts bfp = new BackupFilenameParts(f, basename, suffix, digits, extension);
-        bfHashMap.put(bfp.indexNum(), f);
-      }
-      TreeMap<Integer, File> bfTreeMap = new TreeMap<>();
-      bfTreeMap.putAll(bfHashMap);
+      TreeMap<Integer, File> bfTreeMap = sortBackupFilesAsTreeMap(
+              backupFiles, basename);
 
       if (reverseOrder)
       {
@@ -305,7 +281,7 @@ public class BackupFiles
           // int n = tempMax - m;
           String backupfilename = dir + File.separatorChar
                   + BackupFilenameFilter.getBackupFilename(n, basename,
-                          suffix, digits, extension);
+                          suffix, digits);
           File backupfile_n = new File(backupfilename);
 
           if (!backupfile_n.exists())
@@ -317,11 +293,12 @@ public class BackupFiles
           // if (m == 0 && backupfile_n.exists())
           if ((!noMax) && n == tempMax && backupfile_n.exists())
           {
-            // move the largest (max) rolled file to a temp file and add to the delete list
+            // move the largest (max) rolled file to a temp file and add to the
+            // delete list
             try
             {
-              File temp = File.createTempFile(backupfilename, ".tmp",
-                    dirFile);
+              File temp = File.createTempFile(backupfilename, TEMP_FILE_EXT,
+                      dirFile);
               backupfile_n.renameTo(temp);
 
               oldFilesToDelete = new File[] { temp };
@@ -353,7 +330,6 @@ public class BackupFiles
         // version style numbering (with earliest file deletion if max files
         // reached)
 
-
         bfTreeMap.values().toArray(backupFiles);
 
         // noMax == true means keep all backup files
@@ -372,6 +348,21 @@ public class BackupFiles
       }
     }
 
+    deleteOldFiles(oldFilesToDelete, confirmDeleteExtraInfo);
+
+    // Let's make the new backup file!! yay, got there at last!
+    String latestBackupFilename = dir + File.separatorChar
+            + BackupFilenameFilter.getBackupFilename(nextIndexNum, basename,
+                    suffix, digits);
+    File latestBackupFile = new File(latestBackupFilename);
+    ret = ret && file.renameTo(latestBackupFile);
+
+    return ret;
+  }
+
+  private void deleteOldFiles(File[] oldFilesToDelete,
+          String confirmDeleteExtraInfo)
+  {
     if (oldFilesToDelete != null && oldFilesToDelete.length > 0)
     {
       // delete old backup/version files
@@ -419,28 +410,40 @@ public class BackupFiles
       }
 
     }
+  }
 
-    // Let's make the new backup file!! yay, got there at last!
-    String latestBackupFilename = dir + File.separatorChar
-            + BackupFilenameFilter.getBackupFilename(nextIndexNum, basename,
-                    suffix, digits, extension);
-    File latestBackupFile = new File(latestBackupFilename);
-    ret = ret && file.renameTo(latestBackupFile);
-
-    return ret;
+  private TreeMap sortBackupFilesAsTreeMap(File[] backupFiles,
+          String basename)
+  {
+    // sort the backup files (based on integer found in the suffix) using a
+    // precomputed Hashmap for speed
+    Map<Integer, File> bfHashMap = new HashMap<>();
+    for (int i = 0; i < backupFiles.length; i++)
+    {
+      File f = backupFiles[i];
+      BackupFilenameParts bfp = new BackupFilenameParts(f, basename, suffix,
+              digits);
+      bfHashMap.put(bfp.indexNum(), f);
+    }
+    TreeMap<Integer, File> bfTreeMap = new TreeMap<>();
+    bfTreeMap.putAll(bfHashMap);
+    return bfTreeMap;
   }
 
   public boolean rollBackupsAndRenameTempFile()
   {
     boolean write = this.getWriteSuccess();
-    
+
     boolean roll = false;
-    if (write) {
+    if (write)
+    {
       roll = this.rollBackupFiles();
-    } else {
+    }
+    else
+    {
       return false;
     }
-    
+
     /*
      * Not sure that this confirmation is desirable.  By this stage the new file is
      * already written successfully, but something (e.g. disk full) has happened while 
@@ -452,7 +455,8 @@ public class BackupFiles
       int confirm = JvOptionPane.showConfirmDialog(Desktop.desktop,
               MessageManager.getString(
                       "label.backupfiles_confirm_save_file_backupfiles_roll_wrong"),
-              MessageManager.getString("label.backupfiles_confirm_save_file"),
+              MessageManager
+                      .getString("label.backupfiles_confirm_save_file"),
               JvOptionPane.YES_NO_OPTION, JvOptionPane.WARNING_MESSAGE);
 
       if (confirm == JvOptionPane.YES_OPTION)
@@ -470,9 +474,8 @@ public class BackupFiles
     return rename;
   }
 
-  public static TreeMap<Integer, File> lsBackupFilesAsTreeMap(
-          String fileName,
-          String suffix, int digits)
+  public static TreeMap<Integer, File> getBackupFilesAsTreeMap(
+          String fileName, String suffix, int digits)
   {
     File[] backupFiles = null;
 
@@ -493,28 +496,20 @@ public class BackupFiles
 
     String filename = file.getName();
     String basename = filename;
-    String extension = "";
-    int dotcharpos = filename.lastIndexOf('.');
-    // don't split of filenames with the last '.' at the very beginning or
-    // very end of the filename
-    if ((dotcharpos > 0) && (dotcharpos < filename.length() - 1))
-    {
-      basename = filename.substring(0, dotcharpos);
-      extension = filename.substring(dotcharpos); // NOTE this includes the '.'
-    }
-    
+
     // find existing backup files
-    BackupFilenameFilter bff = new BackupFilenameFilter(basename, suffix, digits, extension);
+    BackupFilenameFilter bff = new BackupFilenameFilter(basename, suffix,
+            digits);
     backupFiles = dirFile.listFiles(bff); // is clone needed?
-    
+
     // sort the backup files (based on integer found in the suffix) using a
     // precomputed Hashmap for speed
-    HashMap<Integer, File> bfHashMap = new HashMap<>();
+    Map<Integer, File> bfHashMap = new HashMap<>();
     for (int i = 0; i < backupFiles.length; i++)
     {
       File f = backupFiles[i];
       BackupFilenameParts bfp = new BackupFilenameParts(f, basename, suffix,
-              digits, extension);
+              digits);
       bfHashMap.put(bfp.indexNum(), f);
     }
     TreeMap<Integer, File> bfTreeMap = new TreeMap<>();
@@ -523,15 +518,4 @@ public class BackupFiles
     return bfTreeMap;
   }
 
-  public static File[] lsBackupFiles(String fileName, String suffix,
-          int digits)
-  {
-    TreeMap<Integer, File> bfTreeMap = lsBackupFilesAsTreeMap(fileName,
-            suffix, digits);
-    File[] backupFiles = new File[bfTreeMap.size()];
-    bfTreeMap.values().toArray(backupFiles);
-    return backupFiles;
-  }
-
 }
-