que pensez vous de ce code ?

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : que pensez vous de ce code ?

Profil supprimé

le but c'est de gerer un systeme d' execution de tâches toutes
les n secondes où à un moment précis.
ca à l'air de fonctionner mais j'aimerais bien avoir votre avis
pour peut etre l'améliorer :jap:

Code :

class Task:
def __init__(self, func, when, id, args):
"""
Arguments:
- func -- la fonction a éxecuter.
- when -- le nombre de secondes d'attente
ou l'interval si c' est une tâche
qui doit se répéter.
- id -- le nom de la tâche.
- args -- les arguments à passer à la fonction.
"""
self.func = func
self.seconds = when
self.when = time.time() + when
self.id = id
self.args = args
def run(self):
"""
Exucte la fonction.
"""
try:
if self.args:
self.func(*self.args)
else:
self.func()
except:
log.error('Error in scheduled task: #id %s, (%s)' % \
(self.id, repr(self.func)))
class Scheduler:
def __init__(self):
"""
Scheduler object.
Contient deux méthodes principales:
- addEvent: executes une tâche à un moment précis.
- addPeriodicEvent: répétes une tâche à intervals réguliers
toutes les n secondes.
"""
# contient les instances de Task équivalent
# aux taches en attente d' execution.
self.schedule = []
def addEvent(self, id, func, t, args):
"""
addEvent(id, function, seconds, ('arg1','arg2',))
"""
task = Task(func, t, id, args)
when = time.time() + t
if self.schedule:
# on trie les tâches en fonction
# de leurs secondes d'execution.
if when > self.schedule[-1].when:
self.schedule.append(task)
return
else:
self.schedule.insert(-2, task)
return
log.debug('Add new scheduled Task, #id: %s' % id)
self.schedule.append(task)
def addPeriodicEvent(self, id, func, t, args):
"""
addPeriodicEvent(id, function, seconds, ('arg1','arg2',))
"""
def event():
self.addEvent(id, func, t, args)
return self.addEvent(id, event, t, ())
return event()
def removeEvent(self, id):
log.debug('Remove scheduled Task, #id: %s' % id)
self.schedule = [i for i in self.schedule if i.id != id]
def ids(self):
"""
return les noms (id) de chaque tâches
contenus dans self.schedule.
"""
L = []
for i in self.schedule:
# on évite de les rajouter en double
# dans le cas d'un event périodique.
if not i.id in L:
L.append(i.id)
return L

Code :

S = Scheduler()
while S.schedule and S.schedule[0].when \
<= time.time():
S.schedule[0].run()
S.schedule.pop(0)
break

Message édité par Profil supprimé le 11-08-2004 à 12:54:15

Publicité

masklinn

í dag viðrar vel til loftárása

:heink:
c'est quoi cette boucle imonde qui bloque tout? je dirais d'utiliser des threads+timers ca marche très bien, ou des fonctions de déclenchement de tasks intégrées à l'OS (type cron) c'est fait pour ca.

