Конструктор. Значения по умолчанию в базе данных

@Zuzik · Регистрация: 11.06.2012

Author24 — интернет-сервис помощи студентам

Как сделать красивее код ниже ( код одной из моделей в базе данных, использую орм peewee( алхимия для моих целей слишком большая)).

Python

class Rating(BaseModel):
# тут описываются поля модели ( столбцы базы данных
    win = IntegerField(default=0)
    lose = IntegerField(default=0)
    win_network = IntegerField(default=0)
    lose_network = IntegerField(default=0)
    win_last = BooleanField(default=True)
    win_lose_seq = TextField(default='1, 2, 3:::1,2,3')
    player = ForeignKeyField(Player, related_name='rating')
 
# передаем нужно ли использовать значения по умолчанию, если нет - используем значения переданные дальше
    def __init__(self, defaults=True, _win=None, _lose=None, _win_network=None,
                 _lose_network=None, _win_last=None, _win_lose_seq=None,
                 _player=None,
                 *args, **kwargs):
        super(Rating, self).__init__(*args, **kwargs)
        if not defaults:
            self.win = _win
            self.lose = _lose
            self.win_network = _win_network
            self.lose_network = _lose_network
            self.win_last = _win_last
            self.win_lose_seq = _win_lose_seq
            self.player = _player

Код выглядит громоздко, а если учесть что моделей 8 штук, то это дело становится печальным. Можно ли сделать короче?

@dondublon · 12.05.2015, 09:07

А стоит ли?
Во-первых, короче значит негибче.
Во-вторых, можно объединить ваши параметры в один класс или словарь, если они всегда идут скопом.
И хранить тоже в словаре. Но потеряете преимущество описанных полей.
В общем, выше головы не прыгнешь.

@0x10 · 12.05.2015, 13:23

Не гонясь за простотой, мне в сигнатуре метода не нравится само наличие параметра default. Избыточно. Если меня, как пользователя класса, устраивает дефолт, то я опускаю параметры. Если не устраивает дефолт, то я пишу их. А тут получается я могу передать параметры, но оставить defaults=True и поведение будет удивлять.

Я бы предложил убрать defaults, а тело __init__ переписать в виде:

Python

1	self.win = _win or self.win

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

@Zuzik · 12.05.2015, 22:22 **[ТС]**

0x10, спасибо, уже воспользовался вашим советом. Честно говоря и не подумал, что так можно.
dondublon, ваш совет тоже во внимание принял.
Оставлю в таком виде + уберу defaults. Вопрос впринципе решен. Будут у кого еще мысли - пишите сюда.

@dondublon 4632 / 2050 / 361 Регистрация: 17.03.2012 Сообщений: 10,134 Записей в блоге: 6
	12.05.2015, 09:07	2
	А стоит ли? Во-первых, короче значит негибче. Во-вторых, можно объединить ваши параметры в один класс или словарь, если они всегда идут скопом. И хранить тоже в словаре. Но потеряете преимущество описанных полей. В общем, выше головы не прыгнешь. 1

@Zuzik 298 / 256 / 57 Регистрация: 11.06.2012 Сообщений: 1,557
	12.05.2015, 22:22 [ТС]	4
	0x10, спасибо, уже воспользовался вашим советом. Честно говоря и не подумал, что так можно. dondublon, ваш совет тоже во внимание принял. Оставлю в таком виде + уберу defaults. Вопрос впринципе решен. Будут у кого еще мысли - пишите сюда. 0