使用视图装饰器来处理用户权限是不好的做法吗？

Question

我正在使用 Django 视图装饰器以相当复杂的方式检查权限，并且开始意识到这可能是一种不好的做法。

假设用户的个人资料处于某种状态，比如说“申请待定”，因此某些视图不应显示给该用户，而应该显示给“申请完成”的用户。我目前正在使用装饰器将待处理的用户重定向到主页，并弹出一个告诉他们他们的应用程序仍处于待处理状态的弹出窗口。

但是，我在google's python best practice 上读到，装饰器应该很简单，而不是依赖于数据库连接、文件等。

这是否意味着在显示视图之前检查借款人应用程序的状态是不好的做法，如果是，还有什么替代方法？

Answer 1

在 Django 中，使用 user_passes_test 或 permission_required 装饰器是正确的做法。

from django.contrib.auth.decorators import user_passes_test, permission_required

@user_passes_test(lambda user: user.is_superuser)
@permission_required('your_perm')
def my_view(request):
    # code

Answer 2

你提到的文字充其量是值得商榷的——至少在它的表述上：

避免装饰器本身的外部依赖（例如，不要依赖文件、套接字、数据库连接等），因为它们在装饰器运行时可能不可用（在导入时，可能来自 pydoc 或其他工具） .

(...)

装饰器是“顶级代码”的特例

这仅适用于装饰器函数的“外部”部分，而不是装饰器返回的包装函数内的代码（通常），即：

def mydecorator(func):
    print("""
       this is the outer decorator code 
       and will be executed when the decorator
       is applied (most often - but not necessarily
       - at import time)
       """
       )
       def wrapper(*args, **kw):
           print("""
              this is the inner decorator code     
              which is only executed when the
              decorated function is called,
              so here it's safe to do whatever
              you would do in the decorated function.
              """
              )
              return func(*args, **kw)
      return wrapper       

IOW，您正在做的事情（在视图中使用装饰器来检查是否允许用户访问内容）不仅完全没问题，而且是装饰器用途的规范示例，甚至是官方的处理方式这在 Django 中，参见 Davide Pizzolato 的回答。