-1

这是我目前正在处理的代码，但我的老板对它的最后一部分不太满意。有人能帮我吗？下面的问题已标记。提前致谢！

#!/usr/bin/python

def codeNameDict(lst, cnDict):
    key = int(lst[0])
    value = lst[1]
    cnDict[key] = value
    return cnDict

def realNameDict(lst, rnDict):
    key = int(lst[0])
    value = lst[2].strip()
    rnDict[key] = value
    return rnDict

def getCodeKey(keys):
    import random
    index = random.choice(keys)
    return index

def searchKey(cnDict,value):
    for x in cnDict.keys():
      if(value == cnDict.get(x)):
        return x
        break;

def namegen(cnDict):
    index1 = getCodeKey(cnDict.keys())
    index2 = getCodeKey(cnDict.keys())

    if(index1 == index2):
     while(index1 == index2):
      index2 = getCodeKey(cnDict.keys())
      indexgen = [index1,index2]

    else:
      indexgen = [index1,index2]
    return indexgen

cnDict = {}
rnDict = {}

f = open("namefile.txt")
try:
    for line in f:
        words = line.split(":")
        cnDict = codeNameDict(words,cnDict)
        rnDict = realNameDict(words,rnDict)
finally:
    f.close()


# PROBLEMATIC SECTION #
for x in range(10):
    indexgen = namegen(cnDict)
    key1 = indexgen[0]
    key2 = indexgen[1]
    #print indexgen
    #print key1
    #print key2
    pokemon1 = cnDict[key1]
    pokemon2 = cnDict[key2]
    realname1 = rnDict[key1]
    realname2 = rnDict[key2]

    print "Partners-> %s:%s" % (pokemon1, pokemon2)
    print "RealNames-> %s:%s" % (realname1, realname2)

    del cnDict[key1]
    del cnDict[key2]
4

2 回答 2

0

你可以制作线条

key1 = indexgen[0]
key2 = indexgen[1]

与：

key1, key2 = indexgen[0], indexgen[1].

与pokemon1, pokemon2 和相同realname1, realname2。我也不会使用魔法值10，而是给它一个名字。

于 2012-08-31T08:27:25.943 回答
0

您的代码有几个改进。让我们举个例子：

def getCodeKey(keys):
import random
index = random.choice(keys)
return index

这实际上是返回random.choice将返回的内容。我建议将import语句移动到文件的开头，并使用random.choice而不是getCodeKey（再次，它是同一件事）

其次，在 中searchKey，不需要break在 return 之后，因为 return 已经从函数返回。但是，如果搜索的值不在字典中，您应该考虑采取一些措施......

第三，

if(index1 == index2):
 while(index1 == index2):
  index2 = getCodeKey(cnDict.keys())
  indexgen = [index1,index2]

else:
  indexgen = [index1,index2]
return indexgen

if没有必要，因为如果它不是真的，while循环将不会被执行。然后您也不需要else，请注意您在if和else语句中为 indexgen bot 设置值。另外 - 没有必要为你将立即返回的值创建变量。我会将其重写为

 while(index1 == index2):
  index2 = getCodeKey(cnDict.keys())

return [index1,index2]

至于这range部分，我唯一看到的是halex建议的，即将所有双重语句放在一行中。

key1, key2 = indexgen[0], indexgen[1]

不过我觉得变化不大

这不完全是你所要求的，但正如你所说 - 你正在学习 python，所以我认为这可能至少有点用

于 2012-08-31T08:36:48.203 回答