Django多字段文件上传异常排查：首次上传正常但无法重新上传及删除

一、重新上传不生效的核心原因与修复 #

你当前的视图在处理POST请求时，每次都会创建新的DocumentInfo实例，但通常一个用户应该对应唯一的一条文档记录。这就导致旧记录始终存在，新上传的内容被存到了新实例里，前端显示的还是旧记录，看起来像是没生效。

修改views.py中的POST处理逻辑：

def document_info(request):
    if request.method == 'POST':
        # 获取当前用户的文档记录，不存在则自动创建
        doc_instance, created = DocumentInfo.objects.get_or_create(user=request.user)
        # 传入instance参数，让表单更新已有记录而非新建
        form = DocumentForm(request.POST, request.FILES, instance=doc_instance)
        if form.is_valid():
            form.save()
            return redirect('document')
    # ... 其余代码保持不变

这样用户再次上传时，会直接覆盖已有实例的对应字段值，而不是生成重复记录。

二、删除功能失效的原因与修复 #

你的删除逻辑调用了未定义的delete_item函数，而且目标是删除整个实例，不是单个文件字段。我们需要调整为删除指定字段的文件，并同步清理服务器上的旧文件。

1. 更新视图中的删除逻辑

def document_info(request):
    # ... 前面的POST逻辑 ...
    
    # 处理单个文件字段的删除请求
    if 'delete_field' in request.GET and 'id' in request.GET:
        try:
            doc_instance = DocumentInfo.objects.get(id=request.GET['id'], user=request.user)
            field_name = request.GET['delete_field']
            # 限制可删除的字段，避免非法操作
            allowed_fields = ['certificate', 'id_card', 'service_certificate', 'educational_certificate']
            if field_name not in allowed_fields:
                raise ValueError("无效的字段名称")
            
            file_obj = getattr(doc_instance, field_name)
            if file_obj:
                # 删除服务器上的物理文件（如果存在）
                if file_obj.path.exists():
                    file_obj.delete(save=False)  # save=False避免提前触发保存
                # 将数据库中的字段值设为None
                setattr(doc_instance, field_name, None)
                doc_instance.save()
        except (DocumentInfo.DoesNotExist, ValueError):
            # 处理记录不存在或非法字段的情况
            pass
        return redirect('document')
    
    # ... 其余代码保持不变

2. 模板中添加删除按钮

在documents.html模板里，给每个已上传的文件添加删除链接：

{% for doc in documents %}
    {% if doc.certificate %}
        <div>
            <span>证书文件：</span>
            <a href="{{ doc.certificate.url }}" target="_blank">{{ doc.certificate.name }}</a>
            <a href="?delete_field=certificate&id={{ doc.id }}" onclick="return confirm('确定要删除这个文件吗？')">删除</a>
        </div>
    {% endif %}

    {% if doc.id_card %}
        <div>
            <span>身份证文件：</span>
            <a href="{{ doc.id_card.url }}" target="_blank">{{ doc.id_card.name }}</a>
            <a href="?delete_field=id_card&id={{ doc.id }}" onclick="return confirm('确定要删除这个文件吗？')">删除</a>
        </div>
    {% endif %}

    <!-- 重复上述结构，处理service_certificate和educational_certificate字段 -->
{% endfor %}

三、额外优化：自动删除旧文件 #

Django的FileField在更新字段时，默认不会自动删除服务器上的旧文件，会导致磁盘空间浪费。我们可以用信号来自动清理旧文件：

在models.py中添加以下代码：

from django.db.models.signals import pre_save
from django.dispatch import receiver
import os

@receiver(pre_save, sender=DocumentInfo)
def delete_old_file_on_update(sender, instance, **kwargs):
    # 仅当实例已存在（非新建）时执行
    if instance.pk:
        try:
            old_instance = sender.objects.get(pk=instance.pk)
            # 遍历所有FileField字段
            file_fields = ['certificate', 'id_card', 'service_certificate', 'educational_certificate']
            for field_name in file_fields:
                old_file = getattr(old_instance, field_name)
                new_file = getattr(instance, field_name)
                # 如果旧文件存在且与新文件不同，删除旧文件
                if old_file and old_file != new_file:
                    if os.path.isfile(old_file.path):
                        os.remove(old_file.path)
        except sender.DoesNotExist:
            # 旧实例不存在（比如刚创建的情况），跳过处理
            pass

四、检查表单的关键属性 #

确保你的表单标签包含enctype="multipart/form-data"，否则文件上传会失败：

<form method="post" enctype="multipart/form-data">
    {% csrf_token %}
    {{ form.as_p }}
    <button type="submit">保存文件</button>
</form>

完成以上修改后，重新上传和删除功能应该就能正常工作了。

内容的提问来源于stack exchange，提问作者Sam