如何随机重命名周邮件列表文件并移动，避免覆盖？

问题根源分析 #

你的代码存在两个关键问题：

1. 变量名错误：random.choice(file_list)中的file_list应为files_list，这会直接触发NameError导致程序中途停止。
2. 重复命名逻辑错误：使用random.choice()随机选取目标文件名，会出现多个源文件被重命名为同一个目标名的情况，进而导致文件覆盖。

修正后的代码 #

import os
import random
import shutil

def shuffle_and_move_files(source_folder, destination_folder):
    if not os.path.exists(source_folder):
        print(f"Source folder '{source_folder}' does not exist.")
        return

    if not os.path.exists(destination_folder):
        os.makedirs(destination_folder)

    # 仅保留源文件夹中的文件，过滤子文件夹
    files_list = [f for f in os.listdir(source_folder) if os.path.isfile(os.path.join(source_folder, f))]
    # 复制文件名列表并打乱，生成唯一的目标文件名映射
    target_names = files_list.copy()
    random.shuffle(target_names)

    # 一一对应源文件和随机后的目标文件，确保无重复
    for source_name, target_name in zip(files_list, target_names):
        source_path = os.path.join(source_folder, source_name)
        destination_path = os.path.join(destination_folder, target_name)
        
        # 容错处理：避免目标文件夹已有同名文件导致的错误
        if os.path.exists(destination_path):
            print(f"Warning: Target file '{target_name}' already exists, skipping.")
            continue
            
        shutil.move(source_path, destination_path)
        print(f"Moved '{source_name}' to '{target_name}'")

核心修正说明 #

• 提前过滤非文件项，避免误处理文件夹。
• 通过复制并打乱文件名列表，再用zip做一一映射，保证每个目标文件名只被使用一次，彻底解决覆盖问题。
• 增加目标文件存在的判断，避免因目标文件夹已有同名文件导致程序崩溃。
• 移除冗余的random.choice()调用，逻辑更简洁高效。

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