Code Review Spring Rest- amator

Code Review Spring Rest- amator

Wątek przeniesiony 2020-07-01 20:14 z Java przez Shalom.

HU
  • Rejestracja:ponad 9 lat
  • Ostatnio:ponad 4 lata
  • Postów:21
0

Witam,

Niedawno zacząłem się bawić w Springu i zrobiłem aplikacje Rest. Nie chciałem też się porywać nie wiadomo na co na początku.
Zanim zacznę jeszcze sprawdzać całą walidację i testy, to prosiłbym o Code Review / ocenę samego projektu

https://github.com/rmaduzia/SpringBootRestApplication-MatchingUserProfile

Za wszelkie merytoryczne uwagi z góry dzięki.

PS. jakby co to właśnie w tym momencie przypomniałem sobie, żeby zautomatyzować Mappera
Nie jestem jeszcze przekonany jak powinien np wyglądać Update User - czy wymagane hasło czy bez ?

edytowany 2x, ostatnio: hugarili
Shalom
  • Rejestracja:około 21 lat
  • Ostatnio:prawie 3 lata
  • Lokalizacja:Space: the final frontier
  • Postów:26433
1

Ale co tu oceniać? Przecież tam nic nie ma. To jest Generic CRUD sklonowany z jakiegoś tutoriala starszego od ciebie.

Przyznam że nie wiem czy to repo to jest trolling/żart czy nie...


"Nie brookliński most, ale przemienić w jasny, nowy dzień najsmutniejszą noc - to jest dopiero coś!"
edytowany 1x, ostatnio: Shalom
HU
Dzięki za opinie, poprawie wspomniane błędy oraz te rzeczy, które napisałem, natomiast prosiłbym wyjaśnienie co jest nie tak z jest mapowanie 1:1 @Entity na DTO , chyba, że chodzi o te ID ? W sprawie tamtego kontrolera, to przyznam się, że nie do końca wiedziałem jak to ugryźć i dalej nie wiem, żeby własnie nie było z tego takiego makaronu. Skoro sam temat w Repo jest strasznie jałowy, podpowiesz na czym lepiej się skupić ? Bo tak jak napisałem, chciałem coś klepnąć, żeby zapoznać się na wstępie z Springiem Za pomoc dzięki
Shalom
Generalnie takie mapowanie zwyczajnie nie ma sensu. Nie masz tu żadnej logiki więc mógłbyś robić encja na twarz i pchasz. Jaki masz zysk z tego przemapowywania w obie strony? Jeśli chodzi o ten kontroler to Spring ma takie coś jak query by example -> https://www.baeldung.com/spring-data-query-by-example jeśli już koniecznie chcesz to JPA tam mieć.
AK
@Shalom: poświęciłem nieco z mojego czasu. Rzeczywiście to się kupy nie trzyma. Ale MIAŁO BYĆ W SPRINGU to jest. To jakas nowa religia?
AK
Łącznie z porównaniem Xxxxx == "" Bosch ....
JK
  • Rejestracja:około 11 lat
  • Ostatnio:około 10 godzin
  • Postów:107
1

Pan u góry chyba dawno nie był początkującym ;)
Pamiętam jak ja zaczynałem - konfiguracja samego springa (jeszcze tylko XML był dostępny) żeby w ogóle pozbyć się wyjątków to były +/- 3-4 tygodnie. Nawet nie wiedziałem czym rózni się POST/GET.

Fakt, kod jest słaby, nie ma co oceniać ALE sam plus że COŚ już jest - dodawaj logike, refaktoruj, pisz proste testy i za 3 miesiące będziesz się śmiał, że takie proste rzeczy sprawiały Ci problemy.

HU
  • Rejestracja:ponad 9 lat
  • Ostatnio:ponad 4 lata
  • Postów:21
0
Jan Ko napisał(a):

Fakt, kod jest słaby, nie ma co oceniać ALE sam plus że COŚ już jest - dodawaj logike, refaktoruj, pisz proste testy i za 3 miesiące będziesz się śmiał, że takie proste rzeczy sprawiały Ci problemy.

Sam post nie miał na celu pokazać, jak zajebisty jest to projekt, czy coś. Chciałem tylko poznać wstępnie opinie. Stwierdziłem, że lepiej będzie napisać prosty projekt niż od razu zabrać się za coś skomplikowanego(Rozumiem uczenie się na błędach etc, ale czemu tracisz czas na różne obejścia, zmieniać praktycznie pół projektu, bo nagle okazuje się inne rozwiązanie jest lepsze(a powinno być to oczywiste), temu wstępnie ten projekt służy.

Dopiszę testy, walidację, wprowadzę dodatkowe funkcje matchowania, ale nie wiążę z tym jakiejś przyszłości, to miał być wstęp na zapoznanie się z Springiem.

Po prostu chciałem się dowiedzieć, ile tak naprawdę błędów popełniłem, zanim przystąpię do czegoś większego.

AK
  • Rejestracja:ponad 6 lat
  • Ostatnio:około rok
  • Postów:3561
1

@hugarili: Ugruntuj Javę, naucz się jej porządnie, bez kwiatków jak tutaj (porównania i sprawdzanie stringów). Naucz się myśleć obiektowo, a nie proceduralnie.
Uczenie się na tym etapie Springa zrobi ci krzywdę.


Bo C to najlepszy język, każdy uczeń ci to powie
HU
Faktycznie teraz dopiero zdałem sobie sprawę z tego porównania, a co masz dokładniej na myśli z sprawdzaniem stringów ? Z tym ugruntowaniem wiedzy z Javy masz na pewno rację, bo sam wiem, jak wiele nie wiem. Po prostu myślałem, że pogodzę w parze naukę Javy i Springa przy rozwiązywaniu napotkanych problemów.
Grzyboo
@hungarili Stringi (a nawet ogólnie: Obiekty) porównujemy metodą equals, nie operatorem ==. Ten drugi porównuje referencje, więc 2 identyczne ciągi znaków mogą nie być równe i zazwyczaj nie będą. No i nauka Springa i Javy jednocześnie niestety nie idzie w parze.
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)