Как упростить однострочный if?

Question

[danyaxaxaxaxa]

Опять же пытаюсь научиться писать код чище. Как можно упростить этот кусок кода?

if int(conversation_id) != event.object['message']['peer_id'] - 2000000000:
     continue

Comments

[danyaxaxaxaxa]

Всем спасибо за ответы!
Теперь я знаю немного больше об упрощении кода!

Answer 1

[Сергей П]

CONVERSATION2PEER_GAP = 2_000_000_000
conversation_id = int(conversation_id)  # Почему у вас где-то идентификаторы хранятся интом,
# а где-то текстом? Нужно стремиться к стандартизации и избегать таких неконсистентных нюансов.
# Если там не может быть другого текста, то следует приводить к числу сразу когда возможно.
peer_id = event.object['message']['peer_id']
if peer_id != conversation_id + CONVERSATION2PEER_GAP:
     continue

Answer 2

[LXSTVAYNE]

Можно попробовать написать для себя такой код, мне кажется читается проще:

import enum


class SenderType(enum.Enum):
    FROM_USER = enum.auto()
    FROM_CHAT = enum.auto()
    FROM_GROUP = enum.auto()


def get_sender_type(peer_id: int) -> SenderType:
    SHIFT_VALUE = 2_000_000_000

    if peer_id > SHIFT_VALUE:
        return SenderType.FROM_CHAT
    elif 0 < peer_id < SHIFT_VALUE:
        return SenderType.FROM_USER
    else:
        return SenderType.FROM_GROUP


if __name__ == '__main__':
    test_id = 345343111

    if get_sender_type(test_id) == SenderType.FROM_USER:
        print('Работает)')

Answer 3

[U235U235]

Не сильно проще, но как вариант:

if sum((int(conversation_id), -event.object['message']['peer_id'], 2e9)):
    continue

Comments

[Сергей П]

Этот вариант гораздо хуже оригинала по причинам:
- меньшая читаемость и понятность
- использование вещественных чисел там, где были целые.