Python循环返回函数问题：为何自定义duplicates函数始终返回True？

嘿，我一眼就发现你这段代码的核心问题啦！你的duplicates函数逻辑存在一个关键错误——它在循环的第一次迭代时就直接返回结果了，根本没来得及检查字符串里的所有字符。

具体问题拆解 #

我们拿你的测试用例来举例：

• 当传入"goo"时，函数首先取出第一个字符'g'，调用s.count('g')得到结果1，满足else分支的条件，直接返回True，完全没去检查后面重复的'o'。
• 同样，传入"good"时，第一个字符'g'的计数是1，函数立刻返回True，完全忽略了后续出现的重复字符。

你的代码逻辑相当于“只要第一个字符不重复，就判定整个字符串没有重复”，这显然和你想要的“检测整个字符串是否存在重复字符”的需求不符。

修正后的代码方案 #

方案1：修复原有循环逻辑

只需要把return True移出循环体，确保函数只有在遍历完所有字符、确认没有任何重复的情况下才返回True：

def duplicates(s):
    for c in s:
        if s.count(c) > 1:
            return False
    # 所有字符都检查完毕，无重复才返回True
    return True

方案2：更高效的集合判断法

利用Python集合自动去重的特性，直接比较集合和原字符串的长度，这个方法的时间复杂度是O(n)，比原方案的O(n²)高效得多：

def duplicates(s):
    # 如果集合长度等于原字符串长度，说明无重复字符
    return len(set(s)) == len(s)

现在测试这两个修正后的函数，就能得到正确结果了：

print(duplicates("god"))   # 输出True
print(duplicates("goo"))   # 输出False
print(duplicates("good"))  # 输出False

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