Python : palindrome을 확인하면 False가 반환됩니다.

Question

저는 프로그램을 작성하려고하는데, 제가해야할 일 중 하나는 숫자가 회문인지 확인하는 것입니다. 어떤 이유 는, 내가 쓴 작동하지 않습니다

flag = True 
for j in range(0, len(listnum)): 
    for t in range(len(listnum) -1, -1, -1): 
     if listnum[j] == listnum[t]: 
      flag = True 
      print "false" 
     else: 
      flag = False 
      print "false" 
      break 
if flag == True: 
    return True 
else: 
    return False 

인쇄 "거짓"/ "진정한"부분은 디버깅을위한 것입니다. 필자가 작성한 함수에서이 코드를 제거 했으므로 "return True/False"가 표시됩니다. 마지막 번호가 목록에있는 다른 다른 경우

내가 확인하고있어 수는 906,609

Answer 1

귀하의 기능을 검사합니다. 당신이 원하는 것은 아마도 :

def palindrome(listnum): 
    for j in range(0, len(listnum)): 
     t = len(listnum) - j - 1 
     if listnum[j] != listnum[t]: 
      return False    

    return True 

assert palindrome([9,0,6,6,0,9]) 
assert palindrome("ABBA") 
assert palindrome([]) 
assert palindrome("1") 
assert palindrome([1.0, 2, 1]) 
assert not palindrome("lolz") 
assert not palindrome([1,2,3]) 

Answer 2

그것은 당신의 코드가 작동하지 않는 이유에 대한 답변 아니지만, 당신은 단어가 회문 여부를 확인하는 훨씬 쉬운 방법이 있다고 알고 관심이있을 수도 있습니다. 정의에 따르면, 회문은 그것을 뒤집어서 변하지 않는 단어입니다. 따라서 :

def is_palindrome(word): 
    return word == ''.join(reversed(word)) 

print is_palindrome('906609') # True 

설명이 필요할 수도있는 부분은 join입니다. reversed이 단어의 문자를 하나씩 반환하기 때문에 필요합니다. 먼저 단어를 다시 묶어야합니다.

의견에서 지적한 바와 같이, 또 다른 작성 방법은 word == word[::-1]입니다. 그것은 똑같은 것을 의미하지만 틀림없이 조금 더 수수한 것입니다.

Answer 3

여기서 중요한 문제는 for 루프의 중첩입니다. 잠금 단계에서 j와 t를 업데이트하려고하지만 j = 0으로 시작한 다음 t의 모든 값을 확인하면됩니다. 그러면 j = 1이고 t의 모든 값을 다시 확인하십시오.

중첩 루프 대신 루프 카운터를 사용하여 단어를 조사하는 거리를 추적 한 다음 그 중 j 및 t를 계산할 수 있습니다 . 예 :

for d in range(len(listnum)/2): 
    j = d 
    t = len(listnum) - d - 1 
    #letter equality check here, return if false 

return True