michat90 / praca_dyplomowa

0 stars 0 forks source link

Uwagi ogólne #3

Open mmiklas opened 1 year ago

mmiklas commented 1 year ago

Po sprawdzeniu projektu, kilka uwag ogólnych:

michat90 commented 1 year ago

Będę pracował nad tymi punktami przez weekend. Co do GUI podszedłem do tematu z myślą że chce sam pisać css żeby nauczyć się płynnie w nim poruszać co uważam będzie pomocne w przyszłości.

śr., 6 wrz 2023, 14:18 użytkownik Marcin Mikłas @.***> napisał:

Po sprawdzeniu projektu, kilka uwag ogólnych:

  • nie ma sensu definiować typu transakcji w kategorii, bo i tak zawsze zostaje ona nadpisana przy wprowadzaniu samej transakcji,
  • w tego typu aplikacji cennym dodatkiem byłaby, graficzna reprezentacja transkacji, np. wizualizacja wydatków po kategoriach itp.,
  • warto więcej uwagi poświęcić na GUI/UX, schludny interfejs pozwala użytkownikowi łatwiej poruszać się po aplikacji i sprawia, że nie trzeba się dodatkowo jej uczyć lub czytać dokumentację (do poprawy rozmiszczenie okien, etykiet, przycisków, czcionek itp.) warto rozważyć skorzystanie z gotowych rozwiązań jak frameworki css,
  • po zapisaniu formularza użytkownik spodziewa się przekierowania do listy elementów lub komunikatu, że wszystko poszło dobrze,
  • im prostszy interfejs, lepiej opisany tym lepiej, warto korzystać ze sprawdzonych znanych schematów tworzenia elementów jak CRUD,
  • dokumentacja wymaga dopracowania,
  • przy rozwijaniu projektu warto inwestować w testy jednostkowe, ich koszty szybko się zwracają,
  • warto zadbać o utrzymanie kodu w lepszej kondycji, używamy repozytorium więc zamiast komentować można usunąć niepotrzebne fragmęnty kodu i stosować automatyczne formatowanie, sprawi to, że projekt będzie łatwiejszy w utrzymaniu.

— Reply to this email directly, view it on GitHub https://github.com/michat90/praca_dyplomowa/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/AOFS3ABCP4LFWFB3X5XVGPTXZBS3DANCNFSM6AAAAAA4NHWTRU . You are receiving this because you are subscribed to this thread.Message ID: @.***>