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

4 Июнь 2017, 15:05

Отклики: 15

4 Июнь 2017, 15:05

Отклики: 15

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

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

Пример:

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

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

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

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

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

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

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

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

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

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

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

Ответ | 0

4 Июнь 2017, 16:30 hich Отклики: 165 Местонахождение: hich	4 Июнь 2017, 16:30 hich Отклики: 165 Местонахождение: hich MPTT hich
	Ответ \| 0

4 Июнь 2017, 20:11

lampslave

Отклики: 512

4 Июнь 2017, 20:11

lampslave

Отклики: 512

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

Обновлено 4 Июнь 2017, 20:14 lampslave.

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

Ответ | 0

9 Июнь 2017, 11:23

ModPhoenix

Отклики: 15

9 Июнь 2017, 11:23

ModPhoenix

Отклики: 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 потому что мне нужны будет комментарии под разными моделями.

Обновлено 9 Июнь 2017, 11:28 ModPhoenix.

Ответ | 1

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

9 Июнь 2017, 12:42 gruy Отклики: 481 Местонахождение: Феодосия	9 Июнь 2017, 12:42 gruy Отклики: 481 Местонахождение: Феодосия django-debug-toolbar
	Ответ \| 1

10 Июнь 2017, 19:45

ModPhoenix

Отклики: 15

10 Июнь 2017, 19:45

ModPhoenix

Отклики: 15

проверил django-debug-toolbar количество SQL запросов 4,20 ms (14 запросов включая дублей: 11 )

откуда берутся эти дубли и нормально ли это? Если нет то как от них избавится?

enter image description here

Обновлено 10 Июнь 2017, 19:45 ModPhoenix.

Ответ | 0

11 Июнь 2017, 18:12

ModPhoenix

Отклики: 15

11 Июнь 2017, 18:12

ModPhoenix

Отклики: 15

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

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

Ответ | 0

12 Июнь 2017, 14:01

oleg_n

Отклики: 257
Местонахождение: Kyiv ** jf2@ua.fm

12 Июнь 2017, 14:01

oleg_n

Отклики: 257
Местонахождение: Kyiv ** jf2@ua.fm

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

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

Ответ | 0

13 Июнь 2017, 8:14

ModPhoenix

Отклики: 15

13 Июнь 2017, 8:14

ModPhoenix

Отклики: 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 запросом, потом сортируем по вложенности и выводим в шаблоне.

Обновлено 13 Июнь 2017, 8:14 ModPhoenix.

Ответ | 2

13 Июнь 2017, 13:40 oleg_n Отклики: 257 Местонахождение: Kyiv ** jf2@ua.fm	13 Июнь 2017, 13:40 oleg_n Отклики: 257 Местонахождение: Kyiv ** jf2@ua.fm ModPhoenix - Спасибо! Очень познавательно и полезно!
	Ответ \| 1

14 Июнь 2018, 8:02 goltsev Отклики: 1	14 Июнь 2018, 8:02 goltsev Отклики: 1 CommentForm где фигурирует еще подскажите?
	Ответ \| 0