Ensuite pour le code, à première vu en tant que programmeur (très) moyen (si Taz vient il saura surement plus t'en dire que moi... si il a le courage de le faire)

-> les arguments par défaut, def addEvent(self, id, func, t, args=()) ca évite self.addEvent(id, event, t, ()) qui est crade et ne veut pas dire grand chose
-> pourquoi devoir saisir l'ID des tâches? celui ci devrait être généré dynamiquement
-> les lists disposent d'une fonction sort() si tu veux les trier
-> j'arrive pas trop a comprendre comment fonctionne/a quoi sert addPeriodicEvent
-> dans les listes, il y a pop() et remove() si tu veux retirer des éléments, reconstruire toute la liste sans remettre l'élément à enlever c'est débile

---------------
Stick a parrot in a Call of Duty lobby, and you're gonna get a racist parrot. — Cody

Profil supprimé

Masklinn a écrit :

:heink:
c'est quoi cette boucle imonde qui bloque tout? je dirais d'utiliser des threads+timers ca marche très bien, ou des fonctions de déclenchement de tasks intégrées à l'OS (type cron) c'est fait pour ca.

elle bloque rien du tout ma boucle si il y a aucune taches
la boucle est ignorée et on passe à la suite, si il y a des tâches seulement la première est executée et break est appelé.

Sinon pour les threads, pour l'instant je continues comme ca, si je vois que ca fonctionne pas comme je veux je regarderais de plus près Timer

Ensuite pour le code, à première vu en tant que programmeur (très) moyen (si Taz vient il saura surement plus t'en dire que moi... si il a le courage de le faire)

:jap:

-> les arguments par défaut, def addEvent(self, id, func, t, args=()) ca évite self.addEvent(id, event, t, ()) qui est crade et ne veut pas dire grand chose
corrigé :jap:

-> pourquoi devoir saisir l'ID des tâches? celui ci devrait être généré dynamiquement

le problème c est si je veux les supprimer, je suis pas censé savoir quelle nom à telle tàche, bien que ce serait mieux, c'est sur.

-> les lists disposent d'une fonction sort() si tu veux les trier
euh oué :whistle:
-> j'arrive pas trop a comprendre comment fonctionne/a quoi sert addPeriodicEvent
en fait je rajoutes un event simple qui a comme fonction la sous fonction event() de addPeriodicEvent, donc à chaque fois que l'event est executé un nouvel event est crée. mais là faut tester pour comprendre, je dois surement pas étre tres clair.

-> dans les listes, il y a pop() et remove() si tu veux retirer des éléments, reconstruire toute la liste sans remettre l'élément à enlever c'est débile
oué a voir mais je crois que ca me posait problème dans le cas des events ajoutés avec addPeriodicEvent().

en tous les cas merci pour tes remarques je vais essailer d'améliorer tout ca :jap:

masklinn

í dag viðrar vel til loftárása

Citation :

elle bloque rien du tout ma boucle :heink: si il y a aucune taches
la boucle est ignorée et on passe à la suite, si il y a des tâches seulement la première est executée et break est appelé.

Si il y a une tâche, tu ne peux rien faire pendant qu'elle s'exécute
Si tu lances plusieurs tâches en simultanné elles s'exécutent l'une après l'autre
D'ailleurs t'es obligé de remplir ta liste de tâches avant de lancer l'exécution globale, tu peux pas ajouter de tâche quand le truc se déroule [:aloy]

Citation :

le problème c est si je veux les supprimer, je suis pas censé savoir quelle nom à telle tàche, bien que ce serait mieux, c'est sur.

Dans Tasks tu crée un accesseur GetId() et dans Scheduler un GetIdList(), GetId renvoie l'id de la tâche en cours et GetIdList renvoie les Ids de toutes les tâches enregistrées

Citation :

en fait je rajoutes un event simple qui a comme fonction la sous fonction event() de addPeriodicEvent, donc à chaque fois que l'event est executé un nouvel event est crée. mais là faut tester pour comprendre, je dois surement pas étre tres clair.

Ah oui, c'est bien se faire chier pour rien quand tu sais que les timers font ce genre de trucs pour toi [:itm]

Citation :

oué a voir mais je crois que ca me posait problème dans le cas des events ajoutés avec addPeriodicEvent().

Ca n'enlève qu'un élément à la fois, donc il faut boucler jusqu'a ce que tous les éléments à détruire soient partis si il y en a plusieurs

Message édité par masklinn le 11-08-2004 à 14:10:14

---------------
Stick a parrot in a Call of Duty lobby, and you're gonna get a racist parrot. — Cody

FORUM HardWare.fr

Programmation

Python

que pensez vous de ce code ?

Sujets relatifs
[Access]cacher le code	Récupérer un code SQL
Intégrer du code (x)html à un autre xml	[Rationnal Rose][Java]Reverse de code source
Exécuter une partie de code sur une même page en utilisant un bouton	Génération de documentation de code C++
inclure du code javascript en vb.net	cadrer une photo sur un visage humain.. sdk? bout de code?
[jsp] vous pensez quoi de mon code ???	Que pensez vous de ce code
Plus de sujets relatifs à : que pensez vous de ce code ?

Page générée en 0.061 secondes