Sprawdzenie aplikacji w Springu

Sprawdzenie aplikacji w Springu
N1
  • Rejestracja:ponad 8 lat
  • Ostatnio:prawie 7 lat
  • Postów:18
0

Witam

Jestem w trakcie pisania aplikacji z wykorzystaniem Springa. Miałbym do Was prośbę - chciałbym abyście spojrzeli na mój kod i ocenili go (wskazali co robię źle, co powinienem poprawić itd.)
Jest to harmonogram pracowników. Mam w nim obiekt Employee, który reprezentuje pracownika. Poza polami jak imię, nazwisko czy adres, zawiera też listę dni. Moja koncepcja jest taka, że pracownik posiada dni, które są dla niego robocze (nie dla wszystkich dany dzien jest roboczy). Dzień z kolei posiada kilka pól - datę, ilość zadeklarowanych godzin, ilość godzin faktycznie przepracowanych, obecność planowana, obecność faktyczna. Aplikację chciałby oprzeć o REST. Udostępniłem już kilka metod i obecnie aplikacja umożliwia:

  • dodanie pracownika - wiadomo,
  • pobranie wszystkich pracowników - jw.,
  • pobranie pracownika na podstawie id - jw.,
  • pobranie dni roboczych w danym miesiącu dla danego pracownika (EmployeeService.getDaysFromMonth) - metoda zwraca listę obiektów typu dzień, który posiada wskazany pracownik z wskazanym miesiącem i rokiem
  • pobranie par pracownik, lista dni - zwraca mapę <Employee, List<Day>> zawierającą wszystkie dni robocze dla danego miesiąca i roku
  • druga metoda robi to co powyższa z tym, że nie zwraca mapy tylko listę obiektów EmployeeAndDaysList - ma to na celu umożliwienie przekazania tego jako JSON RESTowy
  • ustawienie dla danego pracownika, w danym miesiącu ilości planowanych godzin

Aplikakcja to na razie sam backend i to jeszcze nie gotowy, ale nie chce podawać ocenie od razu całej aplikacji, tylko część, abym mógł poprawić błędy i dalej pisac już z uwzględnieniem moich błędów.

Ogólnie najbardziej zależy mi żebyście ocenili strukturę samego projektu (podział na warstwy domeny, usług, repozytorium) i koncepcję tzn., że pracownika posiada listę dni (przy tych założeniach - dzień posiada dodatkowe wtrybuty - wydało mi się to najlepszą opcją).
Repozytorium to na razie taka atrapa w pamięci, ale docelowa chciałbym wykorzystać Spring Data. Sama aplikacja nie ma zbyt wiele konfiguracji bo jest to Spring Boot

https://github.com/nlo123/szp

S9
  • Rejestracja:ponad 10 lat
  • Ostatnio:5 miesięcy
  • Lokalizacja:Warszawa
  • Postów:3573
1

Blędne jest używanie Chociażby java.util.Date
Brak odpowiedniego podziału na pakiety- zazwyczaj mamy taki podział na 3 grupy
-data
-domain
-web
w data są modele bazodanowe a w domain DTO


"w haśle <młody dynamiczny zespół> nie chodzi o to ile masz lat tylko jak często zmienia się skład"
edytowany 1x, ostatnio: scibi92
N1
  • Rejestracja:ponad 8 lat
  • Ostatnio:prawie 7 lat
  • Postów:18
0

Dziękuję za odpowiedź

Rozumiem, że zamiast java.util.Date lepiej jest użyć Calendar? Ale czy do samego przechowywania daty Date nie jest odpowiedni? Wszystkie operacji na datach wykonuje z użyciem klasy Calendar.

Spotkałem się z podziałem na pakiety taki jak zastosowałem, czyli:

  • Domain - obiekty POJO
  • Repository - warstwa DAO
  • Service - usługi, logika aplikacji
  • Controller - kontrolery

Czy taki podział też jest stosowany czy w praktyce stosowany jest tylko ten podany przez Ciebie?

jarekr000000
Nie rób operacji na datach z Calendar - on się do tego nie nadaje. Dlatego w Java 8 powstało nowe API do dat - zacznij od wyszukania LocalData (i tego się trzymaj).
S9
  • Rejestracja:ponad 10 lat
  • Ostatnio:5 miesięcy
  • Lokalizacja:Warszawa
  • Postów:3573
0

Ech jak zapisujesz date do bazy danych to korzystaj z java.sql.Date albo z nowego Java 8 Date API
Ja stosuje podział:
-config
-common
--exception
-data
--dao
--entity
-domain
--dto
--service
-web
--controller

Plus jakiegoś dodatkowe pakiety np. do common walidacje,do web security itp


"w haśle <młody dynamiczny zespół> nie chodzi o to ile masz lat tylko jak często zmienia się skład"
edytowany 1x, ostatnio: scibi92
jarekr000000
Co do ustalenego podziału to polecam obejrzenie super filmiku z Uncle Bobem i przemyślenie.... https://vimeo.com/97530863
P7
  • Rejestracja:około 9 lat
  • Ostatnio:3 dni
  • Lokalizacja:Warszawa
  • Postów:89
0

odnośnie struktury projektu i organizacji pakietów ze względu na warstwy vs funkcjonalności to niedawno toczyła się ciekawa dyskusja tu na forum, polecam poczytać:
Podział na pakiety ze względu na warstwy czy funkcjonalność.

jeszcze odnośnie kodu patrząc na szybko rzuciła mi się w oko metoda

Kopiuj
getEmployeesByNames

z repo. - jest do poprawy bo:

  1. przesłoniłeś sobie w metodzie pole
Kopiuj
employees

i porównujesz z pustą listą

  1. porównujesz Stringi == niedobrze, bardzo..
edytowany 1x, ostatnio: Pablitto77
FI
FI
  • Rejestracja:około 10 lat
  • Ostatnio:prawie 4 lata
  • Postów:471
0

a jak to jest z @ResponseBody , czy metody void powinny miec taka anotacje?

Shalom
  • Rejestracja:około 21 lat
  • Ostatnio:prawie 3 lata
  • Lokalizacja:Space: the final frontier
  • Postów:26433
0

A jaki to by miało sens? o_O Zresztą teraz zamiast @ResponseBody lepiej po prostu zrobić osobny @RestController jak chcesz jsony zwracać.


"Nie brookliński most, ale przemienić w jasny, nowy dzień najsmutniejszą noc - to jest dopiero coś!"
PI
  • Rejestracja:ponad 9 lat
  • Ostatnio:3 miesiące
  • Postów:2787
0

@Shalom to wcale nie jest takie bezsensowne pytanie. Jak metoda void nie ma adnotacji 'ResponseBody', to zamiast pustego JSona dostaniesz 404 not found html stronkę ^^ #sprawdzone_info

Kliknij, aby dodać treść...

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj <u>underline</u>.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, <b>, <i>, <kbd>, <del>, <strong>, <dfn>, <pre>, <blockquote>, <hr/>, <sub>, <sup> oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H<sub>2</sub>O i m<sup>2</sup> otrzymasz: H2O i m2.

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript
document.write('Hello World');
```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy  by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

  • Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
  • Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
  • Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
  • Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

  • Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

  • Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

  • Alt+L - zaznaczenie całej linii
  • Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
  • Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
  • Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

  • Ctrl+Enter - dodaj post
  • ⌘+Enter - dodaj post (MacOS)