2
sku_ = re.search("Артикул товара: (\d+)", item_info.text)
sku = sku_[1] if sku_ else ""

Можно ли данный код сократить до одной строчки?

0

4 ответа 4

3

У меня так получилось, но наверняка можно ещё лучше сделать:

sku = (re.search("Артикул товара: (\d+)", item_info.text) or ('',''))[1]
1
  • 1
    Если индекс от конца сделать ([-1]), то можно после or записать список из одного элемента, получится немного короче (хотя это уже код-гольфом попахивает): sku = (re.search("Артикул товара: (\d+)", item_info.text) or [""])[-1]
    – insolor
    28 июл 2022 в 14:05
3

Кое-кто считает оператор := некрасивым. Кто-то полагает что оператор or нехорошо применять к не булевым значениям. Обойдёмся без некрасивых и нехороших вещей с помощью лямбды:

sku = (lambda m: m[1] if m else "")(re.search("Артикул товара: (\d+)", item_info.text))

P.S. Это шутка. Определённая и сразу вызванная лямбда - такой же или худший костыль как := или or.

P.P.S. Механизм наподобие этого позволяет записать в одну строку вообще любое вычисление (исключения, асинхронность и т.п. отложим пока в сторону).

2

Мой вариант

sku = sku[1] if (sku := re.search("Артикул товара:(\d+);", item_info)) else ''
1
  • Позволю себе добавить, что данное решение будет актуально для версий python 3.8 и выше
    – Namerek
    24 июл 2022 в 11:56
2

Вариант решения через контейнер Maybe из библиотеки dry-python/returns

import re

from returns.maybe import Maybe


def get_sku(text):
    return (
        Maybe.from_optional(re.search("Артикул товара: (\d+)", text))
        .bind_optional(lambda x: x[1])  # Или .bind_optional(itemgetter(1))
        .value_or("")
    )


assert get_sku("Артикул товара: 12323423") == "12323423"
assert get_sku("sdfwefew") == ""

Это похоже на класс Optional из Java, но названия методов совершенно другие, и по ним сразу не угадаешь что делает каждый метод.

На Java выглядело бы примерно так:

// Псевдокод
sku = Optional.ofNullable(re.search("Артикул товара: (\d+)", text))
    .map(lambda x: x[1])
    .orElse("");

Тут довольно очевидно, что map применяет функцию к непустому результату, но что делает bind_optional глядя на код так просто не угадаешь.

Так что скорее не рекомендую такой способ.

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.