Discussion :

[Balbuzard]

Bonjour!

j'ai un script qui marche, et en 50 lignes parse un fichier de configuration plutôt complexe;

J'aimerais ne pas me trainer ce script dans les autres scripts qui utilisent ce fichier de conf, et du coup j'ai pensé à le mettre dans une classe à part et de créer un fichier Configuration lorsque j'en ai besoin.

J'ai rajouté "class Configuration:" puis mis les attributs de class dans la méthode __init__, au final ça donne:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Configuration(object):
 
  def __init__(self):
    self.config_file = "/usr/local/scripts/servers.conf" 
    self.hosts_services = list()
    self.configuration = dict()
    self.groups = dict()
 
  def get_conf(self):
    # et là le travail se fait
 
  def says_blah(self):
    # Ça c'est pour me ramener à un cas trivial
    print "blah"

dans un shell python, j'importe le module (la classe est dans un fichier à part) et de ce module ma classe:

Code :

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

1
2
3
4
5
6
>>>from config import Configuration
>>>c = Configuration()
>>>c.says_blah()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: unbound method says_blah() must be called with Configuration instance as first argument (got nothing instead)

Je ne comprends pas, la ligne c = Configuration() a bien créé une instance? De toute manière les autres tentatives

Code :

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

1
2
3
4
5
6
7
8
says_blah(c)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'says_blah' is not defined
>>> Configuration.says_blah(c)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: unbound method says_blah() must be called with Configuration instance as first argument (got type instance instead)

donnent les mêmes erreurs ou bien ne sont carrément pas valables :-S

Quelqu'un pourrait me donner une piste?

La POO est souvent présentée comme quelque chose qui simplifie la vie, et pourtant, quelque soit le language, les premiers pas sont durs

Merci!

[VinsS]

Salut, es-tu sur que c'est le code que tu nous montres, parce que il fonctionne sans problème (si on ajoute 'pass' dans la deuxième fonction)

>>> from conf import Configuration
>>> c = Configuration()
>>> c.config_file
'/usr/local/scripts/servers.conf'
>>> c.says_blah()
blah

[Sve@r]

Bonjour!

J'aimerais ne pas me trainer ce script dans les autres scripts qui utilisent ce fichier de conf, et du coup j'ai pensé à le mettre dans une classe à part et de créer un fichier Configuration lorsque j'en ai besoin.

J'ai rajouté "class Configuration:" puis mis les attributs de class dans la méthode __init__, au final ça donne:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Configuration(object):
 
  def __init__(self):
    self.config_file = "/usr/local/scripts/servers.conf" 
    self.hosts_services = list()
    self.configuration = dict()
    self.groups = dict()
 
  def get_conf(self):
    # et là le travail se fait
 
  def says_blah(self):
    # Ça c'est pour me ramener à un cas trivial
    print "blah"

dans un shell python, j'importe le module (la classe est dans un fichier à part) et de ce module ma classe:

Code :

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

1
2
3
4
5
6
>>>from config import Configuration
>>>c = Configuration()
>>>c.says_blah()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: unbound method says_blah() must be called with Configuration instance as first argument (got nothing instead)

Je ne comprends pas, la ligne c = Configuration() a bien créé une instance? De toute manière les autres tentatives

Code :

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

1
2
3
4
5
6
7
8
says_blah(c)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
NameError: name 'says_blah' is not defined
>>> Configuration.says_blah(c)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: unbound method says_blah() must be called with Configuration instance as first argument (got type instance instead)

donnent les mêmes erreurs ou bien ne sont carrément pas valables :-S

Quelqu'un pourrait me donner une piste?

Bonjour

Sauf si c'est pour ta culture perso, pourquoi réinventer la roue ? Tu as à ta disposition le module ConfigParser qui te permet d'analyser un fichier de config type windows, structuré ainsi

[chapitre1]
param1=val1
param2=val2
param3=val3

[chapitre2]
param1=val1
param2=val2
param3=val3

