Проектирование системы комментирования | Forum

4 June 2017, 15:05

Posts: 15

4 June 2017, 15:05

Posts: 15

Добрый день, я новичок в программировании и хотел бы получить совет по поводу проектирования системы комментирования для сайта.

Хочу создать простенькую систему с двойной вложенностью как на youtube.

Пример:

-- Комментарий 1

-- Комментарий 4

-- Комментарий 6

-- Комментарий 2

-- Комментарий 3

-- Комментарий 5

Комментировать могут только авторизированные пользователи.

В моделе будут поля:

пользователь - привязка к моделе пользотеля
коментарий - текст самого коментария
дата создания
дата последнего изменения
карма - оценка пользователей комментария

Наконец можно подойти к сути темы.

Как посоветуете реализовать вложенность комментариев, что бы у вложенных комментариев была какая то привязка к родительскому комментарию?

Answer | 0

4 June 2017, 16:30 hich Posts: 164 Location: hich	4 June 2017, 16:30 hich Posts: 164 Location: hich MPTT hich
	Answer \| 0

4 June 2017, 20:11

lampslave

Posts: 517

4 June 2017, 20:11

lampslave

Posts: 517

mptt для этой цели более чем избыточен, достаточно простого fk на родительский комментарий. Дерево строится как-то так https://github.com/lampslave/blog/blob/master/lbe/utils.py#L8-L22, посмотрите там во вьюхе и шаблоне как всё это работает.

Updated 4 June 2017, 20:14 by lampslave.

Answer | 0

9 June 2017, 11:23

ModPhoenix

Posts: 15

9 June 2017, 11:23

ModPhoenix

Posts: 15

Спасибо за исходники, нашел еще в интернете видео урок и еще исходники совместил все и на данный момент вышло следующее.

Модель комментариев:

from django.conf import settings
from django.db import models
from django.contrib.contenttypes.fields import GenericForeignKey
from django.contrib.contenttypes.models import ContentType

class CommentManager(models.Manager):

    def filter_by_instance(self, instance):
        content_type = ContentType.objects.get_for_model(instance.__class__)
        object_id = instance.id
        qs = super(CommentManager, self).filter(content_type=content_type, object_id=object_id).filter(parent=None)
        return qs

class CommentAbstractModel(models.Model):
    content_type = models.ForeignKey(ContentType, on_delete=models.CASCADE)
    object_id = models.PositiveIntegerField()
    content_object = GenericForeignKey('content_type', 'object_id')

    class Meta:
        abstract = True

class Comment(CommentAbstractModel):
    user = models.ForeignKey(settings.AUTH_USER_MODEL)
    content = models.TextField(unique=False)
    date = models.DateTimeField(auto_now_add=True)
    modified = models.DateTimeField(auto_now=True)
    karma = models.IntegerField(default=0)
    parent = models.ForeignKey('self', on_delete=models.CASCADE, blank=True, null=True)

    objects = CommentManager()

    class Meta(CommentAbstractModel.Meta):
        db_table = 'comments'
        ordering = ['date']

    def __str__(self):
        return "%s: %s..." % (self.user.username, self.content[:50])

    def children(self):
        return Comment.objects.filter(parent=self)

Views:

def single_post(request, id, slug):

    post = get_object_or_404(Post, id=id)

    comments = post.get_comments

    initial_data = {
        "content_type": post.get_content_type,
        "object_id": post.id
    }

    comment_form = CommentForm(request.POST or None, initial=initial_data)
    if comment_form.is_valid():
        c_type = comment_form.cleaned_data.get("content_type")
        content_type = ContentType.objects.get(model=c_type)
        object_id = comment_form.cleaned_data.get("object_id")
        content_data = comment_form.cleaned_data.get("content")
        parent_obj = None

        try:
            parent_id = int(request.POST.get("parent_id"))
        except:
            parent_id = None

        if parent_id:
            parent_qs = Comment.objects.filter(id=parent_id)
            if parent_qs.exists():
                parent_obj = parent_qs.first()

        new_comment = Comment.objects.create(
                                user = request.user,
                                content_type = content_type,
                                object_id = object_id,
                                content = content_data,
                                parent = parent_obj,
                            )
        return HttpResponseRedirect ("%s#comment-%s" % (new_comment.content_object.get_absolute_url(), new_comment.id))

    context = {
        "post": post,
        "comments": comments,
        "comment_form": comment_form,
    }

    return render(request, "blog/single.html", context)

