Django自定义GameManager调用show_pc_game提示Manager无该属性如何解决?
报错修复方案
报错的核心原因是GameManager类下的show_pc_game方法缩进错误,没有被识别为管理器的成员方法,同时还存在两处逻辑错误需要一并修复:
- 方法缩进错误
- 多对多字段筛选逻辑错误
- 模板变量引用错误
1 修复models.py的缩进和筛选逻辑
原代码中
show_pc_game没有缩进在GameManager类的作用域内,且多对多字段不能直接传字符串筛选,需要跨表查询关联的平台名称。
修改后代码:
class GameManager(models.Manager): # 方法缩进至类内部 def show_pc_game(self): # 多对多字段用双下划线跨表查询Platform的title字段 return self.filter(platform__title='PC') # 其余Model代码保持不变
2 修复模板变量引用错误
原模板中循环
PC_games时用的迭代变量是pcgame,但输出时错误引用了game.title,同时当前逻辑会在所有平台的子菜单下都展示PC游戏,不符合需求。
修改后base.html代码:
<li class="nav-item dropdown"> <a class="nav-link dropdown-toggle" href="#" data-bs-toggle="dropdown">Platform</a> <ul class="dropdown-menu"> {% for platform in platforms %} <li><a class="dropdown-item" href="#"> {{platform.title}} » </a> {% if platform.title == 'PC' %} <ul class="submenu dropdown-menu"> {% for pcgame in PC_games %} <!-- 改用迭代变量pcgame取值 --> <li><a class="dropdown-item" href="#">{{pcgame.title}}</a></li> {% endfor %} </ul> {% endif %} </li> {% endfor %} </ul> </li>
推荐优化方案
如果需要在所有平台下都展示对应游戏,不需要单独定义管理器方法,利用Django多对多反向查询即可实现,代码更简洁易维护:
第一步:修改context_processors.py,删除PC_games相关代码
from .models import Platform, Game def add_variable_to_context(request): return { 'platforms': Platform.objects.all(), 'games': Game.objects.all(), }
第二步:修改base.html代码,直接遍历每个平台的关联游戏
<li class="nav-item dropdown"> <a class="nav-link dropdown-toggle" href="#" data-bs-toggle="dropdown">Platform</a> <ul class="dropdown-menu"> {% for platform in platforms %} <li><a class="dropdown-item" href="#"> {{platform.title}} » </a> <ul class="submenu dropdown-menu"> {% for game in platform.game_set.all %} <li><a class="dropdown-item" href="#">{{game.title}}</a></li> {% empty %} <li><a class="dropdown-item disabled" href="#">暂无该平台游戏</a></li> {% endfor %} </ul> </li> {% endfor %} </ul> </li>
内容的提问来源于stack exchange,提问作者saleh
相关产品推荐
相关产品推荐

