Prośba o code review projektu webowego

Question

Witam,

od października zainteresowałem się aplikacjami webowymi i zacząłem od JavyEE.
Potem jakoś od końca października zacząłem pisać proste CRUD'y w springu.

Pierwsza moja 'większa' aplikacja to było napisanie API dla bankowej aplikacji.
Druga napisana aplikacja to jest też aplikacja bankowa ale napisana w springu + thymeleaf. Po prostu wyciągnąłem wnioski bo napisaniu tamtego restowego API i chciałem to napisać lepiej i nauczyć się troche front-end'u bo słabo u mnie z tym, a przy tym i tak muszę napisać projekt na zaliczenie na studia przedmiotu. Prosiłbym żebyście zrobili code review na mojej aplikacji :) W aplikacji mam zamiar napisać jeszcze testy najprawdopodobniej w spocku. Link do githuba -> Bank-Web-Application

Mam jeszcze pytanie co do mojego githuba. Przypiąłem sobie te 2 napisane aplikacje + repo z zadaniami algorytmicznymi ze spoja. Kod w tym repo jest z przed roku, początki mojego programowania i nie wiem czy usunąć to repo z przypietych czy zostawić jak jest. Co myślicie?

Jeszcze pytanie odnośnie nauki. Ogólnie to celuje w dostanie się na jakiś płatny staż na wakacje, dopiero marzec więc jeszcze czasu trochę mam. W thymeleafie raczej nie chce już nic robić, bo teraz tylko REST'owe aplikacje. Jest sens uczyć się javy + np. angulara? Będę startował na jakieś juniorskie stanowisko z javy więc myślę, że lepiej umieć dobrze jave, niż umiec średnio jave i troche angulara. Co myślicie? Czekam na konstruktywne opinie, pozdrawiam :)

Comments

Czy ja dobrze widzę, że trzymasz zahashowane i nie-zahashowane hasło obok siebie?

---

Tak, zapomniałem to zmienić bo sprawdzałem coś, nie pamietam juz co. Zfixuje to, dzieki :)

Answer 1

Największy brak to taki, że nie ma testów jednostkowych.

Zatrzymałem się także nad tym:

@Component
class TransactionFactory {

    Transaction create(SendTransactionDTO dto, BankAccount bankAccount) {

        return Transaction
                .builder()
                .title(dto.getTitle())
                .description(dto.getDescription())
                .senderAccountNumber(dto.getSenderAccountNumber())
                .receiverAccountNumber(dto.getReceiverAccountNumber())
                .date(LocalDateTime.now())
                .amount(dto.getAmount())
                .account(bankAccount)
                .build();
    }
}

Bo zwykle factory robi się tak, że metoda create jest statyczna i nie robi się z tego springowego komponentu:

public class TransactionFactory {

  public static  Transaction create(SendTransactionDTO dto, BankAccount bankAccount) {

        return Transaction
                .builder()
                .title(dto.getTitle())
                .description(dto.getDescription())
                .senderAccountNumber(dto.getSenderAccountNumber())
                .receiverAccountNumber(dto.getReceiverAccountNumber())
                .date(LocalDateTime.now())
                .amount(dto.getAmount())
                .account(bankAccount)
                .build();
    }
}

Taka uwaga na temat użycia typu do pieniędzy, bo używasz zdaje się Double:

https://www.youtube.com/watch?v=1USy_dRBmsc

Zastanów się także, czy nazwa klasy SendTransactionDTO jest ok (chodzi o słówko Send) Dlaczego nie może być po prostu TransactionDTO.

Comments

Wszystkie factory poprawie, Double zamienie na Stringa. A co do nazwy, zastanawialem sie nad nia, i wybralem SendTransactionDTO bo nie potrzebuje wszystkich pol z klasy Transaction dlatego uznalem ze Send bedzie lepsze od samego TransactionDTO, nie wiem czy wiesz co mam na mysli. Dzieki za poswiecony czas :)

---

Własnie dlatego to się nazywa DTO:

https://stackoverflow.com/questions/1051182/what-is-data-transfer-object

Słowo Send powoduje zamieszanie ponieważ jest czasownikiem. Jeszcze przyszło mi do głowy np. TransactionRequest.

---

Ok, poprawie