JSPでリストからひとつひとつ要素を抽出したいが、まとめての抽出になってしまう

現在、Webアプリケーションを作っており、
Beanに格納されている要素を抽出し、
それらをリストに格納した後、
JSP上で順番に取り出していきたいと考えております。

現状のコードは下記の通りです。

lang
1LinkedList<OfferEngineerBeans>offerAllList = new LinkedList<>();
2				ArrayList<Integer> offerClientid = new ArrayList<>();
3				ArrayList<Integer> offerEngineerid = new ArrayList<>();
4				ArrayList<Integer> offerNumber = new ArrayList<>();
5				offerAllList.addAll(offerdao.selectOfferSendAllList(myuserid));
6				for(int i = 0; i < offerAllList.size(); i++){
7					int clientid = offerdao.selectOfferSendAllList(myuserid).get(i).getClientid();
8					int engineerid = offerdao.selectOfferSendAllList(myuserid).get(i).getEngineerid();
9					offerClientid.add(clientid);
10					offerEngineerid.add(engineerid);
11					boolean open_check = offerAllList.get(i).isOpen_check();
12					if(open_check == true){
13						offerNumber.add(i, 1);
14					}else{
15						offerNumber.add(i, 0);
16					}
17				}
18				request.setAttribute("offerList", offerAllList);
19				request.setAttribute("clientid", offerClientid);
20				request.setAttribute("engineerid", offerEngineerid);
21				request.setAttribute("open_check", offerNumber);
22				offerdao.close();
23				request.getRequestDispatcher("OfferSendList_all.jsp").forward(request, response);

JSPの方では、
配列として出力されるようです。
その配列から要素を一つだけとりだしていけるようにしたいです。

取り出したい要素は
clientidとengineeridの二つです。

説明が下手で申し訳ございませんが、
ご教授の程よろしくお願い致します。

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

回答1件

出ている情報が少ないので、推測でコメントします。
見当違いでしたら、すいません。

for(int i = 0; i < offerAllList.size(); i++){
int clientid = offerdao.selectOfferSendAllList(myuserid).get(i).getClientid();
int engineerid = offerdao.selectOfferSendAllList(myuserid).get(i).getEngineerid();

ここが気になります。
selectOfferSendAllList()で何をしているかは分かりませんが、(SQL発行してselect？)
for文の１行前で

offerAllList.addAll(offerdao.selectOfferSendAllList(myuserid));

しているので、offerAllListからget()するようにしたほうがいいのではないかと思います。

もし、selectOfferSendAllList()でSQLを発行しているのであれば、
何度もDBアクセスすることになり、サーバへの負荷がかかり、処理がものすごく遅くなります。
DBアクセスはなるべく少なくすることをオススメします。

offerdao が OfferDAO型と仮定してコード書いてみると、

lang
1offerAllList.addAll(offerdao.selectOfferSendAllList(myuserid));
2for(int i = 0; i < offerAllList.size(); i++){
3 OfferDAO odao = (OfferDAO)offerAllList.get(i);
4 int clientid = odao.getClientid();
5 int engineerid = odao.getEngineerid();
6･･･
7}

まとめて処理すると混乱してしまうので、
慣れるまでは分割してコーディングしてみたほうがいいと思います。
デバッグもしやすいですし。

投稿2015/07/08 00:45