Witam, jeżeli masz chwilę to prosiłbym abyś o sprawdzania mojego projektu i o porady co do projektu lub czego się uczyć albo to i to. Z góry dzięki :). Ogólnie to celuje w Java Developera, bo jednak te stronki to chyba nie dla mnie.
Link do strony:
- stronka na heroku: https://home-budget-site.herokuapp.com/home
Link do repozytoriów:
- strona: https://github.com/Oziaka/home-budget-site
- REST API: https://github.com/Oziaka/home-budget-api
- Rejestracja:ponad 5 lat
- Ostatnio:prawie 2 lata
- Postów:26
0
Angular 
Spring 
Java edytowany 1x, ostatnio: Oziaka
- Rejestracja:ponad 9 lat
- Ostatnio:4 miesiące
- Postów:2787
2
Fajnie, że masz rozdzielone repozytoria na frontend i backend. Z angulara się nie znam, ale popatrzyłem na backendową część i parę uwag ode mnie:
- https://github.com/Oziaka/home-budget-api/blob/master/dev/database/dropTables.sql - ciekawi mnie, czemu jest tylko jeden plik sqlowy, który dropuje bazę? :) Często stosowanym podejściem są skrypty migracyjne, np
Flyway(ihibernate.ddl-autoustawione na false) - Wersja javy definiowane osobno w
pom.xmli wsystem.properties? - https://github.com/Oziaka/home-budget-api/blob/master/src/main/java/pl/security/user_role/UserRoleResource.java - pusty controller?
- https://github.com/Oziaka/home-budget-api/blob/master/src/main/java/pl/security/user_role/UserRoleDto.java - trochę jak dla mnie to logika powinna iść w drugim kierunku -> Czyli zamiast robić tak, że rola usera ma w sobie set userów, wolałbym, żeby to każdy z userów miał set roli. Bo rola, sama w sobie, nie powinna wiedzieć którzy userzy ją mają (jest sobie rola, i tyle jej wystarczy, że istnieje)
- https://github.com/Oziaka/home-budget-api/blob/master/src/main/java/pl/user/User.java -> relacja
@ManyToManyz Listą jest bardzo nieefektywna - see https://vladmihalcea.com/14-high-performance-java-persistence-tips/ punkt 7 - https://github.com/Oziaka/home-budget-api/blob/master/src/main/java/pl/user/User.java (ale w sumie i reszta aplikacji) - unidirectional
@OneToManyjest również nieefektywna - see https://vladmihalcea.com/the-best-way-to-map-a-onetomany-association-with-jpa-and-hibernate/
To tak na pierwszy rzut oka, nie przejrzałem wszystkich klas ;) ale ogólnie skoro dopiero "celujesz" w java developera, to jak dla mnie bardzo ładnie.
- Rejestracja:ponad 5 lat
- Ostatnio:prawie 2 lata
- Postów:26
0
No poprawiłem te relacje i dałem przy mapach sety zamiast list i dodałem wartość mappedby. Chciałem jeszcze zapytać, co dalej robić żeby dostać pracę jako programista i czy ten projekt spełni wymagania rekruterów?
NeutrinoSpinZero
Moim zdaniem jest to pierwszy krok i zaczątek do dalszego szlifowania skilla. Według mojej obserwacji nikt nie ma czasu patrzeć na projekt. Poza tym to jest crud, niemal bez logiki, więc wymyśl sobie albo drugi projekt, albo ten rozbuduj tak, aby coś robił, liczył, planował, nie wiem, cokolwiek i refaktoryzuj na bieżąco
Oziaka
@NeutrinoSpinZero: Aha, chyba rozumiem zrobić jakiś "super" projekt i liczyć że akurat jak ktoś spojrzy na niego nawet przypadkiem bo wylał kawę na klawiaturę i tak jakoś sam wyskoczyła moje strona i ona ma przez ten ułamek sekundy go zainteresować zanim pójdzie po ręcznik, no to nieźle :), no to jestem w du***
Oziaka
@NeutrinoSpinZero: a jest sens robienie samego api, no bo jednak dużo czasu idzie na ten frontend
NeutrinoSpinZero
Olej frontend przynajmniej na razie. Podstawy znasz, wiesz jak to mniej więcej działa. Skup się na programowaniu w Java. Poznaj ten język i API biblioteki standardowej. Naucz się dobrze JPA i Hibernate oraz Springa. Poznaj wzorce projektowe, SOLID i jak pisać czysty kod.
Oziaka
Dzięki za pomoc :), cóż nie zostaje mi nic innego jak wziąć się do roboty. Do zobaczenia :)
- Rejestracja:około 9 lat
- Ostatnio:4 minuty
- Postów:160
1
https://github.com/Oziaka/home-budget-site/blob/master/src/app/register/register.component.ts - sprawdzenie zgodności haseł można fajnie ograć za pomoca validatora
https://github.com/Oziaka/home-budget-site/blob/master/src/app/models/user.model.ts - do takich modeli lepiej używać interfejsów niż klas
- z tego co zauważyłem w projektach dobrą praktyką jest explicite podawanie modyfikatorów dostępu nawet jeśli domyślnie są public
- dodatkowo masz dużo pustych plików css, konstruktorów i metod cyklu życia w komponentach. Jeśli to jest zostawione na poczet dalszej pracy to ok ale ogólnie sie takie rzeczy usuwa