code review - Hotel Reservation System

code review - Hotel Reservation System
Alvaro26
  • Rejestracja: dni
  • Ostatnio: dni
0

Cześć,

Po przeczytaniu tutoriala na stronie Oracle postanowiłem spróbować swoich sił i napisać coś większego, niż kalkulator.
Ostatecznie postanowiłem odłożyć, póki co, GUI, bo wolałem się skupić na dobrze zaprojektowanych klasach, niż na bajerach.

Gorąco proszę Was o przejrzenie kodu mojego projekciku i wytknięcie mi wszystkich błędów, jakie znajdziecie.

Link do Gita: https://github.com/alvaro26/HotelReservationSystem

Alvaro26
  • Rejestracja: dni
  • Ostatnio: dni
0

W ostatniej wersji programu nie ma tej klasy, już usuwam z Gita :D Myślałem, że automatycznie usuwa klasy usuniete w IDE

niezdecydowany
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Bieszczady
1
  1. Nie używa się clone(), to jakaś patologia w javie, poczytaj o tym.
  2. String f - co mi to mówi ? gó12no nic mi to nie mówi, nie wiem co to jest f.
  3. komentarze po polsku i po angielsku... zrób jedną wersje językową.'
  4. Room.java - raz używasz type inference, raz nie. Ta klasa ma też wielki konstruktor, nie podoba mi się.
  5. DataStorage.java to jest jeden wielki WTF'ak

poczytaj o single responsibility principle - za dużo rzeczy robisz w jednej metodzie.

Alvaro26
  • Rejestracja: dni
  • Ostatnio: dni
0

@niezdecydowany dzięki za wskazówki. Na DataStorage bym w ogóle nie patrzył, poprawię to i będę ładować dane z zewnętrznych plików :D

Czy takie coś można wpisać do CV na praktyki? Na studiach mamy C++ obiektowe, i przerabiamy mniej złożone programy

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.