IsambardKingdomBrunel
#python
#django
#naukaprogramowania

Przepisałem jeden ze swoich projektów i dodałem do niego kilka bajerów. Jakby ktoś mógł mi podpowiedzieć co jest nie tak, albo jak ulepszyć taką apkę to byłbym wdzięczny.

https://github.com/JohnCleese/Shelter
  • Najlepsze
  • Wszystkie komentarze
wqdqweff
wqdqweff
@IsambardKingdomBrunel: stoi to gdzieś, żeby można było zobaczyć?
IsambardKingdomBrunel
@wqdqweff: Mam notorycznie jakiś problem z heroku, którego nie potrafię rozwiązać. Nie może mi przejść static i nie mogę zrobić migrate bazy danych.
croppz
@IsambardKingdomBrunel: to ustaw sobie to na pythonanywhere, zmieścisz się na darmowym koncie. I mają bardzo łopatologiczną instrukcję jak ustawić u nich stronę.
L.....k
L.....k
Dalej to samo co ostatnio:
- Nie używasz linterów
- Nazwy twoich commitów są nieczytelne
- Nie sortujesz importów
- Nie korzystasz z PEP8
- ... ( ͡° ʖ
IsambardKingdomBrunel
@Lunatik:

Nie używam linterów bo po prostu nie wiem jak ich użyć

Z nazwami commitów się zgodzę i je
morsik
morsik
@IsambardKingdomBrunel: nazwy modułów (aplikacji Djangowych) masz rozpocząte wielkimi literami :P
Toż to tak wali po oczach, że ciężko nie rzucić argumentu o PEP8 ;)
IsambardKingdomBrunel
@morsik: No ok moja apka nie jest jakaś zajebista jeśli chodzi o PEP8, ale czy np. są jakieś rzeczy, jakie warto dodać pod względem samej funkcjonalności?
L.....k
L.....k
@IsambardKingdomBrunel: Jak postawisz demo to będziemy w stanie mówic więcej o funkcjonalnosci. Niestety stylistyka jest niewdzięczna, i im dłużej pracujesz tym naturalniej wyłapujesz takie niescisłosci. Nikt nie będzie na poważnie traktował aplikacji w której na pierwszy rzut oka brakuje konsekwencji w stylu, bo to stety-niestety podstawa. Jakikolwiek by on nie był, obierz jedne wytyczne i się ich trzymaj.

Jeżeli chodzi o lintery - isort i pylint są do ogarnięcia w