Ну, не только у вас есть 3 цикла, но этот подход не будет работать, если у вас более 3 факторов:)
Один из возможных способов:
def genfactors(fdict):
factors = set([1])
for factor, count in fdict.iteritems():
for ignore in range(count):
factors.update([n*factor for n in factors])
# that line could also be:
# factors.update(map(lambda e: e*factor, factors))
return factors
factors = {2:3, 3:2, 5:1}
for factor in genfactors(factors):
print factor
Кроме того, вы можете избежать дублирования некоторой работы во внутреннем цикле: если ваш рабочий набор равен (1,3) и вы хотите применить к 2 ^ 3 факторам, мы выполняем:
(1,3) U (1,3)*2 = (1,2,3,6)
(1,2,3,6) U (1,2,3,6)*2 = (1,2,3,4,6,12)
(1,2,3,4,6,12) U (1,2,3,4,6,12)*2 = (1,2,3,4,6,8,12,24)
Посмотрите, сколько дубликатов у нас во вторых наборах?
Но мы можем сделать вместо:
(1,3) + (1,3)*2 = (1,2,3,6)
(1,2,3,6) + ((1,3)*2)*2 = (1,2,3,4,6,12)
(1,2,3,4,6,12) + (((1,3)*2)*2)*2 = (1,2,3,4,6,8,12,24)
Решение выглядит еще лучше без комплектов:
def genfactors(fdict):
factors = [1]
for factor, count in fdict.iteritems():
newfactors = factors
for ignore in range(count):
newfactors = map(lambda e: e*factor, newfactors)
factors += newfactors
return factors