Discussion :

[Bktero]

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 :

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
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 !

[wiztricks]

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 :

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

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

[__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 :

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

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)

[fred1599]

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

Code :

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

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 :

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
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...