7

因此,正如标题所暗示的那样,这是该书中的一个示例。我对编程仍然很陌生,并且难以调试。话虽如此,欢迎任何批评,特别是如果它显示了一种更有效的编码方式;请记住,我还是新手,所以如果你向我扔一个新的内置函数或其他东西,我很可能不知道你指的是什么。

所以这个练习的重点是编写一个函数,给它三个参数,以确定这三个参数是否形成一个三角形。这是我的代码:

def is_triangle(a,b,c):
    num_list = [a,b,c]
    biggest = max(num_list)
    other_two = num_list.remove(biggest)
    sum_of_two = sum(other_two)

    if sum_of_two > biggest:
        print 'Congrats, %d, %d, and %d form a triangle!' % (a,b,c)
    elif sum_of_two == biggest:
        print 'That forms a degenerate triangle!'
    else:
        print 'That does\'t make any sort triangle... >:['


def sides():
    side1 = raw_input('Please input side numero Juan: ')
    side2 = raw_input('Now side two: ')
    side3 = raw_input('...aaaannnd three: ')
    import time
    time.sleep(1)
    print 'Thanks >:]'
    side1 = int(side1)
    side2 = int(side2)
    side3 = int(side3)
    is_triangle(side1,side2,side3)

sides()

但是,每当我运行它时,我都会得到以下信息:

Traceback (most recent call last):
  File "A:/Python/is_triangle.py", line 27, in <module>
    sides()
  File "A:/Python/is_triangle.py", line 25, in sides
    is_triangle(side1,side2,side3)
  File "A:/Python/is_triangle.py", line 5, in is_triangle
    sum_of_two = sum(other_two)
TypeError: 'NoneType' object is not iterable

我的猜测是 sum_of_two 行,但我不知道它有什么问题。有人可以帮我调试吗?

我花了一个小时在没有内置函数的情况下重写它(以各种方式,or到处都是一堆 s)。但它看起来很糟糕,我宁愿学着这样写。

4

3 回答 3

5

问题在于remove修改基础列表的事实 - 它不返回新列表。将其更改为:

num_list.remove(biggest)
sum_of_two = sum(num_list)

要了解发生这种情况的确切原因,请在 IDLE 中尝试以下操作:

>>> x = [1,2,3,4,5]
>>> x.remove(1)
>>> x
[2,3,4,5]
于 2013-05-28T04:51:05.323 回答
2

由于num_list.remove(biggest)返回None,请考虑这个

other1, other2, biggest = sorted(num_list)
sum_of_two = other1 + other2

看起来if块也需要缩进

def is_triangle(a, b, c):
    num_list = [a, b, c]
    other1, other2, biggest = sorted(num_list)
    sum_of_two = other1 + other2

    if sum_of_two > biggest:
        print 'Congrats, %d, %d, and %d form a triangle!' % (a,b,c)
    elif sum_of_two == biggest:
        print 'That forms a degenerate triangle!'
    else:
        print 'That does\'t make any sort triangle... >:['
于 2013-05-28T04:52:49.160 回答
2

你只是犯了一个非常常见的错误,一个简单的错误。在 Python 中,当函数导致数据结构发生更改时,它不会返回更改后的结构。通常它会返回None。如果函数获得新的数据结构或新值,则返回它。

所以,str.lower()实际上并没有改变一个字符串;它返回一个新字符串,其中字符为小写。如果您有一些名为的列表lst并运行sorted(lst),它不会更改列表;它返回一个已排序的新列表。但是lst.sort()对列表进行就地排序,因此它不会返回对列表的引用;它返回None

在下面的评论中,@lvc 指出list.pop()从列表中删除一个值并返回该值。因此,这是一个更改数据结构并返回非 的函数的示例None,但它仍然绝对不会返回对更改后的数据结构的引用。

list.remove()函数更改一个列表,并返回None. 您需要做的就是更改您的函数以对所有内容使用相同的列表:首先使用列表查找最大值,然后从列表中删除最大值,然后将列表传递给sum().

Python 这样做的原因是为了遵守“命令/查询分离”原则。

http://en.wikipedia.org/wiki/Command%E2%80%93query_separation

于 2013-05-28T04:52:49.253 回答