std :: logic_error вместо return false

Question

Я искал мнение об использовании std::logic_error вместо использования сложного списка вложенных if/elseif return true/false.

Я хотел бы перейти от многих аналогичных функций, как один ниже

bool validate_data(){ 
    std::vector<int> v; 
    //fill with data 
    if(v.find(10) == v.end()){ 
     return false; 
    } 
    // other checks that return false 
} 

в

bool validate_data(){ 
    std::vector<int> v; 
    //fill with data 
    if(v.find(10) == v.end()){ 
     throw std::logic_error("error message"); 
    } 
    // other checks that return false 
} 

и назвать все это список функций в одном try-catch блоке.

Поскольку это производное от std::exception, вероятно, я не знаю, хорошая ли это идея.

Кто-нибудь может использовать пример ниже?

Большое спасибо

AFG

Answer 1

Вы должны использовать исключения только для исключительных обстоятельств. Использование (и проверка) возвращаемых значений намного более эффективно, когда вероятность того, что он является истинным или ложным, является нетривиальным. Исключения являются более эффективными только тогда, когда вероятность броска настолько мала, что позволяет взвесить стоимость проверки возвращаемого значения.

Так что если вероятность недействительных данных очень низкая, то перейдите с исключениями. В противном случае текущее решение должно быть не только точным, но и немного более эффективным (как бросание и обработка относительно дороги).

Answer 2

Поскольку ваша функция называется validate_data() я только бросить исключение, если есть какая-то внутренняя ошибка в функции и использовать true или false, чтобы указать, что функция сделал Validate вход, но он был либо valid (return true), либо недействительным (return false).

Это не помешает вам создать несколько конструкций if() else if() else, но это сделает код более чистым и легче отличить, были ли данные недопустимы или произошла внутренняя ошибка.

try { 
    bool valid = validate_data(foo); 
    /* process data or break if invalid */ 
} catch (std::exception &ex) { 
    /* internal error happened */ 
} 

как вы можете видеть, это сделает ваш код длиннее, ИМХО чище.

Answer 3

Используйте исключения только для исключительных ситуаций.

Не находите значение 10 исключительной ситуации? Или это нормальная ситуация?

Переименование вашего метода из validate_data в is_data_valid делает его более понятным. Если метод возвращает true, он действителен. Если он возвращает false, это не так. Не нужно использовать исключения для этого.