Django リレーション（一対多 → 一対多）について教えてください。

Question

実現したいこと

Djangoでアプリ開発をしている初学者です。
複数のリレーションを組みたいのですが
エラーが発生してしまうため、解決方法をご指導ください。

どうぞよろしくお願いします。

前提

リレーションの詳細
User（１）対　Corp（多）
Corp（１）対　Staff（多）

Djangoアプリケーションを
「accounts」「corp」に分けて作成しています。

app
|-accounts
|-corp
|-app

発生している問題・エラーメッセージ

staffを参照するリンク(staff_list)を叩くと以下のエラーが発生します。

AttributeError at /corp/1/staff-list/
'WSGIRequest' object has no attribute 'corp'

該当のソースコード

accounts/models.py

from django.contrib.auth.models import AbstractUser


class CustomUser(AbstractUser):

    class Meta:
        verbose_name_plural = "CustomUser"

---

corp/models.py

from accounts.models import CustomUser
from django.db import models


class Corp(models.Model):

    user = models.ForeignKey(CustomUser, verbose_name='ユーザー', on_delete=models.PROTECT)

    corp_name = models.CharField(
        verbose_name='会社名',
        max_length=50,
    )
    corp_name_kana = models.CharField(
        verbose_name='会社名カナ',
        max_length=100,
    )

    class Meta:
        verbose_name_plural = 'Corp'

    def __str__(self):
        return self.corp_name


class Staff(models.Model):

    corp = models.ForeignKey(Corp, verbose_name='会社名', on_delete=models.PROTECT)

    last_name = models.CharField(
        verbose_name='姓',
        max_length=10,
    )

    first_name = models.CharField(
        verbose_name='名',
        max_length=10,
    )

    class Meta:
        verbose_name_plural = 'Staff'

    def __str__(self):
        return self.last_name

---

追記）corp/ulrs.py

#corp/ulrs.py
# (省略) 
    path('corp-list/', views.CorpListView.as_view(), name="corp_list"),
    path('corp/<int:pk>/', views.CorpView.as_view(), name="corp"),

    path('corp/<int:pk>/staff-list/', views.StaffListView.as_view(), name='staff_list'),

追記）corp/views.py

from .models import Corp, Staff
from django.views import generic

#CorpListViewは正常に取得できる
class CorpListView(generic.ListView):
    model = Corp
    template_name = 'corp_list.html'
    paginate_by = 5

    def get_queryset(self):
        corps = Corp.objects.filter(user=self.request.user).order_by('corp_name')
        return corps

#CorpViewも正常に表示できる
class CorpView(generic.DetailView):
    model = Corp
    template_name = 'corp.html'


#StaffListViewはエラーになってしまう
class StaffListView(generic.ListView):
    model = Staff
    template_name = "staff_list.html"
    paginate_by = 10

    def get_queryset(self):
        staff_list = Staff.objects.filter(corp=self.request.corp).order_by('last_name')
        return staff_list

---

staff/templates/staff_list.html

<!-- 省略 -->
<a href="{% url 'staff:staff_list' %}">従業員情報</a>
<!-- 省略 -->

試したこと

マイグレーションをやり直したりデータベースの確認、
インターネットで同様の解説を検索など

補足情報（FW/ツールのバージョンなど）

Django 4.1.7
Python 3.10
PostgreSQL 15.1

Comments

[dameo]

staff_listの取得方法が分かりませんが、
Staff.objects.filter(corp__user__username='hoge')
みたいな感じで、CustomUserのusernameが'hoge'であるようなStaffのQuerySetが取得できます。
QuerySetはqueryプロパティを取得してstrにするとSQLを見れます。なので
str(Staff.objects.filter(corp__user__username='hoge').query)
とかすると、SQLを見る事ができます。joinしてるかどうかはそこで分かると思います。

[Te_PiN]

コメントありがとうございます。
ご教授いただいた内容をviewsに書くと次のエラーになってしまいます。

NoReverseMatch at /corp/staff_list/
Reverse for 'corp' with arguments '('',)' not found. 1 pattern(s) tried: ['corp/(?P<pk>[0-9]+)/$']

[dameo]

ちゃんと行ったことを全部載せてもらえますか？あと、ここでなく質問文に書いてください。
そしてviewではなくdjango shellを使用した操作で明快にしてください。

Answer 1

以下の方法で解決できました。

staff_list.htmlのリンクでcorp pkを埋め込む

 <div class="post-preview">
        <a href="{% url 'corp:staff_list' corp.pk %}" class="btn btn-primary float-right">従業員情報</a>
    </div>

viewsで .filter(corp__pk=self.kwargs["pk"])にてデータを取得

