java サーブレット内でListに追加したタイトルのみ検索をかけたい

お世話になっております。
java初心者です。

サーブレット内でListをいくつかaddしました。containsを用いてブラウザ上で検索した文字を含むタイトル名達だけブラウザ上に出したいです

つたない分ですいません

JSP

<%@page import="java.util.List"%>
<%@page import="net.diamondf.ewb.stydy2.model.Comic"%>
<%@ page language="java" contentType="text/html; charset=UTF-8" pageEncoding="UTF-8"%>
<% List<Comic> comicList = (List<Comic>)request.getAttribute("comicList"); %>

<!DOCTYPE html> <html> <head> <meta charset="UTF-8"> <title></title> </head> <body>

<form action="/web-study2/comic" method="get">
	<input type="text" name="Title">
	<input type="submit" value="検索">
</form>
//<input type="text" name="ttt">


<table border="1">
	<tr>
		<th>タイトル</th>
		<th>作者</th>
		<th>価格</th>
		<th>在庫</th>
	</tr>

	<% for(Comic comic:comicList){ %>
		<tr>
			<td><a href="/web-study2/comic/detail"><%= comic.getTitle() %></a> </td>
			<td><%= comic.getAuthor() %></td>
			<td><%= comic.getPrice() %></td>
			<td><%= comic.getStock() %></td>
		</tr>
	<%} %>









</table>

</body> </html>

サーブレット

package net.diamondf.ewb.stydy2.servlet;

import java.io.IOException;
import java.util.ArrayList;
import java.util.List;

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 net.diamondf.ewb.stydy2.model.Comic;

/**

Servlet implementation class ComicServlet

*/
@WebServlet("/comic")
public class ComicServlet extends HttpServlet {
private static final long serialVersionUID = 1L;

/**
 * @see HttpServlet#HttpServlet()
 */
public ComicServlet() {
	super();
	// TODO Auto-generated constructor stub
}

/**
 * @see HttpServlet#doGet(HttpServletRequest request, HttpServletResponse response)
 */
protected void doGet(HttpServletRequest request, HttpServletResponse response)
		throws ServletException, IOException {
	// TODO Auto-generated method stub

	//response.getWriter().append("Served at: ").append(request.getContextPath());
	response.setContentType("text/html; charset=UTF-8");//文字化け対策

	Comic co1 = new Comic("java1", "山田", 300, 100);
	//↑このようなnewをのち9個作成しました。

	String title = request.getParameter("title");
	Integer length = 0;

	if (title != null) {
		length = title.length();
		Boolean b = title.contains("abc");
		System.out.println("title : " + title + " b : " + b);
	}


	List<Comic> comicList = new ArrayList<>();
	if (length == 0) {

		comicList.add(co1);
		comicList.add(co2);
		comicList.add(co3);
		comicList.add(co4);
		comicList.add(co5);
		comicList.add(co6);
		comicList.add(co7);
		comicList.add(co8);
		comicList.add(co9);
		comicList.add(co10);
	}
	for(Comic comic:comicList) {
		comic.getTitle();
		System.out.println(comic.getTitle());

	

	comicList.size();





	request.setAttribute("comicList", comicList);

	request.getRequestDispatcher("/WEB-INF/view/comic1.jsp").forward(request, response);
}

/**
 * @see HttpServlet#doPost(HttpServletRequest request, HttpServletResponse response)
 */
protected void doPost(HttpServletRequest request, HttpServletResponse response)
		throws ServletException, IOException {
	// TODO Auto-generated method stub
	doGet(request, response);

}

}

dice142

2018/10/23 02:05

コードをご提示ください。

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

回答2件

リクエスト属性へ格納するComicのListは、titleの値でフィルタした後のListに集約するのが良いでしょう。

すべての処理を1つのメソッドに書かずに、以下のように行いたい処理単位でメソッドにわけると、見やすいコードになります。検索条件がない場合とある場合とで処理内容が異なるので、その分岐も別メソッドを作ってわけてしまい、サーブレットの入り口である doGetやdoPostでは、分岐処理を書かなくするのも、わかりやすいサーブレットのコードになります。

java
1import java.io.IOException;
2import java.util.ArrayList;
3import java.util.List;
4import java.util.stream.Collectors;
5
6import javax.servlet.ServletException;
7import javax.servlet.annotation.WebServlet;
8import javax.servlet.http.HttpServlet;
9import javax.servlet.http.HttpServletRequest;
10import javax.servlet.http.HttpServletResponse;
11
12@WebServlet("/ComicServlet")
13public class ComicServlet extends HttpServlet {
14
15	protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
16		String title = request.getParameter("title");
17
18		List<Comic> filteredComics = getComics(title);
19
20		request.setAttribute("comicList", filteredComics);
21		request.getRequestDispatcher("/WEB-INF/view/comic1.jsp").forward(request, response);
22	}
23
24	/**
25	 * 指定したタイトルを含むComic一覧を返す。タイトルが指定されていない場合は全件返す。
26	 * @param title 抽出したいタイトル
27	 * @return Comic一覧
28	 */
29	private List<Comic> getComics(String title) {
30		if (title == null || "".equals(title)) {
31			return getAllComics();
32		}
33
34		return getAllComics().stream()
35				// コミック全件のタイトルからパラメータのtitleを含むものを抽出
36				.filter(comic -> comic.getTitle().contains(title))
37				// Listに集約する
38				.collect(Collectors.toList());
39	}
40
41	/**
42	 * 全部のComicを取得する。
43	 * @return Comic全部
44	 */
45	private List<Comic> getAllComics() {
46		List<Comic> comicList = new ArrayList<>();
47
48		comicList.add(new Comic("タイトルA", ...(省略)...));
49		comicList.add(new Comic("タイトルB", ...(省略)...));
50		comicList.add(new Comic("タイトルC", ...(省略)...));
51		...(省略)...
52		comicList.add(new Comic("タイトルJ", ...(省略)...));
53
54		return comicList;
55	}
56}
57

投稿2018/10/23 13:58

A-pZ

総合スコア12011

コードの適切な箇所で、次のようにすれば、Stringのcontains()で絞込みできます。

java
1String title = request.getParameter("title");
2
3Iterator<Comic> iterator = comicList.iterator();
4while(iterator.hasNext()){
5  Comic comic = iterator.next();
6
7  //指定した文字列が含まれない場合、リストから要素を削除する
8  if(!comic.getTitle().contains(title)){
9    iterator.remove();
10  }
11
12}

なお、ラムダ式を用いると、次のような書き方もできます。

java
1//指定した文字列が含まれない場合、リストから要素を削除する
2comicList.removeIf(comic -> !comic.getTitle().contains(title));

投稿2018/10/23 10:32

編集2018/10/23 10:36

退会済みユーザー

総合スコア0

あなたの回答

tips

プレビュー

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

質問の解決につながる回答をしましょう。サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。また、読む側のことを考えた、分かりやすい文章を心がけましょう。

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう！

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう！

java サーブレット内でListに追加したタイトルのみ検索をかけたい

関連した質問