Discussion :

[lg_53]

Je souhaite dériver une classe de dictionnaire pour lui ajouter de la sémantique. Admettons, j'ai un dictionaire contenant des configurations, chaque config portant un nom, les noms etant les clés du dico. Pour être plus clair vis à vis de mon utilisateur, s'il me demande une clé qui n'est pas dans le dico, je veux lui soulever une exception plus explicite qu'un KeyError. Je procède ainsi :

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
class UnexistantConfigError(KeyError): pass
 
class Configs(dict):
    def __getitem__(self, i):
        try :
            return super(Configs, self).__getitem__(i)
        except KeyError :
            raise UnexistantConfigError("%s is an unknown configuration"%i)
        except Exception as e :
            raise e
 
class ConfigA(): pass
class ConfigB(): pass
 
configs = Configs( { "1" : ConfigA, "2" : ConfigB } )
 
print(configs["3"]) ### OK : Raise UnexistantConfigError

Ca fonctionne bien, et ca fait le job voulu. Maintenant la question est : y a t il un moyen plus propre de faire cela ? Un moyen sans devoir réécrire le __getitem__ ?

[Sve@r]

Bonjour

Maintenant la question est : y a t il un moyen plus propre de faire cela ?

Supprimer ce except Exception as e : raise e qui signifie littéralement "intercepte toute autre exception pour la renvoyer comme une exception" (pourquoi alors l'intercepter ???).
Et peut-être si ton "UnexistantConfigError" est juste une exception liée à ton "Configs", l'intégrer alors comme sous-classe de "Configs"...

Un moyen sans devoir réécrire le __getitem__ ?

Pas à ma connaissance. Il me semble qu'on peut redéfinir des trucs élémentaires (comme les int, les str, etc) en jouant avec __new__ mais je ne sais pas le faire. Et est-ce que cela permettrait de supprimer ton __getitem__ (qui est quand-même nécessaire pour que ton objet puisse être invoqué via un appel avec indices) ?
Donc me semble que ta solution est déjà pas mal. Bon effectivement tu réécris le __getitem__ mais bon, Python est un beau langage mais qui nécessite tout de même parfois quelques concessions à la trop grande facilité.

[wiztricks]

Salut,

Un moyen sans devoir réécrire le __getitem__ ?

Surcharger une méthode pour en modifier le comportement (ici en attrapant l'exception levée pour la remplacer par une autre) est "normal".
Le soucis est qu'on ne la remplace pas: on en ajoute une autre.
Pourquoi ne pas juste "customiser" le message d'erreur:

Code :

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

1
2
3
4
5
6
    def __getitem__(self, i):
        try :
            return super(Configs, self).__getitem__(i)
        except KeyError as e:
            e.args = ("%s is an unknown configuration"%i,)
            raise

- W

[lg_53]

Supprimer ce except Exception as e : raise e qui signifie littéralement "intercepte toute autre exception pour la renvoyer comme une exception" (pourquoi alors l'intercepter ???)

C'est pas faux !

Pourquoi ne pas juste "customiser" le message d'erreur:

Code :

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

1
2
3
4
5
6
    def __getitem__(self, i):
        try :
            return super(Configs, self).__getitem__(i)
        except KeyError as e:
            e.args = ("%s is an unknown configuration"%i,)
            raise

Hehe, un point de la doc que j'avais oublié : l'attribut args des exceptions (faut dire que je ne m'en suis jamais servi).


Vos 2 remarques me permettent de raccourcir un peu mon code. Bon ceci dit ca reste bien long pour une si petite fonctionalité. J'avais espéré qqch de plus efficace... Comme tu dis Sve@r, on ne peut pas gagner sur tous les tableaux.

Merci à vous deux.

[Bktero]

En fait, je ne vois pas trop ce qui ne te va pas dans ton code.

Tu étends une classe, tu souhaites adapter le comportement d'une méthode, tu le fais. C'est exactement le principe d'overrider une fonction de la classe mère dans une classe fille.

Tout va bien, non ?

[Sve@r]

En dehors de la solution de wiztricks, ne peut-on pas remplacer class UnexistantConfigError(KeyError): pass par un plus simple UnexistantConfigError=KeyError ?

[wiztricks]

ne peut-on pas remplacer class UnexistantConfigError(KeyError): pass par un plus simple UnexistantConfigError=KeyError ?

On peut toujours le faire, mais çà ne changera pas le libellé de l'exception qui sera toujours KeyError.

- W