class StaffListView(LoginRequiredMixin, OnlyYouMixin, generic.ListView, Corp):
    model = Staff
    template_name = "staff_list.html"

    def get_queryset(self):
        staff_list = Staff.objects.filter(corp__pk=self.kwargs["pk"])
        return staff_list

このように修正したところ、取得できるようになりました。
なかなか情報がなかったので、同様のエラーが発生している
どなたかのお役にたてば幸いです。

Comments

[dameo]

ここで普通と違うのはfilterの使い方だけですよね。私の回答があれば難しいことは何もないと思いますが、ほんの少しても役に立つようにしたいのであれば、まずはテンプレート、次にurl.pyやpkの指定方法など、動く形で具体的に載せないといけませんね。あなたがどういう画面を作りたいのかが誰にも分からないからです。djangoは特にうまく動かない理由が分かりにくいフレームで、エラーから原因にも到達しにくいので、全コードを載せる、動くモノを載せるのがとても大事です。役に立ちたいならね。

あなたの問題は、まずリレーションがない形でのviewとテンプレートの使い方を分かっていなかった点。
加えてリレーションがある形でのmodelの使い方をまるで理解していなかった点の2点でした。
そこにアプローチしてることと、原因の明記、対策の明記が必須で、理解してもらうためには最小限の現象再現コードが必要です。

もう少し早く理解できるといいですね。

[Te_PiN]

今後の参考にさせていただきます

Answer 2

回答ではありません。

質問者さんはview＋テンプレートの使い方が分からないようなので、タイトルにあるリレーションとは関係ない部分に引っかかっているようです。なので今回の質問は考慮せず、現在タイトルにある「Django リレーション（一対多　→　一対多）について教えてください。」に対して回答します。

モデル抜粋

質問者さんのモデルは必要以上に複雑なので、単純化しました。

mysite/myapp/models.py

from django.db import models
from django.contrib.auth.models import AbstractUser
class CustomUser(AbstractUser):
    def __str__(self):
       return f'{self.username}({self.first_name} {self.last_name})'
class Corp(models.Model):
    user = models.ForeignKey(CustomUser, on_delete=models.PROTECT)
    name = models.CharField(max_length=50)
    def __str__(self):
        return self.name
class Staff(models.Model):
    corp = models.ForeignKey(Corp, on_delete=models.PROTECT)
    name = models.CharField(max_length=10)
    def __str__(self):
        return self.name

名前やメンバ自体は削除・変更していますが骨格自体は変えていません。

クエリセット

viewからmodelを参照する際は、Modelオブジェクトを元にQuerySetを作成して、データベースから値を取得します。詳細は以下に記載があります。
https://docs.djangoproject.com/en/4.1/ref/models/querysets/

例えば、CustomUserのusernameが'user1'であるユーザーのCustomUser-Corp-Staffリストの取得は以下のようになります。

mysite/queryset_check_script.py

from myapp.models import CustomUser, Corp, Staff
queryset_user1 = Staff.objects.filter(corp__user__username='user1').values_list('corp__user__username', 'corp__name', 'name')
print(queryset_user1)
print(queryset_user1.query)

表の中身は以下のとおりです(本回答ではsqlite3を使っています)。

sqlite> select * from myapp_customuser;
id          password    last_login  is_superuser  username    first_name  last_name   email       is_staff    is_active   date_joined               
----------  ----------  ----------  ------------  ----------  ----------  ----------  ----------  ----------  ----------  --------------------------
1                                   0             user1                                           0           1           2023-03-20 21:34:33.274055
2                                   0             user2                                           0           1           2023-03-20 21:34:33.274081
sqlite> select * from myapp_corp;
id          name        user_id   
----------  ----------  ----------
1           corp1       1         
2           corp2       2         
3           corp3       1         
sqlite> select * from myapp_staff;
id          name        corp_id   
----------  ----------  ----------
1           staff1      1         
2           staff2      2         
3           staff3      3         
4           staff4      2         
sqlite> 

この表のデータを先のスクリプトを使い、django shellから流すと以下のように出力されます。

$ python manage.py shell <queryset_check_script.py
<QuerySet [('user1', 'corp1', 'staff1'), ('user1', 'corp3', 'staff3')]>
SELECT "myapp_customuser"."username", "myapp_corp"."name", "myapp_staff"."name" FROM "myapp_staff" INNER JOIN "myapp_corp" ON ("myapp_staff"."corp_id" = "myapp_corp"."id") INNER JOIN "myapp_customuser" ON ("myapp_corp"."user_id" = "myapp_customuser"."id") WHERE "myapp_customuser"."username" = user1
$

環境構築スクリプト

import os, venv, subprocess
ENV = 'env'
if os.name == 'posix':
    BIN = 'bin'
else:
    BIN = 'Scripts'
