formatting
[jalview.git] / src / jalview / analysis / CrossRef.java
index 1c14240..6c63f1c 100644 (file)
@@ -1,23 +1,24 @@
 /*\r
- * Jalview - A Sequence Alignment Editor and Viewer (Version 2.6)\r
- * Copyright (C) 2010 J Procter, AM Waterhouse, G Barton, M Clamp, S Searle\r
- * \r
+ * Jalview - A Sequence Alignment Editor and Viewer (Version 2.7)\r
+ * Copyright (C) 2011 J Procter, AM Waterhouse, J Engelhardt, LM Lui, G Barton, M Clamp, S Searle\r
+ *\r
  * This file is part of Jalview.\r
- * \r
+ *\r
  * Jalview is free software: you can redistribute it and/or\r
- * modify it under the terms of the GNU General Public License \r
+ * modify it under the terms of the GNU General Public License\r
  * as published by the Free Software Foundation, either version 3 of the License, or (at your option) any later version.\r
- * \r
- * Jalview is distributed in the hope that it will be useful, but \r
- * WITHOUT ANY WARRANTY; without even the implied warranty \r
- * of MERCHANTABILITY or FITNESS FOR A PARTICULAR \r
+ *\r
+ * Jalview is distributed in the hope that it will be useful, but\r
+ * WITHOUT ANY WARRANTY; without even the implied warranty\r
+ * of MERCHANTABILITY or FITNESS FOR A PARTICULAR\r
  * PURPOSE.  See the GNU General Public License for more details.\r
- * \r
+ *\r
  * You should have received a copy of the GNU General Public License along with Jalview.  If not, see <http://www.gnu.org/licenses/>.\r
  */\r
 package jalview.analysis;\r
 \r
 import java.util.Enumeration;\r
+import java.util.List;\r
 import java.util.Vector;\r
 import java.util.Hashtable;\r
 \r
@@ -66,12 +67,12 @@ public class CrossRef
   public static Hashtable classifyDbRefs(DBRefEntry[] rfs)\r
   {\r
     Hashtable classes = new Hashtable();\r
-    classes.put(DBRefSource.PROTEINDBS, jalview.util.DBRefUtils.selectRefs(\r
-            rfs, DBRefSource.PROTEINDBS));\r
+    classes.put(DBRefSource.PROTEINDBS,\r
+            jalview.util.DBRefUtils.selectRefs(rfs, DBRefSource.PROTEINDBS));\r
     classes.put(DBRefSource.DNACODINGDBS, jalview.util.DBRefUtils\r
             .selectRefs(rfs, DBRefSource.DNACODINGDBS));\r
-    classes.put(DBRefSource.DOMAINDBS, jalview.util.DBRefUtils.selectRefs(\r
-            rfs, DBRefSource.DOMAINDBS));\r
+    classes.put(DBRefSource.DOMAINDBS,\r
+            jalview.util.DBRefUtils.selectRefs(rfs, DBRefSource.DOMAINDBS));\r
     // classes.put(OTHER, )\r
     return classes;\r
   }\r
@@ -486,79 +487,82 @@ public class CrossRef
       System.err.println("Empty dataset sequence set - NO VECTOR");\r
       return false;\r
     }\r
