Python字符列表与字符串对比函数返回结果异常求助

我来帮你看看这段代码的问题所在，你遇到的情况是因为函数的逻辑完全走偏啦！

核心问题分析 #

咱们先拆解一下你的compare函数里的bug：

• 循环只执行一次就返回结果：你的for循环里，只要第一次比较orig[i]和sofar[i]相等，就直接return True，不等就return False——这意味着函数只会检查第一个字符，后面的字符根本没机会被对比！比如你测试的例子里，第一个字符都是'a'，所以直接返回True，完全不管第二个字符'v'和'b'不相等的情况。
• 索引越界风险：range(len(orig)+1)会生成从0到len(orig)的索引，但字符串和列表的索引都是从0到len()-1的，当i等于len(orig)时，访问orig[i]或sofar[i]会直接抛出索引错误（只不过你的代码因为提前return了，没触发这个问题）。

修复后的代码 #

正确的逻辑应该是：遍历所有对应位置的字符，只要发现有一个不相等就立即返回False；如果所有字符都对比完且全部相等，再返回True。修改后的代码如下：

def compare(sofar, orig):
    if len(sofar) != len(orig):
        return False
    # 遍历每一个索引位置
    for i in range(len(orig)):
        if orig[i] != sofar[i]:
            return False
    # 所有字符都相等才返回True
    return True

测试验证 #

用你的测试用例跑一下：

compare(['a','v','c','c','s'], 'abccs')

这次会返回False，完全符合你的预期。

另外，Python里还有更简洁的写法，直接利用可迭代对象的特性完成对比：

def compare(sofar, orig):
    return len(sofar) == len(orig) and all(a == b for a, b in zip(orig, sofar))

zip会把两个可迭代对象的对应元素配对，all函数会检查所有配对的元素是否都相等，代码更简洁且可读性更强。

内容的提问来源于stack exchange，提问作者mozzie