This contains Pat's bug fixes and updates. It represents mothur v.1.3.0

[mothur.git] / onegapignore.h

diff --git a/onegapignore.h b/onegapignore.h
index ee6cc5267931816a654c99f4e748d19bb2940901..6421741c0d4c153794ab40efc9318af6d64349a4 100644 (file)
--- a/onegapignore.h
+++ b/onegapignore.h
@@ -25,27 +25,27 @@ public:
                int start = 0;
                int end = 0;
                
-               for(int i=0;i<A.getLength();i++){
-                       if(A.getAligned()[i] == '.' || B.getAligned()[i] == '.' || A.getAligned()[i] == '-' || B.getAligned()[i] == '-'){
-                       }
-                       else{
+               string seqA = A.getAligned();
+               string seqB = B.getAligned();
+               int alignLength = seqA.length();
+               
+               // this assumes that sequences start and end with '.'s instead of'-'s.
+               for(int i=0;i<alignLength;i++){
+                       if(seqA[i] != '.' && seqB[i] != '.'){
                                start = i;
                                break;
                        }
                }
-               for(int i=A.getLength()-1;i>=0;i--){
-                       if(A.getAligned()[i] == '.' || B.getAligned()[i] == '.' || A.getAligned()[i] == '-' || B.getAligned()[i] == '-'){
-                       }
-                       else{
+               for(int i=alignLength-1;i>=0;i--){
+                       if(seqA[i] != '.' && seqB[i] != '.'){
                                end = i;
                                break;
                        }
                }
                
-               
                for(int i=start;i<=end;i++){
-                       if(A.getAligned()[i] == '-' && B.getAligned()[i] == '-'){}
-                       else if(A.getAligned()[i] == '-' && B.getAligned()[i] != '-'){
+                       if(seqA[i] == '-' && seqB[i] == '-'){   ;       }
+                       else if(seqB[i] != '-' && seqA[i] == '-'){
                                if(openGapA == 0){
                                        difference++;
                                        minLength++;
@@ -53,7 +53,7 @@ public:
                                        openGapB = 0;
                                }
                        }
-                       else if(A.getAligned()[i] != '-' && B.getAligned()[i] == '-'){
+                       else if(seqA[i] != '-' && seqB[i] == '-'){
                                if(openGapB == 0){
                                        difference++;
                                        minLength++;
@@ -61,20 +61,16 @@ public:
                                        openGapB = 1;
                                }
                        }
-                       else if(A.getAligned()[i] != '-' && B.getAligned()[i] != '-'){
-                               if(A.getAligned()[i] != B.getAligned()[i]){
+                       else if(seqA[i] != '-' && seqB[i] != '-'){
+                               if(seqA[i] != seqB[i]){
                                        difference++;
-                                       minLength++;
-                                       openGapA = 0;
-                                       openGapB = 0;
-                               }
-                               else{
-                                       minLength++;
-                                       openGapA = 0;
-                                       openGapB = 0;
                                }
+                               minLength++;
+                               openGapA = 0;
+                               openGapB = 0;
                        }
                }
+
                if(minLength == 0)      {       dist = 1.0000;                                                  }
                else                            {       dist = (double)difference / minLength;  }
        }