質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.51%
Django

DjangoはPythonで書かれた、オープンソースウェブアプリケーションのフレームワークです。複雑なデータベースを扱うウェブサイトを開発する際に必要な労力を減らす為にデザインされました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

755閲覧

【Django】filter()メソッドを用いているのにも関わらず、MultipleObjectsReturnedエラーが出てしまう

Mr_PONPON_MARU

総合スコア35

Django

DjangoはPythonで書かれた、オープンソースウェブアプリケーションのフレームワークです。複雑なデータベースを扱うウェブサイトを開発する際に必要な労力を減らす為にデザインされました。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

0クリップ

投稿2023/04/30 10:02

編集2023/05/01 14:04

実現したいこと

会員制の記事公開アプリケーションをDjangoで作成しており、マイペイージ上で自身のプロフィール情報と併せて自身が投稿した記事も表示させるページを作成しています。

前提

プロフィール情報に関するモデルはaccounts/models.pyのCustomUserモデルに、記事に関するモデルはarticle/models.pyのArticleモデルで定義しており、一対多(0以上)のリレーション関係にあります。

accounts.models.py

1from django.db import models 2from django.contrib.auth.models import AbstractUser 3from django.utils.translation import gettext_lazy as _ 4 5class CustomUser(AbstractUser): 6 """拡張ユーザーモデル""" 7 username = models.CharField( 8 _("username"), 9 max_length=30, 10 help_text='Required 30 characters or fewer.', 11 unique=True, 12 error_messages={ 13 'unique': _("This Username already exists."), 14 },) 15 email = models.EmailField( 16 _('email'), 17 unique=True, 18 error_messages={ 19 'unique': _("A user with that email address already exists."), 20 },) 21 fb_link = models.URLField(verbose_name='Facebook Link', null=True, blank=True) 22 ig_link = models.URLField(verbose_name='Instagram Link', null=True, blank=True) 23 tw_link = models.URLField(verbose_name='Twitter Link', null=True, blank=True) 24 bg_image = models.ImageField(verbose_name='Backgroung Image', null=True, blank=True, upload_to='bgimage/') 25 icon = models.ImageField(verbose_name='Icon Image', null=True, blank=True, upload_to='icon/') 26 profession = models.CharField(verbose_name='Profession', null=True, blank=True, max_length=20) 27 introduction = models.TextField(verbose_name='Introduction', null=True, blank=True, max_length=500) 28 29 class Meta: 30 verbose_name_plural = 'CustomUser'

ariticle.models.py

1from django.db import models 2from accounts.models import CustomUser 3from mdeditor.fields import MDTextField 4 5class Article(models.Model): 6 """記事タグ""" 7 tag_choices = ( 8 ('ELECTRONICS','ELECTRONICS'), 9 ('INSTALLATION','INSTALLATION'), 10 ('SERVICES','SERVICES'), 11 ('CRAFT','CRAFT') 12 ) 13 14 post_user = models.ForeignKey(CustomUser, verbose_name='Post User', on_delete=models.CASCADE,) 15 title = models.TextField(verbose_name='title', max_length=50,) 16 content = MDTextField() 17 photo = models.ImageField(verbose_name='photo', null=True, blank=True,) 18 thumbnail = models.ImageField(verbose_name='thumbnail', null=True, blank=True,) 19 tag = models.CharField(verbose_name='article tag', choices=tag_choices, max_length=30,) 20 created_at = models.DateField(verbose_name='created_at', auto_now_add=True,)

該当のソースコード

マイページの表示にはDetailViewを継承したクラスを用いました。get_querysetメソッドのオーバライドは、Articleモデルに対してアクセスユーザーでフィルターを掛けてレコードを抽出するしてテンプレートへ渡す処理を行っています。

article/views.py

