тонкие контроллеры - толстые модели

вьюху можно считать контроллером

спс

или лучше во вьюхи вынести?

Я бы в менеджер определил

Я бы в менеджер определил

+

кстати от / можно избавиться если cls.objects.filter написать на одном уровне. дальше всё-равно открываются скобки у тебя. читаться будет лучше (имхо)

Хотел разделить filter и exclude просто

Я бы в менеджер определил

Спасибо, а то некоторые классы затрагивают несколько моделей, которые, по сути, описываю один объект

A view is a place where we put the "logic" of our application

Из мануала джангогирлс

подскажите, плз, как правильно делать кнопку "отмены" в форме подтверждения (допустим, удаления объекта) для пользователя? и делать ли ее вообще?

Ссылку на страницу назад

т.е. форма типа <form action="{% url 'comparator:delete_case' comparecase.id %}" method="post"> блаблабла уверены ли вы

Посмотри как в админке сделано при удалении

внизу <input type="submit" value="Confirm">

а кнопка отмены input'ом?

Посмотри как в админке сделано при удалении

а, ок, гляну, пасиб

Вроде просто a, хотя могу ошибаться.

а если просто а, то она же будет не как кнопка, а как ссылка

некрасиво

Возможно онклик в button

во, я так и хотел сначала сделать, но засомневался

Да вроде там онклик был

а вообще ее принято делать? в принципе юзер сам может назад нажать в браузере

если передумал удалять например

И жс на страницу назад.

Лучше явно, а то за сомневается и вообще закроет страницу.

А потом будет мучиться и проходить всю иерархию заново.

да, согласен, сделаю через онклик, спасибо

можно воткнуть дополнительный параметр типа _confirm=true

в форму

при запросе на /delete/ проверять наличие, без него возвращать форму конфирма, а с ним удалять

блин, как всетаки весело клепать шаблончики

какого-же это всетаки погрязнуть в API'шках

одни JSON, да XML

шедуллеры

здорово же

верстка - это вторично

я забыл как называется чувство очень сильного удовлетворения

эйфория

во

эйфория - это когда потреление рамы проектом снес с 16 гигабайт до 100 МБ

вот это здорово

?

и ничего не поломалось

О_о ты там чего крутил?

резал одну большую транзакцию

подкажите, плз. имею конструкцию вида last_unit = CompareUnit.objects.filter(case_id__exact=case.id).reverse()[:1] в надежде получить из нее некий "последний юнит", принадлежащий определенному "кейсу". затем мне нужно взять у этого последнего юнита свойство "position", т.е. last_unit.position. но может так оказаться, что в кейсе нет ни одного юнита и вывалится AttributeError. как правильно делать проверку в таком случае?

я сначала хотел сделать if last_unit: но ведь фильтр вернет QuerySet в любом случае, как я понимаю

можешь сделать проверку на каунт например filter(...).count() > 0

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

ну т.е. я выходит должен сделать проверку if last_unit.count== 1 но как-то коряво это смотрится

типа убедиться, что фильтр вернул мне в QuerySet значение, и оно всего одно...

а в чем задача вообще? убедиться что есть хоть одно значение в таблице? много ли там записей сейчас и в будущем ?

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

в базе сейчас пусто, в теории должно быть много много кейсов, но мало-мало юнитов у каждого, в среднем 3-5

можно еще так попробовать получить последний юнит

Если я правильно понял задачу, то можно сообразить что-то типа этого: position = 0 if case.units.all() position = case.units.last().position Unit.objects.create(position=position, case=case)

Unit.objects.get(case=case, pk=Unit.objects.aggregate(Max("pk"))["pk__max"])

и обрабатывать уже Unit.DoesNotExist ексепшн

а если все ок - вернет последний созданный юнит

Если цель вернуть именно последний созданный юнит то отлично подойдет Unit.objects.last()

Если цель вернуть именно последний созданный юнит то отлично подойдет Unit.objects.last()

сорри, отвлекли. так вот да, .last() - то, что нужно. и еще моя ошибка была в том, что я искал поле у QuerySet, а надо было из него взять еще [0]

Unit.objects.get(case=case, pk=Unit.objects.aggregate(Max("pk"))["pk__max"])

та зачем так! Unit.objects.annotate(pk_max=Max("pk")).get(case=case, pk=pk_max)

и он запилит нормальный запрос с аггрегациоными функциями без повторного селекта

так же и по лукапам можно пробежаться

та зачем так! Unit.objects.annotate(pk_max=Max("pk")).get(case=case, pk=pk_max)

кстати наверное с max - еще изящнее, выглядит очень читаемо

а то у меня тут еще сомнения возникли, по какому полю он будет их сортировать в фильтре. я вроде читал, что это где-то задается, но забыл как

как задашь, так и будет сортировать

а вот я забыл где задается поле для сортировки

там можно много изящных запросиков замутить

order_by() и передаешь арги по каким полям сортировать

а разве нет возможности задать поле для сортировки по умолчанию?

В модели можно

вот и мне так казалось. а напомните параметр плз

https://docs.djangoproject.com/en/1.10/ref/models/options/#ordering

а, да, спасибо

та зачем так! Unit.objects.annotate(pk_max=Max("pk")).get(case=case, pk=pk_max)

а зачем вообще нужен annotate? почему не просто .objects.get(case=case, pk=Max('pk'))? так нельзя?

что-то я делаю не так. с annotate() выдает name 'pk_max' is not defined

F('pk_max') но мне кажется всё равно не будет работать

в итоге вот так сделал: last_unit = CompareUnit.objects.filter(case=case).aggregate(Max('position'))['position__max'] new_unit_position = 0 if last_unit is None else last_unit+1 надеюсь не очень по-индийски получилось

можешь сделать проверку на каунт например filter(...).count() > 0

.exists()

подкажите, плз. имею конструкцию вида last_unit = CompareUnit.objects.filter(case_id__exact=case.id).reverse()[:1] в надежде получить из нее некий "последний юнит", принадлежащий определенному "кейсу". затем мне нужно взять у этого последнего юнита свойство "position", т.е. last_unit.position. но может так оказаться, что в кейсе нет ни одного юнита и вывалится AttributeError. как правильно делать проверку в таком случае?

.last()

.last()

уже было, но нет. я фундаментально спутал понятия "последний элемент" и "элемент с наибольшим значением в поле... "

выше написал, как решил, работает

Ааа.

Ок

Но все равно не пренебрегайте экзистом и ластом

Они генерируют разные запросы

Разные, чем каунт

И слайс

Вроде как

Да и более осмысленно выглядят

подскажите, плз, по базовой концепции, в ту ли сторону копаю. есть некая основная модель и пара других моделей, которые с ней связаны через ForeignKey. в views есть CBV для основной модели, наследует от generic.DetailView. если я хочу в шаблоне для этой вьюхи показывать все связанные с основной моделью другие модели (объекты), мне надо переопределять get_context_data? в принципе работает, но не уверен, хорошая ли это практика

Нет, не надо. Если не ошибаюсь, вам нужно это https://docs.djangoproject.com/en/1.10/ref/models/fields/#django.db.models.ForeignKey.related_name

хм, да, похоже! спасибо, сейчас попробую переделать

Может кто подсказать надо после сохранения файла распарсить его. как это лучше сделать? Сейчас вьюшка на CreateView сделана