Python文件归档功能代码可行性验证及简化实现方案问询

首先，你的代码存在几个关键问题，导致它无法完全实现你想要的剪切功能：

• 用了shutil.copy而非shutil.move：copy是复制文件，而你需要的是剪切（移动）操作。
• try-except逻辑错误：只有当Archive文件夹已存在时才执行复制，新建文件夹后反而没有执行移动操作，完全搞反了流程。
• 不必要的os.chdir：切换工作目录会影响后续代码的路径解析，完全可以避免。
• 路径拼接方式不安全：用字符串拼接路径容易出现跨平台问题（比如Windows用\而不是/），也容易出错。
• 文件判断不准确：仅通过p.suffix判断是否是文件不够严谨——有些目录名也可能带点（比如my.dir），应该实际检查路径对应的实体是否是文件。

修正后的可行代码 #

我们可以结合pathlib（更现代的路径处理方式）和shutil.move来修复这些问题，同时保持逻辑清晰：

import os
from pathlib import Path
from shutil import move

def archive_file(file_path):
    # 转换为Path对象，方便后续操作
    file = Path(file_path)
    
    # 先检查是否是有效文件
    if not file.is_file():
        print("输入的路径不是一个有效文件")
        return
    
    # 定义Archive文件夹路径
    archive_dir = file.parent / "Archive"
    
    # 创建Archive文件夹（已存在则忽略）
    archive_dir.mkdir(exist_ok=True)
    
    # 移动文件到Archive文件夹
    target_path = archive_dir / file.name
    move(file, target_path)
    print(f"文件已成功移动到：{target_path}")

# 测试调用
path = 'C:/Users/Lampard/Desktop/Folder1/File.py'
archive_file(path)

更简洁的实现（甚至单行函数） #

如果追求简洁，我们可以利用pathlib的链式调用和mkdir的exist_ok参数，把逻辑压缩得更紧凑：

简洁版函数 #

from pathlib import Path
from shutil import move

def archive_file(file_path):
    f = Path(file_path)
    if f.is_file():
        (f.parent / "Archive").mkdir(exist_ok=True)
        move(f, f.parent / "Archive" / f.name)

极简单行版本（适合一次性调用，可读性稍差） #

from pathlib import Path
from shutil import move

f = Path('C:/Users/Lampard/Desktop/Folder1/File.py'); f.is_file() and (f.parent/"Archive").mkdir(exist_ok=True) and move(f, f.parent/"Archive"/f.name)

关键优化点说明 #

• 使用Path对象处理路径：自动适配Windows/Linux的路径分隔符，路径拼接更安全（用/运算符）。
• mkdir(exist_ok=True)：替代原来的try-except，一行代码搞定文件夹创建（已存在则不报错）。
• shutil.move：实现真正的剪切（移动）操作，符合你的需求。
• 严谨的文件校验：is_file()确保路径指向的是实际存在的文件，而不是目录或不存在的路径。

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