self.mapping = {}
Насколько есть смысл делать методы по типу in_mapping? Чтобы проверять наличие элемента не через smth in self.mapping
никакого
осуждается или просто бессмысленно?
скорее всего бессмысленно. Но тут вопрос, что вообще за класс и какие реально нейминги
class MeanTimer: def __init__(self, gap_interval): """ :param gap_interval: Max gap between messages in time, e.g 2 secs """ self.chats = {} def add_message(self, message): """ :param message: Message to process :type message: Message :return: """ peer_id = message.peer_id # type: int Вот исходный класс, и мне просто нужно проверить есть ли peer_id в чатах и запихнуть в chats по ключу peer_id другой объект
ну хз, может и осмысленно, но тогда и _chats назвать, чтобы туда вообще руками не лазили. А может вместо проверки наличия сделать более высокоуровневый метод?
например? некий add_time, который сам это все проверит и если нужно создаст?
Обсуждают сегодня