1from django.views import generic 2from django.contrib.auth.mixins import LoginRequiredMixin 3from accounts.models import CustomUser 4from .models import Article 5 6class MyPageView(LoginRequiredMixin, generic.DetailView): 7 template_name = 'mypage.html' 8 model = CustomUser 9 10 def get_queryset(self): 11 articles = CustomUser.objects.filter(article__post_user=self.request.user).order_by('-created_at') 12 return articles

テンプレートは下記の様に作成しており、Work Contentブロック内にviewsで抽出したArticleモデルのレコード表示しています。

mypage.html

1{% block contents %} 2<section> 3 <div class="container"> 4 <!-- Profile Block --> 5 <div class="row"> 6 <div class="col-md-4 mx-auto"> 7 <div class="u-pull-half text-center"> 8 {% if object.icon %} 9 <img class="img-fluid u-avatar u-box-shadow-lg rounded-circle mb-3" width="200" height="auto" src="{{ object.icon.url }}" alt="Image Description"> 10 {% else %} 11 <img class="img-fluid u-avatar u-box-shadow-lg rounded-circle mb-3" width="200" height="auto" src="{% static 'img/default_icon.png' %}" alt="Image Description"> 12 {% endif %} 13 </div> 14 </div> 15 </div> 16 <!-- End Profile Block --> 17 18 <!-- About --> 19 <div class="row u-content-space-bottom"> 20 <div class="col-lg-6 mb-5 mb-lg-5 pl-lg-5 mx-auto"> 21 {% if object.introduction %} 22 <h4 class="mb-3">About me</h4> 23 <p>{{ object.introduction }}</p> 24 {% endif%} 25 </div> 26 </div> 27 <!-- End About --> 28 </div> 29</section> 30<!-- End About Section --> 31 32 33<!-- Portfolio --> 34<section class="u-content-space"> 35 <div class="container"> 36 <header class="text-center w-md-50 mx-auto mb-8"> 37 <h2 class="h1">Prototyping Works</h2> 38 </header> 39 40 <!-- Work Content --> 41 <div class="js-shuffle u-portfolio row no-gutters mb-6"> 42 {% for article in object.article_set.all %} 43 <figure class="col-sm-6 col-md-4 u-portfolio__item" data-groups='["its-illustration"]'> 44 <img class="u-portfolio__image" src="{{ article.thumbnail.url }}" alt="Image Description"> 45 <figcaption class="u-portfolio__info"> 46 <h6 class="mb-0">{{ article.title }}</h6> 47 <small class="d-block">Branding</small> 48 </figcaption> 49 <a class="js-popup-image u-portfolio__zoom" href="assets/img-temp/portfolio/img1.jpg">Zoom</a> 50 </figure> 51 {% endfor %} 52 <!-- sizer --> 53 <figure class="col-sm-6 col-md-4 u-portfolio__item shuffle_sizer"></figure> 54 </div> 55 <!-- End Work Content --> 56 </div> 57</section> 58<!-- End Portfolio --> 59{% endblock %}

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

python manage.py runseruverを実行するとブラウザ上で下記のエラーメッセージが表示されてしまいます。

MultipleObjectsReturned at /mypage/username/ get() returned more than one CustomUser -- it returned 2! Request Method: GET Request URL: http://127.0.0.1:8000/mypage/username/ Django Version: 3.0.3 Exception Type: MultipleObjectsReturned Exception Value: get() returned more than one CustomUser -- it returned 2! Exception Location: /Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/db/models/query.py in get, line 422 Python Executable: /Users/xxx/anaconda3/envs/venv_prototyping/bin/python Python Version: 3.7.6 Python Path: ['/Users/xxx/Desktop/prototyping', '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python37.zip', '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7', '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/lib-dynload', '/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages']
Traceback (most recent call last): File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/core/handlers/exception.py", line 34, in inner response = get_response(request) File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/core/handlers/base.py", line 115, in _get_response response = self.process_exception_by_middleware(e, request) File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/core/handlers/base.py", line 113, in _get_response response = wrapped_callback(request, *callback_args, **callback_kwargs) File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/base.py", line 71, in view return self.dispatch(request, *args, **kwargs) File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/contrib/auth/mixins.py", line 52, in dispatch return super().dispatch(request, *args, **kwargs) File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/base.py", line 97, in dispatch return handler(request, *args, **kwargs) File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/detail.py", line 106, in get self.object = self.get_object() File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/views/generic/detail.py", line 52, in get_object obj = queryset.get() File "/Users/xxx/anaconda3/envs/venv_prototyping/lib/python3.7/site-packages/django/db/models/query.py", line 422, in get num if not limit or num < limit else 'more than %s' % (limit - 1), Exception Type: MultipleObjectsReturned at /mypage/username/ Exception Value: get() returned more than one CustomUser -- it returned 2!