И шаблон:

{% for comment in comments %}

    <div id="comment-{{ comment.id }}" class="card p-3">
    <blockquote class="card-block card-blockquote">
    <div>
    <small class="text-muted">
    {{ comment.user }} <cite title="Source Title">{{ comment.date|timesince }} назад</cite>
    </small>
    </div>
    <p>{{ comment.content }}</p>
    </blockquote>
    <form method="POST" action="">
    {% csrf_token %}
    {{ comment_form }}
    <input type="hidden" name="parent_id" value="{{ comment.id }}">
    <p><input type="submit" class="btn btn-primary btn-comment" value="Ответить"></p>
    </form>

    </div>

    {% for child_comment in comment.children %}

        <div id="comment-{{ child_comment.id }}" class="reply card p-3">
        <blockquote class="card-block card-blockquote">
        <div>
        <small class="text-muted">
        {{ child_comment.user }} <cite title="Source Title">{{ child_comment.date|timesince }} назад</cite>
        </small>
        </div>
        <p>{{ child_comment.content }}</p>
        </blockquote>
        <form method="POST" action="">
        {% csrf_token %}
        {{ comment_form }}
        <input type="hidden" name="parent_id" value="{{ comment.id }}">
        <p><input type="submit" class="btn btn-primary btn-comment" value="Ответить"></p>
        </form>
        </div>

    {% endfor %}

{% endfor %}

Все работает нормально, но это только тестовая реализация. Нужно еще удаление комментариев корректное сделать.

Использовал GenericForeignKey потому что мне нужны будет комментарии под разными моделями.

Updated 9 June 2017, 11:28 by ModPhoenix.

Answer | 1

9 June 2017, 11:24 ModPhoenix Posts: 15	9 June 2017, 11:24 ModPhoenix Posts: 15 Хотел еще спросить как можно в django посмотреть сколько запросов к БД идет?
	Answer \| 0

9 June 2017, 12:42 gruy Posts: 608 Location: Феодосия	9 June 2017, 12:42 gruy Posts: 608 Location: Феодосия django-debug-toolbar
	Answer \| 1

10 June 2017, 19:45 ModPhoenix Posts: 15	10 June 2017, 19:45 ModPhoenix Posts: 15 проверил django-debug-toolbar количество SQL запросов `4,20 ms (14 запросов включая дублей: 11 )` откуда берутся эти дубли и нормально ли это? Если нет то как от них избавится? Updated 10 June 2017, 19:45 by ModPhoenix.
	Answer \| 0

11 June 2017, 18:12

ModPhoenix

Posts: 15

11 June 2017, 18:12

ModPhoenix

Posts: 15

Была проблема с очень большим количеством SQL запросов, но при помощи django-debug-toolbar и документации удалось все пофиксить.

Теперь 1 SQL запрос при любом количестве комментариев, а до этого было 2 запроса на 1 комментарий. Причем делал запросы как на ютубе показывали я уже думал что это норма для джанго при 50 комментах 150+ запросов к бд.

Answer | 0

12 June 2017, 14:01

oleg_n

Posts: 311
Location: Kyiv

12 June 2017, 14:01

oleg_n

Posts: 311
Location: Kyiv

Теперь 1 SQL запрос при любом количестве комментариев, а до этого было 2 запроса на 1 комментарий. Причем делал запросы как на ютубе показывали я уже думал что это норма для джанго при 50 комментах 150+ запросов к бд.

