Discuss / Python / 交作业

交作业

Topic source

酱油richer

#1 Created at ... [Delete] [Delete and Lock User]

def is_palindrome(n): n=str(n) for i in range(len(n)): return n[i]==n[len(n)-1-i]

你的这种方法有局限,只适用于1000以内的

酱油richer

#3 Created at ... [Delete] [Delete and Lock User]

1000以外不行么?求教

ThinkTik

#4 Created at ... [Delete] [Delete and Lock User]

不好意思,我刚学这个。想问下为什么for循环里面的return语句不会提前终止循环。不是应该是第一次循环执行return语句体后就会终止函数跳出整个函数体的么?

ThinkTik

#5 Created at ... [Delete] [Delete and Lock User]

喔喔你错了,的确是只能判断3位数以下的回数。 如下:

def is_palindrome(n): n=str(n) for i in range(len(n)): return n[i]==n[len(n)-1-i]

output = filter(is_palindrome, range(1, 1000)) print(list(output)) 结果: [1, 2, 3, 4, 5, 6, 7, 8, 9, 11, 22, 33, 44, 55, 66, 77, 88, 99, 101, 111, 121, 131, 141, 151, 161, 171, 181, 191, 202, 212, 222, 232, 242, 252, 262, 272, 282, 292, 303, 313, 323, 333, 343, 353, 363, 373, 383, 393, 404, 414, 424, 434, 444, 454, 464, 474, 484, 494, 505, 515, 525, 535, 545, 555, 565, 575, 585, 595, 606, 616, 626, 636, 646, 656, 666, 676, 686, 696, 707, 717, 727, 737, 747, 757, 767, 777, 787, 797, 808, 818, 828, 838, 848, 858, 868, 878, 888, 898, 909, 919, 929, 939, 949, 959, 969, 979, 989, 999]

验证: out = is_palindrome(9999) print(out)

有:True

再验证: out = is_palindrome(9919) print(out)

有:True 这就错了,9919显然不是。解答是我上面提出的问题

酱油richer

#6 Created at ... [Delete] [Delete and Lock User]

哦哦。感谢。明白了

东派小六

#7 Created at ... [Delete] [Delete and Lock User]

原因应该是那个fifter是根据Ture和false来判断,is_palindrome程式返回的不是具体的数,而是ture或false,直接调用它并不会返回结果

def is_palindrome(n): n=str(n) for i in range(len(n)): return n[i]==n[len(n)-1-i]

在第一次执行 n[0] == n[-1] 就结束了. 所以你这代码, 别说三位数, 其实根本就是错了. 对3位数以下的正确是误打误撞. 事实上对任意第一个数和最后一个数相同的数, 都会 return True.

建议修改如下:

def is_palindrome(n):
    n=str(n)
    for i in range(len(n) / 2):
        if n[i] == n[-i-1]:
            pass
        else:
            return False
    return True

ericwang_1992

#9 Created at ... [Delete] [Delete and Lock User]

range里面有可能会出现float类型的数应该强制转化为int类型的。 应该修改为range(int(len(n) / 2))


  • 1

Reply