Script nepřičítá k proměnné – Python – Fórum – Programujte.com

Dobrý den , mám problém se scriptem na hledání prvočísel.  Script funguje tak jak má , dokud nepřijde na přičítání a následnému  opakování cyklu. 

cis = int(input("xd: "))
list1 = []
list2 = []
num = 2
while (num in range(0,cis)):
    list1.append(num)
    num += 1
    continue

num2 = 2

for i in list1:
    if list1[0] in list2:
        None
    else:
        list2.append(list1[0])
    if i % num2 == 0:
        list1.remove(i)
        continue
    if all(i % num2 in list1):
        num2 += 1
        continue

print (list1)
print (list2)

Děkuji za pomoc. 

Máš tam dvoje přičítání. Obojí podmíněné. Asi není ta podmínka splněna. U prvního není, co řešit (kromě toho, že ten cyklus vůbec nepotřebuješ :-)). Tak koukneme na ten for...

První if(+else) -- nevím, k čemu to má být dobré, ale budiž.

Druhý if -- to je optimalizace pro sudá čísla? Fajn, ale měníš ten list, který právě procházíš. Ve výsledku ti to bude jedno číslo přeskakovat, takže půjdeš pořád jen po sudých.

Třetí if -- k tomu ti to teda zatím ani nedojde a netuším, co od toho čekáš, ale all nejspíš nedělá to, co si myslíš :-)

#2 gna
Jsem začátečník , tak se omlouvám , někde jsem četl , že all znamená jako pro všechny prvky v tom listu. Ten druhý if odebírá čísla dělitelná tou proměnnou , takže by to ideálně mělo odebrat postupně všechny čísla , co nejsou prvočísla. Ten třetí if by měl ověřit , že už v listu nejsou žádná v tomhle případě sudá čísla a postoupit na čísla co jsou dělitelná třemi. 

#3 P3p111n0

Neomlouvej se, na zetptání není nic špatného.

Jo, neuvědomil jsem si, že se num2 má měnit a nebude pořád 2. To je celkem jedno, šlo o to, že ten list měníš zatímco ho procházíš.

Tomu all nedáváš list, ale True nebo False (výsledek toho in).

#4 gna
Úplně se nechytám , jak by to tedy mělo být ? Jak tedy ukončit měnění toho listu než ho začnu procházet a jak na to all ? 

No, ideálně ten list uvnitř toho cyklu vůbec neměň. Nevím, jak to chceš řešit, ale nějak k tomu ani nevidím důvod.

Úplně primitivně by to bylo třeba takhle:

cis = int(input("xd: "))

prvocisla = []
for i in range(2, cis):
	madelitele = False
	for j in range(2, i):
		if i % j == 0:
			madelitele = True
			break
	if not madelitele:
		prvocisla.append(i)

print(prvocisla)

To k čemu možná směřuješ by bylo třeba takhle:

prvocisla = []
for i in range(2, cis):
    if all(i % j != 0 for j in range(2, i)):
        prvocisla.append(i)

A nebo ještě zhuštěněji takhle:

prvocisla = [i for i in range(2, cis) if all(i % j != 0 for j in range(2, i))]

Pokud to máš do školy a netušíš, co děláš, tak bych asi čekal první možnost.

#6 gna
Kdepak do školy , znuděnej samouk :D Ale koukám na tu tvoji verzi a ty jsi asi génius. Základní myšlenka byla , že naplním jeden list číslama a postupně z něj odebírat čísla , než mi zbyde jen list s prvočísly. Děkuji moc za pomoc !