Discussion :

[jurassic pork]

hello,
je suis plutôt novice en python et je suis loin de connaître toutes les subtilités de ce langage. Mon problème est le suivant :
je dispose d'un résultat qui est une liste de tuples comme celle-ci :

le but du code c'est de repérer dans les tuples ceux qui ont en premier élément un 1 et suivant la valeur du deuxième élément de remplir un tableau.
Voici le code que j'ai écrit et qui fonctionne (data représentant les données ci dessus) :

Code :

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

1
2
3
4
5
6
7
8
9
 bits = []
        for a,b in data:
        	print(b)
        	if a ==1 and 20 <= b <= 28:
        		bits.append(0)
        	if a ==1 and 65 <= b <= 75:
        		bits.append(1)
        print("longueur bits : %d " % len(bits))
        print(bits)

je me dis qu'il doit y avoir une façon plus élégante de traiter le problème. Qu'en pensez-vous ?

Ami calmant, J.P

[flapili]

bonjour, pas grand chose à changer pour moi ( je suis pas un pro du python hein ^^ )

personnellement je ferait ç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
 
bits = ""
if data:
	for a,b in data:
		print(b)
		if a:
			if b in range(20, 28)
				bits += "0"
			if b in range(65, 75)
				bits += "1"
	print("longueur bits : %d " % len(bits))
	print(bits)
else:
	print("aucune donnée")

plutôt que d'utiliser if a == 1: / if a != 0: / if a == True:, on préfèrera if a: ( ou if not a: si a est False / 0 / -0 / 0.0 / ect).

ATTENTION : si a est différant de 0 : a vaut True (et non pas si a vaut 1) en gros si a=0 / a =[] / a =() / a ={} / a ="" -> False, si a vaut autre chose: a=12 , a=[""] , a ="0" , a="False" ...en vrai faut être débile pour avoir une chaine de caractère "False" vus que elle vaut True... En cas de doute tu peut utiliser bool(variable) dans l'IDLE , puis après if variable: ou if not variable:.

aussi un simple if data: permet de vérifier que la liste n'est pas vide.

Après que bits soit une liste ou une chaine de caractère ça revient au même ici, for i in bits: retournera le même résultat, l'avantage d'une chaine de caractère est que tu peut directement l'enregistrer dans un fichier puis a relire directement (alors que avec une liste tu aura si tu relie une chaine de caractère du style "[1,0,1, ect...]"

Après si dans ton programme t'as besoin obligatoirement d'une liste et pas d'une chaine de caractère utilise une liste, mais vus qu'avec une boucle for on passe de l'un à l'autre (si une fonction existe pour convertir une liste en str et vice versa je suis preneur ^^ )

sinon pour passer de l'un à l'autre :

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
 
def list_to_str(list):
	str = ""
	for i in list:
		str += i
	else:
		print("liste vide")
	return str
 
def str_to_list(str):
	list = []
	for i in str:
		list.append(i)
	else:
		print("str vide")
	return list

y'a pas de raison que ça marche pas, même si je les ais pas testées

[__dardanos__]

Salut,

je me dis qu'il doit y avoir une façon plus élégante de traiter le problème. Qu'en pensez-vous ?

Avec itertools.groupby :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
>>> l = [(1,3),(0,5),(1,2),(2,0),(1,1),(2,2)]
>>> from operator import itemgetter as ig
>>> l.sort(key=ig(0))
>>> from itertools import groupby
>>> for k, g in groupby(l, key=ig(0)): print(k, list(g), sep="\n")
... 
0
[(0, 5)]
1
[(1, 3), (1, 2), (1, 1)]
2
[(2, 0), (2, 2)]
>>>

[flapili]

Avec itertools.groupby :

Le but de l'auteur est de convertir une liste en turple en binaire de la forme (x , y) x vaut soit 1 soit 0, et y soit entre (20,28) soit (65,75)
du code est généré que si x =1, et après 0 si y dans (20,28), 1 dans (65,75)

Je voit mal comment ton code peut marcher sans même savoir dans quelle intervalle est y ...

[BufferBob]

salut,

une méthode que je trouve à titre personnel élégante et/ou résolument pythonesque :

Code :

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

bits = [0 if 20 <= b <= 28 else 1 for a, b in data if a == 1 and (20 <= b <= 28 or 65 <= b <= 75)]

la principale différence c'est qu'on effectue 2 fois le test 20 <= b <= 28, ça n'est donc probablement pas mieux que la version dépliée, ça a juste le mérite d'être concis.

[fred1599]

Le but de l'auteur est de convertir une liste en turple en binaire de la forme (x , y) x vaut soit 1 soit 0, et y soit entre (20,28) soit (65,75)
du code est généré que si x =1, et après 0 si y dans (20,28), 1 dans (65,75)

Je voit mal comment ton code peut marcher sans même savoir dans quelle intervalle est y ...

C'est une bonne méthode, son code n'est pas là pour présenter la solution, mais plus pour le guider vers sa solution. Ça demande une réflexion supplémentaire, mais à le mérite de faire réfléchir, ce que ne fait pas une solution toute faîte.

[jurassic pork]

hello,
merci pour vos réponses. Je m'aperçois que mon idée de départ n'était pas si mauvaise que çà. Même si il y a des solutions plus concises ou plus performantes, le gain ne sera pas trop important et la lisibilité sera plus difficile.

Ami calmant, J.P