Python文件排序器运行异常求助：单文件目录正常、多文件目录报错问题排查

我帮你找出了代码里的几个关键问题，还有一些可以优化的地方，让我们一步步来看：

核心问题1：循环只处理第一个文件就终止 #

你在每个if/elif/else分支里都加了break语句，这意味着程序读完目录里的第一个文件，处理完就直接跳出循环了，根本不会处理剩下的文件。这也是为什么只有单个图片时能正常运行——第一个（也是唯一一个）文件刚好匹配图片的条件，处理完就结束了；但如果第一个文件是其他类型，就会触发后续的问题。

核心问题2：else分支的文件路径错误 #

在else块里，你写的是shutil.move(file, "...")，但file只是单纯的文件名，不是完整的文件路径。Python会在当前脚本的运行目录下去找这个文件，自然找不到，所以抛出了FileNotFoundError。其他分支虽然用了f"{path1}\\{file}"是对的，但else分支漏掉了这个路径拼接。

优化后的完整代码 #

我把你的代码重构了一下，解决了上述问题，还增加了一些实用的功能：

import os
import shutil

def moveFile(path):
    # 用字典整理各类文件的后缀和目标路径，后续修改更方便
    file_categories = {
        "images": ((".gif", ".jfif", ".jpg", ".jpeg", ".png"), "C:\\Users\\CLEMENT.LAPORTE\\Pictures\\Pictures\\"),
        "videos": ((".mp4", ".mkv", ".avi"), "C:\\Users\\CLEMENT.LAPORTE\\Videos\\Videos\\"),
        "music": ((".mp3", ".wav", ".m4a"), "C:\\Users\\CLEMENT.LAPORTE\\Music\\Songs\\"),
        "apps": ((".exe",), "C:\\Users\\CLEMENT.LAPORTE\\App\\"),
        "documents": ((".txt", ".docx", ".pptx", ".pdf"), "C:\\Users\\CLEMENT.LAPORTE\\Work\\"),
        "code": ((".py", ".c", ".cpp", ".java", ".js", ".html", ".css"), "C:\\Users\\CLEMENT.LAPORTE\\Code\\")
    }
    default_dir = "C:\\Users\\CLEMENT.LAPORTE\\Other\\"

    # 提前创建所有目标目录，不存在则自动生成，避免目录缺失报错
    for _, (_, dest_path) in file_categories.items():
        os.makedirs(dest_path, exist_ok=True)
    os.makedirs(default_dir, exist_ok=True)

    # 遍历目标路径下的所有条目
    for file in os.listdir(path):
        file_full_path = os.path.join(path, file)
        # 只处理文件，跳过子目录
        if not os.path.isfile(file_full_path):
            continue

        file_moved = False
        # 检查文件属于哪一类别
        for _, (extensions, dest_path) in file_categories.items():
            # 转成小写避免大小写后缀的问题（比如.PNG和.png）
            if file.lower().endswith(extensions):
                try:
                    shutil.move(file_full_path, dest_path)
                    print(f"Moved:\t{file}\t → {dest_path}")
                    file_moved = True
                    break
                except Exception as e:
                    print(f"Failed to move {file}: {str(e)}")
        # 如果不属于任何预设类别，移动到默认目录
        if not file_moved:
            try:
                shutil.move(file_full_path, default_dir)
                print(f"Moved:\t{file}\t → {default_dir}")
            except Exception as e:
                print(f"Failed to move {file}: {str(e)}")

if __name__ == "__main__":
    target_path = input("Enter the path you want to sort: ")
    # 先检查输入的路径是否有效
    if os.path.isdir(target_path):
        moveFile(target_path)
    else:
        print("Error: The specified path does not exist!")

关键修改说明 #

1. 移除了所有break语句：现在循环会处理目录里的每一个文件，而不是只处理第一个
2. 统一使用完整文件路径：用os.path.join拼接路径，比直接写\\更安全，也兼容不同操作系统
3. 整理文件类别为字典：让代码结构更清晰，后续要添加新的文件类型或修改路径时，直接修改字典即可
4. 自动创建目标目录：用os.makedirs(..., exist_ok=True)确保所有目标目录都存在，避免因为目录缺失导致的报错
5. 跳过子目录：只处理文件，不会尝试移动文件夹
6. 异常捕获：单个文件移动失败（比如文件正在被占用）时，会打印错误信息但不会终止整个程序
7. 路径合法性检查：先验证用户输入的路径是否存在，避免无效路径导致的崩溃

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