В Django - Inheritance модели - позволяет ли он переопределить атрибут родительской модели?

Я хочу сделать это:

class Place(models.Model):
   name = models.CharField(max_length=20)
   rating = models.DecimalField()

class LongNamedRestaurant(Place):  # Subclassing `Place`.
   name = models.CharField(max_length=255)  # Notice, I'm overriding `Place.name` to give it a longer length.
   food_type = models.CharField(max_length=25)

Это версия, которую я хотел бы использовать (хотя я открыт для любых предложений): http://docs.djangoproject.com/en/dev/topics/db/models/

Это поддерживается в Django? Если нет, есть ли способ достичь подобных результатов?

9 ответов

Решение

Обновленный ответ: как отмечают люди в комментариях, первоначальный ответ не отвечал правильно на вопрос. Действительно, только LongNamedRestaurant модель была создана в базе данных, Place не было.

Решение состоит в том, чтобы создать абстрактную модель, представляющую "Место", например. AbstractPlace и наследовать от него:

class AbstractPlace(models.Model):
    name = models.CharField(max_length=20)
    rating = models.DecimalField()

    class Meta:
        abstract = True

class Place(AbstractPlace):
    pass

class LongNamedRestaurant(AbstractPlace):
    name = models.CharField(max_length=255)
    food_type = models.CharField(max_length=25)

Пожалуйста, прочитайте также ответ @Mark, он дает отличное объяснение, почему вы не можете изменить атрибуты, унаследованные от неабстрактного класса.

(Обратите внимание, что это возможно только начиная с Django 1.10: до Django 1.10 модификация атрибута, унаследованного от абстрактного класса, была невозможна.)

Оригинальный ответ

Начиная с Django 1.10 это возможно! Вы просто должны сделать то, что вы просили:

class Place(models.Model):
    name = models.CharField(max_length=20)
    rating = models.DecimalField()

    class Meta:
        abstract = True

class LongNamedRestaurant(Place):  # Subclassing `Place`.
    name = models.CharField(max_length=255)  # Notice, I'm overriding `Place.name` to give it a longer length.
    food_type = models.CharField(max_length=25)

Нет, это не так

Название поля "скрытие" не допускается

В обычном наследовании классов Python дочернему классу разрешено переопределять любой атрибут родительского класса. В Django это не разрешено для атрибутов, которые Field случаи (по крайней мере, не в данный момент). Если базовый класс имеет поле с именем author, вы не можете создать другое поле модели с именем author в любом классе, который наследует от этого базового класса.

Это невозможно без абстракции, и вот почему: LongNamedRestaurant также Placeне только в классе, но и в базе данных. Таблица мест содержит запись для каждого чистого Place и для каждого LongNamedRestaurant, LongNamedRestaurant просто создает дополнительную таблицу с food_type и ссылка на таблицу мест.

Если вы делаете Place.objects.all()Вы также получаете каждое место, которое является LongNamedRestaurantи это будет экземпляр Place (без food_type). Так Place.name а также LongNamedRestaurant.name использовать один и тот же столбец базы данных и, следовательно, должны быть одного типа.

Я думаю, что это имеет смысл для нормальных моделей: каждый ресторан - это место, и в нем должно быть как минимум все, что есть в этом месте. Может быть, эта согласованность также объясняет, почему это было невозможно для абстрактных моделей до 1.10, хотя это не создавало бы проблем с базой данных. Как отмечает @lampslave, это стало возможным в 1.10. Я лично рекомендовал бы осторожность: если Sub.x переопределяет Super.x, убедитесь, что Sub.x является подклассом Super.x, в противном случае Sub нельзя использовать вместо Super.

Обходные пути: вы можете создать пользовательскую модель (AUTH_USER_MODEL), который включает в себя довольно много дублирования кода, если вам нужно только изменить поле электронной почты. В качестве альтернативы вы можете оставить письмо как есть и убедиться, что оно требуется во всех формах. Это не гарантирует целостность базы данных, если ее используют другие приложения, и не работает наоборот (если вы хотите, чтобы имя пользователя не требовалось).

См. /questions/30367924/kak-pereopredelit-znachenie-po-umolchaniyu-dlya-polya-modeli-iz-abstraktnogo-bazovogo-klassa/30367942#30367942:

