Discussion :

[neeoo11]

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 :

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
 
#!/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,

[mont29]

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 :

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

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

[PauseKawa]

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.

@+

[wiztricks]

Salut,

Personnellement, je préfère la construction:

Code :

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

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

[PauseKawa]

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 ?

@+