[已解决]30课课后练习第1题

一二二四四

请帮忙看下下面这段代码问题在哪里，会报错。谢谢。

import os
def file_size():
    files = dict()
    all_files = os.listdir(os.curdir)
    for each_file in all_files:
        if os.path.isfile(each_file):
            files[each_file] = os.path.getsize(each_file)
        else:
            file_size()
            os.chdir(os.pardir)
    for each in files:
        print('%s 【%d】' %(each,files(each)))

file_size()

最佳答案

月排行榜 / 总排行榜

hldh214

2016-3-7 16:45:30

一二二四四 发表于 2016-3-7 16:18
当对象是文件夹的时候，用的递归

好吧, 仔细分析了你的代码, 存在这么几个问题:
1. 定义dict的时候需要在函数外部, 不然递归调用的时候会清空dict里面的内容
2. 取字典里面的值是用中括号, 像这样

1. files[each]

复制代码

而不是用小括号
3. else里面的递归调用存在逻辑错误, 这个错误和函数整体的参数设计有关, 递归函数一般都需要参数的, 这里定义一个目录名作为形参, 并且赋初值为当前目录, 把进入目录的操作移到函数开头, 我贴上我的代码:

1. import os
   
2. 
   
3. files = dict()
   
4. 
   
5. 
   
6. def file_size(dirName='.'):
   
7.     global files
   
8.     os.chdir(dirName)
   
9.     all_files = os.listdir(os.curdir)
   
10.     for each_file in all_files:
    
11.         if os.path.isfile(each_file):
    
12.             files[each_file] = os.path.getsize(each_file)
    
13.         else:
    
14.             file_size(each_file)
    
15.     os.chdir('..')
    
16.     for each in files:
    
17.         print('%s 【%d】' % (each, files[each]))
    
18. 
    
19. 
    
20. file_size()
    

复制代码

跳转到最佳答案楼层

hldh214

没懂为什么else里用递归

一二二四四

hldh214 发表于 2016-3-7 16:05
没懂为什么else里用递归

当对象是文件夹的时候，用的递归

hldh214

一二二四四 发表于 2016-3-7 16:18
当对象是文件夹的时候，用的递归

好吧, 仔细分析了你的代码, 存在这么几个问题:
1. 定义dict的时候需要在函数外部, 不然递归调用的时候会清空dict里面的内容
2. 取字典里面的值是用中括号, 像这样

1. files[each]

复制代码

而不是用小括号
3. else里面的递归调用存在逻辑错误, 这个错误和函数整体的参数设计有关, 递归函数一般都需要参数的, 这里定义一个目录名作为形参, 并且赋初值为当前目录, 把进入目录的操作移到函数开头, 我贴上我的代码:

1. import os
   
2. 
   
3. files = dict()
   
4. 
   
5. 
   
6. def file_size(dirName='.'):
   
7.     global files
   
8.     os.chdir(dirName)
   
9.     all_files = os.listdir(os.curdir)
   
10.     for each_file in all_files:
    
11.         if os.path.isfile(each_file):
    
12.             files[each_file] = os.path.getsize(each_file)
    
13.         else:
    
14.             file_size(each_file)
    
15.     os.chdir('..')
    
16.     for each in files:
    
17.         print('%s 【%d】' % (each, files[each]))
    
18. 
    
19. 
    
20. file_size()
    

复制代码

一二二四四

hldh214 发表于 2016-3-7 16:45
好吧, 仔细分析了你的代码, 存在这么几个问题:
1. 定义dict的时候需要在函数外部, 不然递归调用的时候会 ...

感谢感谢，可是为什么会把所有的文件都统计两遍呢？

hldh214

一二二四四 发表于 2016-3-7 17:44
感谢感谢，可是为什么会把所有的文件都统计两遍呢？

我这边没出现这个问题, 倒是遇到不同文件夹中有重名的文件只会统计一次