Django、ClassVIewでif文を使い、'AnonymousUser' object is not iterableを回避したい。

以下のエラーを回避するためにviews.pyで条件分岐を行いたいのですが、上手くいきません。
どなたかアドバイス頂けないでしょうか？

python
1'AnonymousUser' object is not iterable

views.py

python
1class PostDetailView(DetailView):
2    model = Post
3
4    def get_object(self, queryset=None):
5        obj = super().get_object(queryset=queryset)
6        if not obj.is_public and not self.request.user.is_authenticated:
7            raise Http404
8        return obj
9
10    def get_context_data(self, **kwargs):
11        context = super().get_context_data(**kwargs)
12        context['check'] = Like.objects.filter(user=self.request.user, post=self.kwargs.get('pk')).exists()
13        return context

行動規範の内容に同意します

回答1件

ベストアンサー

views.pyのユーザーログインの有無のコードをけして、HTMLを

HTML
1{% if request.user.is_authenticated %}
2・・・
3{% else %}
4・・・
5{% endif %}
6```view.pyで何も渡さなくても、template内で`{{request.user}}`とやれば、ログイン中のユーザーを取得できることに気づきました。で、上記で分岐できましたよ。(class、関数どちらでも)

投稿2020/05/30 04:06

編集2020/06/01 22:42

ForestSeo

総合スコア2720

退会済みユーザー

2020/05/30 04:35

おはようございます。これだと同じエラーが出てしまうんです。悩んでいます！

ForestSeo

2020/05/30 04:44 編集

ちょっと気になったのが、あなたのDetailViewに self.request.user.is_authenticated とありますが、 self.request.user.is_authenticated() では？編集しときました

退会済みユーザー

2020/05/30 05:00

ご指摘ありがとうございます！修正致しました。ただ、以前のエラーは解消されないです。。。

ForestSeo

2020/05/30 05:03

context['check'] = Like.objects.filter(user=self.request.user,・・・) のエラーですよね。 self.request.userはAnonymousUserで、iterableじゃないよ！ってことですよね。じゃあifで条件分岐して、ログインしていたら以前のままで、ログインしていなかったらFalseにしてあげればいいのでは？

退会済みユーザー

2020/05/30 05:10

条件分岐をテンプレート側でしてるんですが、エラーが出てしまうんですよね。じゃあview側でやればいいのかなとも考えたんですけど、その辺りやり方が分からないのです。。。

ForestSeo

2020/05/30 05:13

僕の回答みたいな感じに

退会済みユーザー

2020/05/30 06:30

ありがとうございます。上手く行きました！ただ一点確認で、私もこれから調べるのですが、if self.request.user.is_authenticated()部分の最後のかっこを外すと上手く動作します。カッコをつけると、'bool' object is not callableが出てしまいます。

ForestSeo

2020/05/30 06:47

?? チュートリアルにはis_authenticated()って書いてあったんですけどね... template内はis_authenticatedじゃなきゃダメだった気がしますが。

退会済みユーザー

2020/05/30 08:58

私も調べてみます！分かったらこちらにコメントしますね！

ForestSeo

2020/05/30 22:14

思ったんですけど、やっぱ関数でやりません？w なんとなくここまで複雑になればDetailViewの威力が発揮できないというか...

ForestSeo

2020/06/01 22:42

上記でどうですか？回答を編集しました。