Django ImageField上传文件时文件夹重复嵌套问题排查

你遇到的文件夹嵌套问题，核心原因是Profile的save方法被反复触发，且每次处理时都会重复叠加图片路径：

1. 第一次保存Profile时，Django会结合upload_to="profile_pics/"和图片原始文件名，生成profile_pics/xxx.jpg作为image.name。
2. 你的post_save信号绑定了save_profile函数，每次User实例保存（哪怕只是修改User的其他字段），都会调用instance.profile.save()，再次触发Profile的save方法。
3. 第二次执行save时，当前的image.name已经是profile_pics/xxx.jpg，而make_thumbnail函数直接用这个值作为新File的name。Django会再次把这个name和upload_to="profile_pics/"拼接，变成profile_pics/profile_pics/xxx.jpg。
4. 多次触发后，就会形成多层嵌套的目录结构。

至于博客文章没有这个问题，大概率是因为博客的save逻辑只在创建文章/上传图片时执行一次，没有像Profile这样被信号反复触发，不会重复处理同一个图片文件。

这里提供几个更优雅的解决思路，比你临时用uuid重命名的方案更贴合Django的逻辑：

方案一：仅处理新上传/修改的图片 #

利用Django文件字段的_committed属性，判断图片是否是新上传（未提交到数据库）的，只在这种情况下生成缩略图，避免每次save都重复处理：

def save(self, *args, **kwargs):
    # 只有当图片是新上传或被修改时才生成缩略图
    if self.image and not self.image._committed:
        self.image = make_thumbnail(self.image, size=(200, 200))
    super().save(*args, **kwargs)

方案二：修改缩略图函数，只保留文件名 #

在make_thumbnail中，只提取image.name的纯文件名部分，去掉前面的路径，这样即使多次处理，也不会叠加路径：

from io import BytesIO
from django.core.files import File
from PIL import Image
import os

def make_thumbnail(image, size=(600, 600)):
    im = Image.open(image)
    # 只保留文件名，丢弃前面的路径
    filename = os.path.basename(image.name)
    
    if im.format == "JPEG":
        im.convert("RGB")
        im.thumbnail(size)
        thumb_io = BytesIO()
        im.save(thumb_io, "JPEG", quality=85)
        image = File(thumb_io, name=filename)
    else:
        im.convert("RGBA")
        im.thumbnail(size)
        thumb_io = BytesIO()
        im.save(thumb_io, "PNG", quality=85)
        image = File(thumb_io, name=filename)
    return image

这个方案不需要修改Profile的save方法，直接从根源避免路径叠加。

方案三：优化信号逻辑，减少不必要的save调用 #

你的save_profile信号每次User保存都会强制保存Profile，其实很多时候Profile并没有修改。可以优化信号逻辑，只在Profile需要保存时执行：

@receiver(post_save, sender=User)
def save_profile(sender, instance, **kwargs):
    try:
        profile = instance.profile
        # 只有当Profile是新建的，或者有字段被修改时才保存
        if profile._state.adding or profile._changed_fields:
            profile.save()
    except Profile.DoesNotExist:
        # 如果Profile不存在，调用创建函数
        create_profile(sender, instance, created=True, **kwargs)

你用uuid生成新文件名的方法是有效的，效率上也没问题——uuid生成非常快，不会成为性能瓶颈。唯一的小缺点是会丢失原始文件名，如果你不需要保留原始文件名，这个方案完全可以用。

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