Django, modèle Borg, appels d'API, résultats de la mise en cache

Question

J'utilise une API d'un site différent qui renvoie deux adresses URL de prix que mes utilisateurs utilisent pour acheter des biens virtuels.

Je suis censé mettre en cache ces résultats pendant au moins une heure, car ils ne modifient pas beaucoup les prix sur leur système. (Et nous voulons sauver nos deux années et leur bande passante.)

Après avoir cherché années singleton en Python j'ai découvert le modèle de borg, qui semble encore plus cool, donc ce que je l'ai fait:

def fetchPrices(): 
    #uses urllib2.urlopen() to fetch prices 
    #parses results with ElementTree 
    return prices 

class PriceStore(): 
    __shared_state = {} 

    def update(self): 
     if self.lastUpdate is not None and (datetime.now() - self.lastUpdate).seconds >= 3600: 
      self.prices = fetchPrices() 
      self.lastUpdate = datetime.now() 
     elif self.lastUpdate is not None: 
      return 
     else: 
      self.lastUpdate = datetime.now() - timedelta(hours=1) 
      self.update() 

    def __init__(self): 
     self.__dict__ = self.__shared_state 
     self.lastUpdate = None 
     self.update() 

L'idée est d'utiliser ce de la manière suivante:

store = PriceStore() 
url = store.prices['2.9900']['url'] 

et le magasin doit initialiser correctement et ne chercher de nouvelles informations de point de prix, si l'information existante est plus d'une heure. Il semble que je tape sur leur API à chaque fois que PriceStore est initialisé, par contre. Quelqu'un peut-il repérer mon problème? Puis-je utiliser une variable globale comme __shared_state dans django et m'attendre à ce qu'elle contienne toujours des informations de prix?

Merci!

Answer 1

me semble être frapper leur API avec chaque fois que PriceStore est initialisé, cependant. Quelqu'un peut-il repérer mon problème?

Eh oui, il est facile à repérer:

def __init__(self): 
    self.__dict__ = self.__shared_state 
    self.lastUpdate = None 

le self.lastUpdate = Nonegarantit absolument que l'appel suivant immédiatement à self.update() trouvera la valeur de self.lastUpdate être None - vous venez obligé à être alors!

Retirer self.lastUpdate = None que dans le __init__ et, par exemple, utiliser à la place un corps

lastUpdate = None 

à classe niveau, par exemple, juste après l'affectation __shared_state = {} et avec le même alignement que cette affectation. Que fera fonctionner les choses comme vous l'avez prévu.

Answer 2

Dans __init__, vous définissez self.lastUpdate = None. Ne fais pas ça.

Plus précisément, considérez le code suivant:

A = PriceStore() 

# some time later 

B = PriceStore() 

maintenant A.lastUpdate == Aucun, que vous ne vouliez pas! Au lieu de cela, essayez

if "lastUpdate" not in self.__dict__: 
    self.lastUpdate = None 

De cette façon, vous ne l'écraser.

Answer 3

Votre problème principal est que lorsque vous créez un nouveau PriceStore, vous définissez self.lastUpdate sur None (dans l'avant-dernière ligne). Donc, bien qu'ils partagent tous l'état, chaque nouvel objet clobbers l'état.

faire ce lieu:

class PriceStore(): 
    __shared_state = {'lastUpdate': None} 

Un autre problème que vous pourriez faire face est que, selon la façon dont votre Django est déployé, vous pouvez utiliser plus d'un processus. Chacun aurait sa propre copie de l'état partagé.