Discussion :

[fma38]

Bonjour,

Dans le cadre de mon framework python, j'aimerais pouvoir faire la chose suivante :

Code :

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

1
2
3
@trigger.group(ga="1/1/1").changed()
def rule(event):
    pass

mais visiblement, ce n'est pas possible :

Code :

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

1
2
3
4
5
$ python decorators.py 
  File "decorators.py", line 117
    @trigger.group(ga="1/1/1").changed()
                              ^
SyntaxError: invalid syntax

Est-ce que c'est parce que je n'utilise pas la syntaxe kivabien, ou est-ce vraiment impossible de faire ce genre de chose ?

Dans ce dernier cas, que me proposeriez-vous ? Je peux bien sûr faire :

Code :

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

1
2
3
@trigger.group.changed(ga="1/1/1")
def rule(event):
    pass

mais c'est moins sioux, car changed() prendra d'autres argument, et ga n'est pas vraiment en relation direct avec cette partie du décorateur... J'aurais aimé bien séparer les choses, tout en conservant une syntaxe sympa.

Merci d'avance pour vos suggestions.

PS : pour info, voici comment je comptais procéder pour l'implémentation des décorateurs :

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
18
19
class Group():
    def __call__(self, ga=None, id=None):
        # TODO: save ga and id, to be able to use them in the decorating methods
        return self
 
    def changed(self, from_=None, to=None):
        def decorated(func):
            # TODO: register func in the triggering system
            return func
 
        return decorated
 
 
class Trigger():
    def __init__(self):
        self.group = Group()
 
 
trigger = Trigger()

Le truc était de transformer Group an callable, se retournant lui-même, afin de pouvoir accéder à ses méthodes décoratrices...

Chuis déçu que ça ne puisse pas fonctionner Bouuh !

[fma38]

Bon, en faisant ceci :

Code :

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

1
2
3
4
5
trig = trigger.group(ga="1/1/1").changed()
 
@trig
def rule(event):
    pass

ça marche, mais bon, ça perd quand même beaucoup de son charme

D'autres suggestions ?

[dividee]

On dirait qu'il s'agit bien d'une limitation du parseur Python. Dans la spécification de la grammaire, on trouve d'ailleurs ceci:

Code :

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

decorator: '@' dotted_name [ '(' [arglist] ')' ] NEWLINE

Ce qui confirme cette limitation.
Je ne vois pas de bonne raison à priori, sinon pour simplifier l'analyse syntaxique.
Ma seule suggestion est de ne pas utiliser de décorateur:

Code :

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

1
2
3
def rule(event):
    pass
rule = trigger.group(ga="1/1/1").changed()(rule)

[fma38]

Bon OK pour la syntaxe. Dommage, tant pis !

Je peux effectivement faire comme tu préconises, mais c'est moins sympa

En fait, si je pars là dessus, je n'ai même plus besoin de décorateur du tout : le but est simplement d'enregistrer la fonction dans un table, qui permettra par la suite d'appeler cette fonction suivant des évènements (temps, changement d'état d'un objet KNX ou du système)...

C'est avant tout la syntaxe des décorateur qui m'intéressait (qui m'intéresse toujours !). Faut que je trouve une autre façon de faire, qui reste quand même conviviale, en limitant le nombre de choses à faire...

Je vais réfléchir aussi à une autre voie, en définissant une classe et des méthodes, plutôt qu'une simple fonction. Y'a peut-être plus de souplesse à ce niveau.

Genre métaclass (bien que ce soit un truc que je n'ai jamais utilisé) ? Qu'en pensez-vous ?

Je rappelle que le but est de fournir quelque chose qui soit facilement utilisable par des non développeurs, et également qui soit clair à relire. Ces scripts de rules seront une bonne partie du coeur de l'appli finale, permettant d'étendre les fonctionalités du bus domotique (genre simulation de présence, ou faire des scénarios complexes, des séquences, etc...).

[fma38]

J'ai trouvé une autre façon de contourner le problème, mais qui reste un peu du bricolage :

Code :

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

1
2
3
@eval("trigger.group(ga='1/1/1').changed()")
def rule(event):
    pass

[wiztricks]

Code :

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

1
2
3
4
5
trig = trigger.group(ga="1/1/1").changed()
 
@trig
def rule(event):
    pass

ça marche, mais bon, ça perd quand même beaucoup de son charme

Je ferais peut etre:

Code :

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

1
2
3
4
5
group = trigger.group(ga="1/1/1")
 
@group.changed()
def rule(event):
    pass

Il faudrait peut etre regarder du cote des frameworks qui realisent une syntaxe declarative (SQLAlchemy par exemple. Ca pourrait vous donner des idees.
- W

[fma38]

Effectivement, cette forme est pas mal. J'envisage même de pouvoir passer une liste de GA au constructeur...

Je vais regarder SQLAlchemy, pour voir les décorateurs disponibles.

Merci !

[wiztricks]

Je vais regarder SQLAlchemy, pour voir les décorateurs disponibles.

SQLAlchemy, c'est pour le cote interface "declarative" pas pour les decorateurs.
- W

[fma38]

Ah, au temps pour moi....