Комментарии пользователя Орхан Гасанлы

Как вынести отдельное количества продукта с листа java?

Орхан Гасанлы @azerphoenix Куратор тега Java

Добрый день!
Удалось добраться до вашего проекта и глянуть на него.

После принятия, общая сумма должна сняться с баланса покупателя и в history отметить, что он взял такой-то товар таким-то количеством. Но servlet не считывает, что находиться в input и выдаёт значение null

Это происходит потому, что вы ничего не отправляете на сервер при POST запросе на /payment
Вот, скрин:
https://i.imgur.com/Q6FT81z.png
Вот, например, на этом скрине атрибут name отсутствует. Потому на сервер ничего и не отправляется
https://i.imgur.com/5AJ0Zq5.png
Добавил атрибуты name в input'ы (paymentForm.jsp)

paymentForm

<%@ page contentType="text/html;charset=UTF-8" language="java" %>

<form action="payment" method="post">
    <div class="container">
        <div class="row">
            <div class="col-md-4 order-md-2 mb-4">

                <h4 class="d-flex justify-content-between align-items-center mb-3">
                    <span class="text-muted">Ваша корзина</span>
                </h4>

                <ul class="list-group mb-3">
                    <c:forEach var="product" items="${cartList}" varStatus="productId">
                        <li class="list-group-item d-flex justify-content-between lh-condensed ">
                            <div>
                                <h6 class="my-0 w-75">${product.title} ${product.model}</h6>
                                <small class="text-muted">${product.description}</small>
                            </div>
                            <div class="d-flex flex-column w-100 justify-content-center">
                                <span class="text-muted"
                                      style="margin-left: auto">${product.price}€ X ${quantity}</span>
                                <small class="text-muted float-right"
                                       style="font-size: 12px; margin-left: auto">${product.price * quantity}€</small>
                            </div>
                        </li>
                    </c:forEach>

                    <li class="list-group-item d-flex justify-content-between bg-light">
                        <div class="text-success">
                            <h6 class="my-0">Промо-код</h6>
                            <small>kapatichloh</small>
                        </div>
                        <span class="text-success">-5€</span>
                    </li>

                    <li class="list-group-item d-flex justify-content-between">
                        <span>Total (USD)</span>
                        <span id="totalPrice" name="totalPrice" class="font-weight-bold">${totalPrice}</span>
                    </li>
                </ul>
            </div>

            <div class="col-md-8 order-md-1">
                <h4 class="mb-3">Платежный адрес</h4>
                <form class="needs-validation" novalidate="">
                    <div class="row">
                        <div class="col-md-6 mb-3">
                            <label for="name">Имя</label>
                            <input name="name" type="text" class="form-control" id="name" placeholder="" value="${user.buyer.name}"
                                   required>
                        </div>

                        <div class="col-md-6 mb-3">
                            <label for="lastname">Фамилия</label>
                            <input name="lastname" type="text" class="form-control" id="lastname"
                                   value="${user.buyer.lastname}" required>
                        </div>
                    </div>

                    <div class="mb-3">
                        <label for="email">Email<span class="text-muted"></span></label>
                        <input name="email" type="email" class="form-control" id="email"
                               value="${user.buyer.email}" required>
                    </div>

                    <div class="row">
                        <div class="col-md-5 mb-3">
                            <label for="address">Адрес</label>
                            <input name="address" type="text" class="form-control" id="address"
                                   value="${user.buyer.address}" required>
                        </div>

                        <div class="col-md-3 mb-3">
                            <label for="zip">Zip-код</label>
                            <input name="zip" type="text" class="form-control" id="zip" required>
                        </div>
                    </div>

                    <div class="mb-3">
                        <label for="address2">Адрес 2<span class="text-muted"> (необязательно)</span></label>
                        <input name="address2" type="text" class="form-control" id="address2">
                    </div>

                    <hr class="mb-4">

                    <h4 class="mb-3">Оплата</h4>

                    <div class="row">
                        <div class="col-md-6 mb-3">
                            <label for="cc-name">Имя владельца карты</label>
                            <input name="cc-name" type="text" class="form-control" id="cc-name" placeholder="" required="">
                            <small class="text-muted">Полное имя написано на карте.</small>
                        </div>

                        <div class="col-md-6 mb-3">
                            <label for="cc-number">Номер карты</label>
                            <input name="cc-number" type="text" class="form-control" id="cc-number" placeholder="" required="">
                        </div>
                    </div>
                    <div class="row">
                        <div class="col-md-3 mb-3">
                            <label for="cc-expiration">Срок действия</label>
                            <input name="cc-expiration" type="text" class="form-control" id="cc-expiration" placeholder="" required="">
                        </div>

                        <div class="col-md-3 mb-3">
                            <label for="cc-expiration">CVV</label>
                            <input name="cc-cvv" type="text" class="form-control" id="cc-cvv" placeholder="" required="">
                        </div>
                    </div>

                    <hr class="mb-4">

                    <input class="btn btn-primary btn-lg btn-block" value="Продолжить" type="submit">
                </form>
            </div>
        </div>
    </div>
</form>

И теперь, данные отправляются на сервер:
https://i.imgur.com/Rjak1t7.png
Только вот, у вас нет сервлета, чтобы обработать запросы поступающие в /payment
Напишите сервлет, который примет данные из указанной формы, обработает их и вернет соответствующий результат

Ниже несколько советов по улучшению вашего проекта:
1) https://i.imgur.com/mRZmW7V.png
Файлов конфигурации IDE (.idea и т.д.) не должно быть в гит
Build директорий тоже не должно быть в гит (например, target)
Соответственно, лишние файлы нужно убрать из гита. А также используйте файл .gitignore
2) Комментарии к коммитам должны быть осмысленными. Неважно является ли проект тестовым или нет, старайтесь всегда указывать корректные комментарии.
3) https://i.imgur.com/TNbSCB4.png
Желательно, чтобы внутри директории java был пакет с указанием доменного имени и самого проекта. Например,
com.example.website
com.example.website.entity
com.example.website.servlets
4) Является ли проект рабочим? У вас в pom указана зависимость без версии, но при этом parent pom spring'a откуда он может взять актуальную версию отсутствует.

<dependency>
            <groupId>org.springframework.security</groupId>
            <artifactId>spring-security-taglibs</artifactId>
        </dependency>

Добавил туда:
<version>5.4.5</version>
5) Подобные конфиги лучше вынести в yml или properties

String uploadFolder = "D:\\UploadJPTVR19WebShop/avatars";

Иначе приложение будет не devOPS friendly.
6) У вас в файле footer.jspf отсутствуют закрывающие теги

</body>
</html>

Как должно быть: https://i.imgur.com/1hc4znt.png

7) Я добавил зависимость:

<dependency>
            <groupId>javax.servlet</groupId>
            <artifactId>javax.servlet-api</artifactId>
            <version>3.0.1</version>
            <scope>provided</scope>
        </dependency>

Иначе не работал сайт

8) Нельзя вкладывать тег form в другой тег form

Войдите на сайт