생성자 오류, 충돌하기 전에 '는'다음 삼각형을 출력

Question

QuBiEngine::QuBiEngine(ifstream& dnaFile) 
{ 
    int i = 0; 
    while(!dnaFile.eof()) //while the file isn't at its end 
    { 
     dna.push_back(""); //creates an element 
     if(!dnaFile.good())//checks for failbits and other errors 
     { 
      dna[i] = "Not a valid sequence"; 
      i++; 
      continue; 
     } 
     getline(dnaFile, dna[i]); 
     //checks to see if the character is valid ie: a, t, c, g 
     for(int j=0; j<dna[i].length(); j++) 
     { 
      dna[i][j] = putchar(tolower(dna[i][j])); 
      if((dna[i][j]!='a')||(dna[i][j]!='t')||(dna[i][j]!='c')||(dna[i][j]!='g')) 
      { 
       dna[i] = "Not a valid sequence"; 
       i++; 
       break; 
      }    
     } 
     i++;   
    } 
} 

이것은 dnaFile ifstream의 각 라인을 소요하고 테스트를 통과하면 그렇지 않은 경우, 벡터에 넣습니다 그러면 그것은 단지 유효하지 않은 것을 벡터에 넣을뿐입니다.

Answer 1

나는 두 번째의 i++ 문이 두 배 증가 때문에 오버플로 만드는 경우를 파악하지 내 벡터.

Answer 2

당신이 표시된 곳에서 break;를 누락 될 수 있음을 보인다

보다

QuBiEngine::QuBiEngine(ifstream& dnaFile) 
{ 
    int i = 0; 
    while(!dnaFile.eof()) //while the file isn't at its end 
    { 
     dna.push_back(""); //creates an element 
     if(!dnaFile.good())//checks for failbits and other errors 
     { 
      dna[i] = "Not a valid sequence"; 
      i++; 
      continue; 
     } 
     getline(dnaFile, dna[i]); 
     bool bad = false; 
     //checks to see if the character is valid ie: a, t, c, g 
     for(int j=0; j<dna[i].length(); j++) 
     { 
      dna[i][j] = putchar(tolower(dna[i][j])); 
      if((dna[i][j]!='a')||(dna[i][j]!='t')||(dna[i][j]!='c')||(dna[i][j]!='g')) 
      { 
       dna[i] = "Not a valid sequence"; 
       break; 
      }    
     } 
     i++; 
    } 
} 

기타 : dna 변수는 무엇인가? 선언과 초기화를 보여줄 수 있습니까? 당신의 dna가 선언 된 경우 그 외에는

, 당신은 아웃 오브 바운드하지 않도록 (어떤 검사)