Obtenir DRY avec les conditions Python et les retours Django

Question

Ce que je vais partager est un modèle courant dans mes projets. Certains cas sont plus longs que d'autres mais le motif est plus ou moins le même et je me demande comment je peux être plus SEC. La seule chose qui se passe ici est de définir un message basé sur le statut. Theres doit être un moyen d'appeler le retour qu'une seule fois et peut-être même resserrer le conditionnel.

Des idées?

Answer 1

Se pourrait-il quelque chose dans la ligne de:

Module lib

def add_error_message(request, status, result=0): 
    if status == 'ACTIVE' and not result: 
     msg = 'Active Message... bla bla bla' 
    elif status == 'TOO MANY FAILURES': 
     msg = 'Failed. Foooobarrrrr' 
    elif status == 'DEACTIVATED BY MERCHANT': 
     msg = 'Deactivated derpa derp' 
    messages.add_message(request, messages.ERROR, _(msg)) 

Ce ne serait pas pour la condition de 'p.result', un dictionnaire aurait été beaucoup mieux:

MESSAGES = { 
    'ACTIVE': 'Active Message... bla bla bla', 
    'TOO MANY FAILURES': 'Failed. Foooobarrrrr', 
    'DEACTIVATED BY MERCHANT': 'Deactivated derpa derp', 
} 
def add_error_message(request, status): 
    try: 
     messages.add_message(request, messages.ERROR, _(MESSAGES[status])) 
    except KeyError: 
     pass 

vue Module

from lib import add_error_message 

def your_view(request, *args, **kwargs): 
    # ... 
    add_error_message(request, status, p.result if p else 0) 
    return HttpResponseRedirect(reverse('billing_update')) 

Answer 2

Je remarque que votre dernière déclaration est conditionnelle, et devinez basée sur mon propre code similaire que si aucun des contrôles déclencheurs, vous voulez aller et retour quelque chose d'autre, peut-être même pas ... redirect

Voilà comment je pense que je le ferais:

def checkForMessage(): 
    if status == 'ACTIVE' and p.result != "0": 
     return 'Active Message... bla bla bla' 
    if status == 'TOO MANY FAILURES': 
     return 'Failed. Foooobarrrrr' 
    if status == 'DEACTIVATED BY MERCHANT': 
     return 'Deactivated derpa derp' 
    return None 

msg = checkForMessage() 
if msg: 
    messages.add_message(request, messages.ERROR, _(msg)) 

    return HttpResponseRedirect(reverse('billing_update')) 

// ... go on and do some other stuff ... 
return direct_to_template('some/template.html', {some: stuff}) 

Notez que la fonction checkForMessage est définie dans la fonction de vue, donc nous ne pas passer tout ce qui concerne un test en tant que paramètre (dans Si les tets sont nombreux et variés, s'ils ont juste besoin d'un statut et d'une variable "p", il peut tout aussi bien être déclaré outsi de la fonction de vue et prenez ces paramètres). Le point principal est que l'alternative "fall through" où l'on ne veut ajouter aucun message peut être traitée en retournant None dans la méthode check et en vérifiant l'existence d'un message dans la méthode view.