X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fanalysis%2FAnnotationSorter.java;h=1f2e78f9ad01b30c7d5e8057aaafba3d093cc890;hb=80b889f0cca49103e1b20ed806755a0719789906;hp=81398ebcd8683cdf5bbc54c5d3ae28ecef58a438;hpb=2273eba5668e5340354da60fed329c6c716cc439;p=jalview.git

diff --git a/src/jalview/analysis/AnnotationSorter.java b/src/jalview/analysis/AnnotationSorter.java
index 81398eb..1f2e78f 100644
--- a/src/jalview/analysis/AnnotationSorter.java
+++ b/src/jalview/analysis/AnnotationSorter.java
@@ -1,5 +1,27 @@
+/*
+ * Jalview - A Sequence Alignment Editor and Viewer ($$Version-Rel$$)
+ * Copyright (C) $$Year-Rel$$ The Jalview Authors
+ * 
+ * This file is part of Jalview.
+ * 
+ * Jalview is free software: you can redistribute it and/or
+ * modify it under the terms of the GNU General Public License 
+ * as published by the Free Software Foundation, either version 3
+ * of the License, or (at your option) any later version.
+ *  
+ * Jalview is distributed in the hope that it will be useful, but 
+ * WITHOUT ANY WARRANTY; without even the implied warranty 
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR 
+ * PURPOSE.  See the GNU General Public License for more details.
+ * 
+ * You should have received a copy of the GNU General Public License
+ * along with Jalview.  If not, see <http://www.gnu.org/licenses/>.
+ * The Jalview Authors are detailed in the 'AUTHORS' file.
+ */
 package jalview.analysis;
 
+import java.util.Locale;
+
 import jalview.datamodel.AlignmentAnnotation;
 import jalview.datamodel.AlignmentI;
 import jalview.datamodel.SequenceI;
@@ -30,8 +52,8 @@ public class AnnotationSorter
   public enum SequenceAnnotationOrder
   {
     // Text descriptions surface in the Preferences Sort by... options
-    SEQUENCE_AND_LABEL("Sequence"), LABEL_AND_SEQUENCE("Label"), NONE(
-            "No sort");
+    SEQUENCE_AND_LABEL("Sequence"), LABEL_AND_SEQUENCE("Label"),
+    NONE("No sort");
 
     private String description;
 
@@ -46,7 +68,8 @@ public class AnnotationSorter
       return description;
     }
 
-    public static SequenceAnnotationOrder forDescription(String d) {
+    public static SequenceAnnotationOrder forDescription(String d)
+    {
       for (SequenceAnnotationOrder order : values())
       {
         if (order.toString().equals(d))
@@ -87,7 +110,8 @@ public class AnnotationSorter
    * <ul>
    * <li>annotations with a reference to a sequence in the alignment are sorted
    * on sequence ordering</li>
-   * <li>other annotations go 'at the end', with their mutual order unchanged</li>
+   * <li>other annotations go 'at the end', with their mutual order
+   * unchanged</li>
    * <li>within the same sequence ref, sort by label (non-case-sensitive)</li>
    * </ul>
    */
@@ -109,16 +133,45 @@ public class AnnotationSorter
         return 1;
       }
 
+      // TODO how to treat sequence-related autocalculated annotation
+      boolean o1auto = o1.autoCalculated && o1.sequenceRef == null;
+      boolean o2auto = o2.autoCalculated && o2.sequenceRef == null;
       /*
        * Ignore label (keep existing ordering) for
        * Conservation/Quality/Consensus etc
        */
-      if (o1.sequenceRef == null && o2.sequenceRef == null)
+      if (o1auto && o2auto)
       {
         return 0;
       }
-      int sequenceOrder = compareSequences(o1, o2);
-      return sequenceOrder == 0 ? compareLabels(o1, o2) : sequenceOrder;
+
+      /*
+       * Sort autocalculated before or after sequence-related.
+       */
+      if (o1auto)
+      {
+        return showAutocalcAbove ? -1 : 1;
+      }
+      if (o2auto)
+      {
+        return showAutocalcAbove ? 1 : -1;
+      }
+      int computedOrder = compareSequences(o1, o2);
+      if (computedOrder == 0)
+      {
+        computedOrder = compareLabels(o1, o2);
+      }
+      if (computedOrder == 0)
+      {
+        computedOrder = compareDescriptions(o1, o2);
+      }
+      return computedOrder;
+    }
+
+    @Override
+    public String toString()
+    {
+      return "Sort by sequence and label";
     }
   };
 
@@ -127,7 +180,8 @@ public class AnnotationSorter
    * <ul>
    * <li>annotations with a reference to a sequence in the alignment are sorted
    * on label (non-case-sensitive)</li>
-   * <li>other annotations go 'at the end', with their mutual order unchanged</li>
+   * <li>other annotations go 'at the end', with their mutual order
+   * unchanged</li>
    * <li>within the same label, sort by order of the related sequences</li>
    * </ul>
    */
