Проблема, использующая for loop и goto statement

Question

Я использую C++ 11 std::list и список итераторов. Я использую вложенные циклы и проверяю условие, и если это удастся, я использую оператор goto, чтобы выйти. Псевдокод показано ниже:

std::list<myclass>::iterator tmpItr; 
std::list<myclass>::iterator tmpItr2; 
std::list<std::list<myclass>::iterator>::iterator radItr; 
double cutoffVal = someVal; 
double currVal = 0.0; 

for (radItr=radList.begin();radItr!=radList.end();radItr++) { 
    tmpItr2 = *radItr; 
    for (tmpItr=pList.begin();tmpItr!=pList.end();tmpItr++) { 
     if (tmpItr == tmpItr2) 
      continue; 

     /* Some Operation Here */ 
     currVal += NewVal; 
     if (currVal >= cutoffVal) 
      goto BREAKLOOP; 
    } 
} 

BREAKLOOP: 
// Use tmpItr and tmpItr2 

Моя проблема заключается в том, что когда-то значение tmpItr идет к list.end() и я получаю ошибку сегментации. Описывает ли заявление goto сообщение о приращении здесь?

Answer 1

Я не уверен, что это исправит проблему с вашими итераторами, но большинство людей, с которыми я говорю, считают, что goto's в C++ - это плохой код. Почему бы не заменить Гото с:

(bool)breakout=true; 
    break; 
    } 
if(breakout)break; 

Answer 2

Я думаю, ваша проблема возникает, когда goto является не достиг:

for (...; radItr != radList.end(); ...) { 
    for (...; tmpItr != pList.end(); ...) { 
     if (...) 
      goto BREAKLOOP; 
    } 
} 

// At this point, radItr==radList.end() && tmpItr==pList.end() 

BREAKLOOP: