Python实现Isogram判断函数始终返回True，结果异常求助

首先，你的代码始终返回True的核心原因是判断逻辑的循环在第一次迭代就直接return了，根本没检查完所有字母的计数情况。

具体问题拆解 #

看这段判断代码：

for values in dict:
    if dict[values] > 1:
        return False 
    else:
        return True

• 首先，for values in dict遍历的是字典的键，不是值，变量名起错了，但这不是关键。
• 关键是：循环第一次执行时，不管第一个字母的计数是1还是大于1，都会直接return结果。比如字符串是"abac"，第一个字母a的计数是1，代码直接返回True，完全没检查后面的a重复情况。

修正判断逻辑 #

你需要先遍历完所有字母的计数，只有当所有字母的计数都不超过1时，才返回True；只要发现任何一个字母计数大于1，就立即返回False。修正后的代码如下：

def is_isogram(string):
    letter_counts = {}  # 不要用dict当变量名，这是Python内置类型
    for letter in string.lower():
        if letter in letter_counts:
            letter_counts[letter] += 1
        else:
            letter_counts[letter] = 1
    
    # 修正后的判断逻辑
    for count in letter_counts.values():
        if count > 1:
            return False
    # 所有字母都不重复，返回True（按常规Isogram判断逻辑）
    return True

额外优化：用集合简化代码 #

其实不需要字典计数，利用集合元素唯一的特性，可以更简洁地实现判断：

def is_isogram(string):
    # 转小写并过滤非字母字符（如果需要忽略空格、符号等）
    cleaned_letters = [c.lower() for c in string if c.isalpha()]
    # 集合长度等于原列表长度说明无重复
    return len(cleaned_letters) == len(set(cleaned_letters))

关于需求笔误 #

你提到的原需求描述“当字符串是Isogram时返回False，否则返回False”明显有错误，常规的Isogram判断函数应该是：当字符串是Isogram时返回True，否则返回False，上面的代码都是按这个逻辑实现的。如果实际需求是反过来，把return的结果取反即可。

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