試したこと・調べたこと

Djangoの管理画面から試しに当該ユーザに紐づけられるArticleモデルのレコードを2つではなく1つで実行してみたところ、正常に表示されました。

このエラーがどういう状況で発生するか下記ページで調べてみたところ、get()メソッドを使用した際にレコードが2つ以上ある場合に発生するエラーで、get()メソッドをfilter()メソッドにすることで解決することが分かりました。
https://torajirousan.hatenadiary.jp/entry/2019/02/08/230027

しかし今回はget()メソッドを使わずにfilter()メソッドを使用しており、なぜこのようなエラーが出てしまうのか分かりませんでした。

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

開発環境は下記の通りです
・python 3.7.6
・Anacondaローカル内
・Django3系
・MacOS
・Chromeブラウザ

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

get_queryset()が問題ではなく、DetailViewが使用しているget_object()メソッドが原因です。
DetailViewは、デフォルトでget_object()メソッドを使用して、クエリセットから単一のオブジェクトを取得しようとします。
get_object()メソッドは内部で get()を使用しており、ここで複数のオブジェクトが返されると MultipleObjectsReturnedエラーが発生します。
MyPageViewクラスでget_queryset()をオーバーライドしても、get_object()メソッドはオーバーライドされていないため、デフォルトの実装が使用されます。
ですのでget()メソッドが使用され、エラーが発生しています。

投稿2023/04/30 10:12

jp-seemore.com

総合スコア62

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

Mr_PONPON_MARU

2023/04/30 11:16

コメントありがとうございます。DetailViewでget_object()メソッドがデフォルトで働いていることは知りませんでした。 修正案として、MyPageViewをListViewを継承したクラスビューへと変更するか、get_queryset()メソッドを削除して、下記のようにget_object()メソッドを修正する必要がある、という認識でよろしいでしょうか? ```views.py class MyPageView(LoginRequiredMixin, generic.DetailView): template_name = 'mypage.html' model = CustomUser def get_object(self): articles = CustomUser.objects.filter(article__post_user=self.request.user).order_by('-created_at') return articles ```
jp-seemore.com

2023/04/30 11:21

その認識であっています。 ただしDetailViewを使用する場合は、単一のオブジェクトを返すことが期待されていますので、記事のリストを返すためには、代わりにListViewを使用することをオススメします。
Mr_PONPON_MARU

2023/04/30 12:43 編集

ありがとうございます。 もう一点素人質問で恐縮ですが、従テーブルのArticleからは複数オブジェクトを、主テーブルのCustomUserからは単一のオブジェクトを返す今回の事例のように、常に主テーブルのデータは単一で返され、主テーブルから複数のデータを返すことが想定されない場合でもListViewを継承したクラスビューを用いる事は、ビュー設計の観点的には問題はありませんでしょうか?
jp-seemore.com

2023/04/30 22:02

返信遅れて申し訳ございません。 はい。特に問題はございません。
Mr_PONPON_MARU

2023/05/01 05:04

ListViewに変えて試してみます。 ありがとうございました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.51%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問