2014-10-28 5 views
-3

Я иду через эйлер проекта, и я застрял в этом вопросе.Продукт подмножества чисел в python

Я собираюсь опубликовать свой код с комментариями, чтобы каждый мог следить за моим мышлением и видеть, где я ошибся. Все предложения приветствуются :)

# need to find the largest product in a series 

import time # brings time into the code 

start = time.time() # creates a start time for the code 

list = [] # where I'm going to store the multipules 

# the '\ takes the grid and converts it into one long number' 

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

i = 0 # start of the ticker 

for i in range(0, 1000, 1): # the length of the large block of numbers is 1000 

    hold_num = int(num[i])*int(num[i+1])*int(num[i+2]) # im creating a number of every three consecutive numbers ... this is the subset 
    list.append(hold_num) # storing the products in a list 
    i =+ 1 

print max(list) # finding the max of the products in the stored list 

я в конечном итоге получаю эту ошибку:

Traceback (most recent call last): 
    File "/Users/robertdefilippi/Documents/Python/Euler/8eu.py", line 37, in <module> 
    hold_num = int(num[i])*int(num[i+1])*int(num[i+2]) 
IndexError: string index out of range 
[Finished in 0.0s with exit code 1] 

Любые предложения о том, что я делаю не так?

+2

'num [i + 2]' когда i 1000. problem.loop до 998. – vks

+1

Рассмотрите возможность использования 'zip (num, num [1:], num [2:])', чтобы получить список всех 3-кортежи, не беспокоясь о конце особых проблем. Попробуйте с помощью 'num = range (10)', чтобы увидеть, что он делает. – Alfe

ответ

0

Когда вы делаете for i in range(0, 1000, 1), достигнув 999, num[i+1] и num[i+2] будет вне диапазона.

Вместо этого вы должны использовать range(0,998,1). Которая может быть сокращена до range(998). И делая это, i будет увеличиваться и начинается с 0. Вам не нужно делать i+=1 или i=0.

+0

Использование этого '998' не является хорошим стилем; не вычисляйте во время программирования, пусть компьютер сделает это за вас. Фактически, ваш путь - это скрытый код, потому что этот 998 непосредственно получен из 1000 и 3 (и эффекта «один за другим»). Поэтому, если код будет изменен, чтобы не облегчить 3, а 6, вам придется настроить 998 на 995 и, вероятно, это будет забыто. – Alfe

Смежные вопросы