Быстрые вопросы. У меня проблемы с моей петлей «Пока»

Question

У меня проблемы с раздражением. Проблема, с которой я сталкиваюсь; когда я вхожу в -1 в первый cin >> statement, время не выходит, как я хочу.

Я стараюсь не использовать "break" в цикле, потому что мне сообщили о его дурной привычке.

FYI moveOn инициализируется значением true. Вот код:

while(moveOn){ 

cout << "Enter info:"; 
cin >> weight; 
if(weight == -1){ 
    moveOn = false; 
} 

weight = pack.weight; 
cin >> pack.sides[0] >> pack.sides[1] >> pack.sides[2]; 

    while(pack.weight > 0 || pack.sides[0] < 0 || pack.sides[1] < 0 || pack.sides[2] < 0){ 
     cout << "Enter Vaild info:"; 
     cin >> pack.weight >> pack.sides[0] >> pack.sides[1] >> pack.sides[2]; 
    } 

girth = getGirth(pack.sides); 
float price; 
price = getPackageAmount(pack.weight, weights, prices); 
cout << price << endl; 

if(pack.weight <= 50){ 
    if(pack.sides[0] <= 36 && pack.sides[1] <= 36 && pack.sides[2] <= 36){ 
     if(girth <= 60){ 
      pack.accepted = true; 
      accepted++; 
     } 
    } 
} 
else{ 
    pack.accepted = false; 
    rejected++; 

} 
} 

Спасибо заранее.

Answer 1

while(moveOn) не означает «немедленно остановиться, когда moveOn будет false» - тест выполняется только в начале каждой итерации цикла.

Вы должны добавить else пункт:

while (moveOn) 
{ 
    cout << "Enter info:"; 
    cin >> weight; 
    if(weight == -1){ 
     moveOn = false; 
    } 
    else 
    { 
     weight = pack.weight; 
     // ... 
    } 
} 

(Вы, вероятно, также необходимо изменить weight = pack.weight к pack.weight = weight.)