プルダウンの表示について。

Question

前提・実現したいこと

SpringMVCフレームワークを使用しています。

初期画面と明細追加画面があり、初期画面の「商品」プルダウンでiteratorを使って表示したいと考えています。現在は、<form:select name="id" path="name" items="${nameList}" />のように表示させています。、SalesSystemController.javaで呼び出しをし、商品を選択して明細追加画面に行くのですがSystem.out.println(form.getId());で取得したidがnullとなってしまいます。

希望する動作としては、
1、itaretorでプルダウンを表示する。
2、取得するidがnullにならないようにする。

です。わかる方がいましたらご教授お願いします。

該当のソースコード

SalesSystemController.java

package jp.practice.sales;

@Controller
@RequestMapping(value = "/system")
public class SalesSystemController {

	// private MessageSource messageSource;
	private static final Logger logger = LoggerFactory.getLogger(SalesSystemController.class);

	private static final String INIT = "init";
	private static final String ADD = "add";
	private static final String FIX = "fix";

	private static final String ERRMSG = "1以上の数字を入力してください";

	@RequestMapping(value = "/start")
	public String init(SalesForm form, Model model) {
//		RecordManagerにItemListを格納する
		model.addAttribute("ItemList", RecordManager.getItemList());
//		RecordManagerにNameListを作成する
		model.addAttribute("nameList", RecordManager.makeNameList());
		return INIT;
	}

	@RequestMapping(params = "add", method = RequestMethod.POST)
	public String add(@Valid SalesForm form, BindingResult result, Model model) {

		logger.info("Post : " + form.getQuantity());
System.out.println(form.getId());

		model.addAttribute("allList", RecordManager.selectItem(form.getName()));
//		RecordManagerにNameListを作成する
		model.addAttribute("nameList", RecordManager.makeNameList());
		if (result.hasErrors()) {
			model.addAttribute("message", ERRMSG);
			return INIT;
		} else {
			model.addAttribute("addData", form.getQuantity());
			return ADD;
		}

	}
}

SalesForm.java

package jp.practice.sales;

public class SalesForm implements Serializable {
	private static final long serialVersionUID = 1L;

	/** 商品id */
	private String id;
	/** 商品名 */
	private String name;
	/** 単価 */
	private int price;
	/** 個数 */
	@Min(1)
	//@Pattern(regexp = "\d")
	@NotNull
	private int quantity;
	/** 小計 */
	private int subtotal;
以下、上記のコンストラクタとgetterとsetterがあります

Item.java

package jp.practice.sales;



/**
 * 売上明細1件分のデータを保持するクラス
 */
public class Item {

	/** 商品id */
	private String id;
	/** 商品名 */
	private String name;
	/** 単価 */
	private int price;

	/** 個数 */

	private int quantity;
	/** 小計 */
	private int subtotal;
	/*
	 * (非 Javadoc)
	 *
	 * @see java.lang.Object#hashCode()
	 */
	@Override
	public int hashCode() {
		final int prime = 31;
		int result = 1;
		result = prime * result + quantity;
		return result;
	}

	/*
	 * (非 Javadoc)
	 *
	 * @see java.lang.Object#equals(java.lang.Object)
	 */
	@Override
	public boolean equals(Object obj) {
		if (this == obj) {
			return true;
		}
		if (obj == null) {
			return false;
		}
		if (!(obj instanceof Item)) {
			return false;
		}
		Item other = (Item) obj;
		if (quantity != other.quantity) {
			return false;
		}
		return true;
	}

	/**
	 * @return id
	 */
	public String getId() {
		return id;
	}

	/**
	 * @param id
	 *            セットする id
	 */
	public void setId(String id) {
		this.id = id;
	}

	/**
	 * @return name
	 */
	public String getName() {
		return name;
	}

	/**
	 * @param name
	 *            セットする name
	 */
	public void setName(String name) {
		this.name = name;
	}

	/**
	 * @return price
	 */
	public int getPrice() {
		return price;
	}

	/**
	 * @param price
	 *            セットする price
	 */
	public void setPrice(int price) {
		this.price = price;
	}

	/**
	 * @return quantity
	 */
	public int getQuantity() {
		return quantity;
	}

	/**
	 * @param quantity
	 *            セットする quantity
	 */
	public void setQuantity(int quantity) {
		this.quantity = quantity;
	}

	/**
	 * @return subtotal
	 */
	public int getSubtotal() {
		return subtotal;
	}

	/**
	 * @param subtotal
	 *            セットする subtotal
	 */
	public void setSubtotal(int subtotal) {
		this.subtotal = subtotal;
	}



	/**
	 * コンストラクターです。
	 *
	 * @param id
	 *            商品id
	 * @param name
	 *            商品名
	 * @param price
	 *            単価
	 * @param quantity
	 *            個数
	 * @param subtotal
	 *            小計
	 */
	public Item(String id, String name, int price, int quantity, int subtotal) {
		this.id = id;
		this.name = name;
		this.price = price;
		this.quantity = quantity;
		this.subtotal = subtotal;
	}

}
// Copyright 2015 FUJITSU APPLICATIONS LIMITED

RecordManager.java

//商品一覧
/**
 * 商品データ全体を保持するクラス
 */
public final class RecordManager {

