项目Euler＃8，答案错误

Question

我被困在Problem 8 of Project Euler上，似乎无法弄清楚自己在犯什么错误。

我的答案是70573265280。我是编程新手。

我将数字复制了两次以检查是否有复制错误，但仍然得到相同的错误答案。

def count_nums(y):
    largest_product = 0

    while len(y) >= 13:
        sub_list = y[:14]
        product = 1

        for x in sub_list:  
            product *= x

        if product > largest_product:
            largest_product = product
        else:
            largest_product

        y.pop(0)

    return largest_product

x = "7316717653133062491922511967442657474235534919493496983520312774506326239578318016984801869478851843858615607891129494954595017379583319528532088055111254069874715852386305071569329096329522744304355766896648950445244523161731856403098711121722383113622298934233803081353362766142828064444866452387493035890729629049156044077239071381051585930796086670172427121883998797908792274921901699720888093776657273330010533678812202354218097512545405947522435258490771167055601360483958644670632441572215539753697817977846174064955149290862569321978468622482839722413756570560574902614079729686524145351004748216637048440319989000889524345065854122758866688116427171479924442928230863465674813919123162824586178664583591245665294765456828489128831426076900422421902267105562632111110937054421750694165896040807198403850962455444362981230987879927244284909188845801561660979191338754992005240636899125607176060588611646710940507754100225698315520005593572972571636269561882670428252483600823257530420752963450"

x = [int(y) for y in x]
print(count_nums(x))

Answer 1

欧拉问题＃8：

  

在1000位数字中找到五个连续数字的最大乘积   数字。

73167176531330624919225119674426574742355349194934
96983520312774506326239578318016984801869478851843
85861560789112949495459501737958331952853208805511
12540698747158523863050715693290963295227443043557
66896648950445244523161731856403098711121722383113
62229893423380308135336276614282806444486645238749
30358907296290491560440772390713810515859307960866
70172427121883998797908792274921901699720888093776
65727333001053367881220235421809751254540594752243
52584907711670556013604839586446706324415722155397
53697817977846174064955149290862569321978468622482
83972241375657056057490261407972968652414535100474
82166370484403199890008895243450658541227588666881
16427171479924442928230863465674813919123162824586
17866458359124566529476545682848912883142607690042
24219022671055626321111109370544217506941658960408
07198403850962455444362981230987879927244284909188
84580156166097919133875499200524063689912560717606
05886116467109405077541002256983155200055935729725
71636269561882670428252483600823257530420752963450

---

解决您的问题

首先，给变量起实际名称，y并不是一个好的参数名称，并且在函数内部也不能重用x。数字长度应为5而不是13。我还删除了sub_list变量，因为您只使用了一次，因此无需为其使用变量，并删除了一个无操作的else。

def count_nums(items, n=5):
    largest_product = 0

    while len(items) >= n:
        product = 1
        for item in items[:n+1]:  
            product *= item
        if product > largest_product:
            largest_product = product
        items.pop(0)

    return largest_product


x = "7316717653133062491922511967442657474235534919493496983520312774506326239578318016984801869478851843858615607891129494954595017379583319528532088055111254069874715852386305071569329096329522744304355766896648950445244523161731856403098711121722383113622298934233803081353362766142828064444866452387493035890729629049156044077239071381051585930796086670172427121883998797908792274921901699720888093776657273330010533678812202354218097512545405947522435258490771167055601360483958644670632441572215539753697817977846174064955149290862569321978468622482839722413756570560574902614079729686524145351004748216637048440319989000889524345065854122758866688116427171479924442928230863465674813919123162824586178664583591245665294765456828489128831426076900422421902267105562632111110937054421750694165896040807198403850962455444362981230987879927244284909188845801561660979191338754992005240636899125607176060588611646710940507754100225698315520005593572972571636269561882670428252483600823257530420752963450"

x = [int(y) for y in x]
print(count_nums(x))

---

我的解决方案

我将提出另一项改进：使用范围进行迭代而不是弹出项目。

def count_nums(items, n=5):
    largest_product = 0

    for i in range(len(items)-n):
        product = 1
        for x in items[i:i+n]:
            product *= x
        if product > largest_product:
            largest_product = product

    return largest_product


x = "7316717653133062491922511967442657474235534919493496983520312774506326239578318016984801869478851843858615607891129494954595017379583319528532088055111254069874715852386305071569329096329522744304355766896648950445244523161731856403098711121722383113622298934233803081353362766142828064444866452387493035890729629049156044077239071381051585930796086670172427121883998797908792274921901699720888093776657273330010533678812202354218097512545405947522435258490771167055601360483958644670632441572215539753697817977846174064955149290862569321978468622482839722413756570560574902614079729686524145351004748216637048440319989000889524345065854122758866688116427171479924442928230863465674813919123162824586178664583591245665294765456828489128831426076900422421902267105562632111110937054421750694165896040807198403850962455444362981230987879927244284909188845801561660979191338754992005240636899125607176060588611646710940507754100225698315520005593572972571636269561882670428252483600823257530420752963450"

x = [int(y) for y in x]
print(count_nums(x))