forEach文を用いてJSPで表示させています。しかし、データが一つしか表示されません。

Question

プログラミング初心者で色々と悩んでおります。
ご教授願います。

まず、MySQLにString型で"1,2,3"のように値が入っており、
それをcandidateAreaに格納しています。
そしてそれをsplitでカンマ毎に分けて、
それぞれに対応したDAOのSELECT文を繰り返し抽出します。
SELECTされたデータをJSPのforEach文で繰り返し処理して表示したいのですが、
最後にSELECTされたデータだけしか表示されないので困っています。
説明が下手、かつ初歩的な質問で申し訳ございません。
何卒よろしく御願い致します。

下記にJAVAのソースコードとJSPの表示ページを記載致します。
他に記載した方がいいものがございましたら、
おっしゃってください。

int myPairId = (int)ses.getAttribute("MY_PAIR_ID");
					// 候補条件を抽出
					dao.connect();
					request.setAttribute("candidateValue", dao.selectCandidate(myPairId));
					JavaBeans candidate = (JavaBeans)request.getAttribute("candidateValue");
					String candidateArea = candidate.getCan_area();
					int canAgeMax = candidate.getCan_age_max();
					int canAgeMin = candidate.getCan_age_min();
					int canTreat = candidate.getCan_treat();
					String[] str = candidateArea.split("," , 0);
					for(int i = 0; i < str.length; i++){
					if(canAgeMax == 50){
						ses.setAttribute("pair_today", dao.select50pair(myPairId, str[i], canTreat, canAgeMin));
					}
					else if(canAgeMin >= 18 && canAgeMax <= 49){
						ses.setAttribute("pair_today", dao.selectPair18to50(myPairId, canAgeMax, canAgeMin, str[i], canTreat));
						}
					}

<table border="2">
<tr><th>写真</th><th>写真</th><th>名前</th><th>歳</th><th>名前</th><th>歳</th><th>お店</th></tr>
<c:forEach var="beans" items="${ sessionScope.pair_today }">
	<tr>
		<td>${ beans.photo1 }</td>
		<td>${ beans.photo2 }</td>
		<td>${ beans.name1 }</td>
		<td>${ beans.age1 }</td>
		<td>${ beans.name2 }</td>
		<td>${ beans.age2 }</td>
		<td>${ beans.shop }</td>
	</tr>
</c:forEach>

Comments

[argius]

dao.select50pair,selectPair18to50からは何が返されますか？　配列ですか？

[Sfidante]

どちらも配列で返しています。

Answer 1

Java側のforループで、pair_todayに毎回上書きしていますね。
いったん、Listなどに全部格納した後で、ses.setAttribute("pair_today", records)すればできると思います。

例：

// import java.util.*;

List<JavaBeans> records = new ArrayList<>();
String[] str = candidateArea.split("," , 0);
for (int i = 0; i < str.length; i++) {
    if (canAgeMax == 50) {
        Collections.addAll(records, dao.select50pair(myPairId, str[i], canTreat, canAgeMin));
    }
    else if (canAgeMin >= 18 && canAgeMax <= 49) {
        Collections.addAll(records, dao.selectPair18to50(myPairId, canAgeMax, canAgeMin, str[i], canTreat));
    }
}
ses.setAttribute("pair_today", records);

Comments

[Sfidante]

返信が遅くなり申し訳ございません。
回答ありがとうございます。

ご指摘通り、"today_pair"に毎回上書きされておりました。
ですが、うまく配列に格納できず、またしても詰まってしまいました。

理由と致しましては、addAllメソッドで下記のエラーがでたことによります。
The method addAll(Collection<? super T>, T...) in the type Collections is not applicable for the arguments (List<JavaBeans>, List<JavaBeans>)

他にも調べて配列への格納を色々試してみたのですが、
うまくいきませんでした。

大変申し訳ないのですが、今一度ご教授願います。

[argius]

配列とおっしゃっていたので、例ではJavaBeans[]が返される想定で書きました。Listは配列と似ていますが、配列ではありません。
DAOの検索結果がList<JavaBeans>なら、records.addAdd(dao.select50pair(myPairId, str[i], canTreat, canAgeMin));のように書けます。

[argius]

間違えました。records.addAllです。

[Sfidante]

ありがとうございます。
目的のデータを取得できました。
大変ご迷惑をおかけしました。