Конструктор Error, печатает «а», а затем треугольник, прежде чем врезаться

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 объявляется так, что вы не выйти за пределы (без проверки)