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ć?
- Rejestracja:ponad 8 lat
- Ostatnio:7 miesięcy
- Postów:408
0
Hibernate 
Spring 
Java 
- Rejestracja:ponad 9 lat
- Ostatnio:2 dni
- Postów:373
1
- Jak używasz mavena to zrób tak aby
pom.xmlbył 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. - 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.
- Pakietów też mógłbyś używać według jakieś konwencji bez wrzucania całego śmietnika w src.
- Używaj lomboka i nie pisz samemu getterów, setterów i konstruktorów bo szkoda na to życia.
edytowany 1x, ostatnio: MrMadMatt
kixe52
Ad 4. Skąd wiesz, że pisze sam? Może klika 3 klawisze i mu IDE generuje? :)
danek
po co w ogóle pisać gettery i settery? :P
kixe52
Żeby było więcej linii kodu w całym projekcie :D
MrMadMatt
@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.
kixe52
@MrMadMatt: wiem, po prostu się czepiam słówek ;)
- Rejestracja:ponad 8 lat
- Ostatnio:7 miesięcy
- Postów:408
0
- Choć cały projekt ze wszystkimi plikami jest w jednym archiwum zip, to dzięki za przypomnienie. Już wrzuciłem pom.xml do src.
- 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.
- 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ę.
- 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
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: Czyli chodzi o to, abym nie przypisywał każdej klasy do jednej paczki, która w przypadku mojego projektu to po prostu "OFNotes"?
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: 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.
- Rejestracja:prawie 6 lat
- Ostatnio:około godziny
- Postów:1001
1
return ResponseEntity.status(HttpStatus.OK).body(null);
Czy ten null jest konieczny?
- Dlaczego wszystko robisz w try catchach?
User user = model.getUserByName(principal.getName());
Sprawdziłem metodę getUserByName() z https://github.com/arkadiusz97/OFNotes/blob/master/backend/src/Model.java
Dodatkowo kolejny raz zwracasz nulla.
- Cała klasa Model https://github.com/arkadiusz97/OFNotes/blob/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
- Rejestracja:ponad 8 lat
- Ostatnio:7 miesięcy
- Postów:408
0
- 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?
- Ta linijka nie powinna być w bloku try, ponieważ nie rzuca żadnego wyjątku?
- 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?
- Rejestracja:ponad 10 lat
- Ostatnio:6 miesięcy
- Lokalizacja:Poznań
- Postów:797
1
- w sumie masz kilkam możliwości: możesz zwrócić puste 201, albo zwrócić 303 z linkiem do utworzonego zasobu
- Rejestracja:ponad 8 lat
- Ostatnio:7 miesięcy
- Postów:408
0
@kixe52: Wprowadziłem trochę zmian do projektu, ale nie wiem za bardzo, jak usunąć nadmiarowy kod? Na przykład zrobić maksymalnie jedną metodę, w której będzie kod odpowiedzialny za transakcje? Jak zrobić kontroler, aby nie trzeba było umieszczać kody w try catch?