Не затруднит ли показать решение?

Answer | 0

13 June 2017, 8:14

ModPhoenix

Posts: 15

13 June 2017, 8:14

ModPhoenix

Posts: 15

Вот что я изменил по сравнению с предыдущим показом.

Создание дерева было реализовано в моделе комментариев, путем получения разными запросами родительских и вложенных комментариев. Я перенес создание дерева во вюху (пока что), как советовал lampslave. Вот как это работает:

  def filter_by_instance(self, instance):
        content_type = ContentType.objects.get_for_model(instance.__class__)
        object_id = instance.id
        qs = super(CommentManager, self).filter(content_type=content_type, object_id=object_id)
        return qs

Получаем все комментарии для поста:

comments = Comment.objects.filter_by_instance(post).select_related()

Обязательно нужно использовать .select_related() для получения связанных моделей.

SQL запрос без .select_related()

SELECT ••• FROM "comments" WHERE ("comments"."object_id" = '1' AND "comments"."content_type_id" = '7') ORDER BY "comments"."date" ASC

с ним

SELECT ••• FROM "comments" INNER JOIN "django_content_type" ON ("comments"."content_type_id" = "django_content_type"."id") INNER JOIN "auth_user" ON ("comments"."user_id" = "auth_user"."id") WHERE ("comments"."content_type_id" = '7' AND "comments"."object_id" = '1') ORDER BY "comments"."date" ASC

Сортируем комментарии нулевого и первого уровней

comments_parent = []
comments_children = []

    for comment in comments:

        if comment.parent_id is None:
            comments_parent.append(comment)
        else:
            comments_children.append(comment)

Выводим в шаблоне(тут конечно минус что много логики в шаблоне):

{% for comment in comments_parent %}

    <div id="comment-{{ comment.id }}" class="card p-3">
    <blockquote class="card-block card-blockquote">
    <div>
    <small class="text-muted">
    {{ comment.user }} <cite title="Source Title">{{ comment.date|timesince }} назад</cite>
    </small>
    </div>
    <p>{{ comment.content }}</p>
    </blockquote>
    <form method="POST" action="">
    {% csrf_token %}
    {{ comment_form }}
    <input type="hidden" name="parent_id" value="{{ comment.id }}">
    <p><input type="submit" class="btn btn-primary btn-comment" value="Ответить"></p>
    </form>

    </div>


    {% for child_comment in comments_children %}

    {% if child_comment.parent_id == comment.id %}



        <div id="comment-{{ child_comment.id }}" class="reply card p-3">
        <blockquote class="card-block card-blockquote">
        <div>
        <small class="text-muted">
        {{ child_comment.user }} <cite title="Source Title">{{ child_comment.date|timesince }} назад</cite>
        </small>
        </div>
        <p>{{ child_comment.content }}</p>
        </blockquote>
        <form method="POST" action="">
        {% csrf_token %}
        {{ comment_form }}
        <input type="hidden" name="parent_id" value="{{ comment.id }}">
        <p><input type="submit" class="btn btn-primary btn-comment" value="Ответить"></p>
        </form>
        </div>


    {% endif %}
    {% endfor %}

{% endfor %}

В итоге выходит мы получаем все данные с базы для комментариев 1 запросом, потом сортируем по вложенности и выводим в шаблоне.

Updated 13 June 2017, 8:14 by ModPhoenix.

Answer | 2

13 June 2017, 13:40 oleg_n Posts: 311 Location: Kyiv	13 June 2017, 13:40 oleg_n Posts: 311 Location: Kyiv ModPhoenix - Спасибо! Очень познавательно и полезно!
	Answer \| 1

14 June 2018, 8:02 goltsev Posts: 1	14 June 2018, 8:02 goltsev Posts: 1 CommentForm где фигурирует еще подскажите?
	Answer \| 0