所以我在jsp中有这个简单的网站,包含待办事项列表,我已经做了添加功能,但现在删除功能有问题。
<ol class="list-group">
<c:forEach items="${todos}" var="todo">
<li class="list-group-item">${todo.toDoPosition} <a class="btn btn-light" href="${pageContext.request.contextPath}/delete-todo.do?uuidDelete=${todo.uuid}">Delete</a></li>
</c:forEach>
</ol>
每个toDoPosition都有唯一的UUID和删除功能应该通过比较doPosition uuid与这一个发送参数来删除位置,我已经检查过这两个uuid是否相同而且它仍然不想从列表中删除位置。
有一个支持删除功能的servlet
private TodoService todoService = new TodoService();
protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
System.out.println(request.getParameter("uuidDelete") + "Should be deleted now");
todoService.deleteTodo(new Todo(request.getParameter("uuidDelete")));
response.sendRedirect("/todo.do");
}
它重定向到在doGet中有一个方法returnList()的servlet,它还有doPost方法,它响应添加新的待办事项:
protected void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
request.getSession().setAttribute("todos", todoService.returnList() );
request.getRequestDispatcher("/WEB-INF/views/todo.jsp").forward(request, response);
}
protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
if (request.getParameter("newTodo") != null) {
todoService.addTodo(new Todo(request.getParameter("newTodo")));
}
response.sendRedirect("/todo.do");
}
还有删除服务:
protected List<Todo> toDoList = new ArrayList<>();
public List<Todo> returnList(){
return toDoList;
}
public void addTodo(Todo toDo) {
toDoList.add(toDo);
System.out.println(toDo.uuid);
}
public void deleteTodo(Todo toDo) {
if(toDoList.contains(toDo.uuid)) {
toDoList.remove(toDo);
}
}
单击“删除”后,它会发送带有UUID,网站刷新的请求,但列表中没有更改。似乎删除方法不会从列表中删除位置,因为我将其重定向到/todo.do和应删除的打印列表位置仍然在这里,我不知道为什么
在deleteTodo中,您传入一个Todo对象。然后使用List contains方法中传入的Todo uuid属性,这是不正确的。您需要将Todo对象传递给contains方法,而不仅仅是它的ID,因为列表包含方法并比较对象,而不是它们的uuid属性。如果您调试此代码,您应该看到从未调用List remove方法。您可能还需要在Todo对象中重写equals才能使其工作,使equals对uuid属性进行比较。