[chapitre3]
param1=val1
param2=val2
param3=val3

Et tu l'utilises ainsi

Code python :

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

1
2
3
4
5
import configParser
parse=configParser.configParser()
parse.read(fichier)
chap3_param2=parse.get("chapitre3", "param2")
...etc etc etc

Comme tu vois, tout est déjà fait...

...parse un fichier de configuration plutôt complexe;

Evidemment, si ton fichier de config n'est pas dans le standard de ce qui se fait habituellement, tu ne peux plus utiliser ce module qui s'appuie sur le standard.

La POO est souvent présentée comme quelque chose qui simplifie la vie, et pourtant, quelque soit le language, les premiers pas sont durs

Bah non. La POO est un concept. Une fois bien en main, son implémentation dans un langage ou un autre (sauf trucs vraiment chiadés avec polymorphisme ou autres) n'est pas vraiment compliqué. C'est prendre en main le concept qui peut être ardu oui, pas le langage.

[Balbuzard]

Bonjour!

Je ne connaissais pas le module ConfigParser Sve@r; En effet ça va sacrement simplifier la chose, reste à savoir comment en utiliser la syntaxe pour mettre l'information dedans.

J'ai compris mon erreur pour mes classes, chaque valeur globale doit être préfixée de "self." Maintenant tout baigne.

Merci pour votre aide à tous les deux!

PS: Voici actuellement l'état de mon fichier de configuration:

Code :

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

1
2
3
4
5
host server_1
    services = apache,mysql
      users = lui,moi,elle
    services = ftp
        users = nous

Il y a comme des branches, pour l'instant, pas évident d'utiliser ConfigParser pour le parser :-S

[kango]

bonjour,

des alternatives:

http://www.voidspace.org.uk/python/configobj.html
http://www.red-dove.com/config-doc/
http://pyyaml.org/wiki/PyYAMLDocumentation

et j'en oublie très certainement

[Balbuzard]

Génial, je regarde ça, merci beaucoup kango!

[fatbob]

Perso, j'utilise plutôt pyyaml.
Il est simple et permet de gérer des structures de données complexes (dictionnaires et listes imbriqués...)

[Sve@r]

Bonjour!

Je ne connaissais pas le module ConfigParser Sve@r; En effet ça va sacrement simplifier la chose, reste à savoir comment en utiliser la syntaxe pour mettre l'information dedans.

J'ai compris mon erreur pour mes classes, chaque valeur globale doit être préfixée de "self." Maintenant tout baigne.

Arf oui. Une variable préfixée de self devient membre de la classe et tous les modules de la classe y ont accès. Toutefois, les éléments extérieurs à la classe y ont aussi accès
Exemple:

Code Python :

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

1
2
3
4
5
6
7
class toto:
	def __init__(self):
		self.val=125
 
a=toto()
print a.val			# Affichera 125
a.val=250			# Modifiera le membre val de la variable a

Pour éviter ce soucis d'accès parfois embêtant, on peut alors préfixer la variable de deux underscores __. Ca privatise la variable qui n'estplus accessible de l'extérieur
Exemple:

Code Python :

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

1
2
3
4
5
6
7
class toto:
	def __init__(self):
		self.__val=125
 
a=toto()
print a.__val		# Erreur
a.__val=250		# Erreur

a marche bien entendu aussi avec les méthodes qu'on peut ainsi rendre privées permettant de faire des outils pour la classe elle-même mais invisibles de l'extérieur

PS: Voici actuellement l'état de mon fichier de configuration:

Code :

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

1
2
3
4
5
host server_1
    services = apache,mysql
      users = lui,moi,elle
    services = ftp
        users = nous

Il y a comme des branches, pour l'instant, pas évident d'utiliser ConfigParser pour le parser :-S

Code :

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

1
2
3
4
5
6
7
8
9
[service1]
hostname=server_1
services=apache,mysql
users=lui,moi,elle
 
[service2]
hostname=server_1
services=ftp
users=nous