プルダウンとテキストを同じtableの同じセルに送信したい

Question

前提・実現したいこと

複数のプルダウンとテキストをformでサーブレットに送信して、
それぞれtableに送信したいです。
下記の方法で実行してみたのですが、textの値しか送信されなかったのでどうか知識をお貸し頂きたいです。

試したこと

_form.jsp

<c:if test="${errors != null}">
    <div id="flush_error">
        入力内容にエラーがあります。<br />
        <c:forEach var="error" items="${errors}">
            ・<c:out value="${error}" /><br />
        </c:forEach>

    </div>
</c:if>

<form method="GET" action = "サーブレット名">
<select name="メニュー1">
<option value="選択肢1">選択肢1</option>
<option value="選択肢2">選択肢2</option>
<option value="選択肢3">選択肢3</option>
</select>
<input type="text" name="プルダウン1"　 value="${Dammy.プルダウン1}" <!--textとプルダウンの内容が両方同じ位置に送られるようにしたい-->

<select name="メニュー1">
<option value="選択肢1">選択肢1</option>
<option value="選択肢2">選択肢2</option>
<option value="選択肢3">選択肢3</option>
</select>
<input type="text" name="プルダウン2" value="${Dammy.プルダウン2}" 

<input type="hidden" name="_token" value="${_token}" />
<button type="submit">投稿</button>
</form>

DammyCreateServlet.java

 protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
        String _token = (String)request.getParameter("_token");
        if(_token != null && _token.equals(request.getSession().getId())) {
            EntityManager em = DBUtil.createEntityManager();
　　　　　　
　　　　　　 クラス名 k = new クラス名();
　　　　　　　
　　　　　　  k.setプルダウン1(request.getParameter("プルダウン1"));
            k.setプルダウン2(request.getParameter("プルダウン2"));
          　
　　　　　　　 List<String> errors = DammyValidator.validate(t);
            if(errors.size() > 0) {
                em.close();　
　
　　　　　　　RequestDispatcher rd = request.getRequestDispatcher("/WEB-INF/views/dammy/new.jsp");
                rd.forward(request, response);
            } else {
                em.getTransaction().begin();
                em.persist(t);
                em.getTransaction().commit();
                em.close();
                request.getSession().setAttribute("flush", "登録が完了しました。");

                response.sendRedirect(request.getContextPath() + "/dammy/index");
            }

DammyValidator.java

package models.validators;

import java.util.ArrayList;
import java.util.List;

import models.クラス名;

public class DammyValidator {
    public static List<String> validate(クラス名 k) {
        List<String> errors = new ArrayList<String>();

        String プルダウン1_error = _validateTitle(d.getプルダウン1());
        if(!プルダウン1_error.equals("")) {
            errors.add(プルダウン1_error);
        }

        return errors;
    }

    private static String _validateTitle(String プルダウン1) {
        if(プルダウン1 == null || プルダウン1.equals("")) {
            return "エラーメッセージ";
            }

        return "";
    }

}

new.jsp

<%@ page language="java" contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%>
<%@ taglib prefix="c" uri="http://java.sun.com/jsp/jstl/core" %>
<c:import url="/WEB-INF/views/layout/app.jsp">
    <c:param name="content">
        <h2>新規登録ページ</h2>

        <form method="POST" action="<c:url value='/dammy/create' />">
            <c:import url="_form.jsp" />
        </form>

        <p><a href="<c:url value='/dammy/index' />">一覧に戻る</a></p>
    </c:param>
</c:import>

補足情報（FW/ツールのバージョンなど）

Eclipse 4.6Neon、MySQL

Comments

[m.ts10806]

jspはコピペで提示されたのでしょうか。
一部HTMLとして正しくないようですが。

[K_3578]

すいません、内容などが公的に触れていいものでないので一部内容を切り替えているので
おかしい部分があったらご指摘頂けると幸いです。
_form.jspの方は省略して記載しています。

[m.ts10806]

↓
<input type="text" value="${値2}"

<input type="hidden" name="_token" value="${_token}" />

---------------------
それなら、ひとまず「同じような挙動を再現できるミニマムコード」を改めて組みなおした方が良いでしょうね。

[K_3578]

ご指摘ありがとうございます。
改めてサンプルコードを書き直させて頂きます。

Answer 1

特に問題はなさそうですので、確認の方法が間違っている可能性があります。
以下のように値を取得してもダメでしょうか。

String selectVal1 = request.getParameter("プルダウン1");  // name属性の値をキーとして取得

Comments

[K_3578]

回答ありがとうございます。無知で申し訳ないのですが、
selectval1はこちらで変数名を切り替えればよろしいということでしょうか。

[plasticgrammer]

変数名は自由に改変してください。

「request.getParameterの引数」と「画面側入力要素のname属性値」が一致していれば、送信された値が取得できるはずです。

[K_3578]

申し訳ありません、省略していた部分で、バリデーションを実行する機能の箇所が抜けていました。
回答頂いた通りにコードを書き直して見たところ、内容が入力されているのにエラーと判定されていますので
内容を追加致しましたのでもう少しお力添え頂きたいです。

[plasticgrammer]

「一部内容を切り替えている」とのことですので、どこまで記載内容を信じて良いのかわかりませんが、気になる点をコメントしておきます。

- 画面のtext項目２つにname属性がない
- Java側で値を取得する処理で、request.getParameter("値1")としているが、"値1"ではなくname属性の値とすべき（先頭のselectの値を取得するなら”プルダウン1”）

[K_3578]

ご指摘ありがとうございます。
ご指摘頂いた内容に関しては既に修正させて頂いていたものなのですがこちらに反映するのを失念して
おりました。分かりにくくて申し訳ありませんでした。

[plasticgrammer]

度々すみません。
textとselectが同じname属性「プルダウン1」となっていますが、誤りでしょうか。
また、DammyValidatorも修正が反映されていないようですので対応をお願いします。

[K_3578]

何度もミスしていて申し訳ないです。
修正、反映致しました。

[plasticgrammer]

状況を整理させてください。
現状は画面のtextに値を入力しているのに、request.getParameter("プルダウン1")で値が取得できずエラーとなっているという状況でしょうか。

[K_3578]

値がnullでないかを判定してエラーを出すDammyValidatorによって
判定する値の箇所を入力しているのに入力していない、nullと判定されエラーが出ている、という状況です。

[plasticgrammer]

質問の回答としては はい／いいえ だとどちらなのでしょうか？
状況を記載し直しているということは、私の理解は間違っているのでしょうか？

[K_3578]

返信が遅れて申し訳ありません。
回答としては「はい」になると思います。
－－－－－－－－－－－－－－－－－－－－－
質問などが伸びすぎてるので最初の問題は解消できたので一応この質問は閉じさせて頂いて、
改めていま分からないところを質問させて頂きたいと思います。ご協力ありがとうございました。