Django表单未保存到数据库

Question

我正在编写一个具有HTML页面的应用程序，该页面必须能够将数据保存到2个模型中。 我创建了两个单独的表单，并在视图中引用了它们，但是该信息并未保存到数据库中。

这是views.py

def nuevoingreso(request):
    if request.method == "POST":
        formingreso = NuevoIngreso(request.POST)
        formprodingreso = NuevoProdIngreso(request.POST)
        if formingreso.is_valid():
            ingreso = formingreso.save(commit=False)
            ingreso.idUser = request.user
            ingreso.Condominio = get_object_or_404(Condominios, idCondominio=request.session["idCondominio"])
            ingreso.save()
            ingresoprod = formprodingreso.save()
            for i in range(5):
                if ProductosIngresos.SubtotalP != "" and ProductosIngresos.IvaP != "" and ProductosIngresos.TotalP != "":
                    ingresoprod.ProductosIngresos(Concepto=request.POST.get("Concepto"+str(i), ""), SubtotalP=request.POST.get("SubtotalP"+str(i), ""), IvaP=request.POST.get("IvaP"+str(i), ""), TotalP=request.POST.get("TotalP"+str(i), ""))
            ingresoprod.save()
            return HttpResponseRedirect("/propiedades/")
        else:
            return render(request, "immovelc/nuevoingreso.html",
                          {"formingreso": formingreso, "formprodingreso": formprodingreso})
    propiedadesing = PropiedadesCond.objects.all()
    context = ({"propiedadesing": propiedadesing})
    return render(request, "immovelc/nuevoingreso.html", context)

表格

class NuevoIngreso(ModelForm):
    class Meta:
        model = Ingresos
        fields = ["Numero", "Persona", "Fecha", "Observaciones", "Cobrado", "Subtotal", "Iva", "Total"]
        def clean(self):
            Numero = self.cleaned_data["Numero"]
            Persona = self.cleaned_data["Persona"]
            Fecha = self.cleaned_data["Fecha"]
            if not Numero:
                raise forms.ValidationError("El campo de numero es obligatorio")
            if not Persona:
                raise forms.ValidationError("El campo de cliente es obligatorio")
            if not Fecha:
                raise forms.ValidationError("El campo de fecha es obligatorio")

class NuevoProdIngreso(ModelForm):
    class Meta:
        model = ProductosIngresos
        fields = ["Concepto", "SubtotalP", "IvaP", "TotalP"]

models.py

class Ingresos(models.Model):
    idIngreso = models.AutoField(primary_key=True, null=False, max_length=15)
    idCondominio = models.ForeignKey(Condominios)
    idUser = models.ForeignKey(User)
    Numero = models.CharField(max_length=100)
    Persona = models.CharField(max_length=250, default="Ninguno")
    Cobrado = models.CharField(max_length=100, default=0)
    Observaciones = models.TextField(default="Ninguna")
    Fecha = models.DateField()
    Subtotal = models.CharField(max_length=100)
    Iva = models.CharField(max_length=100)
    Total = models.CharField(max_length=100)
    def __unicode__(self):
        return unicode(self.idIngreso)

class ProductosIngresos(models.Model):
    idProductoIngreso = models.AutoField(primary_key=True, null=False, max_length=15)
    idIngreso = models.ForeignKey(Ingresos)
    Concepto = models.CharField(max_length=500)
    SubtotalP = models.CharField(max_length=100)
    IvaP = models.CharField(max_length=100)
    TotalP = models.CharField(max_length=100)
    def __unicode__(self):
        return unicode(self.idProductoIngreso)

谢谢！

Answer 1

没有违法行为，但是此代码远非正确的。

此外，您还有许多错误可能要删除。

错误：

• 从不调用formprodingreso.is_valid()
• for i in range(5)内部for i in range(5)您使用一个类就好像它是一个实例（ ProductosIngresos.SubtotalP ）
• 表单中的clean方法必须在Meta块之外

我相信您想要在循环中进行的操作是：

producto_ingreso = ProductosIngresos()
producto_ingreso.idIngreso = ingreso # better change to producto_ingreso.ingreso
producto_ingreso.Concepto=request.POST.get("Concepto"+str(i), "") # producto_ingreso.concepto
producto_ingreso.SubtotalP=request.POST.get("SubtotalP"+str(i), "") # producto_ingreso.subtotal_p
producto_ingreso.IvaP=request.POST.get("IvaP"+str(i), "")
producto_ingreso.TotalP=request.POST.get("TotalP"+str(i), ""))
producto_ingreso.save()

为了使它更整洁，您可以使这种逻辑之王覆写ModelForm的save（）方法。 或使用嵌入式表单集 。

混乱：

• 模型FK是Django中的对象，而不是整数。 最好将它们命名为condominio而不是idCondominio
• 小数列（小计，iva，总数）应声明为deciaml，即models.DecimalField(max_digits=10, decimal_places=2)
• clean方法用于跨字段验证（多个字段）。 仅一个字段应由clean_numero fe验证

过于复杂：

• 模型默认情况下具有ID / PK，无需显式它们（称为self.id或self.pk ）
  • 模型unicode函数未提供任何信息
  • clean和ValidationError是多余的：modelform检查是否自动需要属性

约定错误：

• 属性始终为write_in_lowercase（SubtotalP-> subtotal_p）

如果您不希望开发人员维护您的代码讨厌您并让您感到不满意，我将认真尝试修复所有这些问题。