使用for循环计算两数LCM的代码问题：返回值不符预期

你的代码存在几个关键问题，导致结果不符合预期：

1. 未定义变量错误 #

代码里的print(number)调用了从未声明的变量number，正常运行会直接抛出NameError。你应该打印找到的公倍数a或b（此时两者值相等）。

2. 对最小公倍数的认知偏差 #

6和8的最小公倍数是24（6×4=24，8×3=24），并非你预期的48。48只是它们的公倍数之一，并非最小的那个。

3. 循环逻辑缺陷 #

当前代码没有在找到第一个公倍数时终止循环，会继续查找后续的公倍数；同时循环范围设置过小，逻辑上不够严谨（虽然对6和8来说范围足够，但不适用于更大的数值）。

修正后的双层for循环代码 #

以下是能正确找到最小公倍数的版本，找到结果后立即终止所有循环：

number1 = 6
number2 = 8
result = None

# 遍历number1的倍数
for i in range(1, number2 + 1):
    multiple1 = number1 * i
    # 遍历number2的倍数，检查是否匹配
    for j in range(1, number1 + 1):
        multiple2 = number2 * j
        if multiple1 == multiple2:
            result = multiple1
            break  # 跳出内层循环
    if result is not None:
        break  # 跳出外层循环

print(result)  # 输出：24

更高效的简化版本 #

其实不需要双层循环，只需遍历其中一个数的倍数，检查是否能被另一个数整除即可：

number1 = 6
number2 = 8
lcm = None

# 从number1开始，每次增加number1，生成它的倍数
for multiple in range(number1, number1 * number2 + 1, number1):
    if multiple % number2 == 0:
        lcm = multiple
        break

print(lcm)  # 输出：24

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