@@ -149,55 +203,75 @@ public class AnnotationSorter
         return 1;
       }
 
+      // TODO how to treat sequence-related autocalculated annotation
+      boolean o1auto = o1.autoCalculated && o1.sequenceRef == null;
+      boolean o2auto = o2.autoCalculated && o2.sequenceRef == null;
       /*
        * Ignore label (keep existing ordering) for
        * Conservation/Quality/Consensus etc
        */
-      if (o1.sequenceRef == null && o2.sequenceRef == null)
+      if (o1auto && o2auto)
       {
         return 0;
       }
 
       /*
-       * Sort non-sequence-related before or after sequence-related.
+       * Sort autocalculated before or after sequence-related.
        */
-      if (o1.sequenceRef == null)
+      if (o1auto)
       {
         return showAutocalcAbove ? -1 : 1;
       }
-      if (o2.sequenceRef == null)
+      if (o2auto)
       {
         return showAutocalcAbove ? 1 : -1;
       }
       int labelOrder = compareLabels(o1, o2);
       return labelOrder == 0 ? compareSequences(o1, o2) : labelOrder;
     }
+
+    @Override
+    public String toString()
+    {
+      return "Sort by label and sequence";
+    }
   };
 
   /**
-   * noSort leaves sort order unchanged, within sequence- and
-   * non-sequence-related annotations, but may switch the ordering of these
-   * groups. Note this is guaranteed (at least in Java 7) as Arrays.sort() is
-   * guaranteed to be 'stable' (not change ordering of equal items).
+   * noSort leaves sort order unchanged, within sequence- and autocalculated
+   * annotations, but may switch the ordering of these groups. Note this is
+   * guaranteed (at least in Java 7) as Arrays.sort() is guaranteed to be
+   * 'stable' (not change ordering of equal items).
    */
   private Comparator<? super AlignmentAnnotation> noSort = new Comparator<AlignmentAnnotation>()
   {
     @Override
     public int compare(AlignmentAnnotation o1, AlignmentAnnotation o2)
     {
+      // TODO how to treat sequence-related autocalculated annotation
+      boolean o1auto = o1.autoCalculated && o1.sequenceRef == null;
+      boolean o2auto = o2.autoCalculated && o2.sequenceRef == null;
+      // TODO skip this test to allow customised ordering of all annotations
+      // - needs a third option: place autocalculated first / last / none
       if (o1 != null && o2 != null)
       {
-        if (o1.sequenceRef == null && o2.sequenceRef != null)
+        if (o1auto && !o2auto)
         {
           return showAutocalcAbove ? -1 : 1;
         }
-        if (o1.sequenceRef != null && o2.sequenceRef == null)
+        if (!o1auto && o2auto)
         {
           return showAutocalcAbove ? 1 : -1;
         }
       }
       return 0;
     }
+
+    @Override
+    public String toString()
+    {
+      return "No sort";
+    }
   };
 
   /**
@@ -216,7 +290,8 @@ public class AnnotationSorter
     // cache 'alignment sequence position' for the annotations
     saveSequenceIndices(alignmentAnnotations);
 
-    Comparator<? super AlignmentAnnotation> comparator = getComparator(order);
+    Comparator<? super AlignmentAnnotation> comparator = getComparator(
+            order);
 
     if (alignmentAnnotations != null)
     {
@@ -238,9 +313,11 @@ public class AnnotationSorter
           AlignmentAnnotation[] alignmentAnnotations)
   {
     sequenceIndices.clear();
-    for (AlignmentAnnotation ann : alignmentAnnotations) {
+    for (AlignmentAnnotation ann : alignmentAnnotations)
+    {
       SequenceI seq = ann.sequenceRef;
-      if (seq != null) {
+      if (seq != null)
+      {
         int index = AlignmentUtils.getSequenceIndex(alignment, seq);
         sequenceIndices.put(seq, index);
       }
@@ -289,6 +366,31 @@ public class AnnotationSorter
     }
     String label1 = o1.label;
     String label2 = o2.label;
+    return compareString(label1, label2);
+  }
+
+  /**
+   * Non-case-sensitive comparison of annotation descriptions. Returns zero if
+   * either argument is null.
+   * 
+   * @param o1
+   * @param o2
+   * @return
+   */
+  private int compareDescriptions(AlignmentAnnotation o1,
+          AlignmentAnnotation o2)
+  {
+    if (o1 == null || o2 == null)
+    {
+      return 0;
+    }
+    String label1 = o1.description;
+    String label2 = o2.description;
+    return compareString(label1, label2);
+  }
+
+  private int compareString(String label1, String label2)
+  {
     if (label1 == null && label2 == null)
     {
       return 0;
@@ -301,7 +403,8 @@ public class AnnotationSorter
     {
       return 1;
     }
-    return label1.toUpperCase().compareTo(label2.toUpperCase());
+    return label1.toUpperCase(Locale.ROOT)
+            .compareTo(label2.toUpperCase(Locale.ROOT));
   }
 
   /**