[已解决]《三十五讲课后作业第4题》对于我所写代码的疑惑

我总得懂点什么

1. import os
   
2. 
   
3. import easygui as g
   
4. 
   
5. def search(start_path):
   
6. 
   
7.     count = 0   #统计文件数量
   
8. 
   
9.     count1 = 0   #统计行数
   
10.    
    
11.     os.chdir(start_path)
    
12. 
    
13.     each_file = os.listdir(os.curdir)
    
14. 
    
15.     for each in each_file:
    
16. 
    
17.         if os.path.isfile(each):
    
18. 
    
19.             extension = os.path.splitext(each)[1]
    
20. 
    
21.             if extension == '.py':
    
22. 
    
23.                 count += 1
    
24. 
    
25.                 try:
    
26. 
    
27.                     with open(each) as f:
    
28. 
    
29.                         for each_line in f:
    
30. 
    
31.                             count1 += 1
    
32.                            
    
33.                 except (PermissionError, UnicodeDecodeError):
    
34. 
    
35.                     pass
    
36. 
    
37.         if os.path.isdir(each):
    
38. 
    
39.             search(each)
    
40. 
    
41.             os.chdir(os.pardir)
    
42. 
    
43.     global a
    
44. 
    
45.     a = (count,count1)
    
46. 
    
47.     return a
    
48. 
    
49. start_path = g.enterbox('请输入查找的初始目录：')
    
50. 
    
51. search(start_path)
    
52. 
    
53. count2 = a[1] /100000 * 100
    
54. 
    
55. msg = '文件有%d个，一共%d行，已完成%d '%(a[0],a[1],count2)
    
56. 
    
57. g.msgbox(msg)
    
58. 
    
59.             
    

复制代码

代码运行过后，并没有找到任何一个符合的文件，但我打开查找的目录中却有。是变量的作用域搞错了嘛？哪该如何避免？如果不是，请大佬指正，阿里嘎多！

最佳答案

月排行榜 / 总排行榜

Daniel_Zhang

2021-2-9 12:55:56

不知道是因为什么，没仔细看，有可能是作用域问题，你自己看看喔，该你改好了

1. import os
   
2. 
   
3. import easygui as g
   
4. 
   
5. count = 0   #统计文件数量
   
6. 
   
7. count1 = 0   #统计行数
   
8. 
   
9. 
   
10. def search(start_path):
    
11. 
    
12.     global count, count1
    
13.    
    
14.     os.chdir(start_path)
    
15. 
    
16.     each_file = os.listdir(os.curdir)
    
17. 
    
18.     for each in each_file:
    
19. 
    
20.         if os.path.isfile(each):
    
21. 
    
22.             extension = os.path.splitext(each)[1]
    
23. 
    
24.             if extension == '.py':
    
25.                 print(each)
    
26.                 count += 1
    
27. 
    
28.                 try:
    
29. 
    
30.                     with open(each) as f:
    
31. 
    
32.                         for each_line in f:
    
33. 
    
34.                             count1 += 1
    
35.                            
    
36.                 except (PermissionError, UnicodeDecodeError):
    
37. 
    
38.                     pass
    
39. 
    
40.         if os.path.isdir(each):
    
41. 
    
42.             search(each)
    
43. 
    
44.             os.chdir(os.pardir)
    
45. 
    
46. start_path = g.enterbox('请输入查找的初始目录：')
    
47. 
    
48. search(start_path)
    
49. 
    
50. count2 = count1 /100000 * 100
    
51. 
    
52. msg = '文件有%d个，一共%d行，已完成 %d%%'%(count,count1,count2)
    
53. 
    
54. g.msgbox(msg)

复制代码

跳转到最佳答案楼层

Daniel_Zhang

不知道是因为什么，没仔细看，有可能是作用域问题，你自己看看喔，该你改好了

1. import os
   
2. 
   
3. import easygui as g
   
4. 
   
5. count = 0   #统计文件数量
   
6. 
   
7. count1 = 0   #统计行数
   
8. 
   
9. 
   
10. def search(start_path):
    
11. 
    
12.     global count, count1
    
13.    
    
14.     os.chdir(start_path)
    
15. 
    
16.     each_file = os.listdir(os.curdir)
    
17. 
    
18.     for each in each_file:
    
19. 
    
20.         if os.path.isfile(each):
    
21. 
    
22.             extension = os.path.splitext(each)[1]
    
23. 
    
24.             if extension == '.py':
    
25.                 print(each)
    
26.                 count += 1
    
27. 
    
28.                 try:
    
29. 
    
30.                     with open(each) as f:
    
31. 
    
32.                         for each_line in f:
    
33. 
    
34.                             count1 += 1
    
35.                            
    
36.                 except (PermissionError, UnicodeDecodeError):
    
37. 
    
38.                     pass
    
39. 
    
40.         if os.path.isdir(each):
    
41. 
    
42.             search(each)
    
43. 
    
44.             os.chdir(os.pardir)
    
45. 
    
46. start_path = g.enterbox('请输入查找的初始目录：')
    
47. 
    
48. search(start_path)
    
49. 
    
50. count2 = count1 /100000 * 100
    
51. 
    
52. msg = '文件有%d个，一共%d行，已完成 %d%%'%(count,count1,count2)
    
53. 
    
54. g.msgbox(msg)

复制代码

Daniel_Zhang

有几个问题，函数里面写了 return 但是你调用的时候并没有赋值给任何变量

count count1 似乎在 search 函数里面会被更新，serach 迭代，把这两个变量作为全局变量就可以了

Daniel_Zhang

另外，其实不需要迭代的，可以使用 os.walk(path) 让程序自己搜索所有的子文件夹里面的内容