builder = venv.EnvBuilder(with_pip=True)
builder.create(ENV)
cwd = os.getcwd()
python = os.path.join(cwd, ENV, BIN, 'python')
os.environ['VIRTUAL_ENV'] = os.path.join(cwd, ENV)
os.environ['PATH'] = os.path.join(cwd, ENV, BIN) + os.pathsep + os.environ['PATH']
p = subprocess.run([python, '-m', 'pip', 'install', '--upgrade', 'pip', 'setuptools'])
p = subprocess.run([python, '-m', 'pip', 'install', 'django==4.1.7', 'whatthepatch==1.0.4'])
with open('generated.py', 'wt', encoding='utf8') as f:
    f.write("""\
import subprocess, os
p = subprocess.run('django-admin startproject mysite', shell=True)
os.chdir('mysite')
p = subprocess.run('python manage.py startapp myapp', shell=True)
p = subprocess.run('python ../patch.py ../patch.diff', shell=True)
p = subprocess.run('python manage.py makemigrations', shell=True)
p = subprocess.run('python manage.py migrate', shell=True)
p = subprocess.run('python manage.py shell <insert_script.py', shell=True)
p = subprocess.run('python manage.py shell <queryset_check_script.py', shell=True)
""")
with open('patch.diff', 'wt', encoding='utf8') as f:
    f.write("""\
diff --git a/insert_script.py b/insert_script.py
new file mode 100644
index 0000000..c47446e
--- /dev/null
+++ b/insert_script.py
@@ -0,0 +1,7 @@
+from myapp.models import CustomUser, Corp, Staff
+users = [CustomUser(username=f'user{i+1}') for i in range(2)]
+corps = [Corp(name='corp1', user=users[0]), Corp(name='corp2', user=users[1]), Corp(name='corp3', user=users[0])]
+staffs = [Staff(name='staff1', corp=corps[0]), Staff(name='staff2', corp=corps[1]), Staff(name='staff3', corp=corps[2]), Staff(name='staff4', corp=corps[1])]
+for l in [users, corps, staffs]:
+    for o in l:
+        o.save()
diff --git a/myapp/models.py b/myapp/models.py
index 71a8362..aa03eab 100644
--- a/myapp/models.py
+++ b/myapp/models.py
@@ -1,3 +1,16 @@
 from django.db import models
+from django.contrib.auth.models import AbstractUser
+class CustomUser(AbstractUser):
+    def __str__(self):
+       return f'{self.username}({self.first_name} {self.last_name})'
+class Corp(models.Model):
+    user = models.ForeignKey(CustomUser, on_delete=models.PROTECT)
+    name = models.CharField(max_length=50)
+    def __str__(self):
+        return self.name
+class Staff(models.Model):
+    corp = models.ForeignKey(Corp, on_delete=models.PROTECT)
+    name = models.CharField(max_length=10)
+    def __str__(self):
+        return self.name
 
-# Create your models here.
diff --git a/myapp/urls.py b/myapp/urls.py
new file mode 100644
index 0000000..4c9189f
--- /dev/null
+++ b/myapp/urls.py
@@ -0,0 +1,8 @@
+from django.urls import path
+
+from . import views
+
+urlpatterns = [
+    path('', views.index, name='index'),
+]
+
diff --git a/myapp/views.py b/myapp/views.py
index 91ea44a..59a44a4 100644
--- a/myapp/views.py
+++ b/myapp/views.py
@@ -1,3 +1,4 @@
 from django.shortcuts import render
-
-# Create your views here.
+from .models import CustomUser, Corp, CustomUser
+def index():
+    pass
diff --git a/mysite/settings.py b/mysite/settings.py
index 19d4403..80db486 100644
--- a/mysite/settings.py
+++ b/mysite/settings.py
@@ -31,6 +31,7 @@ ALLOWED_HOSTS = []
 # Application definition
 
 INSTALLED_APPS = [
+    'myapp.apps.MyappConfig',
     'django.contrib.admin',
     'django.contrib.auth',
     'django.contrib.contenttypes',
@@ -103,9 +104,9 @@ AUTH_PASSWORD_VALIDATORS = [
 # Internationalization
 # https://docs.djangoproject.com/en/4.1/topics/i18n/
 
-LANGUAGE_CODE = 'en-us'
+LANGUAGE_CODE = 'ja'
 
-TIME_ZONE = 'UTC'
+TIME_ZONE = 'Asia/Tokyo'
 
 USE_I18N = True
 
@@ -121,3 +122,5 @@ STATIC_URL = 'static/'
 # https://docs.djangoproject.com/en/4.1/ref/settings/#default-auto-field
 
 DEFAULT_AUTO_FIELD = 'django.db.models.BigAutoField'
+
+AUTH_USER_MODEL = "myapp.CustomUser" 
diff --git a/mysite/urls.py b/mysite/urls.py
index 8bc0cd4..7229155 100644
--- a/mysite/urls.py
+++ b/mysite/urls.py
@@ -14,8 +14,9 @@ Including another URLconf
     2. Add a URL to urlpatterns:  path('blog/', include('blog.urls'))
 \"\"\"
 from django.contrib import admin
-from django.urls import path
+from django.urls import path, include
 
 urlpatterns = [
+    path('myapp/', include('myapp.urls')),
     path('admin/', admin.site.urls),
 ]
diff --git a/queryset_check_script.py b/queryset_check_script.py
new file mode 100644
index 0000000..25edd00
--- /dev/null
+++ b/queryset_check_script.py
@@ -0,0 +1,4 @@
+from myapp.models import CustomUser, Corp, Staff
+queryset_user1 = Staff.objects.filter(corp__user__username='user1').values_list('corp__user__username', 'corp__name', 'name')
+print(queryset_user1)
+print(queryset_user1.query)
""")
with open('patch.py', 'wt', encoding='utf8') as f:
    f.write("""\
import whatthepatch, os, sys
with open(sys.argv[1], 'rt', encoding='utf8') as fp:
    for diff in whatthepatch.parse_patch(fp):
        header = diff.header
        if header.new_path == '/dev/null':
            os.remove(header.old_path)
        else:
            if header.old_path == '/dev/null':
                old = []
            else:
                with open(header.old_path, 'rt', encoding='utf8') as fo:
                    old = [line.rstrip(os.linesep) for line in fo]
            new = whatthepatch.apply_diff(diff, old)
            try:
                dir_path = os.path.dirname(header.new_path)
                if dir_path == '':
                    dir_path = '.'
                os.makedirs(dir_path)
            except FileExistsError:
                pass
            with open(header.new_path, 'wt', encoding='utf8') as fn:
                for line in new:
                    print(line, file=fn)
""")
p = subprocess.run([python, 'generated.py'])

