這是檢查用戶輸入的好習慣嗎？

Question

do { 
    std::cout << "Enter a valid name" << std::endl; 
    std::cin >> name; 

} while (!validName(name) && std::cout << "It is not a valid name" << std::endl); 

這是一個很好的做法，如果名稱無效，那麼我用這種方式回答一條線？

Answer 1

在循環本身中執行檢查和輸出可能更清晰。你可以通過打破無限循環或繼續一次循環來做到這一點。例如：

do { 
    std::cout << "Enter a valid name" << std::endl; 
    std::cin >> name; 
    if (!validName(name)) { 
     std::cout << "It is not a valid name" << std::endl; 
     continue; 
    } 
} while (false); 

或者相反：

for (;;) { 
    std::cout << "Enter a valid name" << std::endl; 
    std::cin >> name; 
    if (validName(name)) 
     break; 

    std::cout << "It is not a valid name" << std::endl; 
} 

我一般喜歡第二個，因爲它可以很容易地注入反，所以你可以做處理，如果環路一些其它類型的錯誤身體失敗超過一定次數。

for (int attempt = 0; attempt < 3; ++attempt) { 
    std::cout << "Enter a valid name" << std::endl; 
    std::cin >> name; 
    if (validName(name)) 
     break; 

    std::cout << "It is not a valid name" << std::endl; 
} 

if (!validName(name)) { 
    std::cout << "Failed to get a valid name after 3 tries" << std::endl; 
    std::abort(); 
}