	/** 商品データ */
	private static List<Item> list = new ArrayList<>();
	private static List<Item> selectList = new ArrayList<>();
	private static Item item;

	static {
		list.add(new Item("A00101", "油性ボールペン", 60, 0, 0));
		list.add(new Item("A00201", "極細ボールペン", 120, 0, 0));
		list.add(new Item("A00301", "蛍光ペン６色セット", 420, 0, 0));
		list.add(new Item("A00401", "シャープペンシル", 100, 0, 0));

	}
	public static List<Item> selectItem(String name) {
		Item item = new Item("",name, 0, 0, 0);
		selectList.add(list.get(list.indexOf(item)));
		return selectList;
	}

	public static String[] makeNameList() {
		String[] idList = new String[list.size()];
		for (int i = 0; i < list.size(); i++) {
			idList[i] = (list.get(i)).getName();
		}
		return idList;
	}

	public static List<Item> getItemList() {
		return list;
	}

	public static void setItemList() {
		list.set(list.indexOf(item), item);
	}
}

init.jsp

<body>
	<form:form modelAttribute="salesForm" action="/sales/system">
		<div class="header">
			<span class="titleName">直太郎オンラインショップ</span>
			<div class="date"><%=LocalDate.now()%></div>
		</div>

		<div class="main">
			<div>
				商品:
				<form:select path="name">
					<form:option value="${ idList}" items=" ${ nameList}"/>
				</form:select>
				<br/>
			</div>
			<div>
				<span class="itemQuantity">点数:</span>
				<form:input path="quantity" size="20" />
			</div>
		</div>

		<div class="footer">
			<input type="submit" name="add" value="明細追加" />
		</div>

		<div class="message">
			<c:out value="${message}" />
		</div>
	</form:form>
</body>

add.jsp

<body>
	<form:form modelAttribute="salesForm" action="/sales/system">
		<div class="header">
			<span class="titleName">直太郎オンラインショップ</span>
			<div class="date"><%=LocalDate.now()%></div>
		</div>
		<div class="main">
			<div>
				商品:
				<form:select name="id" path="name" items="${nameList}" />
				<br />
			</div>
			<div>
				<span class="itemQuantity">点数:</span>
				<form:input path="quantity" size="20" />
				<form:errors path="quantity" />
			</div>

			<div class="add">
				<div>
					<input type="submit" name="add" value="明細追加" />
				</div>
			</div>
		</div>
	</form:form>
	<form:form modelAttribute="salesForm" action="/sales/system">
		<div class="main2">
			<div class="message">
				<c:out value="${message}" />
			</div>
			<div class="">
				<span class="details">売上明細</span>

				<table class="product">

					<tr>
						<th>削除</th>
						<th>商品ID</th>
						<th>商品名</th>
						<th>単価</th>
						<th>点数</th>
						<th>小計</th>
					</tr>
					<c:forEach items="${allList}" var="allList" varStatus="status">
						<tr>
							<th><form:radiobutton path="name" label=""
									value="radiobtn" /></th>
							<th>${allList.id}</th>
							<th>${allList.name}</th>
							<th>${allList.price}</th>
							<th>${allList.quantity}</th>
							<th>${allList.subtotal}</th>
						</tr>
					</c:forEach>
				</table>
			</div>

			<div class="total">
				合計：
				<fmt:formatNumber value="${total}" groupingUsed="true" />
				円
			</div>
		</div>


		<div class="footer">
			<script type="text/javascript">
				function remove_row(target) {
					var TR = target.parentNode.parentNode;
					TR.parentNode.deleteRow(TR.rowIndex - 1);
				}
			</script>
			<input type="submit" name="remove" value="削除"
				onclick="remove_row(this)" /> <input type="submit" name="firm"
				value="確定" />
		</div>

	</form:form>

</body>
</html>

試したこと

<select><option/><select>でもプルダウン表示をしてみましたが値の変更に苦労すると思いやめました。

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

Eclipse Java EE IDE for Web Developers.
Version: Mars.2 Release (4.5.2)
ローカルホストTomcat8

Answer 1

1、itaretorでプルダウンを表示する。

init.jspの商品プルダウンは

<form:select path="name">
  <form:option value="${idList}" items="${nameList}"/>
</form:select>

のようになっていますが、<form:option>にitemsはありません。
<c:forEach>と<form:option>で実装するか、
コントローラーの戻り値を変更して<form:options>で実装すると良いと思います。

2、取得するidがnullにならないようにする。

コントローラーのadd()メソッドに@ModelAttributeが足りない気がします。

public String add(@Valid @ModelAttribute("salesForm") SalesForm form, BindingResult result, Model model) {

---

なかなか見ごたえのあるコードなので、他にも問題があるかもしれません...