Покритикуйте код, что можно улучшить?

Question

[Solliter]

Добрый день!
Я новичок, не давно начал искать работу junior и для лучшего поиска работы на гит хаб написал не большой блог https://github.com/solliter/blog-with-accounts , покритикуйте пожалуйста, а также буду рад любым замечаниям и предложением, главное конструктивным, спасибо!

Comments

[Павел Аксенов]

1. blog/admin.py - тут либо что-то недоделано, либо написан лишний код. BlogAdmin не используется.
   
2. blog/views.py в blog_new можно только один раз инициализировать форму, тем самым избавиться от else.
   

   def blog_new(request):
       form = BlogForm(request.POST or None)
       if request.method == "POST":
           if form.is_valid():
               ...
       return render(request, 'blog_edit.html', {'form': form})

   
   так во всех вьюхах можно избавиться от дублирующей инициализации.
   
3. придерживаться одного стиля
   

   return render(request, 'blog_edit.html', {'form': form})
   # или
   context = {}
   template = "home.html"
   return render(request, template, context)

   
   
   
4. Форма авторизации смотрится как-то одиноко и не к месту в blogs/forms.py
   Можно использовать стандартную форму django
   

   from django.contrib.auth.forms import AuthenticationForm

   
   
   

[Павел Аксенов]

В settings.py

1. INSTALLED_APPS идёт все вперемешку, лучше как-то разделить.
   

   INSTALLED_APPS = [
   'стандартные джанго'
   ]
   INSTALLED_APPS += [
   'сторонние/установленные'
   ]
   INSTALLED_APPS += [
   'свои, добавленные в проект'
   ]

   
   
   
2. TEMPLATES.DIRS
   такое ('home/solliter/PycharmProjects/djangoproject/app/blog/templates/blog/') не стоит указывать.
   лучше вычислять путь через os.path.join(BASE_DIR, 'относительный путь от корня проекта')
   
   

[Ranwise]

там еще левые файлы в репе os sys на 20 мб

Answer 1

[Roman K]

1. db_table не использовать, если ты не понимаешь, для чего он
   
2. Дублирование названия класса в атрибутах ужасно
   
3. Почитай стайлгайды Django и посмотри, куда нужно класть class Meta и __str__
   
4. Код грязноват. Ненужные комменты, snake_case перемешан с camelCase, несоблюдение PEP8
   
5. Алгоритмические ошибки. Например, вьюха subscribe, если метод не POST, ведёт в никуда
   
6. В той же вьюхе стоит редирект на URL. Используй reverse, в шаблонах же используешь {% url %}. Кстати, шорткат redirect, который ты используешь, умеет редиректить на название вьюхи (name в urls) и на конкретный объект, если у него определён метод get_absolute_url
   
7. Тут указали, что стоит использовать Class Based Views. Для новичка это слишком неочевидные вещи с этими вашими миксинами. Лучше разберись сначала в Function Based Views