Avis sur une manipulation de listes ?

Version imprimable

Bonjour,

J'apprend Python sur le tas. Je m'en sers généralement pour écrire des petits scripts de traitement où je manipule des listes. Sauf que je n'ai pas beaucoup de recul sur mon code. Ca me semble simple et efficace mais il y a peut-être bien mieux à faire : c'est pour ça que je viens vous demander un avis.

Voici le contenu d'un fichier :
0
SwitchDeviceListener.deviceRegistered()
OFF
SwitchDeviceListener.deviceRegistered()
OFF
110
78
ON
390
ON
78
OFF
437
OFF
79
ON
421
ON
94
OFF
421
OFF
95
ON
405
[...]
L'objectif est d'obtenir les valeurs min, max, moyen. C'est écrit ce script pour cela :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 from statistics import mean, median # Read file content f = open("C:\\Users\\pgradot\\Desktop\\data.txt", "r") lines = f.readlines() f.close() # Clean content to_exclude = ["SwitchDeviceListener.deviceRegistered()\n", "ON\n", "OFF\n"] cleaned = [line for line in lines if line not in to_exclude] print(cleaned) # Convert to numbers numbers = list(map(int, cleaned)) print(numbers) # Print statistics print("Min = ", min(numbers)) print("Max = ", max(numbers)) print("Mean = ", mean(numbers)) print("Median = ", median(numbers))
J'aurais souhaité avoir un regard de gens plus expérimentés en Python sur ce code. Notamment sur la ligne pour obtenir cleaned car je fais très souvent ce genre d'écritures et je me demande s'il n'y a pas mieux à écrire.

Merci pour vos avis ! :ccool:

Salut,

Votre code est un peu trop simple pour se prendre le chou avec.
Après on peut s'amuser à être plus concis mais c'est une question de style, goût...
Que pensez vous de:
Code:

1 2 3 4 5 6 7 8 9 10 11 from statistics import mean, median with open("C:\\Users\\pgradot\\Desktop\\data.txt") as f: numbers = [ int(z) for z in f.read().splitlines() if z.isdigit() ] print(numbers) # Print statistics print("Min = ", min(numbers)) print("Max = ", max(numbers)) print("Mean = ", mean(numbers)) print("Median = ", median(numbers))
- W

07/12/2014, 17h39
__dardanos__
Salut.
Je préfère utiliser un générateur pour lire des données, plutôt qu'une liste en intention. Je trouve la syntaxe plus lisible.
Code:

1 2 3 4 5 import re def get_all(filename): with open(filename, "r") as fobj: return re.findall(r"^\d+$", fobj.read(), re.M)

Je suis aussi pour la méthode isdigit, autre solution possible

Code:

1
2
3
4
5
6
7
8
9
10
11
numbers = []with open("C:\\Users\\pgradot\\Desktop\\data.txt", 'r') as f:
    for line in f:
        try:
            value = int(line)
        except ValueError:
            value = None
        if value:
            numbers.append(value)


print(numbers)

Si les temps sont bien trop faibles pour toi, rien n'empêche de faire de l'interfaçage python/C

Voici un exemple

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
from ctypes import *

libc = cdll.LoadLibrary("libc.so.6") # sous Unix, pour windows voir msvcrt


numbers = []


with open("C:\\Users\\pgradot\\Desktop\\data.txt", 'r') as f:
    for line in f:
        l = bytes(line, 'utf8') # python3 on travaille avec des bytes
        val = libc.strtol(l, None, 10) # retourne 0 si pas un entier
        if val:
            numbers.append(val)


print(numbers)

Je n'ai pas fais les mesures de temps...