Как правильно сохранять инлайн модель в админке, если id модели-родителя еще не сохраненно в бд?

Question

[McGrog]

Доброго времи суток, сообщество!
Intro: являюсь новообращенным в линукс и питон. На данный момент практикуюсь пистаь на джанго. Решил отойти от написания бложика и написать "местный air bnb". Суть проста: админ заводит новую квартиру, с описанием, указанием координат и фотографиями интрерьера. Интерьер "подключен" как инлайн в квартире (многие-к-одному). Так же в интерьере есть фото интерьера, с отношением так же многие-к-одному.
Если сначала сохранить flat, потом создать и сохранить interior, а уже потом зааплоадить и сохранить фото, то все нормально
Проблема: Но если же попробовать создать flat и interior (не сохраняя) и загрузить фото и сохранить уже в конце, то система сначала попытается сохранить фото (в моем случае основанное на id квартиры и интерьера, котороые еще не записаны в БД), то в путь будет сформирован вида FlatNone/InteriorNone/...
Варианты встреченные на просторах:
1)по другому задавать названия файлов и не раскладывать по папкам. Перфекционист во мне сопротивляется такому решению.
2)Использовать presave. А именно до момента "стандартного" сохранения сохранить id квартиры, а потом id интерьера. Но получается, что в момоент "стандартного" сохранения будет еще одно обращения к БД. Тот же самый перфекционист бастует. Вот тут интересно услышать мнение сообщества, возможно я переусложняю.
3)Возможен еще один вариант: переписать метод сохранения, что бы сначала сохранялась модель flat, потом interior, а потом фото. Но на этой мысли у меня начианют дрожать колени и хочется посмотреть чей-нибудь пример и услышать совет. Самому вряд ли получится сделать нормальное переопределение.

Да, и извините, что код в репе выглядит не очень, иногда не удердиваюсь и коммичу так как есть, потому что "заработало!" ;)
С уважением.

Answer 1

[Anatoly Scherbakov]

Столкнулся однажды с той же проблемой. Но из лени решил последовать принципу Sorl Thumbnail, который формирует пути так:

/your_media_folder/b1/08/b108d1217ae0ec3d38c1e6c174c032ce.jpg

Иными словами, каждая картинка получает в качестве имени случайный UUID и раскладывается по папкам из первых его символов. Получаем равномерную раскладку по папкам, что избавляет от беспокойства о возможном торможении файловой системы при большом количестве файлов в папке. Избавляемся от необходимости нормализовать имена файлов (а если символы кириллические в имени попадутся? все ли компоненты системы отнесутся к этому благосклонно?). Взамен же теряем соответствие между структурой файловой системы и структурой моделей Django. ПМСМ, это несущественно; может быть, у Вас ситуация иная и такой способ не подойдёт.

Comments

[Haos616]

Я брал имя файла + брал текущее время прогонял через md5 или sha1 получалось уникальное имя (нужно проверить что такого файла нет, если есть повторить процедуру). Для директории использовал по 2 символа на каталог из получившегося названия, в зависимости объема фотографий можно выбрать количество уровней папок. P.S. Не пищите много файлов одну папку это вредно для файловой системы, которая потом будет делать лишнюю нагрузку.

[McGrog]

А какой порядок для цифры "много" в данном случае?

[Haos616]

Я для себя считаю нормальным количеством около 10,000 файлов (100,000 уже не очень хорошо). Ты можешь просто потестировать сам =). Учитуй то что папки тоже файлы и в папках тоже не должно быть много папой. Два символа из 16-ричной системы = 256 комбинаций и того у тебя 256 папок в папке. Увеличивая вложенность ты увеличиваешь количество папой что увеличивает количество иннодов (http://ru.wikipedia.org/wiki/Inode) в файловой системе и если у тебя они ограниченны, то советую не сильно усердствовать с папками.

[McGrog]

@Haos616 кажется вы мало спали) Но спасибо. И спасибо за ссылку на иноды.
@Altaisoft проверил с кириллицей все норм, но я сразу делал латиницу+ID из-ща того что слышал проблемах с кириллицей и спецсимволах в урле на файл.

[Haos616]

@McGrog

def get_photo_path(instance, filename):
    uid = uuid.uuid4().hex
    ext = filename.split('.')[-1]
    while True:
        new_filename = os.path.join('photos', uid[0:2], '%s.%s' % (uid, ext.lower()))
        print new_filename
        if not os.path.exists(os.path.join(settings.MEDIA_ROOT, new_filename)):
            break
    return new_filename

[Haos616]

Но еще я бы проверил ext на допустимые форматы для сохранения.

Answer 2

[Алексей Блышко]

Если хочется побаловать перфекциониста, то однозначно третий вариант. На практике же все используют вариант, описанный Altaisoft-ом и не парятся.

Answer 3

[McGrog]

Спасибо за ответы и советы.
2 Altaisoft:
Да, я пробовал делать нечто похожее. UID выладвал результатом хэшфункции от времени + еще что-нибудь. И имена номрмализовал, никакой кириллицы и пробелов. Но это не удобно для
1) дебага - посмотреть, что сдалал обработчик фото или куда суется JS за фото.
2) для гипотетической поддержки и масштабируемости. Друг который занимаетя разработкой iOS в моменты диспутов поучает меня что надо думать о своих последователях - они могут шарить ся по sftp, смотреть код страницы, делать что-либо еще и сильно удивляться. К тому же это как то не красиво что фреймворк может генрить урлы на страницы в человекочитаемом формате, а вот на медиа файлах разраб сплоховал.
3)для админов/манегеров сайта так же подобная иерархия будет шевелить волосы на одном месте. Но с ними проще, если очень постараться и допилить админку. С чем у меня пока не лады)

2 nekt:
Очень хочется. но страшно. Даже не знаю как подступиться. У вас случаем не завалялось собственной реализации подобных веще? Хорошо подумайте перед ответом, ведь тогда не отстану с расспросами)

На выходных еще раз погуглю и если кто-нибудь отпишется здесь о возможност помочь, предлагаю разобрать какой-нибудь "just working example". Просто на stackoverflow прям таких же вопросов не поднималось.

Comments

[Anatoly Scherbakov]

Делать уникальный идентификатор можно вызовом: uuid.uuid4().hex из модуля uuid - и не надо махинаций со временем. Что касается отладки, - ну, можно писать логи с этим именем. И JavaScript на странице вроде бы никак не должен трогать это имя... нет? Обычно работают с id и классами. Право, надёжней та система, что проще.

[Haos616]

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

[McGrog]

@Altaisoft благодарю, за совет. Можно логгировать, но это аналогично расставлению принтов. Человеко читаемым делать удобнее. Но меня почти убедили в том что я переусложняю.
Единственное, что осталось за границей осознанного что есть формсет и что делает его сэйв(топовый ответ http://stackoverflow.com/questions/8294889/override-save-on-django-inlinemodeladmin), и чем он отличается от сэйва модели (http://catherinetenajeros.blogspot.ru/2013/04/django-admin-override-save-method.html)
Походу у меня еще большие проблемы с теорией.