我知道这篇文章底部的代码的逻辑/语法是关闭的,但是我很难弄清楚如何编写它以获得所需的结果。第一部分创建了这个字典:
sco = {'human + big': 0, 'big + loud': 0, 'big + human': 0, 'human + loud': 0, 'loud + big': 0, 'loud + human': 0}
然后,我的目的是遍历字典中的每个项目" cnt"一次用于x,然后第二次循环遍历字典x每次该项具有与(cnt [x] [1])相同的值,但是不同的键(cnt [x] [0]),创建一个字符串,将匹配格式"%s +%s"在字典中找到" sco。"然后,它应该在sco中找到与分配给变量dnt的键匹配的键,并将sco中该键的值递增1。
# -*- coding: utf-8 -*-
import itertools
sho = ('human', 'loud', 'big')
sco = {}
for a,b in itertools.permutations(sho, 2):
sco["{0} + {1}".format(a,b)] = 0
cnt = [('human', 'ron g.'), ('loud', 'ron g.'), ('big', 'kim p.'), ('human', 'kim p.'), ('loud', 'brian a.'), ('human', 'linda m.'), ('loud', 'linda m.')]
for x in cnt:
upd = cnt[x][0]
who = cnt[x][1]
for x in cnt:
if cnt[x][0] != upd and cnt[x][1] == who:
cpg = cnt[x][0]
dnt = '%s + %s' % (upd, cpg)
for i in sco:
if sco[i][0] == dnt:
sco[i][1] += sco[i][1]
print sco
目前,打印sco不会导致任何值发生变化。代码的期望结果是:
{'human + big': 1, 'big + loud': 0, 'big + human': 1, 'human + loud': 2, 'loud + big': 0, 'loud + human': 2}
非常感谢任何帮助!
修改后的代码如下:
# -*- coding: utf-8 -*-
import itertools
sho = ('human', 'loud', 'big')
sco = {}
for a,b in itertools.permutations(sho, 2):
sco["{0} + {1}".format(a,b)] = 0
cnt = [('human', 'ron g.'), ('loud', 'ron g.'), ('big', 'kim p.'), ('human', 'kim p.'), ('loud', 'brian a.'), ('human', 'linda m.'), ('loud', 'linda m.')]
for x in cnt:
upd = cnt[0]
who = cnt[1]
for x in cnt:
if cnt[0] != upd and cnt[1] == who:
cpg = cnt[0]
dnt = '%s + %s' % (upd, cpg)
sco[dnt] += 1
print sco
以下代码符合我的意图。谢谢@dermen& @abarnert:
# -*- coding: utf-8 -*-
import itertools
sho = ('human', 'loud', 'big')
sco = {}
for a,b in itertools.permutations(sho, 2):
sco["{0} + {1}".format(a,b)] = 0
cnt = [('human', 'ron g.'), ('loud', 'ron g.'), ('big', 'kim p.'), ('human', 'kim p.'), ('loud', 'brian a.'), ('human', 'linda m.'), ('loud', 'linda m.')]
for x in cnt:
upd = x[0]
who = x[1]
for x in cnt:
if x[0] != upd and x[1] == who:
cpg = x[0]
dnt = '%s + %s' % (upd, cpg)
sco[dnt] += 1
print sco
答案 0 :(得分:1)
实际问题是,不是在值中加1,而是将它们加倍:
sco[i][1] += sco[i][1]
无论你多少次加0到0,它仍然是0。
要添加1,只需使用+= 1
。
但是你有另一个问题,至少在你发布的代码中。每个sco[i]
值只是一个数字,而不是两个事物的列表,其中第二个是数字。所以你真正想要的是:
sco[i] += 1
同样,密钥不是sco[i][0]
,它们只是i
。
你有第三个问题。这个实际上并没有打破你的代码,只是让它变得更复杂,更难理解,更慢......但这仍然值得修复。
字典的全部意义在于您不必迭代它们来搜索密钥,您只需直接查找密钥即可。所以,而不是:
for i in sco:
if i == dnt:
sco[i] += 1
......就这样做:
sco[dnt] += 1
它更简单,更难以出错,同时更有效率。
我不能保证你的代码中没有其他错误,但你肯定需要修复这三个错误,除了其他任何错误之外,我很确定第一个是恰好是导致您询问的具体问题。
答案 1 :(得分:0)
除了@abarnert回复之外,词典的行为如下:
>>> D1 = { 'thing1': [1,2,3 ], 'thing2': ( 1, 2,3), 'thing3':'123' }
>>> D1['thing1']
#[1, 2, 3]
>>> D1['thing2']
#(1, 2, 3)
>>> D1['thing3']
#'123'
>>> thing4 = 1
>>> D2 = { thing4:D1 }
>>> D2[thing4]
#{'thing1': [1, 2, 3], 'thing2': (1, 2, 3), 'thing3': '123'}
如果要迭代字典的键,请使用
>>> D1.keys()
#['thing2', 'thing3', 'thing1']
>>> D2.keys()
#[1]
最后,我建议为你的"键"制作一个模板,以便
>>> template = "{0} + {1}"
然后你可以替换
行sco["{0} + {1}".format(a,b)] = 0
与
sco[template.format(a,b)] = 0
和类似的
dnt = '%s + %s' % (upd, cpg)
带
dnt = template.format(upd, cpg)
- 在我看来这是更安全的做法。阅读有关词典here, see section 5.5.
的更多信息答案 2 :(得分:0)
使用defaultdict和按元组中的项分组的解决方案:
import itertools
from collections import defaultdict
sho = ('human', 'loud', 'big')
sco = {}
for a, b in itertools.permutations(sho, 2):
sco["{0} + {1}".format(a, b)] = 0
cnt = [('human', 'ron g.'), ('loud', 'ron g.'), ('big', 'kim p.'), ('human', 'kim p.'), ('loud', 'brian a.'), ('human', 'linda m.'), ('loud', 'linda m.')]
groups = defaultdict(list)
for obj in cnt:
groups[obj[1]].append(obj[0])
final_list = []
# create list of pairs a + b
final_list.extend(map(lambda x: " + ".join(x), groups.values()))
for key in groups.keys():
groups[key].reverse()
# create list of pairs b + a
final_list.extend(map(lambda x: " + ".join(x), groups.values()))
for val in final_list:
if val in sco:
sco[val] += 1
print(sco)
我们在这里:
cnt
元组到词典sco
结果:
{'human + loud': 2, 'loud + human': 2, 'loud + big': 0, 'human + big': 1, 'big + loud': 0, 'big + human': 1}