dracaryS 92 Posted October 1, 2019 Share Posted October 1, 2019 M2 Download Center This is the hidden content, please Sign In or Sign Up ( Internal ) This is the hidden content, please Sign In or Sign Up ( GitHub ) Hi this is timer for python. Like queue in gamesource. 35 1 1 6 14 Link to comment Share on other sites More sharing options...
Forum Moderator VegaS™ 10275 Posted October 2, 2019 Forum Moderator Share Posted October 2, 2019 On 10/2/2019 at 1:32 AM, dracaryS said: class QueuePython(object): def __init__(self): self.eventList=[] def __del__(self): if len(self.eventList) > 0: self.eventList.clear() def Process(self): if len(self.eventList) > 0: for j in xrange(len(self.eventList)): if app.GetTime() > self.eventList[j]["time"]: i = self.eventList[j]["func"]() if i == 0: self.eventList[j].clear() else: self.eventList[j]["time"] = app.GetTime()+i def AppendEvent(self,name,func,time): self.eventList.append({"name":str(name),"func":__mem_func__(func),"time":int(app.GetTime()+time)}) 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. 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. 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. Spoiler 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> 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. 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: Spoiler del self.eventList[j] _______________________________________ 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 method type. 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. Spoiler 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') _______________________________________ The code: This is the hidden content, please Sign In or Sign Up PS: Don't quote this reply, will be updated. 9 4 17 Check my GitHub Profile Click to find all the threads started by me [TOOL] Text file loader + JSON Link to comment Share on other sites More sharing options...
Premium Hunger 472 Posted October 2, 2019 Premium Share Posted October 2, 2019 @VegaS™ sometimes i cannot understand why u spend so much time to write the comments that nobody reads 1 1 3 Link to comment Share on other sites More sharing options...
Premium filipw1 1931 Posted October 2, 2019 Premium Share Posted October 2, 2019 3 godziny temu, Anix napisał: @VegaS™ sometimes i cannot understand why u spend so much time to write the comments that nobody reads To look somewhat smart I actually read most of them as he has good coding habits Link to comment Share on other sites More sharing options...
Premium Hunger 472 Posted October 2, 2019 Premium Share Posted October 2, 2019 14 minutes ago, filipw1 said: To look somewhat smart I actually read most of them as he has good coding habits He knows aye, no doubt. i just like to bother him. Link to comment Share on other sites More sharing options...
Forum Moderator VegaS™ 10275 Posted October 10, 2019 Forum Moderator Share Posted October 10, 2019 V1 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 method type. 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 This is the hidden content, please Sign In or Sign Up library (i heard that some people don't have it) and using builtin functions, instead of types.MethodType now we're using callable(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: t.AppendEvent(eventName='RUN', eventStartTime=5, eventRunCount=10, eventFunc=self.Run, eventFuncArgs=player.GetLevel()) The following things will happen: The function Run(args), will start to run in 5 seconds for 10 times. PS: Check my first reply for code. 3 3 7 Check my GitHub Profile Click to find all the threads started by me [TOOL] Text file loader + JSON Link to comment Share on other sites More sharing options...
Recommended Posts