Question

[社区编辑：原始标题是“python条件”，OP询问下面的代码有什么问题]

我做了一个函数，它应该确定三个边是否理论上可以形成一个三角形。它在我看来工作正常，但是当我在pyschools.com网站上输入代码时，它告诉我在某些测试用例中它不起作用（不幸的是它没有向我展示它不起作用的情况）。我的代码中缺少某些东西，所以在某些特殊情况下我的逻辑会失效吗？非常感谢您的帮助。这是功能：

import math
def isTriangle(x, y, z):
    if x > 0 and y > 0 and z > 0:
        if x > y and x > z:
            c = x
        elif y > x and y > z:
            c = y
        else:
            c = z
        if c == math.sqrt(x**2 + y**2):
            return True
        else:
            return False
    else:
        return False

Answer 1

这样做更容易：

def isTriangle(sides):
    smallest,medium,biggest = sorted(sides)
    return smallest+medium>=biggest and all(s>0 for s in sides)

（编辑：我已经决定说2,2,4在技术上是一个三角形，但是是退化的三角形;如果你不认为它是一个三角形，可以将>=更改为>。）

这正是你正在做的事情。您正在正确计算c = largest = max(x,y,z)，但随后会return math.sqrt(x**2+y**2)检查它是否为直角三角形。

演示：

>>> isTriangle([2,2,6])
False
>>> isTriangle((5,5,9))
True
>>> isTriangle([-1,2,2])
False

下面我提一下如何简化代码：

import math               # from math import * for such common functions
def isTriangle(x, y, z):  # better to pass in a tuple or object, but this works
    if x>0 and y>0 and z>0:  # (then you could do all(s>0 for s in sides))
                             # (you could also do isTriangle(*sides))
                             # (you might need to add checks len(sides)==3
                             #  if your input data might include e.g. squares)
        if x > y and x > z:    # \
            c = x              #  |
        elif y > x and y > z:  #  > This is the same as c = max(x,y,z)
            c = y              #  |
        else:                  #  |
            c = z              # /
        if c == math.sqrt(x**2 + y**2): # \
            return True                 #  | Same as return c==sqrt(x**2+y**2)
        else:                           #  |  
            return False                # /   
    else:
        return False

“如果bool返回True，则返回False”是几乎所有现代编程语言中的“返回bool”。前者是不必要的冗长，永远不应该使用。

三角不等式作为蟒蛇条件的逻辑？

1 个答案: