Ocena projektu sklep online

Hej, może ktoś mógłby zrobić przegląd kodu i ocenić moją pracę, dać wskazówki i co mogę poprawić. Projekt działa, wszystko wygląda poprawnie, w shouldAddProductToBasketTest() jest NoSuchElementException ale test i tak działa. Nie użyłem springa, ponieważ chciałem się obejść bez niego i dopiero teraz będę się go uczyć. Przyjmę wszystkie rady i będę wdzięczny.

Tutaj mój git: https://github.com/LukaszCh233/Store_Online

    private boolean categoryExists(int id_category) {
        Collection<Category> categories = commonRepository.loadCategoriesFromDatabase();
        for (Category category : categories) {
            if (category.getId_category() == id_category) {
                return true;
            }
        }
        return false;
    }

Pierwsza myśl -> szkoda że nie używasz streamów
Druga myśl -> yyyyyy ... czy ty tutaj ładujesz całą tabelę do pamięci tylko po to żeby sprawdzić że rekord istnieje?

    public Collection<Customer> loadCustomersFromDatabase() {
        Collection<Customer> customers = new ArrayList<>();
        String sql = "SELECT * FROM Customers c JOIN Customers_password cp ON c.id_customer = cp.customer_id";
        try (Statement statement = database.getConnection().createStatement()) {
            ResultSet resultSet = statement.executeQuery(sql);
            while (resultSet.next()) {
                customers.add(mapResultSetToCustomer(resultSet));
            }
        } catch (SQLException e) {
            e.printStackTrace();
        }
        return customers;
    }

Dalej nie sprawdzam. Przerób to tak żeby baza szukałą zamiast ciebie. Słowo kluczowe w SQLu to WHERE

Nie bardzo rozumiem, czyli po prostu przerobić categoryExists tak żeby było zapytanie w sql?

Nie możesz pobierać całej tabeli przy każdym zapytaniu, bo jak będziesz miał tam milion rekordów to aplikacja umrze na pierwszym użytkowniku.
Słyszałeś o SELECT (...) FROM (...) WHERE ?

• Głupio że używasz repozytoriów w testach - zbyt są przez to rigid
• te testy są całkowicie useless: https://github.com/LukaszCh233/Store_Online/blob/main/src/test/java/customersTest/CustomersTest.java powinieneś je usunąć
• package który ma jeden plik nie ma specjalnego sensu
• Funckja displayProductsInStore() zmienia status ?
• ten test też jest bardzo zly https://github.com/LukaszCh233/Store_Online/blob/main/src/test/java/databaseTest/DatabaseTest.java

Poprawiłem trochę co mi powiedzieliście i czekam na dalsze komentarze :)

Testy nadal są bardzo słabe.