0

Столкнулся с такой проблемой: передаю в JSP ArrayList из сервлета с помощью request.setAttribute(), но при попытке получить элемент этого списка в JSP получаю NullPointerException.

Вот сервлет

public class TestServlet extends HttpServlet {

File f = new File("Report_not_R5_updated.xlsx");
ArrayList<String> list = new ArrayList<String>();

protected void doPost (HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {

    XSSFWorkbook workbook = new XSSFWorkbook(new FileInputStream(f));
    XSSFSheet sheet = workbook.getSheet("Alarm");

    for(int r = 1; r<= sheet.getLastRowNum(); r++) {
        XSSFRow row = sheet.getRow(r);
        if((row.getCell(9).toString()).equals("Нет R5")) {
           list.add((row.getCell(3)).toString()); 
        }
    }
    request.setAttribute("answer", list);
    request.getRequestDispatcher("/test.jsp").forward(request, response);
    workbook.close();
}
}

Вот страничка JSP:

<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<title>Автоматический подбор R5</title
</head>
<body>
<h2>Автоматический подбор R5</h2>
<form action="TestServlet" method="POST">
<button type="submit">Получить список</button>
</form>
<%@page import="java.util.ArrayList" %>
<%
ArrayList<String> list = (ArrayList<String>)request.getAttribute("answer"); 
out.println(list.get(0));
%>
</body>
</html>

Если я просто вывожу на экран ArrayList (out.println(list)), то вижу, что он не пустой, в нем есть элементы. Подскажите, в чем ошибка?

5
  • 1
    Мне кажется,или вы сначала делаете запрос на jsp,а потом нажимаете на кнопку?Когда вы делаете запрос на jsp страницу,то у вас должен отобразиться ArrayList,но откуда ему взяться если вы не сделали запрос на сервлет?Вам нужно сделать в jsp проверку на пустоту ArrayList.Если = null , то не выводите его,а если != null то выводите.
    – skoriy
    13 ноя 2018 в 9:00
  • @Sckoriy, ааа, вы правы. Я же сначала обращаюсь к JSP, и только после этого запускаю сервлет. Так и знал, что ошибка детская. Спасибо большое! 13 ноя 2018 в 9:31
  • на самом деле лучше отделить java код от вью, так будет намного проще... и ещё правильнее будет так: List<String> list = new ArrayList<String>(); а ещё скорее сперва надо закрыть ресурс/поток и потом форвардить а не наоборот 13 ноя 2018 в 9:34
  • @MuzaffarRasulov, спасибо за советы, учту. По поводу отделения кода от view, вы имеете ввиду, что лучше не использовать скриплеты в JSP? Из-за того, что такой код сложно поддерживать, или есть еще какие-то причины? Я просто не придумал, как это реализовать без кода в JSP. 13 ноя 2018 в 10:48
  • @Eugene White, так точно. пользуйтесь jsp, этого никто не запрещает только без скриплетов вместо них воспользуйтесь с jstl тегами они намного упрощают и намного читабельным будет ваш код. 13 ноя 2018 в 12:05

1 ответ 1

2

В чем ошибка я, к сожалению, не скажу, но могу предложить Вам альтернативу

<c:forEach var="listItem" items="${answer}">

    //Работа с каждым элементом отдельно использую ${listItem} 

</c:forEach>

Вы так же можете для эксперимента попробывать вывести тип элемента list в вашей jsp, чтобы убедиться, что произошла правильная конвертация

1
  • Спасибо большое за совет! Я пока не сталкивался с JSTL, но обязательно разберусь. На первый взгляд это кажется удобнее, чем скриплеты. 13 ноя 2018 в 9:33

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.