Python质数判断代码报错问题求助（涉及functions、list、loops）

嘿，我来帮你排查这段质数判断代码里的问题！先看看你贴的原代码：

m=int(input("Enter an integer: "))

factorlist=[]

x=[]

def primelist(m):

for i in range(1,m+1):

if m%i ==0:

x=factorlist.append(i)

return(x)

def isprime(m):

return(primelist(m)==[1,m])

print(isprime(m))

你运行的时候肯定会遇到语法错误和逻辑问题，我给你逐个拆解：

• 缩进错误：Python对缩进要求非常严格，primelist函数里的for循环、if语句都没有缩进，这会直接触发语法报错，因为解释器分不清哪些代码属于函数体。
• append方法的误用：factorlist.append(i)这个方法是没有返回值的（它会直接修改原列表，返回None），所以你把它赋值给x的话，x最终是None，完全不是你想要的因数列表。
• return位置错误：你的return(x)写在了if语句块里，这会导致函数在找到第一个因数（也就是1）的时候就直接返回了，根本不会遍历完所有可能的因数，返回的结果肯定不对。
• 全局变量的隐患：你把factorlist定义成了全局变量，要是多次调用primelist函数，之前的因数会一直留在列表里，导致结果混乱。

给你改好的代码，你可以直接运行试试：

m = int(input("Enter an integer: "))

def primelist(num):
    # 在函数内部创建因数列表，避免全局变量的残留问题
    factorlist = []
    for i in range(1, num + 1):
        if num % i == 0:
            # 直接向列表添加元素，不需要赋值给变量
            factorlist.append(i)
    # 遍历完所有数后，再返回完整的因数列表
    return factorlist

def isprime(num):
    # 获取目标数的所有因数
    factors = primelist(num)
    # 质数的定义就是因数只有1和它本身
    return factors == [1, num]

print(isprime(m))

简单说下修改的关键点：

1. 给函数内的代码补上正确的缩进，确保代码块归属正确；
2. 把因数列表移到函数内部，每次调用都会重新创建，避免数据残留；
3. 去掉了没用的x变量，直接使用append修改列表；
4. 把return语句移到for循环外面，保证返回完整的因数列表；
5. 给参数换了更清晰的名字（num），代码可读性更好。

备注：内容来源于stack exchange，提问作者velavan suresh