2

Нужно ли проверять аргументы функции?

def divide(a: int, b: int) -> float:
    try:
        return a / b
    except ZeroDivisionError:
        return 0.0

Или

def divide(a: int, b: int) -> float:
    return a / b

3
  • неправильный вопрос. Зависит от того, что хотите. Я бы вообще сделал def divide(a, b) и return a / b :-) Просто 0 не надо передавать. 19 мая 2022 в 14:43
  • @Сергей если есть возможность передать ноль, кто-нибудь этим обязательно воспользуется. Исключение ZeroDivisionError придумано и используется не просто так
    – Dmitry
    28 июн 2022 в 23:13
  • @Dmitry, в массе учебных примеров прямо пишут, что данные будут корректными, чтобы проверки не делали. А тут пример явно учебный. Так что всё же зависит от того, что хотим. 29 июн 2022 в 3:29

2 ответа 2

3

Универсального ответа на этот вопрос нет. Но общее правило гласит - "если пользователь имеет возможность ввести некорректные данные - он их рано или поздно обязательно введет". Поэтому в серьезных программах проверка как входных данных так и всевозможных исключительных ситуаций может занимать процентов 80 а то и больше всего объема написанного кода. А в учебных скриптах - ну можно и не проверять.

4
  • А ссылку дадите на источник данных, что проверки занимают 80%+? :-) 19 мая 2022 в 14:47
  • Практический опыт. Как личный, так и очень многих коллег. Накопленный за 35+ лет практического программирования.
    – passant
    19 мая 2022 в 14:47
  • Давно не читал теорию, но по Бруксу-младшему было 65%. 19 мая 2022 в 14:49
  • Читал, помню. Будем считать, что ему везло. :-) Вот просто посмотрите пример в этом вопросе. И посмотрите, сколько надо было бы в него добавить проверок, если бы этот код использовался бы в действительно серьезных задачах - от управления медицинскими приборами, до мониторинга работы АЭС.
    – passant
    19 мая 2022 в 14:49
2
except ZeroDivisionError:
    return 0.0

Молча "проглатывать" исключения - самый плохой вариант. Если уж ловите исключение, то нужно хотя бы этот факт как-то логировать:

except ZeroDivisionError:
    logging.warning('Случилось деление на ноль!')
    return 0.0

Ну и в целом такой шаблон поведения хорош только его "ленивостью". Если вы точно хотите именно такого поведения функции, то лучше явно проверить случай b == 0 прямо через if, чтобы тот, кто читал код, лучше понимал, что в нём происходит. Хотя в целом это всё-равно вопрос вкуса, конечно. Я бы, наверное, где-то выше по стеку такое исключение ловил. Всё-таки деление - это деление и оно должно быть предсказуемым. На ноль делить нельзя.

Например, вы можете вычислять среднее значение и выводить его куда-то, тогда лучше будет выглядеть наверное так:

from typing import List, Union

def divide(a: int, b: int) -> float:
    return a / b

def get_mean(arr: List) ->  Union[float, str]:
    return divide(sum(arr), len(arr)) if arr else 'недоступно'

print(get_mean([1, 2, 3]))
print(get_mean([]))

Вывод:

2.0
недоступно
1
  • В целом соглашусь с написанным. Однако со сравнением float чисел - я бы рекомендовал бы быть осторожнее. Известный пример : s = str('0.0') f1=s f1==0.0 Угадаете, что выдаст? Правильно, False. А не True, как хотелось бы. Это несколько смущает, особенно новичков.
    – passant
    19 мая 2022 в 15:37

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.