code review - Hotel Reservation System

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

0

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

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.

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

1 użytkowników online, w tym zalogowanych: 0, gości: 1