Discussion :

[wared05]

bonjour à tous,

je tente d'optimiser mon code et je butte sur ce qui doit etre pour vous un détail mais m'a pris 2h de recherche sans resultat, je suis à 2 doigts de le laisser tel quel puisque fontionnel...

j'ai ca :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
sonde1 = 14
sonde2 = 23
sonde3 = 15
 
# Mise en variable la commande OS dans les variables lire_*
lire_temp1 = "sudo nice -20 ./Adafruit_DHT 22 %d |grep Temp |awk '{ print $3 }'" % (sonde1)
lire_hum1 = "sudo nice -20 ./Adafruit_DHT 22 %d |grep Temp |awk '{ print $7 }'" % (sonde1)
lire_temp2 = "sudo nice -20 ./Adafruit_DHT 22 %d |grep Temp |awk '{ print $3 }'" % (sonde2)
lire_hum2 = "sudo nice -20 ./Adafruit_DHT 22 %d |grep Temp |awk '{ print $7 }'" % (sonde2)
lire_temp3 = "sudo nice -20 ./Adafruit_DHT 22 %d |grep Temp |awk '{ print $3 }'" % (sonde3)
lire_hum3 = "sudo nice -20 ./Adafruit_DHT 22 %d |grep Temp |awk '{ print $7 }'" % (sonde3)

que je tente de tranformer en

Code :

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

1
2
3
4
5
6
7
8
9
 
sondes = [
	[1, 14],
	[2, 23],
	[3, 15],
	]
 
for nb, pin in sondes :
	lire_temp{} = "sudo nice -20 ./Adafruit_DHT 22 {} |grep Temp |awk '{ print $3 }'" .format(nb, pin)

une piste svp...?

[VinsS]

salut,

Tu as le choix, liste ou dictionnaire:

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
 
>>> sondes = 14, 23, 15
>>> #liste
>>> commandes = []
>>> for pin in sondes:
...     commandes.append("sudo nice -20 ./Adafruit_DHT 22 %s |grep Temp |awk '{ print $3 }'" % pin)
... 
>>> print(commandes[0])
sudo nice -20 ./Adafruit_DHT 22 14 |grep Temp |awk '{ print $3 }'
>>> print(commandes[1])
sudo nice -20 ./Adafruit_DHT 22 23 |grep Temp |awk '{ print $3 }'
>>> print(commandes[2])
sudo nice -20 ./Adafruit_DHT 22 15 |grep Temp |awk '{ print $3 }'
>>>
>>> # dictionnaire
>>> commandes = {}
>>> for idx, pin in enumerate(sondes):
...     commandes['lire_temp%s' % (idx+1)] = "sudo nice -20 ./Adafruit_DHT 22 %s |grep Temp |awk '{ print $3 }'" % pin
... 
>>> print(commandes['lire_temp1'])
sudo nice -20 ./Adafruit_DHT 22 14 |grep Temp |awk '{ print $3 }'
>>> print(commandes['lire_temp2'])
sudo nice -20 ./Adafruit_DHT 22 23 |grep Temp |awk '{ print $3 }'
>>> print(commandes['lire_temp3'])
sudo nice -20 ./Adafruit_DHT 22 15 |grep Temp |awk '{ print $3 }'
>>>

[wared05]

merci vincent, je crois que la methode avec la liste n'est pas valable car je reutilise lire_temp1 plus tard dans mon script

la methode dictionnaire semble coller, je vais tenter de la comprdendre et de l'adapter

un grand merci!

[wared05]

je suis donc parti sur la solution du dictionnaire.

pourquoi passer par la liste "commandes"?

et il semble y avoir un soucis avec la variable lire_temp1 qui n'est apparement pas connu dans la suite du script

Code :

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

1
2
3
4
5
sondes = 14, 23, 15
commandes = {}
 
for idx, pin in enumerate(sondes):
	commandes['lire_temp%s' % (idx+1)] = "sudo nice -20 ./Adafruit_DHT 22 %s |grep Temp |awk '{ print $3 }'" % pin

l'idée est de passer la commande OS dans la variable lire_temp1 qui sera utilisée plus tard dans le script

[wared05]

autant pour moi ca fontionne, voici comment je l'ai mis

Code :

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

1
2
3
4
5
6
7
sondes = 14, 23, 15
temp = {}
hum = {}
 
for idx, pin in enumerate(sondes):
	temp['lire_temp%s' % (idx+1)] = "sudo nice -20 ./Adafruit_DHT 22 %s |grep Temp |awk '{ print $3 }'" % pin
	hum['lire_hum%s' % (idx+1)] = "sudo nice -20 ./Adafruit_DHT 22 %s |grep Temp |awk '{ print $7 }'" % pin

et je l'apelle plus tard

Code :

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

1
2
t1 = run_cmd(temp['lire_temp1'])				
		h1 = run_cmd(hum['lire_hum1'])

merci vincent de ton aide