0

Нужно было сделать собственный slug, в который бы входил номер объекта (уникальный для каждого адреса) и номер ID в БД (sqlite3), т.к. тип объекта по адресу может быть различным. В итоге хотел получить для себя в БД уникальный slug = №_ID. Все удалось сделать, НО не могу понять где ошибка. Создал к прмиеру 3 записей в БД, удалил две записи (2, 3), создаю новую запись, ID показывает как следующий номер записи в БД (4), но при этом slug = №_2, а должен быть равен ID (4). После чего, создаю еще одну запись (3), все работает нормально ID = 5, slug = №_5.

models.py

def funcc():
    nm = Object.objects.all()
    ob = Object.objects.count()
    ob = ob + 1
    for nn in nm:
        if nn.id > 0:
            ob = Object.objects.latest('id')
            ob = ob.id + 1
        else:
            ob
    return ob


id_object = models.CharField(max_length=50, default=funcc, verbose_name="ID Объекта", unique=True, blank=True, null=True, editable=False)
id_fias = models.CharField(max_length=50, default='', null=False, blank=False, db_index=True)
slug = models.SlugField(unique=True, blank=True, null=True)

def save(self, *args, **kwargs):

        if not self.slug:
            self.slug = str('{}_{}'.format(self.id_fias, self.id_object))
        super().save(*args, **kwargs)

1 ответ 1

0

можно просто получить последний id, а не бегать по всем объектам в цикле

def get_new_id():
    last_obj = Object.objects.order_by('-id').first()
    last_id = last_obj.id if last_obj else 0
    return last_id + 1

Так же можно улучшить код, если вместо получения целого объекта получать только те поля, которые нам нужны, используя Object.objects.values

p.s.: старайся не использовать ничего не значащие названия func, nm, nn, ob. Если программа хоть как-то разрастется, то ты потом задолбаешься читать свой код

2
  • 1
    Стоит заметить, что реально создаваемый id совсем не обязательно будет last_id+1, он может зависеть от настроек базы данных и от наличия конкурирующих insert'ов
    – andreymal
    20 ноя 2021 в 17:40
  • 1
    спасибо за помощь и ценные советы. 23 ноя 2021 в 6:42

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.