Правильная и красивая замена if?

Question

[Ирина Ховтун]

Как правильно и красиво заменить большое количество if в проекте.
Есть огромное количество

if request.POST.get('type') == '11':
    pass
if request.POST.get('type') == '12':
    pass
if request.POST.get('type') == '13':
    pass
if request.POST.get('type') == '14':
    pass
if request.POST.get('type') == '15':
    pass
if request.POST.get('type') == '16':
    pass

В голову приходит это как то аккуратно оформить как словарь.
напрмиер вот так:

sort_map = {
        '12': test(),
        '13': test1(),
    }

    method = request.POST.get('type')

    if method:
        sort_key = sort_map.get(method)

но увы так это не работает. функции выполняются при старте приложения сразу.
как их указать просто ссылками я что то не очень понимаю.
какие вообще есть методы для работы с таким огромным количеством if ?
их более 30 у меня.
хочется оформить правильно и красиво.

Comments

[Николай Медведков]

sort_map = {
        '12': test,
        '13': test1,
    }

    method = request.POST.get('type')

    if method:
        sort_key = sort_map.get(method)
        sort_key()

вот так?

[Ирина Ховтун]

Николай Медведков, можно и так. спасибки работает.
а насколько это вообще красивое решение так делать ?

[Николай Медведков]

Ирина Ховтун, честно говоря, если у функций есть аргументы, то я не знаю как их передавать таким способом.
p.s. я сам новичок в python, просто знаю что так можно делать, но сам это не использую, ifы проще и понятнее))

[Михаил Кростелев]

Ирина Ховтун, Конкретно в вашем случае, когда от значения одной переменной нужно подбирать дальнейшее действие, как раз-таки всегда рекомендовалось использовать словарь с вариантами. Это даже более "правильный" способ, чем через if.

Можно еще через match case, но эта возможность появилась только в 3.10.

[Ирина Ховтун]

Михаил Кростелев, за match case я смотрела, мне показалось это решение ничем не лучше чем куча if будет.

[Ирина Ховтун]

Николай Медведков, вот да, с аргументами и я пкоа не знаю как быть.

[Михаил Кростелев]

Ирина Ховтун, почитайте про *args, **kwargs

Answer 1

[Ирина Ховтун]

Мне подсказали более изяшный вариант как мне кажется.

sort_map = {
        '12': test,
        '13': test2,
    }

    sort = request.GET.get('sort_by')

    if sort:
        sort_map[str(sort)]()

Comments

[Николай Медведков]

Поздравляю, у вас стало на одну переменную меньше.

[Владислав Лысков]

Николай Медведков, ты пытаешься стать троллем? чесно, у тебя пока плохо получается

[Николай Медведков]

Владислав Лысков, я знаю

[Vindicar]

Это вариант, да. Просто и прямолинейно.
Минус - при добавлении/удалении функции придётся бегать в словарь и корректировать его тоже.

[Николай Медведков]

Vindicar, да, а вот с вашим вариантом можно не бегать в словарь, а просто добавить декоратор?

[Dimonchik]

Николай Медведков, ага

но вышепривеленный вариант куда как нагляднее

[Jack444]

на мой взгляд матчинг тут лучше смотрится.

match request.GET.get('sort_by'):
    case '11': test()
    case '12': test2()

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

Answer 2

[Сергей П]

Ну так не вызывайте эти функции, просто указывайте, а вызывайте одну, которую получите из словаря этого. Учить питон надо старательнее. Функция - это объект первого рода в питоне. Функцию можно присвоить в переменную, передать в аргумент.

Comments

[Николай Медведков]

для этого нужно учить ООП?

[Сергей П]

Николай Медведков, для этого не нужно, нужно просто понимать как устроен язык, переменные, присвоение, вызовы функций, передачу аргументов... Это базовые основы, которые нужно понимать до ООП

[Ирина Ховтун]

Учить питон надо старательнее.
Согласна. но все познается в практике. и усваивается также.

[Сергей П]

Ирина Ховтун, Вот конкретно это знание можно усвоить из любой книжки по питону. Масрка Луца, к примеру, возьмите.

[Ирина Ховтун]

Сергей П, спасибо, я ознакомлюсь обязательно.
а подскажите если использовать данный метод, то каким образом можно передавать аргументы в эти функции?

[Сергей П]

Ирина Ховтун, в таком случае функция вызывается только один раз, аргументы туда при вызове передаются как обычно. В словаре каждой строке-ключу соответствует функцияззначение. То, что функция - это объект первого рода в языке и означает, что она (сама функция, а не ее результат выполнения) может быть значением.
Понятно, что при таком подходе у функций должен быть одинаковый набор аргументов (это называется сигнатура). Если бы вы более подробно расписали свой кейс, было бы понятно какие ещё нюансы вам предстоит преодолеть

Answer 3

[Vindicar]

В комментариях уже написали простое решение, а я его чуток разовью.

import typing as tp

Func = tp.Callable[[], None]  # функция без апарметров, возвращает None

class FuncMap:
  def __init__(self):
    self.map: tp.Dict[str, Func] = dict()

  def register(self, key: str) -> tp.Callable[[Func], Func]:
    def decorate(f: Func) -> Func:
      self.map[key] = f
      return f
    return decorate
  
  def get(self, key: str) -> Func:
    return self.map[key]
  
  def run(self, key: str):
    return self.map[key]()

# а вот так этот класс используется
fmap = FuncMap()

@fmap.register('11')
def test():
  print('This is test()')

fmap.run('11')

Comments

[Николай Медведков]

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

[Vindicar]

Николай Медведков, зато вся чёрная магия прячется в класс, а дальше всё красивенько. =)
А typing можно вообще выкинуть, он тут для читаемости.