En général il est déconseillé d'utiliser des objet mutable comme valeur par défaut (car c'est au final une variable globale utilisée par tous les appels à la fonction/méthode).
resolvconf=open("/etc/resolv.conf","r")
Il serait mieux d'utiliser with, ce qui fermerait proprement et automatiquement le fichier.
# Petite revue de code...
Posté par GuieA_7 (site web personnel) . En réponse au journal Mon premier projet open source. Évalué à 6.
...parce que ça m'amuse.
En général il est déconseillé d'utiliser des objet mutable comme valeur par défaut (car c'est au final une variable globale utilisée par tous les appels à la fonction/méthode).
Il serait mieux d'utiliser
with, ce qui fermerait proprement et automatiquement le fichier.Sinon ça me parait OK.