VBC意见的良好做法

时间:2019-10-02 00:50:11

标签: python django django-views

我对基于类的视图有疑问。

我有以下观点:

class UserCourseListView(ListView):
    model = User
    template_name = 'courses/user_course_list.html'

    def get_queryset(self):
        user_inscriptions = self.request.user.get_inscriptions.all()
        courses = [inscription.course for inscription in user_inscriptions]

        return courses

该视图的目的是获得当时登录用户已注册的课程的列表。并能够在上下文中(在模板中)访问该数据。


我有以下疑问:

model类属性是User模型,在get_queryset estoy方法中,返回Course模型的实例,这没有多大意义。

我的问题是,这样做是一种好习惯吗?

还有其他更好的方法可以用于基于类的视图吗?


我想出的另一种方法是,基于函数的视图更加一致,但这并不能说服我很多。

def user_course_list(request):
    user_inscriptions = request.user.get_inscriptions.all()
    courses = [inscription.course for inscription in user_inscriptions]

    return render('courses/user_course_list.html', {'courses': courses})

由于Django文档建议我们尽可能使用基于类的视图。

更新:

我的文件models.py

# --- --- COURSE MODEL --- ---

class Course(models.Model):
    STATUS = [('unpublish', 'No publicar'),('published', 'Publicar')]
    title = models.CharField(max_length = 60)
    subtitle = models.CharField(max_length = 120)
    price = models.DecimalField(max_digits = 5, decimal_places = 2, default = 0)
    image = models.ImageField(upload_to = manage_courses_media_files)
    description = models.TextField()
    status = models.CharField(max_length = 60, choices = STATUS, default = 'unpublish')
    instructor = models.ForeignKey(Instructor, on_delete = models.CASCADE, related_name = 'get_courses')
    slug = models.SlugField(max_length = 60)
    created = models.DateTimeField(auto_now_add = True)
    updated = models.DateTimeField(auto_now=True)

    class Meta:
        ordering = ['-created', '-updated']

    def __str__(self):
        return self.title

# --- --- INSCRIPTION MODEL --- ---

class Inscription(models.Model):
    course = models.ForeignKey(Course, on_delete = models.CASCADE, verbose_name = 'Curso')
    user = models.ForeignKey(User, on_delete = models.CASCADE, verbose_name = 'Usuario', related_name = 'get_inscriptions')
    created = models.DateTimeField(auto_now_add = True, verbose_name = 'Fecha de inscripcion')

    class Meta:
        verbose_name = 'inscripcion'
        verbose_name_plural = 'inscripciones'

    def __str__(self):
        return self.course.title

1 个答案:

答案 0 :(得分:0)

我不能完全确定您的模型是如何设置的,但是当为特定模型创建视图集时,您将按照与我在下面编写的内容进行设置。如果课程是铭文上的外键,则课程为模型,inscription_set为相关名称。现在,您可能已经在模型上自己设置了related_key名称,并且可以将其切换出来。我猜题词存储一个user_id?我不确定您的模型经理对.get_inscriptions做些什么。

class UserCourseListView(ListView):
    model = Course
    queryset = Course.objects.all()
    template_name = 'courses/user_course_list.html'

    def get_queryset(self):
        courses = self.queryset.filter(inscription_set__get_inscriptions=request.user)
        return courses.all()