Django的。记录未正确删除

时间:2017-02-26 04:33:57

标签: python django sqlite

我有问题希望你能帮助我。

我有这些模型:

MODELS = (
('A', 'A'),
('B', 'B'),
)


class person(models.Model):
    nombre = models.CharField(max_length=128)
    model = models.CharField(max_length=128,choices=MODELS, default=True)

    def __str__(self):
        return self.nombre


class person_A(models.Model):
    person = models.ForeignKey(person, on_delete=models.PROTECT, null=True)
    hobbies = models.CharField(max_length=40, default='')

    def __str__(self):
        return self.person.nombre

class person_B(models.Model):
    person = models.ForeignKey(person, on_delete=models.PROTECT, null=True)
    age = models.IntegerField(default=10)

    def __str__(self):
        return self.person.nombre

如您所见,我有person_Aperson_B,它将保存在我的数据库中。

以下是我的观点:

def get_person(request):
    titulo = 'Person'
    qs_a = person_A.objects.all()
    qs_b = person_B.objects.all()
    qs = chain(qs_b,qs_a)
    form = person_aForm(request.POST or None)
    form2 = personForm(request.POST or None)
    form4 = person_bForm(request.POST or None)
    context = {
        "qs_a": qs_a,
        "qs_b": qs_b,
        "qs": qs,
        "form2": form2,
        "form": form,
        "form4": form4,
        "titulo": titulo,
    }
    form2_valid = form2.is_valid()
    form_valid = form.is_valid()
    form4_valid = form4.is_valid()
    if form2_valid:
        person = form2.save()
        if form_valid:
            person_a = form.save(commit=False)
            person_a.person = person
            person_a.save()
            messages.success(request, 'Se ha guardado satisfactoriamente')
            return redirect("get_person")
        if form4_valid:
            person_b = form4.save(commit=False)
            person_b.person = person
            person_b.save()
            messages.success(request, 'Se ha guardado satisfactoriamente')
            return redirect("get_person")
    return render(request, "person.html", context)

您可以在此处看到我的表单已保存为person_Aperson_B

这是我的删除功能:

def delete_person(request,id):
    try:
        qs = person_B.objects.all()
        instance = get_object_or_404(qs, id=id)
        instance.delete()
        messages.success(request, 'B')
    except:
        try: 
            qs = person_A.objects.all()
            instance = get_object_or_404(qs, id=id)
            instance.delete()
            messages.success(request, 'A')
        except:
            pass
    return redirect('get_person')

我的网址:

url(r'^delete_person/(?P<id>\d+)/$',views.delete_person, name='delete_person'),

模板:

<table class="table table-hover">
    <thead>
      <tr>
        <th>ID</th>
        <th>Name</th>
        <th>Last Name</th>
        <th>Delete</th>
      </tr>
    </thead>
    <tbody>
    {% for item in qs  %}
      <tr>
        <td>{{ item.id }}</td>
        <td>{{ item.person.model }}</td>
        <td>{{ item.person.nombre }}</td>
        <td><a href="{% url 'delete_person' item.id %}">Delete</a></td>
      </tr>
    {% endfor %}
    </tbody>
</table>

基本上,我有两个不同的模型,每个人都被保存在数据库中并显示在table中。为此,我使用了chain()中的itertools。我加入了A人和B人,在一个table中展示了他们。这工作正常,现在我的问题是:

如果person_Aid具有相同person_B而我想删除person_A,则会删除person_B。我知道这是因为chain(),但我想知道是否有可能解决这个问题的方法?

谢谢!

1 个答案:

答案 0 :(得分:1)

您的问题在于delete_function()始终首先尝试删除person_B。除非抛出异常,否则永远不会删除person_A。如果抛出异常,您仍将首先删除person_B,因为它位于try:块内,因此尝试删除它总是在尝试删除person_A之前。顺便说一下,现在编写代码的方式,任何异常将导致尝试删除person_A。我建议使用一个更具体的例外,否则你可能会在以后碰到真正令人困惑的错误。

至于解决方案,我确信有更多聪明的方法,但我现在能想到几个快速而肮脏的解决方案:

  1. 定义自定义filter,它将检查person的类型是什么类型并相应地执行(例如,为不同的类调用不同的删除方法;将标记传递为query parameter并且在delete_person()方法中使用它来确定要删除的person_
  2. 我可能会做什么:不是链接模型实例,而是单独迭代它们并调用适合特定模型的删除方法:

     <table>
     <thead>...</thead>
     <tbody>
       ...
       {% for item in qs_a %}
         <tr>
           <td>... delete_person_A() ...</td>
         </tr>
       {% endfor %}
       {% for item in qs_b %}
         <tr>
           <td>... delete_person_B() ...</td>
         </tr>
       {% endfor %}
     </tbody>
     </table>
    

    如果我没记错的话,chain()迭代第一次迭代直到它耗尽,然后才迭代第二次迭代。因此,分别迭代qs_aqs_b不应该对输出产生巨大影响。虽然这种方式你需要编写更多的代码,但它也会更简单明了 - 因此,如果有需要,以后更容易理解和重构。