使用python将文件夹从文件夹列表移动到其他文件夹列表

首先，显然您的使用lambda没有用，glob函数可以执行此过滤。这就是glob真正的作用，所以你基本上是用更多不必要的函数调用来乱扔你的代码，这在性能方面是相当昂贵的。

看看这个例子，和你的一样：

import glob

# Find all .py files
sourcepath= 'C:/my/store/path/*.py'
paths = glob.glob(sourcepath)

# Find files that end with 'codes'
destpath= 'C:/my/store/path/*codes'
paths2 = glob.glob(destpath)

其次，第二个glob函数调用可能会也可能不会返回目录列表以将目录/文件移动到。这使您的代码依赖于 C:/my/store/pathcontains. That is, you must guarantee that 'C:/my/store/path必须仅包含目录而不包含文件的内容，因此glob将仅返回要在shutil.move. 如果用户后来将文件而不是文件夹添加到 C:/my/store/path 恰好以名称“codes”结尾，并且他们没有指定任何扩展名（例如，codes.txt、codes.py...），那么你会在返回的globin列表中找到这个文件paths2。当然，保证一个目录只包含子目录是有问题的，不是一个好主意，根本不是。您可以通过以下方式测试目录os.path.isdir

公告东西，你正在使用lambda的帮助下filter，以过滤掉任何字符串不包含10在你的第一次调用filter，有些东西你可以实现glob自己：

glob.glob('C:/my/store/path/*10*')

现在，任何名称中包含 10 的 C:/my/store/path 文件或子目录都将被收集在glob函数的返回列表中。

第三，zip截断到其参数列表中最短的可迭代对象。换句话说，如果您想将每个路径 in移动paths到 中的每个路径paths2，则需要将 中的len(paths) == len(paths2)每个文件或目录paths都directory移动到 中paths2。

第四，您错过了for循环的分号，并且在shutil.move您的调用中使用i了I. Python 是一种区分大小写的语言，I大写与i小写并不完全相同：

import shutil
for I,j in zip(selected_path,selected_path1)            # missing :
     shutil.move(i, j)                                  # i not I 

更正的代码：

import shutil
for I,j in zip(selected_path,selected_path1)            # missing :
     shutil.move(I, j)                                  # i not I  

据推测，paths2仅包含 C:/my/store/path 目录的子目录，这是编写代码的更好方法，但绝对不是最好的：

import glob

#paths from source folder
sourcepath='C:/my/store/path/*10*'
paths = glob.glob(sourcepath)

#paths from destination folder
destpath='C:/my/store/path/*20*'
paths2 = glob.glob(destpath)

import shutil
for i,j in zip(paths,paths2):
     shutil.move(i, j)

*我上面提到的一些以前的问题仍然适用于这个代码。

现在您已经完成了阅读这个答案的漫长马拉松，您想做什么来改进您的代码？如果您仍然发现一些不明确的地方，我会很乐意提供帮助。

祝你好运 ：）