使い方は空のディレクトリで上記スクリプトを流すだけです。

中身はおよそ以下のとおりになっています。

1. venvを使いカレントディレクトリにenvという名前で仮想環境を構築
2. 仮想環境の中に入る
3. generated.py, patch.diff, patch.pyをカレントに出力
4. generated.pyを実行してpipを更新してdjangoとパッチ用のツールをインストール
5. djangoのプロジェクト(mysite)を作成し、その中にアプリ(myapp)を作成
6. patch.pyとpatch.diffを使ってアプリ用の実装を生成
7. dbをmigrate(初回)
8. django shellとmysite/insert_script.pyを使用し、サンプルデータを作成
9. django shellとmysite/queryset_check_script.pyを使用し、クエリを発行
10. 仮想環境から出る

Comments

[Te_PiN]

コメントありがとうございます。ただ当方のプログラミングの誤っている箇所をご指摘いただきたいです。
データベースには正しくデータがあるのですが、どこのプログラムに問題があるため、エラーが発生しているのかを知りたいです。

[dameo]

私の回答はあなたのためのものではありません。
あなたの質問のタイトルにある内容を検索して来た人に対するものです。

どうしてもあなたのコードに対する回答が欲しいのなら、現象を再現できる「最小限の」全コードをご用意ください。余計なものが入っていたら見ないし、ファイルが足りずに動かないのなら見ません。多分そういう作業をしているうちに自分で気づく類の間違いです。他人に聞く暇があったら手を動かしましょう。

[Te_PiN]

こちらも手を動かしたいのですがわからないので質問しております。
ご指摘いただければ不足のファイル情報は追加しますし、修正もします。

私の質問に対する回答が同様のエラーに苦戦している初学者の参考にもなると思います。
当質問に対する回答では無いのであれば申し訳ありませんが他でお願いします。

ご厚意には感謝しますが私のような初学者が質問しづらくなると思いますので
これ以上攻撃的なコメントを記入するのはお止めください。

[dameo]

私の回答はあなたのためのものではありません。
あなたの質問のタイトルにある内容を検索して来た人に対するものです。

どうしてもあなたのコードに対する回答が欲しいのなら、現象を再現できる「最小限の」全コードをご用意ください。余計なものが入っていたら見ないし、ファイルが足りずに動かないのなら見ません。多分そういう作業をしているうちに自分で気づく類の間違いです。他人に聞く暇があったら手を動かしましょう。

初心者かどうかは関係ありませんし、攻撃的なコメントはしていません。手を動かしましょう。

[Te_PiN]

「他人に聞く暇があったら手を動かしましょう。」

https://teratail.com/tour
プログラミングに関して、わからないことがあれば是非teratailで質問してください。とあります。

質問者の問題解決以外はコメントはお控えください。

[dameo]

ほんとに何もしてないんですね。コメントしてる暇があったら手を動かしましょう。
私はあなたが私の解答欄にコメントしてるから返してるだけですよ。