Problème avec les dico

j'ai une question simple mais qui m'intrigue depuis quelques jours :
mon objectif c'est de faire du reporting, c'est à dire extraire le résultat de la commande "last" et ranger les éléments dans un dico : login, date de connexion, nombre de fois connecté...

voila une partie de mon code :

Code:

---

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

#!/usr/bin/python import re, os resultat = os.popen("last") connexion = {} for ligne in resultat: extraction = re.search("^([a-z]*).*[A-Z]...([A-Z]....\d*)", ligne) if extraction : login = extraction.group(1) date = extraction.group(2)   if connexion.has_key(login) : connexion[login] = connexion[login] + 1 else : connexion[login] = 1   print connexion

---

en effet cela marche bien, mais je comprend pas comment le dictionnaire se rempli par les logins sachant que je l'ai juste instancier au debut vide. j'ai l'impression qu'il y a quelque chose qui se passe au niveau de la methode "has_key()"

Merci par avance,

Bonsoir,

Ce n’est pas au niveau de dict.has_key() que se situe la “magie”, mais au moment de l’assignation (connexion[login] = 1): celle-ci va créer l’entrée demandée, si elle n’existe pas déjà…

En bonus, plutôt que ce verbeux if/else, pourquoi ne pas utiliser setdefault()*?

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#!/usr/bin/python import re, os resultat = os.popen("last") connexion = {} for ligne in resultat: extraction = re.search("^([a-z]*).*[A-Z]...([A-Z]....\d*)", ligne) if extraction : login = extraction.group(1) date = extraction.group(2)   connexion[login] = connexion.setdefault(login, 0) + 1   print connexion

---

PS*: Attention aux indentations, mieux vaut rester logique et cohérent*! :P

Bonjour,

Juste une petite note sur has_key():
Un des changements dans Python 3 c'est dict.has_key(key) > key in dict mais de manière générale je dirais qu'il est préférable d'utiliser in dans tous les cas, lorsque c'est possible.

@+

Salut,

Personnellement, je préfère la construction:

Code:

---

1 2 3

if login not in connexion: connexion[login] = 0 connexion[login] += 1

---

C'est exactement "pareil", mais je trouve que la lecture en est plus facile.

- W

Bonsoir wiztricks,

C'est bien dans ce sens que j'ai citer in:
1) C'est lisible.
2) C'est 'rapide'
3) C'est universel
4) etc, etc, etc.
C'est votre exemple de code que j'utilise dans ce genre de cas.

@neeoo11
En soit c'est bien de connaître ce mécanisme interne (méthode) des dico. La solution de wiztricks est bien plus lisible non ?

@mont29
C'est une grande mode que de tout mettre sur une ligne (faute sans doute aux list comprehension et autre) mais n'est il pas plus lisible de rester aux bases ?

@+