• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Code Review - aplikacja mikropożyczek (zadanie rekrutacyjne)

VPS Starter Arubacloud
0 głosów
545 wizyt
pytanie zadane 11 lutego 2020 w Java przez Czarus0 Obywatel (1,040 p.)
Cześć!

Ostatnio poszukiwałem pracy i od jednej z firm otrzymałem zadanie rekrutacyjne do wykonania (niestety za późno się odezwali i musiałem im odmówić, bo przyjąłem inną ofertę ;) ).

Piszę tego posta, ponieważ prosiłbym o code-review, konstruktywną krytykę, abym wiedział co można jeszcze poprawić. Przy okazji może ktoś chciałby się sprawdzić i sam rozwiązać takie zadanie, które można napotkać w procesie rekrutacji. Zadanie jest następujące:

„Cel: Prosta aplikacja mikro pożyczkowa.

Wymagania biznesowe:

- Klient może wnioskować o pożyczkę wysyłając kwotę oraz termin

- Dokonana jest analiza wniosku pod względem ryzyka. Aplikacja jest uznana za ryzykowną jeśli:

- wniosek jest pomiędzy 0:00 a 6:00 rano i wniosek jest na kwotę maksymalną

- trzykrotne wnioskowanie z tego samego adresu IP

- pożyczka jest przyznana jeśli przechodzi przez proces ryzyka bez błędów

- w trakcie trwania umowy o pożyczkę, klient może jednorazowo wnioskować o odroczenie terminu spłaty (przedłużenie pożyczki) o 14 dni.

Wymagania techniczne:

- SOLID

- tylko backend, zero gui

- REST API

- testy jednostokowe

- test akceptacyjny dla pozytywnego przypadku”

Link do mojego kodu: https://gitlab.com/Sanecki/micro-loan

Z góry dziękuję bardzo!
komentarz 11 lutego 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)
tak z ciekawości, to jest zadanie na juniora?
komentarz 11 lutego 2020 przez Czarus0 Obywatel (1,040 p.)
wiesz co, z tego co pamiętam to na Java Developera (bodajże wymagali min 2 lata na stanowisku Java Developera lub Junior Java Developera)
komentarz 11 lutego 2020 przez Tomek Sochacki Ekspert (227,510 p.)
Pawi125 moim zdaniem na juniora mogłoby to być, nie jest to nic skomplikowanego i wymagaja samych podstaw ale wiadoko że dochodzi stres itp wiec nie przesadzalbym z czyms wiecej...
komentarz 11 lutego 2020 przez Paweł Nąckiewicz Nałogowiec (48,990 p.)
Tak jasne mogło być na juniora. Tak pytam z ciekawości co teraz wymagają na mida a co na juniora :)

1 odpowiedź

+1 głos
odpowiedź 12 lutego 2020 przez Aisekai Nałogowiec (42,190 p.)
wybrane 12 lutego 2020 przez Czarus0
 
Najlepsza

Co ja bym zrobił inaczej:

1. https://gitlab.com/Sanecki/micro-loan/-/blob/master/src/main/java/pl/csanecki/microloan/loan/dto/LoanQuery.java - zrezygnowałbym z typów prostych w DTO. Jeżeli zmienna jest polem danej klasy, to nawet niezainicjowana przyjmuje defaultową wartość (dla inta to 0). 

2. https://gitlab.com/Sanecki/micro-loan/-/blob/master/src/main/java/pl/csanecki/microloan/loan/dto/UserRequest.java - raczej bym nie tworzył UserRequest na podstawie HttpServletu. Zostawiłbym zwykłego Stringa, bo w razie konieczności nie będziesz musiał tworzyć nowego konstruktora.

3. DTO nie powinny mieć metod oraz według mnie. Zastanowiłbym się nad typami pól, w takim sensie że nie wypychałbym obiektu LocalDateTime nazewnątrz, nawet jeśli Spring radzi sobie z tym i mapuje to na Stringa.

4. https://gitlab.com/Sanecki/micro-loan/-/blob/master/src/main/java/pl/csanecki/microloan/loan/LoanController.java - zastosowałbym konwencję nazewnictwa REST'ową. Spring udostępnia bardzo fajny sposób autentykacji adnotacją @PreAuthorize, gdzie mając jedną metodę z jednym URL (korzystając z PathVariables) można łatwo wydzielić logikę do sprawdzania czy User ma prawa żeby "wykonać metodę" (prawa, np jest Adminem lub jest osobą zalogowaną). Nie sprawdzałbym czy coś jest negatywną odpowiedzią wykorzystując operator insatnceof. Raczej bym dodał jakieś Enum.  Zrezygnowałbym też z tekstowych komunikatów błędu (w takim sensie jakim to masz). Raczej bym jakieś customowe kody błędu zastosował - REST'owe kody błędu są dla programisty, frontend powinien je w miarę fajny sposób przedstawić. A dodanie internacjonalizacji do takich błędów, nie będzie zbyt łatwe :D

5. https://gitlab.com/Sanecki/micro-loan/-/blob/master/src/main/java/pl/csanecki/microloan/loan/service/LoanServiceImpl.java - Optional ma metodę orElse (czy jakoś tak) gdzie możesz stworzyć obiekt w przypadku gdy Optional.isEmpty(). Nie musisz sprawdzać ifem czy Optional.isPresent() / Optional.isEmpty(). Polecam plugin Lombok oraz adnotacje takie jak @Data, @Builder itd. Odpada Ci takie coś jak masz w metodzie registerLoan(UserRequest userRequest, LoanQuery loanQuery). Ładniej to wygląda jak Buildera zastosujesz.

Plus na pewno za tworzenie metod/zmiennych które opisują warunki w Ifach :D

komentarz 12 lutego 2020 przez Czarus0 Obywatel (1,040 p.)
Dzięki wielkie! Z przyjemnością zapoznam i zastosuję się do uwag :)

Podobne pytania

0 głosów
0 odpowiedzi 129 wizyt
pytanie zadane 17 sierpnia 2020 w Java przez DaraS Nowicjusz (120 p.)
0 głosów
4 odpowiedzi 489 wizyt
0 głosów
1 odpowiedź 463 wizyt
pytanie zadane 12 września 2019 w Java przez invokeLater Początkujący (310 p.)

92,452 zapytań

141,262 odpowiedzi

319,079 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...