使用for循环实现True计数函数时的ValueError触发问题排查

原代码存在的问题 #

• 类型判断逻辑完全错误：value == int or value == float是把元素值和int/float这两个类型对象做比较，永远不会成立。而且你的需求是校验元素必须是布尔类型（True/False），应该检查元素的类型是否为bool。
• raise语句缩进错误：当前raise和if块同级，导致不管元素是否合法，第一次循环就会直接抛出异常，完全无法执行统计逻辑。
• 冗余的True判断：就算类型判断正确，if value == True对于布尔类型来说多余，直接用if value就能判断是否为True；另外你的需求是错误类型要抛错，所以元素必须是严格的bool类型，不能是int/float这类可隐式转换的类型。

修正后的代码 #

def only_true(bool_list):
    trues_count = 0
    for value in bool_list:
        # 校验元素是否为布尔类型，非bool直接抛错
        if not isinstance(value, bool):
            raise ValueError("values in list must be True or False")
        # 统计True的数量
        if value:
            trues_count += 1
    return trues_count

代码说明 #

• 用isinstance(value, bool)严格校验每个元素类型，确保只有布尔值能通过检查。
• 将raise放在类型校验的分支里，只有元素不合法时才触发异常，合法元素正常执行统计。
• 用if value直接判断是否为True，符合布尔类型的判断逻辑，写法更简洁。
• 用trues_count += 1替代繁琐的赋值写法，代码更易读。

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