这段代码哪里错了[关闭]

Question

f=open('new.txt')
var=f.read()
var=var.lower()
var = var.replace(';','')
var = var.replace(',','')
var = var.replace('.','')
x=var.split()
dict={}
dictList=[]
dict2={}
for i in xrange(0,len(x)-1):

    if dict.items():
        for k in dict.items():
            if x[i] not in k:

                dict[x[i]]=dict2
                dict2[x[i+1]]=1

            else:
                if dict[x[i]].keys():
                    if x[i+1] in dict[x[i]].keys():
                        dict[x[i]][x[i+1]]+=1
                    else:
                        dict[x[i]][x[i+1]]=1
                else:
                    dict[x[i]][x[i+1]]=1

    else:

        dict[x[i]]=dict2
        dict2[x[i+1]]=1

print dict

此代码必须打印每个单词的二元计数。我很确定我不恰当地实例化了 dict2 。我确定这是一个天真的错误，但我不知何故错过了它。有什么建议？ :)

Answer 1

如果我必须重构这段代码，我会看：

• 变量名。像dict 这样的名字不会告诉编码器/维护者。 dict 和 dict2 有什么区别？你今天可能知道，但明天……也许不会
• 为/if/for/if/if 嵌套。很难遵循所有路径并确保没有可能的错误。我会把它分解成单独的函数，可以独立调试和测试。
• 数据结构。它们通常是程序的核心。但在这里，它们是什么？这对我来说并不明显，这意味着代码做什么/为什么做任何事情并不明显
• 错误处理。什么错误是可能的/预期的？哪些会被处理/忽略？