这篇文章是双重的。 我是Python的新手。
这是旧版Google拼图的代码: 'e'连续数字中的前10位数素数 (https://google-tale.blogspot.ca/2008/07/google-billboard-puzzle.html)
euler = '7182818284590452353602874713526624977572470936999595749669676277240766303535475945713821785251664274274663919320030599218174135966290435729003342952605956307381323286279434907632338298807531952510190115738341879307021540891499348841675092447614606680822648001684774118537423454424371075390777449920695517027618386062613313845830007520449338265602976067371132007093287091274437470472306969772093101416928368190255151086574637721112523897844250569536967707854499699679468644549059879316368892300987931277361782154249992295763514822082698951936680331825288693984964651058209392398294887933203625094431173012381970684161403970198376793206832823764648042953118023287825098194558153017567173613320698112509961818815930416903515988885193458072738667385894228792284998920868058257492796104841984443634632449684875602336248270419786232090021609902353043699418491463140934317381436405462531520961836908887070167683964243781405927145635490613031072085103837505101157477041718986106873969655212671546889570350354'
for i in range(len(euler) - 3):
if (euler[i + 3]== '1' or euler[i + 3]== '3' or euler[i + 3]== '7' or euler[i + 3]== '9'):
number = int(euler[i:i + 3])
for a in range(2, round(number**0.5)):
if number % a == 0:
break
else:
print(number),
break
奇怪地使用上面的代码我可以找到正确的答案,即7427466391(编译需要45%)但是当我测试3位数的代码时,它给出了709这是不正确的,因为前3位数的素数是281范围(3,6)。
我该如何解决这个问题?
这是一个算法,它应该是找到素数(https://en.wikipedia.org/wiki/Primality_test)最快的算法(参见页面中间的伪代码)。这是我的代码,但它不起作用,我进入一个无限循环。
for i in range(len(euler) - 3):
if (euler[i + 3]== '1' or euler[i + 3]== '3' or euler[i + 3]== '7' or euler[i + 3]== '9'):
number = int(euler[i:i + 3])
for y in range(2, number-1):
while y * y <= number:
if (number % y != 0 or number % (y+1) != 0):
break
我该如何解决?
答案 0 :(得分:1)
对于第一部分,将&#39; 3&#39; 替换为&#39; 2&#39; ,以便检查第三位是否有奇怪,而不是第四个:
euler[i + 3]== '1' or euler[i + 3]== '3' or euler[i + 3]== '7' or euler[i + 3]== '9'):
更改为
euler[i + 2]== '1' or euler[i + 2]== '3' or euler[i + 2]== '7' or euler[i + 2]== '9'):
同样,考虑for循环中的范围应该延伸到什么(即len-2或len-3),即使它不适用于此示例。
一般情况下,请避免使用magic numbers。定义变量或接受输入以设置N = 3或10 ...
此外,通过添加if
条件,只有潜在的错误(如上所述),您不会获得太多收益。 过早优化是万恶之源
对于第二部分,当你break
你的内循环时,你并没有真正建立任何素性。你可以做一些像设置旗帜一样简单的事情。在这里,我修改了您的程序以提供一个示例:
euler = '7182818284590452353602874713526624977572470936999595749669676277240766303535475945713821785251664274274663919320030599218174135966290435729003342952605956307381323286279434907632338298807531952510190115738341879307021540891499348841675092447614606680822648001684774118537423454424371075390777449920695517027618386062613313845830007520449338265602976067371132007093287091274437470472306969772093101416928368190255151086574637721112523897844250569536967707854499699679468644549059879316368892300987931277361782154249992295763514822082698951936680331825288693984964651058209392398294887933203625094431173012381970684161403970198376793206832823764648042953118023287825098194558153017567173613320698112509961818815930416903515988885193458072738667385894228792284998920868058257492796104841984443634632449684875602336248270419786232090021609902353043699418491463140934317381436405462531520961836908887070167683964243781405927145635490613031072085103837505101157477041718986106873969655212671546889570350354'
for i in range(len(euler) - 2):
if (euler[i + 2]== '1' or euler[i + 2]== '3' or euler[i + 2]== '7' or euler[i + 2]== '9'):
number = int(euler[i:i + 3])
for y in range(2, number-1):
isPrime = True
while y * y <= number:
if (number % y != 0 or number % (y+1) != 0):
isPrime = False # The number is not prime
break # Exit loop
if isPrime: # Nothing was able to factorize the number
print(number) # Print and leave
exit(0)
但是,您的代码确实非常破碎,并且可以对不同边缘情况进行大量关注和测试。
答案 1 :(得分:0)
对于您的第一部分,您正在掌握Python中的范围和切片的右端非包含属性。但是,当您只是索引以查找元素时,您是否仍然关心它?我的意思是,对于从位置i+3
开始的数字,第三位真的是i
的索引吗?
对于你的第二部分,我没有看到你的代码和伪代码之间的映射。
一般提示。您可以将prime_check逻辑划分为返回True或False的函数,并在每次获得数字时调用该函数,然后决定是否要打印它并中断或继续。此外,您正在进行1,3,7,9的比较,并缩写为in
。