Как проверять аргументы функции/метода?

Question

[DUDE]

Всем привет, есть некий метод или функция, которая принимает ограниченное количество входящих аргументов:

def foo(a: str = None, b: str = None, c: int = None):
    ...

Предположим есть несколько сценариев отработки в зависимости от того, какие аргументы даны.

• a != None, b != None, c == None
  
• a == None, b == None, c!= None
  

Как вариант можно разбить по ифам с соответсующими исключениями, но я не уверен что это приемлемо, а если это и приемлемо то писать валидацию внутри функции?

Возможно я вообще что-то делаю не так, если с таким сталкиваюсь...

Comments

[o5a]

DUDE, Вы бы лучше написали, как все это планируется использовать. Для чего такое разделение на параметры с дефолтным None? Какая планируется логика использования этой функции? Исходя из этого можно будет что-то подсказать.

Answer 1

[Vindicar]

Если аргумент None, означает ли это, что должно использоваться некоторое значение по умолчанию? Если да, то можно вынести if по этому аргументу отдельно, а дальше вести себя так, словно аргумент задан.

Если же нет, то стоит спросить себя - а не слишком ли много эта функция на себя берёт? Не стоит ли её разделить на несколько?

Answer 2

[Стефан]

Переписывайте функцию что бы она и None нормально отрабатывала. Так же, как по мне, если вы у вас функция может не принимать аргументы вовсе - то логичней это упаковать все в:

def foo(*args, **kwargs):
    ...

И возможно как раз с такой логикой вам легче будет все это устроить
А вообще все сильно зависит от того что вы делаете внутри функции, я вам сказал лишь свои рекомендации не опираясь на требования, а это может внести коррективы

Comments

[DUDE]

В том и суть что она принимает ограниченное количество аргументов, а не принимать вообще она их не может)

[Стефан]

DUDE, Всм не может? Что вы имеете ввиду, можете поточней описать, вам надо валидировать комбинации аргументов или что?

[DUDE]

Стефан, да комбинации

[Стефан]

DUDE, У вас в зависимости от комбинации разная логика?

[DUDE]

Стефан, ну я ж написал это)

[Стефан]

DUDE, Разбивайте на разные функции это тогда, это самый логичный и правильный вариант, single responsibility principle не просто так придумали