[kaktusowyAloes]

siemanko, mógłby mi ktoś z Was powiedzieć, czy ten test pierwszości jest poprawny?

LINK

def get_prime():
 n = int(input("enter a number: "))
 r = range(1, n+1 )
 divisiors = []

 for i in r:
 if n % i == 0:
 divisiors.append(i)
 print(divisiors)
 if len(divisiors) > 2:
 return"its not prime number"
 else:
 return "its prime number"

while True:
 get_prime()

#naukaprogramowania #python

[T.....s]

@kaktusowyAloes: poprawny, ale nieoptymalny, najprostsza optymalizacja to liczyć tylko do pierwiastka z n

from math import floor, sqrt

def get_prime():
 n = int(input("enter a number: "))
 r = range(2, floor(sqrt(n))+1 )
 divisiors = []

 for i in r:
 if n % i == 0:
 divisiors.append(i)
 print(divisiors)
 if len(divisiors) > 0:
 return"its not prime number"
 else:
 return "its prime number"

while True:
 print(get_prime())

[T.....s]

@kaktusowyAloes: https://pastebin.com/HJcskqvN

[cober]

@kaktusowyAloes: odpowiedź raczej dostaniesz poprawną. Troszeczkę bardziej "pythonic"

for i in range(n+1):
 if n % i == 0:
 divisors.append(i)

return "prime" if len(divisors) > 2 else return "not prime"

[T.....s]

@kaktusowyAloes: a i dodaj jeszcze sprawdzanie czy n=1 bo ona nie jest ani pierwsza ani złożona ( ͡° ͜ʖ ͡°)

[kaktusowyAloes]

@TeamfightTactics: @cober: dziękuje Panowie :)

[kaktusowyAloes]

@TeamfightTactics: @cober: Panowie, a mam jeszcze jedno pytanie, krótkie. Robię sobie ćwicz z practice python. Zerknijcie na to, na wykopie kiedyś mi ktoś pisał zeby w def nie używać print tylko return. a tutaj jak używam return to mam pusty wynik, jesli zamiast return jest print to wynik jest prawidłowy

https://pastebin.pl/view/b593344e

[kaktusowyAloes]

to samo tu: https://pastebin.com/BVfXBiaF
jak return zamienię na print to wszytko dziala

[TheBloody]

@kaktusowyAloes: return nie może wypisać wartości, możesz równie dobrze zrobić print(remove_dupl(names))

[kaktusowyAloes]

@TheBloody: wow, kolejna rzecz o której nie miałem pojęcia. Jutro też mam wolne to rano to obczaje. Dzięki stary

[TheBloody]

@kaktusowyAloes: Chyba że to będzie

return print(names)

Mam nadzieje że cie nie wprowadziłem w błąd, daj znać rano w razie czego