Wpis z mikrobloga

#python #naukaprogramowania

Witam. Właśnie skończyłem pisać swoją drugą aplikację w Django i byłbym wdzięczny za code review, zarówno aplikacji jak i testów. Jest to prosty blog, ale napisany w pełni przeze mnie, chociaż niektóre rozwiązania były inspirowane książka Django by example (np. paginacja). Mam też parę pytań dotyczących kodu:
1) Jak w klasie dziedziczącej po ModelAdmin sprawić, aby dało się sortować pole, które jest pobierane z redisa? Chodzi konkretnie o sortowanie po number of views
2) Niektóre testy są zakomentowane (testviews.py i testmodels.py; wiem, że nie powinienem czegoś takiego zostawiać w kodzie). Są to testy, które nie przechodzą (oprócz TestSearchView, który wywala błąd o niezainstalowanym pg_trgm), mimo to same funkcjonalności działają. Jak to naprawić?
Z góry dziękuję za wszystkie uwagi. Link to repo: https://github.com/rcybulski1122012/django_blog i do wersji live http://rcybulski.herokuapp.com/
  • 2
  • Odpowiedz
  • Otrzymuj powiadomienia
    o nowych komentarzach

@radek1122012: Trochę późnawo, ale rzadko jestem na wykopie. Bardzo na plus jest to, że w ogóle pokusiłeś się o napisanie testów. Prawdopodobnie z tymi testami chodzi o to, że już po fakcie odwiedzenia strony inkrementujesz ilość wyświetleń. Najlepiej zbadać temat odpalając debugger, bo tak to wróżenie z kart. Większość widoków niepotrzebnie zaimplementowałeś funkcyjnie, a można było użyć do tego klas generycznych i nie powielać kodu. Miejscami kod wydaje się jakby
  • Odpowiedz