Mój pierwszy projekt w Javie.

Odpowiedz Nowy wątek
2019-08-26 18:06
0

Witam,
Niedawno skończyłem pisać swój pierwszy projekt w Javie, który oczywiście może rozbuduję za jakiś czas. Jest do backend to internetowej aplikacji przeznaczonej do przechowywania notatek użytkowników, który komunikuje się poprzez REST API. Użyte technologie to Java, Spring oraz Hibernate. Mam w planach zrobienie frontendu do tej aplikacji jako osobnego "pod projektu" oraz innego znacznie większego projektu w tych technologiach. https://github.com/arkadiusz97/OFNotes/tree/master/backend Jak myślicie, co można by w nim lepiej zrobić?

Pozostało 580 znaków

2019-08-26 18:14
1
  1. Jak używasz mavena to zrób tak aby pom.xml był dostępny razem ze źródłami (https://github.com/matadini/hipster-webapp - tak jak tutaj zrobiłem). Wtedy jest szansa dwoma poleceniami: zbudować i uruchomić aplikację w terminalu.
  2. Plik konfiguracyjny dla każdej encji z mapowaniem jest dla mnie mocno passe i pierwszy raz się spotykam z czymś takim. Używaj adnotacji w klasach.
  3. Pakietów też mógłbyś używać według jakieś konwencji bez wrzucania całego śmietnika w src.
  4. Używaj lomboka i nie pisz samemu getterów, setterów i konstruktorów bo szkoda na to życia.
edytowany 1x, ostatnio: MrMadMatt, 2019-08-26 18:15
Ad 4. Skąd wiesz, że pisze sam? Może klika 3 klawisze i mu IDE generuje? :) - kixe52 2019-08-26 19:28
po co w ogóle pisać gettery i settery? :P - danek 2019-08-26 19:30
Żeby było więcej linii kodu w całym projekcie :D - kixe52 2019-08-26 19:31
@kixe52: Nawet jak mu IDE generuje to i tak musi mu wydać polecenie IDE do wygenerowania, koniec końców marnuje czas - to było sedno mojej wypowiedzi. - MrMadMatt 2019-08-26 19:45
@MrMadMatt: wiem, po prostu się czepiam słówek ;) - kixe52 2019-08-26 19:53

Pozostało 580 znaków

2019-08-26 18:34
0

@MrMadMatt:

  1. Choć cały projekt ze wszystkimi plikami jest w jednym archiwum zip, to dzięki za przypomnienie. Już wrzuciłem pom.xml do src.
  2. W sumie też dobry pomysł. Jak będę wprowadzał jakieś zmiany w projekcie, to użyję adnotacji. XMLi użyłem, bo widziałem je w ogromnej ilości przykładów.
  3. Masz na myśli, że wrzuciłem cały kod do jednego katalogu? Zrobiłem to, aby łatwiej i szybciej mógł to przejrzeć na przykład rekruter. Paczka zip z projektem jest zorganizowana "normalnie". Ale jak i to dla wielu osób będzie wyglądać źle, to oczywiście to zmienię.
  4. Nie jestem pewien czy to dobry pomysł, bo opinie na ten temat są podzielone: Czy używacie lomboka? Ale na pewno pomyślę o tym, jeśli będę modyfikował ten backend, bo z pewnością klasy będą wyglądały lepiej bez chociażby getterów i setterów.
    Dzięki za rady w każdym razie ;)
edytowany 1x, ostatnio: arkadiusz97, 2019-08-26 18:35
Ad 3. Paczka zip z projektem? Co masz na myśli? Jara? A co rozumiesz przez normalnie? Posłuchaj sie @MrMadMatt i zastosuj jakąś konwencję, albo po warstwach, albo po "właściwości". https://dzone.com/articles/project-package-organization - kixe52 2019-08-26 19:34
@kixe52: Czyli chodzi o to, abym nie przypisywał każdej klasy do jednej paczki, która w przypadku mojego projektu to po prostu "OFNotes"? - arkadiusz97 2019-08-27 09:25
Dokładnie tak. Musisz ogarnąć pakiety, tak jak np foldery na swoim kompie. Masz flder muzyka, w środku rap, pop, techno itp ;) W każdym z tych folderów piosenki o pasującym typie. Poczytaj link, który podesłałem. - kixe52 2019-08-27 09:32
@kixe52: W archiwum zip jest pełen projekt, który stworzyłem w Eclipse z wybranym Mavenem jako systemem budowania. Tam wszystkie pliki z rozszerzeniem *.java są w jednym folderze, są tylko rozdzielone względem innych plików. - arkadiusz97 2019-08-27 09:48

Pozostało 580 znaków

2019-08-26 19:50
1
  1. https://github.com/arkadiusz9[...]r/backend/src/Controller.java

    return ResponseEntity.status(HttpStatus.OK).body(null);

    Czy ten null jest konieczny?

  2. Dlaczego wszystko robisz w try catchach?

    User user = model.getUserByName(principal.getName());

Sprawdziłem metodę getUserByName() z https://github.com/arkadiusz9[...]master/backend/src/Model.java
Dodatkowo kolejny raz zwracasz nulla.

  1. Cała klasa Model https://github.com/arkadiusz9[...]master/backend/src/Model.java jest dla mnie zbyt przesadzona.
    Nadmiarowy kod dotyczacy transakcji, duża powtarzalność, dziwne konstrukcje metod, wszędzie try catche.

EDIT: poprawiłem swoje słowa co do transakcji, toż to hibernate.

edytowany 1x, ostatnio: kixe52, 2019-08-26 20:07

Pozostało 580 znaków

2019-08-27 09:40
0

@kixe52:

  1. Ten return jest na przykład w metodach odpowiedzialnych za utworzenie notatki i użytkownika. Czy poza kodem 200, metoda ta powinna zwracać jakiś komunikat? To miałeś na myśli?
  2. Ta linijka nie powinna być w bloku try, ponieważ nie rzuca żadnego wyjątku?
  3. Aby nie było zbyt dużo kodu związanego z transakcjami, powinno to wyglądać tak, że pomiędzy początkiem transakcji a końcem wywoływana jest jakaś metoda operująca na bazie? Co do reszty, co masz dokładnie na myśli? Co na przykład jest dziwnego w konstrukcji metod, poza powtarzalnością? Tych try catchy nie powinno być w ogóle czy powinny być jakoś inaczej umieszczone?

Pozostało 580 znaków

2019-08-27 09:57
1

@arkadiusz97:

  1. w sumie masz kilkam możliwości: możesz zwrócić puste 201, albo zwrócić 303 z linkiem do utworzonego zasobu

Spring? Ja tam wole mieć kontrole nad kodem ᕙ(ꔢ)ᕗ
Haste - mała biblioteka do testów z czasem.

Pozostało 580 znaków

Odpowiedz
Liczba odpowiedzi na stronę

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