[已解决]关于闭包的小问题 求解答

爱笑的无赖

以下是35讲最后一题的一个简化版的答案。
我想在文件夹中递归找出所有'.py'的文件，计算总共的文件数和代码行数
import easygui as g
import os




def count_code(dir1):
    os.chdir(dir1)
    files = os.listdir(os.curdir)
    global count_files
    global count_lines
    for eachfile in files:
        if os.path.isfile(eachfile) and os.path.splitext(eachfile)[1] == '.py':
            f = open(eachfile,'r')
            count_files +=1
            for eachlines in f:
                count_lines += 1
            f.close()
        elif os.path.isdir(eachfile):
            count_code(eachfile)
            os.chdir(os.pardir)
    return count_files,count_lines

count_files = 0
count_lines = 0
dir1 = g.diropenbox(msg = '请选择您的代码库:')
print(count_code(dir1))

这是我的代码。 现在的问题是， count_files放在全局域代码是有效的。
但是介于小甲鱼在视频中说，建议大家将全局域和内部域的参数分开。
所以我应该怎么改？
直接在内部域定义count_files 和 count_lines 每次递归的时候， 他们都会被重置呀！

最佳答案

月排行榜 / 总排行榜

Twilight6

2022-5-12 17:39:48

直接定义成参数：

1. import os
   
2. import easygui as g
   
3. 
   
4. def count_code(dir1, count_files, count_lines):
   
5.     os.chdir(dir1)
   
6.     files = os.listdir(os.curdir)
   
7. 
   
8.     for eachfile in files:
   
9.         if os.path.isfile(eachfile) and os.path.splitext(eachfile)[1] == '.py':
   
10.             f = open(eachfile,'r')
    
11.             count_files +=1
    
12.             for eachlines in f:
    
13.                 count_lines += 1
    
14.             f.close()
    
15.         elif os.path.isdir(eachfile):
    
16.             count_files, count_lines = count_code(eachfile, count_files, count_lines)
    
17.             os.chdir(os.pardir)
    
18.     return count_files,count_lines
    
19. 
    
20. dir1 = g.diropenbox(msg = '请选择您的代码库:')
    
21. print(count_code(dir1, 0, 0))

复制代码

跳转到最佳答案楼层

Twilight6

直接定义成参数：

1. import os
   
2. import easygui as g
   
3. 
   
4. def count_code(dir1, count_files, count_lines):
   
5.     os.chdir(dir1)
   
6.     files = os.listdir(os.curdir)
   
7. 
   
8.     for eachfile in files:
   
9.         if os.path.isfile(eachfile) and os.path.splitext(eachfile)[1] == '.py':
   
10.             f = open(eachfile,'r')
    
11.             count_files +=1
    
12.             for eachlines in f:
    
13.                 count_lines += 1
    
14.             f.close()
    
15.         elif os.path.isdir(eachfile):
    
16.             count_files, count_lines = count_code(eachfile, count_files, count_lines)
    
17.             os.chdir(os.pardir)
    
18.     return count_files,count_lines
    
19. 
    
20. dir1 = g.diropenbox(msg = '请选择您的代码库:')
    
21. print(count_code(dir1, 0, 0))

复制代码

爱笑的无赖

Twilight6 发表于 2022-5-12 17:39
直接定义成参数：

1. import easygui as g
   
2. import os
   
3. 
   
4. 
   
5. 
   
6. 
   
7. def count_code(dir1,count_files,count_lines):
   
8.     os.chdir(dir1)
   
9.     files = os.listdir(os.curdir)
   
10.     for eachfile in files:
    
11.         if os.path.isfile(eachfile) and os.path.splitext(eachfile)[1] == '.py':
    
12.             f = open(eachfile,'r')
    
13.             count_files +=1
    
14.             for eachlines in f:
    
15.                 count_lines += 1
    
16.             f.close()
    
17.         elif os.path.isdir(eachfile):
    
18.             count_files, count_lines = count_code(eachfile,count_files,count_lines) #为什么要重新赋值
    
19.             os.chdir(os.pardir)
    
20.     return count_files,count_lines
    
21. 
    
22. 
    
23. dir1 = g.diropenbox(msg = '请选择您的代码库:')
    
24. print(count_code(dir1,0,0))
    

复制代码

版主大大！问一个感觉很愚蠢的问题。
上面用#标出来的那行代码 为什么要给count_files 和count_lines 重新赋值。  
我试了一下， 不赋值的话， 好像他们会重新被赋值为0。
为啥会重新被赋值为零呢？

Twilight6

爱笑的无赖 发表于 2022-5-12 19:05
版主大大！问一个感觉很愚蠢的问题。
上面用#标出来的那行代码 为什么要给count_files 和count_l ...

因为递归过程中会将 count_files 和  count_lines 进行计算并返回，如果不接收返回后的 count_files

count_lines 那么相当于只算 过了一遍的 for eachfile in files 计算出来的结果

爱笑的无赖

Twilight6 发表于 2022-5-12 19:10
因为递归过程中会将 count_files 和  count_lines 进行计算并返回，如果不接收返回后的 count_files
...

原来如此。。。