Как организовать в django отношение «родитель -- дочерняя запись» в models красиво?

Question

[Сергей Еремин]

Собственно хочется "красивую" models для отношений "родитель -- дочерняя запись" внутри одной таблице. Вот так ругается:

class Person (models.Model):
    FullName = models.CharField ( default=u"Иванов И.И.", max_length=256 )
    # ...
    Parent = models.OneToOneField ( Person )

    def __unicode__(self):
        return self.FullName

Скорее всего получится устанавливать отношение "родитель -- дочерняя запись" в ручную, например:

class Person (models.Model):
    id = models.IntegerField ( primary_key=True )
    FullName = models.CharField ( default=u"Иванов И.И.", max_length=256 )
    # ...
    Parent = models.IntegerField ( default=0, db_index=True )

    def __unicode__(self):
        return self.FullName

Но очень хочется "красивости".

Собственно все это нужно чтобы не переписывать записи, в которых много индексированных полей. Перезапись индексов будет подтормаживать, а если просто добавлять новую запись а у старой менять Parent , то это перестроение всего одного индекса (Parent же тоже придется индексировать, чтобы после быстро выбирать записи с Parent == 0)

Answer 1

[sim3x]

Ты бежишь впереди паровоза

Вот такое не надо
id = models.IntegerField ( primary_key=True )
оно само делается ОРМ

OneToOne это не про то. Это поле, которое связывает две модели, которые по некоторым обстоятельствам были разделены. Раньше их часто употребляли для связи захардкоженной модели User и UserDetails с доп полями, которые нужны были разрабу

FullName
CamelCase - для классов
under_score - для переменных, полей модели и тд

class Person(models.Model):
    full_name = models.CharField( default=u"Иванов И.И.", max_length=256 )
    # ...
    parent = models.ForeighKey('self', blank=True, null=True)
    # or
    # parent = models.ForeighKey("Person", blank=True, null=True)

то это перестроение всего одного индекса

Но как и раньше - решаешь проблемы, которой еще нет

Comments

[Сергей Еремин]

Cпасибо. Заработал вариант:

parent = models.ForeighKey("Person", blank=True, null=True)
# and
# parent = models.ForeighKey("self", blank=True, null=True)

На счет "перестроение всего одного индекса" не понял. Когда я в былые года программировал это было важно.

P.S. До 2004 года я много программировал (Cи, ColdFusion) и всяких граблей насобирал. Понятно, что тогда железо было не такое, но как-то уже на старте думаешь как все это крутиться будет. Особенно старая школа Cи под DOS на PC/XT сказывается. Когда всего 640К, а надо данных обработать на сотню мегабайт задумываешься даже о том как с оптимизировать позиционирование в файле, чтобы диск не сыпался и лента не долго перематывалась.

[sim3x]

>Когда я в былые года программировал это было важно.
сейчас бд умнее, памяти больше - если не платят за оптимизацию, то и не стоит ее делать

[Сергей Еремин]

За оптимизацию не платят (сам себе задачи ставлю)...
Но лучше же сразу делать хорошо, а плохо, оно, само получится :)

[Сергей Еремин]

@sim3x Спасибо. Очень помог. Хочу задать еще один вопрос про авторизацию через django-social-auth ... Похоже вопрос "нубский", но что-то никак не решается... Но из-за своей "нубовости" светить его на ТОСТЕР-е не хочется. Куда тебе постучать, чтобы связаться? URL у тебя в профиле протух.

[sim3x]

@Sergei_Erjemin с эти аппом я не работал - так что наврядли помогу

Answer 2

[Rrooom]

А погуглить?

Отношение к самому себе устанавливается с помощью строки "self".

А еще лучше использовать django-mptt.

Comments

[Сергей Еремин]

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

[sim3x]

@Sergei_Erjemin ну ты ж модель изменил, а бд не уведомил

[Артем Лисовский]

@Sergei_Erjemin python manage.py syncdb

[Сергей Еремин]

Делал-делал... Через South Схема же заработала... И админка... а вот когда входишь внутрь записи все падает! Ошибка рендера админского шаблона.

[Alexander]

@torrie syncdb не изменяет таблицу в базе данных, только удалить или создать новую. Тут или руками или South для миграций.

[Артем Лисовский]

@syschel ну мало ли, вдруг не синкался вовсе :)

Answer 3

[denizen]

попробуйте django-mptt