前提・実現したいこと

functionにある配列（hairetu）をpostしてサーブレットに送りたい

発生している問題・エラーメッセージ

NullPointerException

該当のソースコード

java
1<%@ page language="java" contentType="text/html; charset=UTF-8"
2    pageEncoding="UTF-8"%>
3<!DOCTYPE html>
4<html>
5<head>
6<meta charset="UTF-8">
7<title>午前回答入力</title>
8<script type="text/javascript">
9	function hairetu(var select){
10		var hairetu;
11		hairetu.push(select);
12	}
13</script>
14</head>
15<body>
16	<h1>午前回答</h1>
17    <FORM method="POST" action="AmPrint" name = "amanswer">
18       <% for(int i = 0; i < 80; i++) { %>
19       <%= "問" + (i+1) %>
20       <select name="gozen">
21			<option onclick = hairetu("ア"); value="ア">ア</option>
22			<option onclick = hairetu("イ"); value="イ">イ</option>
23			<option onclick = hairetu("ウ"); value="ウ">ウ</option>
24			<option onclick = hairetu("エ"); value="エ">エ</option>
25			</select>
26			<br>
27      <% } %>
28       <INPUT type="submit" value="次へ">
29    </FORM>
30</body>
31</html>

java
1package pack;
2
3import java.io.IOException;
4import java.io.PrintWriter;
5
6import javax.servlet.ServletException;
7import javax.servlet.annotation.WebServlet;
8import javax.servlet.http.HttpServlet;
9import javax.servlet.http.HttpServletRequest;
10import javax.servlet.http.HttpServletResponse;
11
12/**
13 * Servlet implementation class AmPrint
14 */
15@WebServlet("/AmPrint")
16public class AmPrint extends HttpServlet {
17	private static final long serialVersionUID = 1L;
18	private PrintWriter out;
19
20    /**
21     * @see HttpServlet#HttpServlet()
22     */
23    public AmPrint() {
24        super();
25        // TODO Auto-generated constructor stub
26    }
27
28	/**
29	 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
30	 */
31	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
32		// TODO Auto-generated method stub
33		response.getWriter().append("Served at: ").append(request.getContextPath());
34	}
35
36	/**
37	 * @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
38	 */
39	protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
40		// TODO Auto-generated method stub
41		doGet(request, response);
42
43		//response.setContentType("text/html; charset=UTF-8");
44		//out.println("<html><head></head><body>");
45		try {
46			String am[]	= request.getParameterValues("hairetu");
47			for(int i = 0; i < 80; i++) {
48
49
50				System.out.println(am[i]);
51
52			}
53
54			//out.println("</body></html>");
55
56		}catch(Exception e) {
57			e.printStackTrace();
58		}
59
60	}
61}
62

試したこと

hairetu()hairetu;

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

Eclipse(Photon)

m.ts10806

2019/07/18 08:22

起きている問題、質問は何でしょうか

takoyaking

2019/07/19 00:30

発生している問題はNullPointerExceptionです。サーブレットに変数を渡せていないと思うのでコード上にあるhairetu変数をサーブレットに送るやり方を教えてもらいたいです。

m.ts10806

2019/07/19 00:31

質問本文に追記してください。（エラーはそのままご提示ください）

行動規範の内容に同意します

回答2件

ベストアンサー

幾つか問題があります。

１つめ

下記ではhairetu()関数内で変数hairetuが定義されるため、
毎回空の変数にpushしていることになります。

js
1<script type="text/javascript">
2    function hairetu(var select){
3        var hairetu;
4        hairetu.push(select);
5    }
6</script>

２つめ

formで送信するのはあくまで<form>内にある入力コントロール（input,textarea,select,button）なので幾らJavaScriptの変数に詰め込んでも送信はしません。

故に、request.getParameterValues("hairetu")としても送られてくるデータがないのでNullです。

３つめ

forで80個select を作っているようですが、nameが同じであるため、一番最後の要素の値しか送られません（ラジオボタンを想像してみると良いです。同じnameで選択して、1個しか送れませんよね）

ではどうするか

既に出ている方法でも対応は可能です。
別の方法もありますが、いずれにしてもJavaScriptで作った配列を送信ということはしません
（hidden設けてやる方法もあるけど回りくどいし、受け取った側でも加工が必要）

簡単なのはname自体を配列にする方法です。

html
1       <% for(int i = 1; i <= 80; i++) { %>
2       <%= "問" + i %>
3       <select name="gozen[<%=i%>]">
4            <option value="ア">ア</option>
5            <option value="イ">イ</option>
6            <option value="ウ">ウ</option>
7            <option value="エ">エ</option>
8            </select>
9            <br>
10      <% } %>
11

チェックボックスで見たことがあると思います。

あとはチェックボックスの値を取り出すときと同じようにすればいけるはずです。
整合性とらせるためにキーにiを入れたので、キー＝問番号と合うはず。

投稿2019/07/19 00:44

編集2019/07/19 01:35

m.ts10806

総合スコア80850

takoyaking

2019/07/19 01:32

分かりやすく指摘、解説ありがとうございます。参考にしつつ実行してみたいと思います。

m.ts10806

2019/07/19 01:34

iそのまま入れたら0なので＋1しないといけませんね。でも＋1は煩わしいので下記の方が良いです <% for(int i = 1; i <= 80; i++) { %>

m.ts10806

2019/07/19 01:35

上記で最後のコード修正しました。

行動規範の内容に同意します

NullPointerExceptionが発生している場所も書いていただけると理解しやすいです．

submitでPOSTされる値はFormタグの中のinputやセレクトです.
selectを全てformに異なるnameとして含め，それを取得してはいかがでしょうか．
※ 手元に確認環境がないので動作確認はできていません．

HTML
1<% for(int i = 0; i < 80; i++) { %>
2       <%= "問" + (i+1) %>
3       <select name="gozen<%= (i+1) %>">
4            <option value="ア">ア</option>
5            <option value="イ">イ</option>
6            <option value="ウ">ウ</option>
7            <option value="エ">エ</option>
8       </select>
9<% } %>

Java
1try {
2        for(int i = 0; i < 80; i++) {
3                System.out.println(request.getParameter("gozen"+i));
4            }
5        }
6}

投稿2019/07/18 12:59

rcftdbeu

総合スコア167

takoyaking

2019/07/19 01:06

実行した結果できました！ありがとうございました。 javascriptは使わないは使わない方がよいのでしょうか？

行動規範の内容に同意します

あなたの回答

tips

プレビュー

行動規範の内容に同意します

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

functionの配列変数をpostしたい

前提・実現したいこと

発生している問題・エラーメッセージ

該当のソースコード

試したこと

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

１つめ

２つめ

３つめ

ではどうするか

関連した質問