我有将日志发送到公开API的类。现在我想要发生的是在列表中保存/保存失败的日志,以便可以再次将其重新发送到服务器。
这是我到目前为止所拥有的。 您可能会注意到我已经声明了一个类变量,并且我read这是不可取的。
我担心的是,是否有更好的方法来保留列表或队列?
from collections import *
import time
import threading
import requests
import json
URL_POST = "http:/some/url/here"
class LogManager:
listQueue = deque()
def post(self, log):
headers = {"Content-type": "application/json",
"Accept": "text/plain", "User-Agent": "Test user agent"}
resp = requests.post(URL_POST, data=json.dumps(log), headers=headers)
return resp.status_code
def send_log(self, log):
try:
print ("Sending log to backend")
self.post(log)
except: # sending to server fails for some reason
print ("Sending logs to server fail, appending to Queue")
LogManager.listQueue.append(log)
def resend_log(self, log):
print ("checking if deque has values")
if LogManager.listQueue:
logToPost = LogManager.listQueue.popleft()
try:
self.post(logToPost)
print ("resending failed logs")
except: #for some reason it fails again
LogManager.listQueue.appendleft(logToPost)
print ("appending log back to deque")
def run(self,log):
t1 = threading.Thread(target=self.send_log, args=(log,))
t2 = threading.Thread(target=self.resend_log,args=(log,))
t1.start()
time.sleep(2)
t2.start()
t1.join()
t2.join()
if __name__ == "__main__":
while True:
logs = LogManager()
logs.run({"some log": "test logs"})
答案 0 :(得分:1)
由于LogManager存储的唯一变量是持久变量,因此似乎不需要每次都重新实例化该类。我可能会将logs = LogManager()
行移到while循环之外,并将list_queue
更改为在类'中创建的实例变量self.list_queue
。 __init__
方法。这样,您将拥有一个LogManager,其中一个队列每个循环只调用其run
方法。
也就是说,如果你将它保留在循环中,因为你的类将具有需要每个循环的新实例的更多功能,那么使用类变量来跟踪实例之间的列表正是类变量的用途。 。它们根本不是不可取的;您链接到的文档中给出的示例很糟糕,因为他们使用类变量来处理应该特定于实例的数据。