Javaでfor文のコードのみ処理がされません

Question

###Javaのfor文の処理が反映されません
まずは、下記に該当するクラスを載せます

package search;

import java.io.IOException;
import java.util.HashMap;
import java.util.LinkedList;
import java.util.Map;

import javax.servlet.ServletException;
import javax.servlet.annotation.WebServlet;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.HttpSession;

import profile.Ngram;
import beans.ClientBeans;
import beans.EngineerBeans;
import dao.SearchClientDao;

@WebServlet(name = "SearchClientList" , urlPatterns = { "/SearchClientList" })
public class SearchClientList extends HttpServlet{
	private static final long serialVersionUID = 1L;
	Map<Integer , Integer> industryMap = new HashMap<>();
	Map<Integer , Integer> typeOfIndustryMap = new HashMap<>();
	
	@Override
	protected void doGet(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
		request.setCharacterEncoding("utf8");
		HttpSession ses = request.getSession(true);
		int engineerid = (int)ses.getAttribute("engineerid");
		SearchClientDao searchClientDao = new SearchClientDao();
		EngineerBeans bengineerBeans;
		LinkedList<ClientBeans> clientList = new LinkedList<>();
		try{
			searchClientDao.connect();
			searchClientDao.selectSearchClient(engineerid);
			String place = searchClientDao.selectSearchClient(engineerid).getDemand_place();
			Map<String, Integer> map = Ngram.createNgram(place, 2);
			int price = searchClientDao.selectSearchClient(engineerid).getDemand_price();
			int price_min = 0;
			int price_max = 0;
			if(price == 1){
				price_min = 21;
				price_max = 40;
			}else if(price == 2){
				price_min = 41;
				price_max = 60;
			}else if(price == 3){
				price_min = 61;
				price_max = 80;
			}else if(price == 4){
				price_min = 81;
				price_max = 100;
			}else if(price == 5){
				price_min = 101;
				price_max = 200;
			}
			searchClientDao.selectSearchIndustry(engineerid);
			int industry1 = searchClientDao.selectSearchIndustry(engineerid).getIndustry1();
			industryMap.put(1, industry1);
			int industry2 = searchClientDao.selectSearchIndustry(engineerid).getIndustry2();
			industryMap.put(2, industry2);
			int industry3 = searchClientDao.selectSearchIndustry(engineerid).getIndustry3();
			industryMap.put(3, industry3);
			
			searchClientDao.selectSearchTypeOfIndustry(engineerid);
			int type_of_industry1 = searchClientDao.selectSearchTypeOfIndustry(engineerid).getType_of_industry1();
			typeOfIndustryMap.put(1, type_of_industry1);
			int type_of_industry2 = searchClientDao.selectSearchTypeOfIndustry(engineerid).getType_of_industry2();
			typeOfIndustryMap.put(1, type_of_industry2);
			int type_of_industry3 = searchClientDao.selectSearchTypeOfIndustry(engineerid).getType_of_industry3();
			typeOfIndustryMap.put(1, type_of_industry3);
			
			for (Map.Entry<String, Integer> entry : map.entrySet()) {
				String place_piece = entry.getKey();
				System.out.println(place_piece);
				for(int i = 1; i < industryMap.size(); i++){
					for(int j = 1; j < typeOfIndustryMap.size(); j++){
								int industry_num = industryMap.get(i);
								int type_of_industry_num = typeOfIndustryMap.get(j);
								System.out.println(industry_num);
								System.out.println(type_of_industry_num);
								clientList.addAll(searchClientDao.searchClientList(place_piece , price_min , price_max , type_of_industry_num , industry_num));
							}
						}
					}
			
			request.setAttribute("clientList", clientList);
			searchClientDao.close();
			request.getRequestDispatcher("MatchingList_engineer.jsp").forward(request, response);
		}catch(Exception e){
			e.printStackTrace();
		}
	}
	
	@Override
	protected void doPost(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
		doGet(request, response);
	}
}

上記のクラスでデータベースのデータを取得して、
JSPでそのデータを表示させようとしています。

該当の処理がされない箇所が以下の部分です。

for (Map.Entry<String, Integer> entry : map.entrySet()) {
				String place_piece = entry.getKey();
				System.out.println(place_piece);
				for(int i = 1; i < industryMap.size(); i++){
					for(int j = 1; j < typeOfIndustryMap.size(); j++){
								int industry_num = industryMap.get(i);
								int type_of_industry_num = typeOfIndustryMap.get(j);
								System.out.println(industry_num);
								System.out.println(type_of_industry_num);
								clientList.addAll(searchClientDao.searchClientList(place_piece , price_min , price_max , type_of_industry_num , industry_num));
							}
						}
					}

ここが丸々処理されずにJSPにディスパッチしてしまいます。

処理自体は間違っている可能性大ですので、気にしないで下さい笑

原因が全くわかりませんので、
ご教授いただけるとありがたいです。

Answer 1

for文がマルっと処理されないのなら、
条件にしているmapが空っぽなのではないでしょうか？
検索条件にヒットするデータがないとか、SQLが間違っているとか。
Ngram.createNgram()で何を処理しているか分かりませんが、
そこで作成しているMapに何も入っていないのだと思います。

Comments

[Sfidante]

ご回答ありがとうございます。
mapが処理されるfor文を省いて処理させてみましたが、
内側のfor文の方も処理がありませんでした。

また、エラーが全く出力されていないのですが、
mapが空っぽということはあるのでしょうか？

[OpShig]

for文の直前でmap.size()で中身のサイズを調べてみてはいかがでしょうか。
ちなみにMapが空でもfor文はエラー出ないです。0周のfor文がエラーにならないのと一緒です。

[kaputaros]

> for(int i = 1; i < industryMap.size(); i++){
> for(int j = 1; j < typeOfIndustryMap.size(); j++){

の部分ですね。
確かにフィールドでnewしているので、map自体には値がputされていますが、
typeOfIndustryMapにputしている部分が全て key = 1 に対して値を紐付けているため、
typeOfIndustryMap.size() が 1になっていると思います。
（１つのkeyに上書きしています。）

for文の初期値 j に１がきているので、条件式 j < typeOfIndustryMap.size() の部分が 1 < 1 となり、for文内の処理が一度も実行されることなく進んでしまっているのだと思います。

industryMap と typeOfIndustryMap を取り扱っている for文を
条件部 ⇒ map.size()+1 とするか、

for (Integetr key : map.keySet()) {
System.out.println(map.get(key));
}
のようにするか、にしたら良いかと思います。

[Sfidante]

お二方共ありがとうございました。

>for文の直前でmap.size()で中身のサイズを調べてみてはいかがでしょうか。
>ちなみにMapが空でもfor文はエラー出ないです。0周のfor文がエラーにならないのと一緒です。

おっしゃる通り、map.sizeが0でありました。
それに対しての処理をおこない改善しました。

>typeOfIndustryMapにputしている部分が全て key = 1 に対して値を紐付けているため、
>typeOfIndustryMap.size() が 1になっていると思います。

凡ミスですね。汗
ありがとうございました。