Code review web-aplikacji

Question

Witam, poszukuję pracy jako junior developer i podsyłam do oceny moje aplikacje z CV.

Będę wdzięczny za porady i krytykę.

 

Server: https://covid-2019-map.herokuapp.com

Code: https://github.com/DarX1221/covid-map

 

Server: https://image-web-app.herokuapp.com

Code: https://github.com/DarX1221/image-web-app

konto admin: Admin1 pass123

Comments

Nie wiele przeglądałem ale to co mi się rzuciło w oczy to nazwa pakietu com.example.demo. Aplikacja image-web-app nie pozwala na upload plików lub ich nie wyświetla. Komunikat o poprawnym uploadzie się wyświetla.

---

Dzięki wielkie, już to naprawiłem.
Krótki opis problemu: aplikacja aby załadować zdjęcie najpierw je pobiera na serwer aplikacji (heroku), a nastepnie na chmure, gdy na serwerze zapisało się kilkanaście zdjęć uniemożliwiało to pobranie nowych (musze ogarnąć walidację upload'u).

---

@DaraS, potworny chaos w kodzie... zakomentowane skrypty kodu, 

zdublowane pakietyzacje, ogarnij chaos bo nie wiadomo czego się po czym spodziewać w 1 paczce masz klasy konfiguracyjne, uruchomieniowe i nagle też pojawia się klasa - serviosowo-biznesowa.
Po robocie siąde to powiem więcej ale nie jest to dobrze wykonany projekt 

---

@DaraS,  ogólnie poprawiłem wcześniejsze błedy i mam kilka konkretnych pytań

1. Czy pozostawienie pakietu com.example to duży błąd, nigdy na to nie zwracałem uwagii

2.  Na tutorialach zawsze pakietyzacja przebiega względem tego czym dana klasa się zajmuje
       pakiety: Contoller, Model, Repo, Service   

ostatnio wyczytałem że powinno się pakietyzować według modeli

        pakiety: User, Book .... i w tych pakietach powinny się znajdować klasy: Controler, Service ...

3. W mojej drugiej aplikacji image-web-app, podwójne klniknięcie, rzuca wyjątkiem, czy taka obsługa tego wyjątku jest poprawna

 

    public boolean removeImageById(Long id) {
        Authentication auth = SecurityContextHolder.getContext().getAuthentication();
        String username = auth.getName();
        // if logged username is different than username in AppImage(Long id) to delete, you can delete image only if you are admin
        if (!imageRepository.findAppImageById(id).getAppUser().getUsername().equals(username)) {
            if (!userService.checkTahtLoggedUserIsAdmin()) {
                return false;
            }
        }
        // double click "delete image", try to remove image two times
        try {
            String cloudinaryId = imageRepository.findAppImageById(id).getCloudinaryId();
            imageUploaderService.deleteImage(cloudinaryId);
            imageRepository.removeAppImageById(id);
            return true;
        } catch (InternalAuthenticationServiceException doubleClickDeleteEsception) {
            throw new InternalAuthenticationServiceException(
                    "This exception was throw by double click on delete image button");
        } catch (Exception ex) {
            ex.printStackTrace();
        }
        return false;
    }

 

Z góry dzięki za odpowiedź