欧拉项目：寻找两个三位数乘积的最大回文数（Python代码问题咨询）

咱们先拆解你代码里的几个关键错误，再给出可行的修正方案：

1. 变量作用域与参数传递错误 #

你的check_number函数直接依赖外部循环的i变量，这不仅违背了函数的封装性，还会导致逻辑混乱——函数应该明确接收要检查的回文数作为参数，而不是“偷偷”使用外部变量。

2. 乘积判断逻辑完全走偏 #

你在check_number里的循环逻辑存在核心错误：

• 你把传入的999赋值给x后不断递减，同时用element * x == i判断，但element本身也是从999倒序遍历的，这相当于重复遍历且逻辑冗余。
• 正确的思路应该是：对回文数i，遍历三位数n，检查i是否能被n整除，且商也是一个三位数，这样就能确认i是两个三位数的乘积。

3. 未及时终止循环 #

因为你是从大到小遍历回文数，所以第一个符合条件的数就是最大的，但你的代码没有在找到结果后终止程序，会继续遍历更小的数，完全没必要。

修正后的代码 #

def is_palindrome(num):
    return str(num) == str(num)[::-1]

def find_largest_palindrome():
    # 从最大的三位数乘积开始倒序遍历
    for num in range(999*999, 100*100 - 1, -1):
        if is_palindrome(num):
            # 从大到小遍历三位数，检查整除性
            for n in range(999, 99, -1):
                if num % n == 0:
                    other_factor = num // n
                    # 确认另一个因子也是三位数
                    if 100 <= other_factor <= 999:
                        print(f"最大的回文数是: {num}，由 {n} * {other_factor} 得到")
                        return num
    return None

find_largest_palindrome()

代码逻辑说明 #

• is_palindrome函数单独负责判断回文数，逻辑清晰且可复用。
• 外层循环从最大的三位数乘积（999*999）开始倒序遍历，一旦找到回文数就进入内层检查。
• 内层循环同样从大到小遍历三位数，找到能整除当前回文数的因子后，直接判断商是否为三位数——符合条件就立即返回结果，因为我们是从大到小找的，第一个符合条件的就是答案。

内容的提问来源于stack exchange，提问作者Mampel