• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

CixoNotes - prośba o ocenę kodu prostego programu w języku python

Fiszki IT
Fiszki IT
+1 głos
60 wizyt
pytanie zadane 13 czerwca w Python przez Cixo Gaduła (4,030 p.)
Hejka, niedawno napisałem program służący do tworzenia prostych notatek, dodając go do .bashrc możemy sobie takie przygotowane wcześniej notatki wyświetlać za każdym razem jak zalogujemy się do tty albo otworzymy terminal. Jeżeli ktoś ma ochotę i chwilę czasu był bym bardzo wdzięczny za ocenę mojego kodu :3

Link do repo git: https://github.com/CixoDevelop/CixoNotes

Dzięki za każdą sugestię

1 odpowiedź

+2 głosów
odpowiedź 13 czerwca przez Benek Szeryf (87,050 p.)
wybrane 13 czerwca przez Cixo
 
Najlepsza

Na szybko:

  • Brak testów
  • Trzymanie konfiguracji lepiej byłoby przenieść do pliku z konfiguracją i zaczytywać taki plik (np. za pomocą modułu confgparser). Sprawdzanie pól poprzez iterowanie po vars to chyba nie najlepszy pomysł. Ostatecznie dałoby się ten kod skrócić poprzez iterowanie po atrybutach klasy Language
  • Niezgodność z PEP-8. Nazwy metod piszemy snake casem, a nie camel casem, to nie JS. Nazwy plików podobnie
  • Brak komentarzy w języku angielskim
  • Import z gwiazdką
  • Brak f-stringów, zamiast tego używasz starego formatowania
  • Brak menadżera kontekstu przy obsłudze plików
  • Do pracy ze ścieżkami/plikami używaj modułu pathlib zamiast os.path
  • Warto napisać w readme w jakimś znany formacie, np. markdown
  • Nic nie mówiące opisy commitów

 

1
komentarz 13 czerwca przez Cixo Gaduła (4,030 p.)
Dzięki wprowadze zmiany
komentarz 15 czerwca przez Cixo Gaduła (4,030 p.)

@Benek, Wprowadziłem zmiany o których pisałeś za wyjątkiem języka angielskiego - zbyt mało znam ten język jeszcze żeby w nim opisywać takie rzeczy. Jeżeli miał byś chwilkę był bym strasznie wdzięczny jak byś powiedział czy o to chodziło

Podobne pytania

0 głosów
5 odpowiedzi 348 wizyt
pytanie zadane 11 września 2017 w Java przez Patryk Moros Początkujący (480 p.)
+1 głos
0 odpowiedzi 75 wizyt
pytanie zadane 7 sierpnia w Inne języki przez mattaha Użytkownik (690 p.)
0 głosów
0 odpowiedzi 45 wizyt
pytanie zadane 7 sierpnia w Inne języki przez mattaha Użytkownik (690 p.)
Porady nie od parady
Odznacz odpowiedź zieloną fajką, jeśli uważasz, że jest ona najlepsza ze wszystkich i umożliwiła ci rozwiązanie problemu.Najlepsza odpowiedź

84,721 zapytań

133,527 odpowiedzi

295,921 komentarzy

55,997 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...