X-Git-Url: http://source.jalview.org/gitweb/?a=blobdiff_plain;f=src%2Fjalview%2Fdatamodel%2FSequenceGroup.java;h=cda2af1c70b8df7edc6e13a65b5f12f3a99de8ad;hb=838e4f91d4a53dd315640dbc9ff6ef7a815ee576;hp=752c6d4335dca76cc37d0b3e04449ca958d94fe9;hpb=2273eba5668e5340354da60fed329c6c716cc439;p=jalview.git diff --git a/src/jalview/datamodel/SequenceGroup.java b/src/jalview/datamodel/SequenceGroup.java index 752c6d4..cda2af1 100755 --- a/src/jalview/datamodel/SequenceGroup.java +++ b/src/jalview/datamodel/SequenceGroup.java @@ -1,6 +1,6 @@ /* - * Jalview - A Sequence Alignment Editor and Viewer (Version 2.8.2) - * Copyright (C) 2014 The Jalview Authors + * Jalview - A Sequence Alignment Editor and Viewer (Version 2.9.0b1) + * Copyright (C) 2015 The Jalview Authors * * This file is part of Jalview. * @@ -27,7 +27,6 @@ import jalview.schemes.ResidueProperties; import java.awt.Color; import java.util.ArrayList; -import java.util.Enumeration; import java.util.Hashtable; import java.util.List; import java.util.Map; @@ -63,7 +62,7 @@ public class SequenceGroup implements AnnotatedCollectionI /** * group members */ - private Vector sequences = new Vector(); + private List sequences = new ArrayList(); /** * representative sequence for this group (if any) @@ -77,8 +76,10 @@ public class SequenceGroup implements AnnotatedCollectionI */ public ColourSchemeI cs; + // start column (base 0) int startRes = 0; + // end column (base 0) int endRes = 0; public Color outlineColour = Color.black; @@ -136,7 +137,7 @@ public class SequenceGroup implements AnnotatedCollectionI * @param end * last column of group */ - public SequenceGroup(Vector sequences, String groupName, + public SequenceGroup(List sequences, String groupName, ColourSchemeI scheme, boolean displayBoxes, boolean displayText, boolean colourText, int start, int end) { @@ -160,13 +161,8 @@ public class SequenceGroup implements AnnotatedCollectionI { if (seqsel != null) { - sequences = new Vector(); - Enumeration sq = seqsel.sequences.elements(); - while (sq.hasMoreElements()) - { - sequences.addElement(sq.nextElement()); - } - ; + sequences = new ArrayList(); + sequences.addAll(seqsel.sequences); if (seqsel.groupName != null) { groupName = new String(seqsel.groupName); @@ -310,16 +306,15 @@ public class SequenceGroup implements AnnotatedCollectionI { if (hiddenReps == null) { + // TODO: need a synchronizedCollection here ? return sequences; } else { - Vector allSequences = new Vector(); - SequenceI seq; - for (int i = 0; i < sequences.size(); i++) + List allSequences = new ArrayList(); + for (SequenceI seq : sequences) { - seq = sequences.elementAt(i); - allSequences.addElement(seq); + allSequences.add(seq); if (hiddenReps.containsKey(seq)) { SequenceCollectionI hsg = hiddenReps.get(seq); @@ -327,7 +322,7 @@ public class SequenceGroup implements AnnotatedCollectionI { if (seq2 != seq && !allSequences.contains(seq2)) { - allSequences.addElement(seq2); + allSequences.add(seq2); } } } @@ -471,14 +466,17 @@ public class SequenceGroup implements AnnotatedCollectionI */ public void addSequence(SequenceI s, boolean recalc) { - if (s != null && !sequences.contains(s)) + synchronized (sequences) { - sequences.addElement(s); - } + if (s != null && !sequences.contains(s)) + { + sequences.add(s); + } - if (recalc) - { - recalcConservation(); + if (recalc) + { + recalcConservation(); + } } } @@ -613,31 +611,37 @@ public class SequenceGroup implements AnnotatedCollectionI */ public void addOrRemove(SequenceI s, boolean recalc) { - if (sequences.contains(s)) - { - deleteSequence(s, recalc); - } - else + synchronized (sequences) { - addSequence(s, recalc); + if (sequences.contains(s)) + { + deleteSequence(s, recalc); + } + else + { + addSequence(s, recalc); + } } } /** - * DOCUMENT ME! + * remove * * @param s - * DOCUMENT ME! + * to be removed * @param recalc - * DOCUMENT ME! + * true means recalculate conservation */ public void deleteSequence(SequenceI s, boolean recalc) { - sequences.removeElement(s); - - if (recalc) + synchronized (sequences) { - recalcConservation(); + sequences.remove(s); + + if (recalc) + { + recalcConservation(); + } } } @@ -683,9 +687,7 @@ public class SequenceGroup implements AnnotatedCollectionI } /** - * DOCUMENT ME! - * - * @return DOCUMENT ME! + * @return number of sequences in group */ public int getSize() { @@ -693,23 +695,17 @@ public class SequenceGroup implements AnnotatedCollectionI } /** - * DOCUMENT ME! - * * @param i - * DOCUMENT ME! - * - * @return DOCUMENT ME! + * @return the ith sequence */ public SequenceI getSequenceAt(int i) { - return sequences.elementAt(i); + return sequences.get(i); } /** - * DOCUMENT ME! - * * @param state - * DOCUMENT ME! + * colourText */ public void setColourText(boolean state) { @@ -769,30 +765,27 @@ public class SequenceGroup implements AnnotatedCollectionI } /** - * DOCUMENT ME! + * computes the width of current set of sequences and returns it * * @return DOCUMENT ME! */ @Override public int getWidth() { - // MC This needs to get reset when characters are inserted and deleted - if (sequences.size() > 0) - { - width = sequences.elementAt(0).getLength(); - } - - for (int i = 1; i < sequences.size(); i++) + synchronized (sequences) { - SequenceI seq = sequences.elementAt(i); - - if (seq.getLength() > width) + // MC This needs to get reset when characters are inserted and deleted + boolean first = true; + for (SequenceI seq : sequences) { - width = seq.getLength(); + if (first || seq.getLength() > width) + { + width = seq.getLength(); + first = false; + } } + return width; } - - return width; } /** @@ -843,39 +836,42 @@ public class SequenceGroup implements AnnotatedCollectionI */ public SequenceI[] getSequencesInOrder(AlignmentI al, boolean trim) { - int sSize = sequences.size(); - int alHeight = al.getHeight(); + synchronized (sequences) + { + int sSize = sequences.size(); + int alHeight = al.getHeight(); - SequenceI[] seqs = new SequenceI[(trim) ? sSize : alHeight]; + SequenceI[] seqs = new SequenceI[(trim) ? sSize : alHeight]; - int index = 0; - for (int i = 0; i < alHeight && index < sSize; i++) - { - if (sequences.contains(al.getSequenceAt(i))) + int index = 0; + for (int i = 0; i < alHeight && index < sSize; i++) { - seqs[(trim) ? index : i] = al.getSequenceAt(i); - index++; + if (sequences.contains(al.getSequenceAt(i))) + { + seqs[(trim) ? index : i] = al.getSequenceAt(i); + index++; + } } - } - if (index == 0) - { - return null; - } - if (!trim) - { - return seqs; - } - if (index < seqs.length) - { - SequenceI[] dummy = seqs; - seqs = new SequenceI[index]; - while (--index >= 0) + if (index == 0) + { + return null; + } + if (!trim) { - seqs[index] = dummy[index]; - dummy[index] = null; + return seqs; } + if (index < seqs.length) + { + SequenceI[] dummy = seqs; + seqs = new SequenceI[index]; + while (--index >= 0) + { + seqs[index] = dummy[index]; + dummy[index] = null; + } + } + return seqs; } - return seqs; } /** @@ -989,23 +985,14 @@ public class SequenceGroup implements AnnotatedCollectionI { SequenceGroup sgroup = new SequenceGroup(this); SequenceI[] insect = getSequencesInOrder(alignment); - sgroup.sequences = new Vector(); + sgroup.sequences = new ArrayList(); for (int s = 0; insect != null && s < insect.length; s++) { if (map == null || map.containsKey(insect[s])) { - sgroup.sequences.addElement(insect[s]); + sgroup.sequences.add(insect[s]); } } - // Enumeration en =getSequences(hashtable).elements(); - // while (en.hasMoreElements()) - // { - // SequenceI elem = (SequenceI) en.nextElement(); - // if (alignment.getSequences().contains(elem)) - // { - // sgroup.addSequence(elem, false); - // } - // } return sgroup; } @@ -1225,27 +1212,30 @@ public class SequenceGroup implements AnnotatedCollectionI // TODO add in other methods like 'getAlignmentAnnotation(String label), // etc' ArrayList annot = new ArrayList(); - for (SequenceI seq : sequences) + synchronized (sequences) { - AlignmentAnnotation[] aa = seq.getAnnotation(); - if (aa != null) + for (SequenceI seq : sequences) { - for (AlignmentAnnotation al : aa) + AlignmentAnnotation[] aa = seq.getAnnotation(); + if (aa != null) { - if (al.groupRef == this) + for (AlignmentAnnotation al : aa) { - annot.add(al); + if (al.groupRef == this) + { + annot.add(al); + } } } } - } - if (consensus != null) - { - annot.add(consensus); - } - if (conservation != null) - { - annot.add(conservation); + if (consensus != null) + { + annot.add(consensus); + } + if (conservation != null) + { + annot.add(conservation); + } } return annot.toArray(new AlignmentAnnotation[0]); } @@ -1308,9 +1298,15 @@ public class SequenceGroup implements AnnotatedCollectionI return false; } + /** + * Remove all sequences from the group (leaving other properties unchanged). + */ public void clear() { - sequences.clear(); + synchronized (sequences) + { + sequences.clear(); + } } private AnnotatedCollectionI context;