关于django view的优雅处理方式?

想想应该先上需求的,目的就是检测用户信息如果不是完整的,则将在用户登录5分钟后弹到提示页面,用户除了点击登出或者补全信息,才能继续浏览,否则查看任何页面都转跳到complete页面
我的实现方法是这样的,在登录的时候,给session加上一个ttl的字段,然后在除了登出和补全信息的页面外,其他都在超时后就自动转跳
已经试着用了楼下给出去的中间件形式,可是发现会无限跳转。不知道是不是代码有问题。
小白,如果有错误请原谅
先上代码

# ttl 是登录后user取出来的last_time做了一下延迟的时间,已经格式化为str    
# app/views.py
def index(request, tag=None, sort=None):
    if datetime.datetime.utcnow().strftime('%Y-%m-%d %H:%M:%S') > request.session['ttl']:
        return redirect(to='complete')
    ...
    return render(request, 'index.html', context)

我想在每一个页面开头(每一个方法开头)都做一个这样的判断

if datetime.datetime.utcnow().strftime('%Y-%m-%d %H:%M:%S') > request.session['ttl']:
        return redirect(to='complete')

但是觉得不是很美观,但是包装成方法

def is_ttd(ttl):
    if datetime.datetime.utcnow().strftime('%Y-%m-%d %H:%M:%S') > ttl:
        return redirect(to='complete')

调用的时候

def index(request, tag=None, sort=None):
    is_ttd(request.session['ttl'])
    ...
    return render(request, 'index.html', context)

是没有用的,但是又不能直接加return在前面
但是如果写成这样,

def index(request, tag=None, sort=None):
    ttd = is_ttd(request.session['ttl'])
    if ttd:return ttd
    ...
    return render(request, 'index.html', context)

就更加不美观了。
写成装饰器的话,index里面才传入的request又不能提前调用。
请问有什么方法可以写的优雅一点呢?想要包装一下,可以给每一个方法都能优雅的调用

阅读 5.4k
5 个回答

写成decorator非常合适这种场景,request也会先传给decorator来处理,如下:

def ttd(func):
    def wrapper(request, *args):
        if datetime.datetime.utcnow().strftime('%Y-%m-%d %H:%M:%S') > request.session['ttl']:
            return redirect(to='complete')
        return func(request, *args)
    return wrapper

@ttd
def index(request, tag=None, sort=None):
    # ...
    # return render(request, 'index.html', context)

每个页面和每个方法都需要的话,你应该使用middleware,实现process_request方法

感觉上可以写一个通用视图,这个是基于类的视图,在父类中写这个方法。

可以使用django.contrib.auth.decorators.user_passes_test方法。

撰写回答
你尚未登录,登录后可以
  • 和开发者交流问题的细节
  • 关注并接收问题和回答的更新提醒
  • 参与内容的编辑和改进,让解决方法与时俱进
推荐问题