Discussion :

[EricStib]

Re-bonjour,

Le code suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
for c in string:
        tab.append(int(c))

me donne l'erreur:

invalid literal for int() with base 10: ''

string est une chaine et je voudrais transformer chaque caractère en int avant de les envoyer dans un tableau.
Je pensais que faire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

int(c)

suffisait mais il semblerait que j'ai oublié quelques chose

En vous remerciant pour l'attention porté à mon code.
ci-dessous, mon code complet.
a+

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
 
#!/usr/bin/env python
# -*- coding: utf-8 -*-
#
# problem 8 - project Euler
 
import time
 
string = """73167176531330624919225119674426574742355349194934
96983520312774506326239578318016984801869478851843
85861560789112949495459501737958331952853208805511
12540698747158523863050715693290963295227443043557
66896648950445244523161731856403098711121722383113
62229893423380308135336276614282806444486645238749
30358907296290491560440772390713810515859307960866
70172427121883998797908792274921901699720888093776
65727333001053367881220235421809751254540594752243
52584907711670556013604839586446706324415722155397
53697817977846174064955149290862569321978468622482
83972241375657056057490261407972968652414535100474
82166370484403199890008895243450658541227588666881
16427171479924442928230863465674813919123162824586
17866458359124566529476545682848912883142607690042
24219022671055626321111109370544217506941658960408
07198403850962455444362981230987879927244284909188
84580156166097919133875499200524063689912560717606
05886116467109405077541002256983155200055935729725
71636269561882670428252483600823257530420752963450
"""
 
 
def main():
    s = time.time()
 
    tab = []
    tab_rep = []
 
 
    for c in string:
        tab.append(int(c))
 
 
    i = 0
    while i <= len(tab)-5:
        prod = (tab[i] * tab[i+1] * tab[i+2] * tab[i+3] * tab[i+4])
        tab_rep.append(prod)
        i += 1
 
    print max(tab_rep)
 
 
    e = time.time()-s
    print "\nRésultat fourni en %f secondes." % (e)
    return 0
 
if __name__ == '__main__': main()

[DelphiManiac]

Une piste : Ta chaîne de caractères n'est pas composé que de chiffres.

[DelphiManiac]

Par contre, quitte à faire du python, essayes de penser "pythonien"

Ton problème se résout en 2 lignes de code.

[EricStib]

Ah, ben oui alors ! Je n'avais pas pensé à vérifié cela.
Quand j'ai refusionné string dans une chaine de caractère simple quote, cela a fonctionné direct.

Bon à savoir pour les prochaines fois

Encore un tout grand merci pour avoir attiré mon attention sur mon erreur.

Tu me dira si il conviens de supprimer mon code puisque qu'il s'agit d'un exo du project Euler.
Si c'est nécessaire j'éditerai le sujet pour supprimer le code.

a+

[DelphiManiac]

Je ne connais pas le projet Euler, j'aurais donc du mal à te dire s'il faut le retirer ou pasr

Par contre, tu as du loupé le post :

Par contre, quitte à faire du python, essayes de penser "pythonien"

Ton problème se résout en 2 lignes de code.

En gros, essayes de faire plus simple avec les outils que fourni python.

[Desintegr]

Je pense que la version en deux lignes n'est pas forcément la plus simple à comprendre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
string = '73167176...'
print max([reduce(lambda x,y: x*y, z) for z in [[int(y) for y in x] for x in [string[i:i+5] for i in range(0, len(string)-4)]]])

[DelphiManiac]

Je la voyais un tout petit peu moins 'pythonienne' !!

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
string = '73167176...'
print max([int(string[i]) * int(string[i+1]) * int(string[i+2]) * int(string[i+3]) * int(string[i+4]) for i in xrange(len(string)-4)])

En gros, ne pas oublier la puissance des list comprehension, mais ne pas en abuser non plus

Ça évite d'avoir des gros pavés de code, avec des listes intermédiaires et c'est tout autant lisible quand on s'est habitué à python.

[EricStib]

Très impressionnant !

J'essaierai de trouver une version plus "Python" pour mes futurs réponses

Pour le projet Euler, c'est ici : http://projecteuler.net/
Quand on a trouvé une réponse au problème, on a accès à un forum ou l'on peut voir le code (dans les différends langages) des participants.

Il y a un participant qui utilises Python comme vous et je suis toujours très impressionné !!

Car il faut trouver l'algo qui fonctionne et, surtout, l'optimiser pour que la résolution
du calcul se fasse sous la minute... et il y arrive dans une concision incroyable !

C'est une bonne méthode pour apprendre Python

Encore un grand merci,
a+

[DelphiManiac]

Dommage que mon niveau en math soit devenu plus que faible et les études sont déjà loin

Ça m'aurait intéresser pour le coté algorithmique par contre.

[Desintegr]

Je ne connaissais pas le Project Euler, je viens de me lancer et j'aime bien. C'est intéressant de résoudre ces petits problèmes.

J'en suis au problème n°15

J'ai remarqué jusque là que j'utilise souvent les compréhensions de listes et les générateurs. Mais bon, à chacun sa méthode