Как проверить что запись удалена | Forum

1 March 2011, 22:42 sokol Posts: 5	1 March 2011, 22:42 sokol Posts: 5 Например вызываю: Model.objects.filter(id=5).delete() В доках написано, что метод ничего не возвращает. Как тогда узнать что запись действительно удалена? Неужели только через повторную выборку?
	Answer \| 0

2 March 2011, 1:48 RaD Posts: 2826 Location: Снегири, МО	2 March 2011, 1:48 RaD Posts: 2826 Location: Снегири, МО Раз метод отработал без ошибок, можете спать спокойно. Если ничего не помогает, прочтите документацию, наконец!
	Answer \| 0

2 March 2011, 23:55

sokol

Posts: 5

2 March 2011, 23:55

sokol

Posts: 5

Ну вот например я пытаюсь удалить несуществующую запись, метод отрабатывает без ошибок но никакие записи при этом не были удалены. Я хочу получать что-то типа mysql_affected_rows, это возможно? Кстати в стандартной админке, джанго выдает сообщение об успешном удалении записи.

Answer | 0

3 March 2011, 1:34 RaD Posts: 2826 Location: Снегири, МО	3 March 2011, 1:34 RaD Posts: 2826 Location: Снегири, МО Пример кода с удалением несуществующей записи в студию! Если ничего не помогает, прочтите документацию, наконец!
	Answer \| 0

3 March 2011, 14:04

alerion

Posts: 5750
Location: Львов

3 March 2011, 14:04

alerion

Posts: 5750
Location: Львов

Если вам нужно количество удаленных: сначала посчитайте их и потом тем же QuerySet удалите. Все волнуються что б случайно не удалить существующие данные, а у вас все на оборот.

Фиксю баги по трейсбеку.

Answer | 0

4 March 2011, 0:07

sokol

Posts: 5

4 March 2011, 0:07

sokol

Posts: 5

Меня не это волнует на самом деле. Меня волнует юзабилити! Если пользователь выполняет операцию удаления, то я должен ему сообщить о результате. Считать до-после неправильно, т.к. одновременно может выполняться несколько операций удаления от разных пользователей.

Answer | 0

4 March 2011, 0:12

sokol

Posts: 5

4 March 2011, 0:12

sokol

Posts: 5

rad Пример кода с удалением несуществующей записи в студию! Ну я уже приводил выше: Model.objects.filter(id=5555).delete() Записи с id=5555 в БД на самом деле нет. Никакой ошибки при этом не будет. Вот я и не пойму как отличить успешное удаление Должно же быть что-то.

Answer | 0

4 March 2011, 1:46

RaD

Posts: 2826
Location: Снегири, МО

4 March 2011, 1:46

RaD

Posts: 2826
Location: Снегири, МО

А какую ошибку вы желаете получить в данном конкретном случае? Вы требуете от Django выбрать набор моделей с несуществующим идентификатором. Django предоставляет вам пустой набор. Затем вы применяете для каждого элемента набора (помните, набор-то пустой) операцию delete(). Это как умножение на ноль. Django абсолютно адекватно ничего вам не возвращает, ошибки нет.

P.S. Решайте реальные проблемы.

Если ничего не помогает, прочтите документацию, наконец!

Answer | 0

4 March 2011, 13:25

alerion

Posts: 5750
Location: Львов

4 March 2011, 13:25

alerion

Posts: 5750
Location: Львов

rad:P.S. Решайте реальные проблемы.

Это точно. Вы тратите время ни на то.

Всегда делают вот так:

try:
    Model.objects.get(id=100500).delete()    
except Model.DoesNotExist:
    print 'Error: Object does not exist'

sokol:Считать до-после неправильно, т.к. одновременно может выполняться несколько операций удаления от разных пользователей.

А сколько же у вас онлайн сайта, если такая большая вероятность что два поьзователя одновременно захотят удалить? Обычно удаляют не так часто. Если это очень волнует - вешайте где-то флаг блокировки и проверяйте его. Но это какой-то непонятный геморой. Таким можно заниматься если все готово, проект на 70% покрыт тестами, багов нет и нужно что-то делать.

Фиксю баги по трейсбеку.

Answer | 0

5 March 2011, 3:13

sokol

Posts: 5

5 March 2011, 3:13

sokol

Posts: 5

rad А какую ошибку вы желаете получить в данном конкретном случае? Вы требуете от Django выбрать набор моделей с несуществующим идентификатором. Django предоставляет вам пустой набор. Затем вы применяете для каждого элемента набора (помните, набор-то пустой) операцию delete(). Это как умножение на ноль. Django абсолютно адекватно ничего вам не возвращает, ошибки нет.

P.S. Решайте реальные проблемы. Да не ошибку я хочу получить, а например кол-во удаленных записей в случае если что-то было удалено, ошибку если несуществующая (ну это уже привели спс. alerion). В данном случае должен вернуть 0 например, но как я понял можно и через исключения, посмотрю какие есть в принципе. Всем спасибо!

Answer | 0