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

爱笑的无赖

以下是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

直接定义成参数：

import os
import easygui as g

def count_code(dir1, count_files, count_lines):
    os.chdir(dir1)
    files = os.listdir(os.curdir)

    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_files, count_lines = count_code(eachfile, count_files, count_lines)
            os.chdir(os.pardir)
    return count_files,count_lines

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

跳转到最佳答案楼层

Twilight6

直接定义成参数：

import os
import easygui as g

def count_code(dir1, count_files, count_lines):
    os.chdir(dir1)
    files = os.listdir(os.curdir)

    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_files, count_lines = count_code(eachfile, count_files, count_lines)
            os.chdir(os.pardir)
    return count_files,count_lines

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

爱笑的无赖

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

import easygui as g
import os




def count_code(dir1,count_files,count_lines):
    os.chdir(dir1)
    files = os.listdir(os.curdir)
    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_files, count_lines = count_code(eachfile,count_files,count_lines) #为什么要重新赋值
            os.chdir(os.pardir)
    return count_files,count_lines


dir1 = g.diropenbox(msg = '请选择您的代码库:')
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
...

原来如此。。。