7

我正在为 Django 的 render_to_response() 编写一个包装函数来添加 CSRF 处理。

逻辑是:

def some_view (request)
    dictionary = {'context_param': some_param}
    dictionary.update(csrf(request))
    # ... view code here
    return render_to_response("a_template.html", dictionary)

render_to_response() 具有以下签名:

def render_to_response(*args, **kwargs)

我想编写透明包装器 - 只需添加一些功能(前面提到过)并保留其他内容。我想我应该这样写:

def csrf_response (request, *args, **kwargs):

    # Here I need to somehow extract dictionary from args or kwargs

    if dictionary is None:
        dictionary = {}
    dictionary.update(csrf(request))

    # And here I somehow need to pass dictionary into render_to_response() fot the furher processing
    return render_to_response(*args, **kwargs)

所以问题是 - 从 args/kwargs 中提取所需参数(然后更改它)并进一步传递它的最佳实践是什么?

顺便说一句,render_to_response() 的代码让我觉得有点奇怪。这里是:

def render_to_response(*args, **kwargs):
    """
    Returns a HttpResponse whose content is filled with the result of calling
    django.template.loader.render_to_string() with the passed arguments.
    """
    httpresponse_kwargs = {'mimetype': kwargs.pop('mimetype', None)}
    return HttpResponse(loader.render_to_string(*args, **kwargs), **httpresponse_kwargs)

如果有人用所有位置参数调用它,所以 kwargs 将为空,但mimetype参数将被指定为最后一个位置参数怎么办?看起来在那种情况下它的行为是错误的。

4

1 回答 1

6

由于如何render_to_response实现,指定 mimetype 的唯一方法是使用命名参数。作为位置参数传递的任何内容都将传递给loader.render_to_string.

如何提取某些论点并传递其他论点的方法实际上取决于您在做什么。没有一种“正确”的方法可以始终做到这一点。自然,不同的作者有自己喜欢的约定。

代替您的评论# Here I need to somehow extract dictionary from args or kwargs,您可以将 kwargs 直接用作字典,将 args 用作元组,因为它们正是如此。对于 args,您别无选择,只能断言(即假设)每个位置的值的含义。

我个人更喜欢如果你编写的代码知道某些参数的值,它们应该在签名中声明,如下所示:

def spam(session, name, *args, clear=True, **kwargs):
  # do something with session, name, clear
  return eggs(name, *args, **kwargs)  # if eggs requires name
于 2011-05-09T18:42:44.977 回答