class BaseMessage(models.Model):
    is_public = models.BooleanField(default=False)
    # some more fields...

    class Meta:
        abstract = True

class Message(BaseMessage):
    # some fields...
Message._meta.get_field('is_public').default = True

Мое решение так же просто, как в следующем monkey patchingобратите внимание, как я изменился max_length атрибут fo name поле в LongNamedRestaurant модель:

class Place(models.Model):
   name = models.CharField(max_length=20)

class LongNamedRestaurant(Place):
    food_type = models.CharField(max_length=25)
    Place._meta.get_field('name').max_length = 255

Вставил ваш код в новое приложение, добавил приложение в INSTALLED_APPS и запустил syncdb:

django.core.exceptions.FieldError: Local field 'name' in class 'LongNamedRestaurant' clashes with field of similar name from base class 'Place'

Похоже, Django не поддерживает это.

Этот фрагмент кода с суперспуском позволяет вам переопределять поля в абстрактных родительских классах.

def AbstractClassWithoutFieldsNamed(cls, *excl):
    """
    Removes unwanted fields from abstract base classes.

    Usage::
    >>> from oscar.apps.address.abstract_models import AbstractBillingAddress

    >>> from koe.meta import AbstractClassWithoutFieldsNamed as without
    >>> class BillingAddress(without(AbstractBillingAddress, 'phone_number')):
    ...     pass
    """
    if cls._meta.abstract:
        remove_fields = [f for f in cls._meta.local_fields if f.name in excl]
        for f in remove_fields:
            cls._meta.local_fields.remove(f)
        return cls
    else:
        raise Exception("Not an abstract model")

Когда поля будут удалены из абстрактного родительского класса, вы можете переопределять их по мере необходимости.

Это не моя собственная работа. Оригинальный код отсюда: https://gist.github.com/specialunderwear/9d917ddacf3547b646ba

Может быть, вы могли бы иметь дело с contrib_to_class:

class LongNamedRestaurant(Place):

    food_type = models.CharField(max_length=25)

    def __init__(self, *args, **kwargs):
        super(LongNamedRestaurant, self).__init__(*args, **kwargs)
        name = models.CharField(max_length=255)
        name.contribute_to_class(self, 'name')

Syncdb работает отлично. Я не пробовал этот пример, в моем случае я просто переопределил параметр ограничения, так что... подождите и посмотрите!

Я знаю, что это старый вопрос, но у меня была похожая проблема, и я нашел обходной путь:

У меня были следующие занятия:

class CommonInfo(models.Model):
    image = models.ImageField(blank=True, null=True, default="")

    class Meta:
        abstract = True

class Year(CommonInfo):
    year = models.IntegerField() 

Но я хотел, чтобы унаследованное поле года было обязательным при сохранении поля изображения суперкласса. В конце я использовал ModelForms для принудительного применения изображения на этапе проверки:

class YearForm(ModelForm):
    class Meta:
        model = Year

    def clean(self):
        if not self.cleaned_data['image'] or len(self.cleaned_data['image'])==0:
            raise ValidationError("Please provide an image.")

        return self.cleaned_data

admin.py:

class YearAdmin(admin.ModelAdmin):
    form = YearForm

Кажется, это применимо только к некоторым ситуациям (конечно, где вам нужно применять более строгие правила к полю подкласса).

В качестве альтернативы вы можете использовать clean_<fieldname>() метод вместо clean()Например, если поле town будет необходимо заполнить:

def clean_town(self):
    town = self.cleaned_data["town"]
    if not town or len(town) == 0:
        raise forms.ValidationError("Please enter a town")
    return town

Вы не можете переопределить поля Model, но это легко достигается путем переопределения / указания метода clean(). У меня была проблема с полем электронной почты, и я хотел сделать его уникальным на уровне модели, и сделал это так:

def clean(self):
    """
    Make sure that email field is unique
    """
    if MyUser.objects.filter(email=self.email):
        raise ValidationError({'email': _('This email is already in use')})

Сообщение об ошибке затем фиксируется в поле формы с именем "электронная почта"

Другие вопросы по тегам