1

我在 Python 2.72 中遇到了一个循环问题，这让我非常沮丧。基本上循环不会在第一个 index 上迭代j，我已经尝试了各种方法来修复它，但没有运气。

def learn(dataSet):
    for i in dataSet.getNext():
        recall = raw_input("Enter all members of %s you are able to recall >>> (separated by commas)  " % (i.getName()))
        missed = i.getMembers()     
        missedString = []       
        for a in missed:
            missedString.append(a.getName())

这是我无法迭代的循环。第一个 for 循环仅经过j拆分字符串列表中的第一次迭代，然后将其从missedString. 我希望将拆分字符串的所有成员recall从missedString.

for j in string.split(recall, ','):
    if j in missedString:
        missedString.remove(j)
        continue
for b in missed:
    if b.getName() not in missedString:
        missed.remove(b)
print 'You missed %d.  ' % (len(missed))
if (len(missed)) > 0:
    print 'Maybe a hint or two will help...' 
    for miss in missed:
        remind(miss.getSecs(), i.getName(), missed)

如何修复上述代码？

4

2 回答 2

5

missedString是一个可怕的列表名称

请注意，您可以在几个地方简化代码

        missedString = []       
        for a in missed:
            missedString.append(a.getName())

可以用列表推导替换

        missedString = [a.getName() for a in missed]

在这里你应该只使用split方法recall而不是string.split)。看起来这个循环应该嵌套在for i循环内，所以我假设它是（如果不是，你将使用错误的数据集召回的错误值）。

for j in string.split(recall, ','):
    if j in missedString:
        missedString.remove(j)
        continue

它也可以用列表推导替换。

recall_set = set(recall.split(','))
missedString = [j for j in missedString if j not in recall_set]

如果例如，这将无法正常工作。用户在输入中输入了额外的空格，因此对strip()这些元素来说是个好主意

recall_set = set(s.strip() for s in recall.split(','))
missedString = [j for j in missedString if j not in recall_set]

这个循环有一个严重的问题。一般来说，从您正在迭代的列表中删除元素并不是一个好主意。你最终会跳过一些元素而不检查它们

for b in missed:
    if b.getName() not in missedString:
        missed.remove(b)

也许列表理解可以再次提供帮助

missed = [b for b in missed if b.getName() in missedString]
于 2012-09-24T04:26:07.040 回答
0

在这段代码中

for j in string.split(recall, ','):
    if j in missedString:
        missedString.remove(j)
        continue

尝试添加一个

    else:
        print 'Didn't remove "%s"' % j

（并摆脱continue. 它没有任何目的）。

您可能在拆分中缺少一些空格。如果是这种情况，请添加j = j.strip()orre.split(r'\s*,\s*', recall)代替string.split(recall, ',').

于 2012-09-24T04:29:07.023 回答