Mikroblog w Django, proszę o code review

Question

Hej, od jakiegoś czasu uczę się Django i postanowiłem, że napiszę aplikację do mikroblogowania. Mam już w miarę działającą wersję, ale jest to w zasadzie mój pierwszy projekt w Django, nie licząc przepisywania czegoś z tutoriala, więc bardzo serdecznie prosiłbym, żeby ktoś zaznajomiony z tematem rzucił okiem, czy jest to jakoś akceptowalnie zrobione, a nie przykładowo łamię wzorzec projektowy. Każdą uwagę przyjmę oczywiście z otwartymi rękoma, jeżeli jest potrzebna wersja live to mogę postawić i wrzucić link. Kod: https://github.com/szafran98/mikroblog_django

Answer 1

Kilka losowych wybiórczych spostrzeżeń:

- masz trochę duplikacji roboty związanej z modelami. Na przykład:

Post.update_post_content(id, new_content_post)

Ale wyżej masz już zmienną edited_post, to czemu nie po prostu nie zmodyfikować tego obiektu?

Podobnie:

user_comments_count = Comment.get_user_comment_count(username)

W środku robisz filtrowanie z joinem, żeby przefiltrować po username. Ale znowu, po co, skoro masz już pod ręką obiekt z tym userem?

Mam wrażenie że jest tego więcej, ale nie będę więcej wymieniał.

Dalej:

def delete_post(request, id):
    post_to_delete = Post.objects.get(id=id)
    if request.method == 'DELETE':

Jest do tego dekorator `require_http_methods`.

        let timeToChange = element.innerHTML;
        let toNumber = parseInt(timeToChange);
        element.innerHTML = time2TimeAgo(toNumber);

Jeśli nie musisz koniecznie operować na surowym HTMLu, unikaj innerHTML. textContent powinien tutaj wystarczyć, nie?

style="margin: auto; max-width: 600px; margin-bottom: 30px;

Inline style są be ;)

Ogólna uwaga:

def edit_post(request, id):
    edited_post = Post.objects.get(id=id)
    post_form = AddPostForm(request.POST or None, instance=request.user)
    if request.POST:

Poczytaj o "klasowych" widokach w Django. Pozwalają uniknąć sporo boilerplate'u takiego jak powyższe klasyczne tworzenia Forma i `if request.POST`.

Comments

Dzięki serdeczne, teraz sprawdzam i faktycznie celne uwagi, kłaniam się.