Listの使い方がいまいちわかっていません。

Question

前提・実現したいこと

閲覧いただき、ありがとうございます。
Eclipse,Spring,jdbcを使ってJAVAの勉強をしている初心者です。
マスタのコード、名称をjsp側に渡したいのですが、うまくいきません。
どなたかご教示いただけないでしょうか。
jsp側ではセレクトボックス形式でマスタの名称を表示しようとしています。

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

①list.next()でコンパイルエラーが発生しているのですが、どう修正すべきかわかりません。
エラーメッセージ：メソッドnext()はList<Map<String, Object>>で未定義です。

②List(list)からList(DeparmentList)に項目を転送していますが、もっとスマートなやり方があるのではないか。(どちらのリストもcd,nameのみなので)

該当のソースコード

(Controller.java)★修正前
		List<Map<String, Object>> list = departmentInfoGet.getDepartmentList();
		List<Departrment> departmentList = new ArrayList<>();

		while (list.next()){
			Departrment department = new Departrment();
			department.setCd(((Departrment) list).getCd());
			department.setName(((Departrment)list).getName());
			departmentList.add(department);
		}


		model.addAttribute("departmentList", departmentList);

(Controller.java)★修正後
	List<Map<String, Object>> departmentList = departmentInfoGet.getDepartmentList();

(DepartmentInfoGet.java)★変更なし
package jp.practice.address;

import java.util.List;
import java.util.Map;

import org.springframework.jdbc.core.JdbcTemplate;
import org.springframework.stereotype.Component;
@Component
public class DepartmentInfoGet {
	private JdbcTemplate jdbcTemplate;
	public  DepartmentInfoGet(JdbcTemplate jdbcTemplate) {
		this.jdbcTemplate = jdbcTemplate;
	}

