O problema, como já disseram, é que está usando extend
, que recebe um iterável e adiciona todos os elementos deste na lista. Mas você está passando um número, que não é iterável. Então deveria usar append
para adicionar o elemento.
Só que você modifica a lista sublist
no mesmo loop que itera sobre ela, e isso nunca é uma boa ideia (leia aqui, aqui e aqui para entender melhor).
Enfim, me parece que você está complicando o código à toa. Se quer criar uma lista contendo sub-listas, sendo que cada sub-lista contém os divisores de um número, daria para fazer assim:
def add_divisors(list_num):
final_list = []
for number in list_num:
sublist = []
for x in range(1, number + 1):
if number % x == 0:
sublist.append(x)
final_list.append(sublist)
return final_list
print(add_divisors([3, 4, 6])) # [[1, 3], [1, 2, 4], [1, 2, 3, 6]]
Claro que essa é a abordagem ingênua, você não precisa fazer um loop até o número, pode ir até a raiz quadrada dele. E pode começar do 2, pois todo número é divisível por 1, então para que testar o 1? Coloque-o direto na lista. O mesmo vale para o próprio número.
E para cada divisor que você encontrar, na verdade você encontrou - potencialmente - dois divisores. Por exemplo, se o número for 100 e você encontra o divisor 2, você também já encontrou o divisor 50 (resultado de 100 / 2). Então basta inserir ambos, economizando uma iteração do loop. Só precisa tomar cuidado para o caso de quadrados perfeitos, para não contar duas vezes o mesmo divisor (por exemplo, se o número for 100, não podemos usar esta lógica com o divisor 10, senão ele será contado duas vezes).
Então ficaria assim:
from math import sqrt
def add_divisors(list_num):
final_list = []
for number in list_num:
divisores = [ 1 ] # é divisor de qualquer número
for x in range(2, int(sqrt(number)) + 1):
divisao, resto = divmod(number, x)
if resto == 0:
divisores.append(x)
if x != divisao:
divisores.append(divisao)
divisores.append(number) # todo número é divisor de si mesmo
final_list.append(divisores)
return final_list
Apesar de ter ficado "com mais linhas", ele faz menos iterações do que se fosse de 1 até o número (claro que para valores pequenos não fará tanta diferença assim, mas enfim).
extend
somente aceita objetos iteráveis como argumento e você está passandox
, que é de tipoint
. O problema está aí pelo que vejo.x
pois fica auto incrementandosublist
com o mesmo valor indefinidamente. Talvez seja melhor remover ou comentar a linhasublist.append(x)
final_list
???