-    Enumeration e = dataset.getSequences().elements();\r
-    while (e.hasMoreElements())\r
+    List<SequenceI> ds;\r
+    synchronized (ds = dataset.getSequences())\r
     {\r
-      SequenceI nxt = (SequenceI) e.nextElement();\r
-      if (nxt != null)\r
-      {\r
-        if (nxt.getDatasetSequence() != null)\r
-        {\r
-          System.err\r
-                  .println("Implementation warning: getProducts passed a dataset alignment without dataset sequences in it!");\r
-        }\r
-        if (nxt != sequenceI && nxt != sequenceI.getDatasetSequence())\r
+      for (SequenceI nxt : ds)\r
+        if (nxt != null)\r
         {\r
-          // check if this is the correct sequence type\r
+          if (nxt.getDatasetSequence() != null)\r
           {\r
-            typer[0] = nxt;\r
-            boolean isDna = jalview.util.Comparison.isNucleotide(typer);\r
-            if ((direct && isDna == dna) || (!direct && isDna != dna))\r
+            System.err\r
+                    .println("Implementation warning: getProducts passed a dataset alignment without dataset sequences in it!");\r
+          }\r
+          if (nxt != sequenceI && nxt != sequenceI.getDatasetSequence())\r
+          {\r
+            // check if this is the correct sequence type\r
             {\r
-              // skip this sequence because it is same molecule type\r
-              continue;\r
+              typer[0] = nxt;\r
+              boolean isDna = jalview.util.Comparison.isNucleotide(typer);\r
+              if ((direct && isDna == dna) || (!direct && isDna != dna))\r
+              {\r
+                // skip this sequence because it is same molecule type\r
+                continue;\r
+              }\r
             }\r
-          }\r
 \r
-          // look for direct or indirect references in common\r
-          DBRefEntry[] poss = nxt.getDBRef(), cands = null;\r
-          if (direct)\r
-          {\r
-            cands = jalview.util.DBRefUtils.searchRefs(poss, xrf);\r
-          }\r
-          else\r
-          {\r
-            poss = CrossRef.findXDbRefs(dna, poss); // \r
-            cands = jalview.util.DBRefUtils.searchRefs(poss, xrf);\r
-          }\r
-          if (cands != null)\r
-          {\r
-            if (!rseqs.contains(nxt))\r
+            // look for direct or indirect references in common\r
+            DBRefEntry[] poss = nxt.getDBRef(), cands = null;\r
+            if (direct)\r
+            {\r
+              cands = jalview.util.DBRefUtils.searchRefs(poss, xrf);\r
+            }\r
+            else\r
             {\r
-              rseqs.addElement(nxt);\r
-              boolean foundmap = cf != null; // don't search if we aren't given\r
-              // a codon map object\r
-              for (int r = 0; foundmap && r < cands.length; r++)\r
+              poss = CrossRef.findXDbRefs(dna, poss); //\r
+              cands = jalview.util.DBRefUtils.searchRefs(poss, xrf);\r
+            }\r
+            if (cands != null)\r
+            {\r
+              if (!rseqs.contains(nxt))\r
               {\r
-                if (cands[r].hasMap())\r
+                rseqs.addElement(nxt);\r
+                boolean foundmap = cf != null; // don't search if we aren't\r
+                                               // given\r
+                // a codon map object\r
+                for (int r = 0; foundmap && r < cands.length; r++)\r
                 {\r
-                  if (cands[r].getMap().getTo() != null\r
-                          && cands[r].getMap().getMap().getFromRatio() != cands[r]\r
-                                  .getMap().getMap().getToRatio())\r
+                  if (cands[r].hasMap())\r
                   {\r
-                    foundmap = true;\r
-                    // get sense of map correct for adding to product alignment.\r
-                    if (dna)\r
-                    {\r
-                      // map is from dna seq to a protein product\r
-                      cf.addMap(sequenceI, nxt, cands[r].getMap().getMap());\r
-                    }\r
-                    else\r
+                    if (cands[r].getMap().getTo() != null\r
+                            && cands[r].getMap().getMap().getFromRatio() != cands[r]\r
+                                    .getMap().getMap().getToRatio())\r
                     {\r
-                      // map should be from protein seq to its coding dna\r
-                      cf.addMap(nxt, sequenceI, cands[r].getMap().getMap()\r
-                              .getInverse());\r
+                      foundmap = true;\r
+                      // get sense of map correct for adding to product\r
+                      // alignment.\r
+                      if (dna)\r
+                      {\r
+                        // map is from dna seq to a protein product\r
+                        cf.addMap(sequenceI, nxt, cands[r].getMap()\r
+                                .getMap());\r
+                      }\r
+                      else\r
+                      {\r
+                        // map should be from protein seq to its coding dna\r
+                        cf.addMap(nxt, sequenceI, cands[r].getMap()\r
+                                .getMap().getInverse());\r
+                      }\r
                     }\r
                   }\r
                 }\r
+                // TODO: add mapping between sequences if necessary\r
+                found = true;\r
               }\r
-              // TODO: add mapping between sequences if necessary\r
-              found = true;\r
             }\r
-          }\r
 \r
+          }\r
         }\r
-      }\r
     }\r
     return found;\r
   }\r