	public List<Map<String, Object>> getDepartmentList() {
		List<Map<String, Object>> list = jdbcTemplate.queryForList("select * from unit02.department order by cd");
	return list;
	}

(update.jsp)★修正前
 			<div>
				<span class="itemName">所属:</span>
	 			<form:select path="department">
    			<form:options items="${departmentList}" itemLabel="name" itemValue="cd"/>
    			</form:select>
			</div>

(update.jsp)★修正後
			<div>
				<span class="itemName">所属:</span>
	 			<form:select path="department">
					<c:forEach var="department" items="${departmentList}">
						<form:option value="${department.cd}" label="${department.name}" />
					</c:forEach>
    			</form:select>
			</div>

試したこと

model.addAttribute("departmentList", departmentList);を
model.addAttribute("departmentList", list);としてjspに渡したところ、
jsp側の表示がうまく表示されなかった（※）ので、Servlet側を修正しようとしています。
※うまくいかなかった時の事象
リンク内容

Comments

[m.ts10806]

エラーはそのままご提示ください。
https://teratail.com/help/question-tips#questionTips3-4-2
また「うまくいかない」は何も伝わらない表現なので起きたことを具体的に記載してください。
https://teratail.com/help/question-tips#questionTips3-4-1

[m.ts10806]

あと「Spring」をタグに追加しておいてください。
前のコメントで追記忘れてましたが、SpringではjspよりもTymeleafですね。

[usako]

mts10806様、ご指摘ありがとうございます。
あと、「SpringではjspよりもTymeleaf」とのこと。ご丁寧にありがとうございます。
聞いたこともなかったので、調べてみたいと思います。

Answer 1

Listから順次要素を取り出したいという場合、やり方はいくつかあります。

• カウンタを使ってforループ

for(int i = 0; i < list.size(); i++) {
    Map<String, Object> map = list.get(i);
}

• 拡張for文

for(Map<String, Object> map : list) {

}

• Iteratorを使う

Iterator<Map<String, Object>> iterator = list.iterator();
while(iterator.hasNext()){
    Map<String, Object> map = iterator.next();
}

簡素さから言えば拡張for文を使う場合が多いと思われます。

Comments

[usako]

swordone様、ご回答ありがとうございます。
拡張for文をためしてみようと思います！

Answer 2

model.addAttribute("departmentList", list);としてjspに渡したところ、

jsp側の表示がうまくいかなかったので、Servlet側を修正しようとしています。

むしろ、詰め替えるとループを何度もすることになり無駄だったりするので、departmentInfoGet.getDepartmentList()の結果をそのままjspに渡すので良いと思います。
もし必要な情報が限定されているのでしたらSQLでselect * from ~~~としている*のところを必要なカラムだけにするというのもありと思います。
例：select cd,name from ~~~
※その場合は「getDepartmentList()」というメソッド名を変更したほうが良さそうです。

あとListはwhileではなく通常はfor、またはforEachかと思います。

• java.util.Listのループの書き方

whileによる.next()はあくまでStatementからResultSetを得たときの手法です。
※そもそもエラーの通りnext()メソッドはListにはありません

Comments

[usako]

mts10806様、コメントありがとうございます！
>あとListはwhileではなく通常はfor、またはforEachかと思います。
勉強になりました。教えていただいたサイトとともに忘れないようにしたいと思います。
>departmentInfoGet.getDepartmentList()の結果をそのままjspに渡すので良いと思います。
はい・・・それを試したところ、jsp側の表示がうまくいかなくて。
１つ前のQ&Aになるのですが、「試したこと」の「リンク内容」にあるように、
jsp側のセレクトボックスに{cd="1",name="ああああ"}と表示されてしまいます。
色々試してServlet側を修正すればよいのかと思っていたのですが、何かよいアドバイスをいただけないでしょうか。

[m.ts10806]

「リンク内容」は当ページをさしていますよ。
jsp側でfor利用すれば良いかと思いますが、jsp側ではどのように書いたのでしょうか。
jspもコードもご提示ください。

[usako]

mts10806様、「リンク内容」が間違っていました。申訳ありません。
jspのソースも関連している個所を追加しました。(update.jsp)
jsp側ではセレクトボックスにマスタの名称のみ表示し、コードは別マスタ（従業員マスタ）更新に使用したいと考えています。

[m.ts10806]

「やりたいことができるようになりました！」とあったのでできたものと思ったのですが・・。
ひとまずforで回してoptionを作っては如何でしょうか。

[usako]

mts10806様、たびたび返信ありがとうございます。
jsp側がうまくいったので、あとはServlet側の問題だと思い、クローズしてしまいました・・・
＞ひとまずforで回してoptionを作っては如何でしょうか。
はい！いつもコメントくださりありがとうございます。

[m.ts10806]

なるほど。一連の流れとして対応したほうがいいかもしれませんね。
スポットスポットでは中々前に進みませんので。
ひとまず、jsp側で渡したデータをforで回してみて結果を教えてください。

[usako]

mts10806様、勘違いしているかもしれないので確認させてください。
for文で回すというのはServlet側を、という風に考えていたのですが、jsp側をfor文で回すということでしょうか？

[m.ts10806]

>jsp側をfor文で回すということでしょうか？
はい。その通りです。
例：for
http://jspnyumon.blog89.fc2.com/blog-entry-6.html
forEach（タグ）
http://tag.main.jp/cforeach.html

まずは「実現可能な手っ取り早い方法」ですね。

[m.ts10806]

回答にも書きましたが、今回は既にListで取れている情報を別のListに入れなおすというループの二度手間が発生しているので、getDepartmentList() の結果をそのままviewに渡したほうが良いです。

[usako]

mts10806様、返信ありがとうございます。
>まずは「実現可能な手っ取り早い方法」ですね。
はい！ご教授いただいたサイトを参考にjsp側に手をいれてみます！

[usako]

mts10806様、時間がかかってしまいましたが、うまく行きました！（今度こそ）
後で、修正したソースコードをアップしますが、取り急ぎお礼まで！
最後まで付き合っていただき、ただただ感謝です。

[m.ts10806]

解決につながったようで何よりです。
Tymeleafの件も、特に指定がない限りはjspよりもTymeleafが採用されるケースが多いはずなので、
Tymeleafも勉強しておいたほうが良いと思います。

[usako]

返信ありがとうございます。
まだちょっとネットで調べただけなのですが、Spring bootはTymeleaf推奨とあったのですが、普通のSpring（たぶんFrameworkのこと？）でもTymeleafがよいのでしょうか？

[m.ts10806]

現場ではWebはBootを採用するケースが多そうです（Webに特化してるから）
ただ、jspはjsp固有の技術を多く覚える必要があったり、生HTMLからの変換が結構面倒という印象があり、
Tymeleafが採用されるケースも多いのではないかと思います。
（稀に共存させているケースもあるようですが）

[usako]

なるほど。ではSpring bootも覚える必要があるということですね（まだまだ遠い道のりですが）。
色々教えていただき、本当にありがとうございます。

Answer 3

Springのバージョンに依存しますが、commons-beanutilsが大抵は含まれていますので

import org.apache.commons.beanutils.BeanUtils;
import java.lang.reflect.InvocationTargetException;

....

List<Map<String, Object>> sourcelist = getDepartmentList();
List<SampleModel> target = new ArrayList<SampleModel>();

try {
	for (Map<String, Object> source : sourcelist) {
		SampleModel model = new SampleModel();
		BeanUtils.copyProperties(model, source.get(i));
		target.add(model);
	}
} catch (IllegalAccessException|InvocationTargetException  e) {
	// 例外ハンドリング
}

ないしは、

List<Map<String, Object>> source = getDepartmentList();
List<SampleModel> target = new ArrayList<SampleModel>();

source.stream().forEach(s -> {
	SampleModel model = new SampleModel();
	try {
		BeanUtils.copyProperties(model, s);
	} catch (IllegalAccessException | InvocationTargetException e) {
		// 例外ハンドリング
	}
	target.add(model);
});

で簡単にマッピングできます。

http://commons.apache.org/proper/commons-beanutils/

Comments

[usako]

A-pZ様、ご回答ありがとうございます。
両方とも試してみたのですが、エラーが出てしまいそれを解決することができませんでした・・・（私のf力不足で）
今回は重複した処理を行っている(同じようなListから別のListへの転記)というご指摘を他の方からいただき、結局jsp側に手をいれることにしました。（質問の仕方も悪かったと思います）
でも、具体的なロジックまでご提示いただき、感謝です。もう少し力がついたらこのロジックを参考にしてプログラムで再度チャレンジしようと思います。

Answer 4

Listにnextメソッドってあったっけ？
hasNextじゃないの？？

Comments

[swordone]

ListはnextもhasNextもありません。
Listから生成されるIteratorにはありますが。

[usako]

swordone様、ご指摘ありがとうございます。
やはり考え方がそもそも間違っているということでしょうか・・・

[swordone]

考え方は間違っていません。
コンピュータへの指示の出し方が間違っているのです。

[usako]

なるほど。
もっとがんばります！

[NEO_PLANETT-777]

swordone様
hasNextってIteratorにあるんですね。
勉強になりました。