Çok eski bir web tarayıcısı kullanıyorsunuz. Bu veya diğer siteleri görüntülemekte sorunlar yaşayabilirsiniz.. Tarayıcınızı güncellemeli veya alternatif bir tarayıcı kullanmalısınız.
Thanks for release, the idea isn't bad, but there're some bad things.
I'll show you the problems part and how can be improved, there're just advices, i hope you'll get them.
Python:
def __del__(self):
if len(self.eventList) > 0:
self.eventList.clear()
If you're using Python 2+ or Python 3.2 and below, you can't use the clear() method (allowed on 3.3+), also as i said in the second message you don't need to check the length of the list, already the clear() method doing that inside and there's no reason to put it to __del__ method, it will be called when the object is garbage collected. if you really want to use in future, something outside of this and want just to check the list if isn't empty, is enough to do it just with if some_list, like a normal boolean, there no need to check the length of the list if you don't use it in your code.
Python:
if len(self.eventList) > 0:
for j in xrange(len(self.eventList)):
[...]
You don't have to check the list if you already did a numeric range loop or iterator based loop.
Python:
self.eventList = []
''' example 1 '''
for i in xrange(len(self.eventList)):
print (self.eventList[i])
''' example 2 '''
for event in self.eventList:
print (event)
#<nothing will be printed>
Python:
app.GetTime() + time
I would say to use app.GetGlobalTimeStamp() instead of app.GetTime(), if you teleport while the event is running, the event function will run after 10 seconds like. While app.GetGlobalTimeStamp() will run after the specific time, because is the server timestamp and is updated on each enter in game.
Python:
if i == 0:
self.eventList[j].clear()
I would put here an big exclamation, with this you creating 999999999 lines in syserr, what you do here is like:
While Process() function is called in OnUpdate, so, your condition trying to get the returned value from an specific function, what means the next update time or 0 to destroy the event/clear it. Everything's fine until you return 0 and event should be stopped yes? But there is a problem, you clear the specific dictionary of event and still remained in the list [{}], and the Process() function will take your self.eventList with the items included, the empty dictionaries from your events, and of course even if you've [{}, {}, {}], that doesn't mean your list is empty, have 3 items, so, the loop will trying to read an empty dictionary and you'll get key errors in each milisecond. The method which you need is to delete the dictionary itself from the list after the result value from the function is 0, like this:
Python:
del self.eventList[j]
[/SPOILER]
_______________________________________ I wrote fast some self extensions, if somebody is interested i'll do another updates in the next days.
You can use unlimited arguments on functions, now is using the apply method which returns the result of a function or class object called with supplied arguments, with the old structure you could use just one argument.
You can lock/unlock an event for being processed, it's like a prevent in some actions, if the event is created and you want to do something, you should lock the event, do some actions then you can unlock it again and the process function will run where remained.
Delete an event instantly and force it to stop the process.
Adding return t.EXIT inside of the running function, will delete the event too.
Functions to check if an event exists or is locked or not.
Check if the function is a
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
.
Delete the events with a properly method.
Using app.GetGlobalTimeStamp() now will give you the chance to run the event after teleport where timer remained instantly.
Python:
t = ui.Queue()
# ex1
t.AppendEvent(eventName='RUN', eventStartTime=0, eventFunc=self.Run)
# ex2
t.AppendEvent(eventName='RUN', eventStartTime=0, eventFunc=self.Run, eventFuncArgs=player.GetLevel())
# ex3
t.AppendEvent(eventName='RUN', eventStartTime=0, eventFunc=self.Run, eventFuncArgs=(player.GetLevel(), player.GetName()))
# ex4
t.AppendEvent('UPDATE', 0, self.Update, {'data' : (1, True, (14, 12), [5, 1], 'Corsair')})
# Others:
if t.GetEvent('RUN'):
print ("The event exists.")
if t.GetIsLockedEvent('RUN'):
print ("The event exists but is locked.")
t.LockEvent('RUN')
t.UnlockEvent('RUN')
t.DeleteEvent('RUN')
[/SPOILER]
_______________________________________ The code:
Python:
import types
class Queue(object):
EXIT = 0
def __init__(self):
self.eventList = []
self.eventLockedList = []
def __del__(self):
del self.eventList
del self.eventLockedList
def GetEvent(self, eventName):
""" Get the event dictionary by specific name. """
for event in self.eventList:
if event['name'] == eventName:
return event
return None
def GetIsLockedEvent(self, eventName):
""" Get a boolean if a specific event is locked. """
for event in self.eventLockedList:
if event['name'] == eventName:
return True
return False
def LockEvent(self, eventName):
""" Lock a specific event by name for being processed. """
event = self.GetEvent(eventName)
if event and not event in self.eventLockedList:
self.eventLockedList.append(event)
def UnlockEvent(self, eventName):
""" Unlock a specific event by name for being processed. """
event = self.GetEvent(eventName)
if event in self.eventLockedList:
self.eventLockedList.remove(event)
def DeleteEvent(self, eventName):
""" Delete a specific event by name instantly. """
event = self.GetEvent(eventName)
if event:
self.eventList.remove(event)
def AppendEvent(self, eventName, eventStartTime, eventFunc, eventFuncArgs = ()):
""" Append a new event by specific arguments. """
if not eventName or not isinstance(eventStartTime, types.IntType) or not isinstance(eventFunc, types.MethodType):
return
if self.GetEvent(eventName):
return
if not hasattr(type(eventFuncArgs), '__iter__'):
eventFuncArgs = tuple([eventFuncArgs])
self.eventList.append({'name' : eventName, 'function' : __mem_func__(eventFunc), 'arguments' : eventFuncArgs, 'run_next_time' : app.GetGlobalTimeStamp() + eventStartTime})
def Process(self):
""" Processing the events. """
for index, event in enumerate(self.eventList):
if event in self.eventLockedList:
continue
if app.GetGlobalTimeStamp() > event['run_next_time']:
result = apply(event['function'], event['arguments'])
if result == self.EXIT:
del self.eventList[index]
continue
event['run_next_time'] = app.GetGlobalTimeStamp() + result
You can use unlimited arguments on functions, now is using the apply method which returns the result of a function or class object called with supplied arguments, with the old structure you could use just one argument.
You can lock/unlock an event for being processed, it's like a prevent in some actions, if the event is created and you want to do something, you should lock the event, do some actions then you can unlock it again and the process function will run where remained.
Delete an event instantly and force it to stop the process.
Adding return t.EXIT inside of the running function, will delete the event too.
Functions to check if an event exists or is locked or not.
Check if the function is a
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
.
Delete the events with a properly method.
Using app.GetGlobalTimeStamp() now will give you the chance to run the event after teleport where timer remained instantly.
V2
Fixed non-returning time for processing, if the specific event function has no value from returning, it runs continuously.
Fixed the check if an event exist, now will be replaced with the new one.
Removed
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
library (i heard that some people don't have it) and using builtin functions, instead of
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
now we're using
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
(object), which check if the event function can be called, now you can insert classes and others callable methods, not just simple functions.
Added a reset time event function.
Next update: (when i'll have some free time again)
Insert a new type of event, which you can run an event by specific counter like:
You can use unlimited arguments on functions, now is using the apply method which returns the result of a function or class object called with supplied arguments, with the old structure you could use just one argument.
You can lock/unlock an event for being processed, it's like a prevent in some actions, if the event is created and you want to do something, you should lock the event, do some actions then you can unlock it again and the process function will run where remained.
Delete an event instantly and force it to stop the process.
Adding return t.EXIT inside of the running function, will delete the event too.
Functions to check if an event exists or is locked or not.
Check if the function is a
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
.
Delete the events with a properly method.
Using app.GetGlobalTimeStamp() now will give you the chance to run the event after teleport where timer remained instantly.
V2
Fixed non-returning time for processing, if the specific event function has no value from returning, it runs continuously.
Fixed the check if an event exist, now will be replaced with the new one.
Removed
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
library (i heard that some people don't have it) and using builtin functions, instead of
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
now we're using
Linkleri görebilmek için Turkmmo Forumuna ÜYE olmanız gerekmektedir.
(object), which check if the event function can be called, now you can insert classes and others callable methods, not just simple functions.
Added a reset time event function.
Next update: (when i'll have some free time again)
Insert a new type of event, which you can run an event by specific counter like:
