Le programme consiste à obtenir le nombre premier le plus proche d'un nombre. J'ai résolu le problème mais je souhaite optimiser le code.
import time
def isprime(a ,d):
b=a
c=0
f=b**(0.5)
for i in range(1,int(f)+1):
if a%i==0 and c<=1:
c+=1
if c==1:
return d.append(b)
else:
b=a+1
isprime(b,d)
start=time.time()
b=[89, 54,36, 74, 44, 19, 12] # Input
d=[]
for i in b:
isprime(i,d) #function call
print(d) #output is [89, 59, 37, 79, 47, 19, 13]
stop=time.time()
print(stop-start) #0.0001347064971923828 Seconds
Aide à l'optimisation du code. Je suis juste un débutant, je sais que le code est inférieur. Aide à apprendre un peu de codage.
3 Réponses :
Le code peut être réécrit pour une meilleure lisibilité et des performances améliorées comme suit.
Refactoring du code
b = [12346, 1920131, 219112, 1423231] count = 1000 # number of timing iterations print(timeit(lambda: calc_next_primes(b), number = count)) print(timeit(lambda: next_prime_batch(b), number = count)) # calc_next_primes: 7.8703 seconds for 1K iterations # next_prime_batch: 0.6200 seconds for 1K iterations
Amélioration des performances strong>
Résultat:
Testé avec timeit pour une plus grande précision
Code original transformé en fonction p >
b = [2, 89, 54,36, 74, 44, 19, 12] count = 10000 # number of timing iterations print(timeit(lambda: calc_next_primes(b), number = count)) print(timeit(lambda: next_prime_batch(b), number = count)) # calc_next_primes: 1.4129 seconds for 10K iterations # next_prime_batch: 0.8683 seconds for 10K iterations
Timing Code
Test 1 - Seulement une légère amélioration pour les petits nombres
def isprime(a, d):
b=a
c=0
f=b**(0.5)
for i in range(1,int(f)+1):
if a%i==0 and c<=1:
c+=1
if c==1:
return d.append(b)
else:
b=a+1
isprime(b,d)
def calc_next_primes(b):
d=[]
for i in b:
isprime(i,d) #function call
return b
Tester une amélioration de 2 à 10 fois pour de plus grands nombres
def is_prime(n):
" Returns True if prime False otherwise "
# Base cases
if n < 2:
return False
if n == 2:
return True
if n % 2 == 0:
return False
# checking odd numbers 3, 5, ...
return next((False for i in range(3, int(n**0.5) + 1, 2) if n % i == 0), True)
def next_prime(n):
" First prime number greater or equal to n "
if n <= 2:
return 2
while not is_prime(n):
if n % 2:
n += 2 # stay on odd
else:
n += 1 # switch to odd
return n
def next_prime_batch(b):
" Finds next prime for a batch of numbers "
return [next_prime(x) for x in b]
La qualité de ce code est autant un problème, de mon point de vue, que la performance. (Les noms de variables a , b , c , d et f , vraiment ?) Le code nécessite une meilleure compréhension des nombres premiers et un peu plus d'attention à la lisibilité du code (et dans une moindre mesure, séparer les tests du code lui-même):
def is_odd_prime(number):
for divisor in range(3, int(number ** 0.5) + 1, 2):
if number % divisor == 0:
return False
return True
def prime_ceiling(number):
if number <= 2:
return 2
number |= 1 # enforce odd
while not is_odd_prime(number):
number += 2
return number
if __name__ == "__main__":
from time import time
numbers = [89, 54, 36, 74, 44, 19, 12]
start = time()
primes = [prime_ceiling(number) for number in numbers]
stop = time()
print(primes)
print(stop - start)
Évitez de mesurer le I / Ô temps d'impression de vos résultats, mesurez simplement le temps qu'il faut pour les calculer.
La réponse à cette question sera.
from math import sqrt
from time import time
def prime_no(n):
if n==1:
return False
if n<=3:
return True
if n%2==0 or n%3==0:
return False
for i in range(5,int(sqrt(n))+1,6):
if n%i==0 or n%(i+2)==0:
return False
return True
a=time()
l=[89, 54,36, 74, 44, 19, 12] #input
maxi=max(l)
while (prime_no(maxi)==False):
maxi+=1
k=[True for i in range(maxi+1)]
p=2
while(p*p<=maxi):
if(k[p]==True):
for i in range(p*p,maxi+1,p):
k[i]=False
p+=1
for i in l:
f=i
while k[f]==False:
f+=1
print(f,end=' ')
z=time()
print("Time: {:.20f}".format(z-a)) #89 59 37 79 47 19 13 Time: 0.00011634826660156250
Ce programme est beaucoup plus optimisé car pour la saisie de 99999 nombres, le temps est de 0,38797283172607421875 secondes
Le code produit le mauvais résultat pour 0 (-> 0) et 1 (-> 1) s'ils se trouvent dans l'entrée.
Vous aurez peut-être plus de chance à Révision de code .
J'ai signalé la migration vers CR par cette méta-publication
Comment migrer cette question vers Code Review. Je suis nouveau dans le stack overflow
@Deepan voir le message vers lequel j'ai inclus un lien dans mon commentaire précédent . J'ai signalé ce message pour une intervention du modérateur, en demandant sa migration.
@Deepan puisque vous semblez avoir posté sur CR j'ai rétracté mon drapeau; J'ai quand même voté pour fermer ce message car il ne semble pas être sur le sujet sur ce site