[已解决]文件遍历与递归的一个问题，请大神给予指导

jasonli_whu

自己写了一个递归函数，希望将一个文件夹中的所有文件取出遍历出来，放在一个列表中，请各位大神看看，问题出在哪里了
import os

a = []
def find_files(file):

    if os.path.isfile(file):
        a.append(file)
        return a
    else:
        for each in os.listdir(file):
            find_files(os.path.join(file, each))

find_files()是主函数，判断，若不是文件名，是路径名，就递归调用find_files函数
问题：使用函数的时候，print（find_files("随便输入一个路径")）打印出的是None，而不是我想的包含所有文件名的列表

最佳答案

月排行榜 / 总排行榜

int2str

2017-3-7 18:17:25

在程序中加入必要的提示字符，有助了解函数调用过程！！！

1. import os
   
2. 
   
3. def find_files(file):
   
4.         if os.path.isfile(file):
   
5.             a.append(file)
   
6.         else:
   
7.                 print(file, '是文件夹')
   
8.                 for each in os.listdir(file):
   
9.                         print('内容：', each)
   
10.                         find_files(os.path.join(file, each))
    
11.         
    
12. 
    
13. file = 'F:\\123'
    
14. global a
    
15. a = []
    
16. find_files(file)
    
17. print(a)
    

复制代码

跳转到最佳答案楼层

jackche0214

学习到了，看到楼主遇到的问题，和大家回复的，学习到了两点：
1,在变量的调用的时候，如果函数中调用了外部的变量，那么应该把变量申明为全局变量。
2，在学习阶段，多把结果打印出来添加相应提示符，有助于了解函数调用过程。。

jasonli_whu

int2str 发表于 2017-3-7 21:41
在Pycharm可以打印出来呀....是不是不支持中文字符呢？开头加# -*-coding:gbk-*-

a.append()已经修改了a ...

多谢指点

jasonli_whu

591821661 发表于 2017-3-7 19:44
贴主，告诉你一个强大的，os.walk() 省去递归了

多谢多谢，我来看看

int2str

在Pycharm可以打印出来呀....是不是不支持中文字符呢？开头加# -*-coding:gbk-*-

a.append()已经修改了a的值，没有必要调用return（个人看法）

591821661

jasonli_whu 发表于 2017-3-7 18:48
还有请问，为什么您的if代码后不将a 的结果return（像我的代码段中的一样）

贴主，告诉你一个强大的，os.walk() 省去递归了

jasonli_whu

int2str 发表于 2017-3-7 18:17
在程序中加入必要的提示字符，有助了解函数调用过程！！！

还有请问，为什么您的if代码后不将a 的结果return（像我的代码段中的一样）

jasonli_whu

int2str 发表于 2017-3-7 18:17
在程序中加入必要的提示字符，有助了解函数调用过程！！！

请问，为什么，我用这个正确的代码在pycharm 打印不出结果呢，但是在python自带的shell中可以得到结果呢？

int2str

在程序中加入必要的提示字符，有助了解函数调用过程！！！

1. import os
   
2. 
   
3. def find_files(file):
   
4.         if os.path.isfile(file):
   
5.             a.append(file)
   
6.         else:
   
7.                 print(file, '是文件夹')
   
8.                 for each in os.listdir(file):
   
9.                         print('内容：', each)
   
10.                         find_files(os.path.join(file, each))
    
11.         
    
12. 
    
13. file = 'F:\\123'
    
14. global a
    
15. a = []
    
16. find_files(file)
    
17. print(a)
    

复制代码

lumber2388779

jasonli_whu 发表于 2017-3-7 18:06
改成这样打印的还是None
a = []
def find_files(file):

a = []

函数中声明下global a 如果有问题在函数外部修改成global a=[]

int2str

你每次递归把a = []，之前的路径都没保存下来。。。

jasonli_whu

lumber2388779 发表于 2017-3-7 17:42
你的问题是不是你的a到最后并没有保存你输入文件夹下所有文件
如果是的话是因为你的a[]是局部变量 每次递 ...

改成这样打印的还是None
a = []
def find_files(file):
    if os.path.isfile(file):
        a.append(file)
        return a
    else:
        for each in os.listdir(file):
            find_files(os.path.join(file, each))

jasonli_whu

lumber2388779 发表于 2017-3-7 17:42
你的问题是不是你的a到最后并没有保存你输入文件夹下所有文件
如果是的话是因为你的a[]是局部变量 每次递 ...

自己怎么看都不觉得有问题，你一说我就大明白了，我先改改看

lumber2388779

你的问题是不是你的a到最后并没有保存你输入文件夹下所有文件
如果是的话是因为你的a[]是局部变量 每次递归都重新创建一个a[]
只要把a[]作为全局变量就可以了

jasonli_whu

newu 发表于 2017-3-7 17:35
可以明确下问题吗？

请在看一下，我修改了

newu

可以明确下问题吗？