Как можно упростить этот Python код?

Question

[MonsterAndrew]

Есть код:

for player in players:
    if player["id"] == user_id: return player
    else: return False

Как можно уместить его в одну строчку?

P.S. Я пробовал вот так: return [d for d in players if d["id"] == user_id], но тогда он возвращает не False, а пустой массив. Хотелось бы чтобы если if не сработает возвращался False.

Comments

[longclaps]

Монстер Эндрю, вы потеряли интерес к предыдущей задаче?

[Евгений]

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

return bool(filter(lambda player: player['id'] == user_id, players))

[DDDsa]

Евгений, имеет смысл. Он возвращает первого player, с заданным id или False, если такого нет. Цикл вполне подходит для решения этой задачи

UPD: я слепой. Увидел, что вы имеете в виду. else срабатывает на первой же проверке и он выходит из цикла.

Корректный вариант с циклом был бы такой:

for player in players:
    if player["id"] == user_id:
        return player
else:
    return False

[Евгений]

DDDsa, он в любом случае прерывается после первой же итерации в ветке else.

[DDDsa]

Евгений, точно, я как раз отредактировал свой комментарий )

[longclaps]

Евгений, хех

Answer 1

[planc]

next((d for d in players if d["id"] == user_id), False)

Comments

[longclaps]

Смишно.
players или пустой, или нет.
Если пустой, цикл не походится ни разу, и результатом функции будет None
Если нет, достаточно проверить первый элемент.

return (players[0] if players[0]["id"] == user_id else False) if players else None

[Евгений]

longclaps, Почему? Тут вообще непонятно, что автор хотел решить. Он, кстати, и не появился ни разу, чтобы пояснить. Мой код тоже неоптимальный, так как просмотрит массив до конца, даже если уже нашел совпадение.

[longclaps]

Евгений, автор дурень, да. Но к чему за него фантазировать, есть вопрос - есть ответ.
Генератор, конечно, ничего до конца не просмотрит (гуглите ленивые вычисления), но тут ему точно не место.

Answer 2

[SexyHair]

Зачем тут вообще цикл, если после первой итерации в любом случае вернется или player или False
return players[0] if players[0]['id'] == user_id else False
P.S Ответ на ваш "P.S"
Попробуйте return [d for d in players if d["id"] == user_id] or False