[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

[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]

Dalej to samo co ostatnio:
- Nie używasz linterów
- Nazwy twoich commitów są nieczytelne
- Nie sortujesz importów
- Nie korzystasz z PEP8
- ... ( ͡° ʖ̯ ͡°)

https://github.com/JohnCleese/Shelter/blob/master/Animals/middleware.py#L5-L73
Tego totalnie nie rozumiem, przecież django dostarcza wbudowane narzędzia do permissionów?

[IsambardKingdomBrunel]

@Lunatik:

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

Z nazwami commitów się zgodzę i je poprawiam

z tego co widzę to importy są w miarę posortowane

Nie wiem juz o co chodzi z zarzutem o PEP8. Przejrzałem i dopasaowałem najlepiej jak potrafię kod do pep8. Nazwy klas są zgodne i nazwy zmiennych. Formatowanie po przecinkach jeśli linijka przekracza ilość znaków też przestrzegam.
Pokaż mi po prostu gdzie

[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]

@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 godzinę.