Quelle est la meilleure façon de vérifier les conditions de demande?

Question

Plusieurs fois dans les codes open source de python django, j'ai vu le scénario comme celui-ci

if request.META and 'HTTP_REFERER' in request.META: 

De même je l'ai vu ces deux si les conditions aussi

if request.POST and 'next' in request.POST: 
if request.GET and 'next' in request.GET: 

Je crois que seule condition est suffire pour vérifier la condition if dans ces cas comme

if 'HTTP_REFERER' in request.META: 
if 'next' in request.POST: 
if 'next' in request.GET: 

Alors pourquoi la plupart du temps les gens utilisent le plus tôt une fois, Est-ce que c'est juste comme une double vérification ou des scénarios dans lesquels les premières conditions de double contrôle peuvent être utiles alors que les conditions de contrôle unique ultérieures peuvent échouer?

Answer 1

Je viens de l'ensemble rassemblés de (git courant) source de django, et n'avait pas trouvé une seule occurrence des trois conditions que vous avez mentionnées.

Et vous avez parfaitement raison avec la seule condition suffisante comme long comme vous pouvez supposer que tous les trois dictionnaires sont définis. Et en regardant le code django, je crois que vous pouvez le supposer.

Modifier: également le django documentation suggère que ces dictionnaires seront toujours définis.

Answer 2

Il y a un (hypothétique) cas où votre solution échouerait:

>>> request.POST = None 
>>> 'next' in request.POST 
Traceback (most recent call last): 
    File "<stdin>", line 1, in <module> 
TypeError: argument of type 'NoneType' is not iterable 

Mais si vous pouvez être sûr que request.POST ne sera jamais None, votre solution serait OK.

Il est juste un peu plus lent dans le cas d'un dictionnaire vide car le test if 'foo' in request.POST peut être ignoré si le test (rapide) de vide échoue déjà. D'un autre côté, ce sera plus rapide si le dictionnaire n'est pas vide.

>>> import timeit 
>>> timeit.timeit(setup="a = {}", stmt="if a and 'next' in a: pass") 
0.028279806566242852 
>>> timeit.timeit(setup="a = {}", stmt="if 'next' in a: pass") 
0.04539217556517272 
>>> timeit.timeit(setup="a = {'foo':'bar'}", stmt="if a and 'next' in a: pass") 
0.07471092295071458 
>>> timeit.timeit(setup="a = {'foo':'bar'}", stmt="if 'next' in a: pass") 
0.045236056421884996 
>>> timeit.timeit(setup="a = {'next':'bar'}", stmt="if a and 'next' in a: pass") 
0.0851067469988891 
>>> timeit.timeit(setup="a = {'next':'bar'}", stmt="if 'next' in a: pass") 
0.0520663758715898 

Donc je suppose que c'est une question de micro-optimisation. Dans ce cas, j'invoque le Zen de Python: Explicit est meilleur qu'implicite.

Answer 3

Il peut être acheived que par la vérification
request.META.get('HTTP_REFERER') et
request.REQUEST.get('next')
au lieu de vérifier pour request.POST ou request.GET