【Django】site_detail() takes exactly 1 argument (2 given) というエラーが出る

Question

DjangoでWebサイトを作成しており、下記の記述をしました。

#models.py
# -*- coding: utf-8 -*-
from django.db import models
class Site(models.Model):
    id = models.IntegerField('番号', blank = True, primary_key = True, unique = True)
    name = models.CharField(u'名称', blank = True, max_length = 100)

#urls.py
from . import views
urlpatterns = [ 
    url(r'^owner/$', views.owner_list),#①
    url(r'^site/([0-9]{4})/$', views.site_detail), #② 
    url(r'^site/$', views.site_list), #③
    url(r'^/$', views.cls_main) #④
]

#views.py
def site_detail(request):
    return HttpResponse("Hi")

この状態で、例えば、
http://127.0.0.1:8000/cls/site/2004/
とすると、
urls.pyで#②のURLが選択されて、
views,pyで設定した、
site_detailが呼び出されて、
「Hi」と表示されるだろうと考えていました。
しかし、
「TypeError at /cls/site/2004/
detail() takes exactly 1 argument (2 given)」
というエラーが出てしまいました。

Tracebackを引用すると、下記のとおりでした。

Installed Applications:
('django.contrib.admin',
'django.contrib.auth',
'django.contrib.contenttypes',
'django.contrib.sessions',
'django.contrib.messages',
'django.contrib.staticfiles',
'bootstrapform',
'cls')
Installed Middleware:
('django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.common.CommonMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.auth.middleware.SessionAuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware')

Traceback:
File "/usr/local/lib/python2.7/dist-packages/Django-1.8.2-py2.7.egg/django/core/handlers/base.py" in get_response
132.
response = wrapped_callback(request, *callback_args, **callback_kwargs)

Exception Type: TypeError at /cls/site/2004/
Exception Value: site_detail() takes exactly 1 argument (2 given)

どこに問題があるのか検討がつかないのですが、お分かりの方、ご教示いただけないでしょうか。
お手数をお掛けしますが、よろしくお願いいたします。

Answer 1

私の方で正確にエラーを発生させたわけではないのですが、ソースコードを拝見する限り以下のように修正したら動くのではないかと思い、回答させていただきました。

DjangoでURLに正規表現を用いる場合、変数を宣言するように名前をつける必要があります。
今回は4桁の数字のようなので、例えば「year」とすると以下のようになります。

url(r'^site/(?P<year>[0-9]{4})/$', views.site_detail),

そしてこの「year」をどこで使用するかというと、views.pyにある関数、今回では「site_detail」の引数に指定します。
よってviews.pyは以下のようになります。

def site_detail(request, year):
    return HttpResponse("Hi")

もちろんこの「year」は関数のブロック内であれば使用できるので、以下のようにするとURLに指定した数字が表示されるはずです。

def site_detail(request, year):
    return HttpResponse(year)

今回のエラーは、本来2つの引数を取る関数なのに「request」の一つしか指定していないために発生したのではないかと思います。

なお、今回は修正箇所のみ表示させていただきました。

もしも、このやり方でもエラーが発生した場合には、今度は私の方でも同じように再現し、また回答させていただきたいと思います。

Comments

[退会済みユーザー]

ありがとうございました。
ご指摘頂いた方法でうまくいきました。
私は、
「site_detail() takes exactly 1 argument (2 given)」
というエラーの表現を見て、
「1つの引数をとるべきなのに、2つ与えられている」
という誤訳をしていました。
「2つの引数をとるべきなのに、1つしか与えられていない」という意味なのですね。
大変わかり易くご説明頂いて、感謝しております。

[takaocs]

こちらこそお役に立てて光栄です。
Djangoは書籍も非常に少なく、ドキュメントも英語がほとんどなのでとても勉強が難しいですが、とても良いフレームワークだと教えられ日々学習しています。