프로젝트 오일러 : 가장 큰 회문 (파이썬). 나는 무엇을 잘못 했는가?

Question

나는 초심자 일뿐 아니라 여러 가지 일을 잘못했을 수도 있습니다. 하지만 누군가 나를 도울까요? 목표는 두 자리 3 자리 숫자의 곱으로 만들어진 최대 회중 전선을 인쇄하는 것입니다. 감사.

palindromes = [] 
for i in range(100, 1000): 
    for x in range(100, 1000): 
     a = x*i 
     str(a) 
     b = a[::-1] 
     if a == b : 
      palindromes.append(a) 

print (palindromes[len(palindromes) - 1]) 

Answer 1

기본 점 : 당신은 문자열로 숫자를 변환 할 때

• , 당신은 결과를 저장해야합니다.
• 완료하면 목록을 정렬하십시오.
• 를 사용하여 오른쪽 끝 색인을 사용하면 회문를 발견하면, 마지막 요소
• 를 얻을 정수 형태를 유지, 그래서 정렬이 제대로 작동합니다. 그렇지 않으면 '99999'가 가장 큰 회상암으로 간주됩니다.
는

는 코드 :

palindromes = [] 
for i in range(100, 1000): 
    for x in range(100, 1000): 
     a = str(x*i) 
     b = a[::-1] 
     if a == b : 
      palindromes.append(int(a)) 

print (sorted(palindromes)[-1]) 

Answer 2

정의하기 전에 코드에 x을 사용하면 해당 코드에 오류가 발생합니다. 다른 오류도 있습니다 : 추적을 사용하여 쉽게 찾을 수 있어야합니다. 하나의 논리 오류는 a 문자열을 찾을 수 있지만 어디에도 저장하지 않으므로 손실됩니다. 또한 a으로 정수로 시작한 다음 문자열로 만드십시오. 유형을 변경해야하는 중요한 이유가 없다면 유형을 별도로 유지하십시오. 더 나은 변수 이름을 사용해야합니다 : a,b,i,x은 형편없는 이름입니다. 그들이 의미하는 것을 적어 라! 더 많은 문자를 입력하는 것은 저렴하며 오류를 방지합니다.

각 회문은 이 세 자리 숫자의 곱해야합니다, 그래서 당신은 range(100, 1000)을 통해 이 중첩 루프가 필요합니다. x을 다른 루프 변수로 사용하면 코드를 많이 변경할 필요가 없습니다.

Answer 3

인쇄하기 전에 줄을 추가

palindrome.sort()