课题打卡，请老师检查有无疏漏

package mgallery.dao;


import java.util.ArrayList;
import java.util.List;
import mgallery.entity.Painting;
import mgallery.utils.PageModel;
import mgallery.utils.XmlDataSource;

public class PaintingDao {
    
    public PageModel pagination(int page, int rows) {
     List<Painting> list = XmlDataSource.getRawData();
     PageModel pageModel  = new PageModel(list,page,rows);    
     return pageModel;
    }
  
    public PageModel pagination(int category, int page, int rows) {
      List<Painting> list = XmlDataSource.getRawData();
      List<Painting> categoryList = new ArrayList();
      for(Painting p: list) {
        if(p.getCategroy() ==category ) {
          categoryList.add(p);
        }        
      }
      PageModel pageModel  = new PageModel(categoryList,page,rows);  
      return pageModel;
    }
    
    public void create(Painting painting) {
      XmlDataSource.append(painting);
    }
    
    public Painting findById(Integer id) {
      List<Painting> list = XmlDataSource.getRawData();
      Painting painting = null;
      for (Painting p:list) {
        if(p.getId() == id) {
          painting = p;
          break;
        }
      }
      return painting;
    }
    
    public void update(Painting painting) {
      XmlDataSource.update(painting);
    }
    
    public void delete(Painting painting) {
      XmlDataSource.delete(painting);
    }
    
}

package mgallery.service;

import javax.management.RuntimeErrorException;
import mgallery.dao.PaintingDao;
import mgallery.entity.Painting;
import mgallery.utils.PageModel;

public class PaintingService {
    private PaintingDao paintingDao = new PaintingDao();
    
    public PageModel pagination(int page,int rows, String...category) {
      if (rows == 0) {
        throw new RuntimeException("无效的rows 参数");
      }      
      if(category.length ==0 || category[0]==null) {
        return paintingDao.pagination(page, rows);
      }else {
        return paintingDao.pagination(Integer.parseInt(category[0]), page, rows);
      }
    }
    
    public void create(Painting painting) {
      paintingDao.create(painting);
    }
    
    public Painting findById(Integer id) {
      Painting p = paintingDao.findById(id);
      if(p == null) {
        throw new RuntimeException("该id不存在");
      }
      return p;
      
    }
    
    public void update(Painting painting, String isPreviewModified) {
       Painting npainting = painting; 
      
      if(isPreviewModified.equals("0")) {
        Painting oldPainting = paintingDao.findById(npainting.getId());
        String preview = oldPainting.getPreview();
        npainting.setPreview(preview);
      }
      
      System.out.println(npainting);
      paintingDao.update(npainting);

    }
    
    public void delete(Painting painting) {
      paintingDao.delete(painting);
    }
    

}

package mgallery.controller;

import java.io.File;
import java.io.IOException;
import java.io.UnsupportedEncodingException;
import java.util.List;
import java.util.UUID;
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 org.apache.commons.fileupload.FileItem;
import org.apache.commons.fileupload.FileItemFactory;
import org.apache.commons.fileupload.FileUploadException;
import org.apache.commons.fileupload.disk.DiskFileItemFactory;
import org.apache.commons.fileupload.servlet.ServletFileUpload;
import com.alibaba.fastjson.JSON;
import com.alibaba.fastjson.JSONObject;
import mgallery.entity.Painting;
import mgallery.service.PaintingService;
import mgallery.utils.PageModel;

/**
 * Servlet implementation class ManagementController
 */
@WebServlet("/management")
public class ManagementController extends HttpServlet {
	private static final long serialVersionUID = 1L;
	private PaintingService paintingService = new PaintingService();
       
    /**
     * @see HttpServlet#HttpServlet()
     */
    public ManagementController() {
        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
	    request.setCharacterEncoding("utf-8");
	    response.setContentType("text/html;charset=utf-8");
	    String method = request.getParameter("method");
	    if(method.equals("list")) {
	      this.list(request,response);
	    }else if(method.equals("show_create")){
	      this.showCreatePage(request,response);
	    }else if(method.equals("create")) {
	      this.create(request, response);
	    }else if(method.equals("show_update")) {
	      this.showUpdatePage(request,response);
	    }else if(method.equals("update")) {
	      this.update(request,response);
	    }else if(method.equals("delete")) {
	      this.delete(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);
	}

	private void list(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
	  String p = request.getParameter("p");
	  String r = request.getParameter("r");
	  if(p==null) {
	    p="1";
	  }
	  if(r == null) {
	    r ="6";
	  }
	  PageModel pageModel = paintingService.pagination(Integer.parseInt(p),Integer.parseInt(r));
	  request.setAttribute("pageModel", pageModel);
	  request.getRequestDispatcher("/WEB-INF/jsp/list.jsp").forward(request, response);
	  
	}
	
	private void showCreatePage(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
	    request.getRequestDispatcher("/WEB-INF/jsp/create.jsp").forward(request, response);
	}
	
	private void create(HttpServletRequest request, HttpServletResponse response) {
	  FileItemFactory factory = new DiskFileItemFactory();
	  ServletFileUpload sf = new ServletFileUpload(factory);
	  
	  try {
      List<FileItem> formData = sf.parseRequest(request);
      Painting painting = new Painting();
      for (FileItem fi:formData) {
        if(fi.isFormField() ) {
          //System.out.println(fi.getFieldName() );
          switch (fi.getFieldName()) {
            case "pname":
              painting.setPname(fi.getString("UTF-8"));
              break;
            case "category":
              painting.setCategroy(Integer.parseInt(fi.getString("UTF-8")));
              break;
            case "price":
              painting.setPrice(Integer.parseInt(fi.getString("UTF-8")));
              break;
            case "description":
              painting.setDescrption(fi.getString("UTF-8"));
              break;
            default:
              break;
          }
          
          
        }else {
          String path = request.getServletContext().getRealPath("/upload");
          //System.out.println(path);
          String fileName = UUID.randomUUID().toString();
          String suffix = fi.getName().substring(fi.getName().lastIndexOf("."));
          fi.write(new File(path,fileName + suffix));
          painting.setPreview("/upload/" + fileName + suffix);
          
        }
      }
      
      paintingService.create(painting);
      response.sendRedirect("/management?method=list");
    } catch ( Exception e) {
      // TODO Auto-generated catch block
      e.printStackTrace();
    }
	  
	}
	
	private void showUpdatePage(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
	  String id = request.getParameter("id");
	  Painting painting = paintingService.findById(Integer.parseInt(id));
	  request.setAttribute("painting", painting);
	  request.getRequestDispatcher("/WEB-INF/jsp/update.jsp").forward(request, response);
	}
	
	private void update(HttpServletRequest request, HttpServletResponse response) {
	   FileItemFactory factory = new DiskFileItemFactory();
	   ServletFileUpload sf = new ServletFileUpload(factory);
	   String isPreviewModified = null;
	   try {
      List<FileItem> formData = sf.parseRequest(request);
      Painting painting = new Painting();
      for(FileItem fi:formData) {
        switch (fi.getFieldName()) {
          case "pname":
            painting.setPname(fi.getString("UTF-8"));
            break;
          case "category":
            painting.setCategroy(Integer.parseInt(fi.getString("UTF-8")));
            break;
          case "price":
            painting.setPrice(Integer.parseInt(fi.getString("UTF-8")));
            break;
          case "description":
            painting.setDescrption(fi.getString("UTF-8"));
            break;
          case "id":
            painting.setId(Integer.parseInt(fi.getString("UTF-8")));
            break;
          case "isPreviewModified":
            isPreviewModified = fi.getString("UTF-8");
            break;
          default:
            break;
        }       
      }
      
      if(isPreviewModified.equals("1")) {
        for(FileItem fi:formData) {
          if(!fi.isFormField()) {
            String path = request.getServletContext().getRealPath("/upload");
            String fileName = UUID.randomUUID().toString();
            String suffix = fi.getName().substring(fi.getName().lastIndexOf("."));
            fi.write(new File(path,fileName + suffix));
            painting.setPreview("/upload/" + fileName + suffix);
            
          }
        }       
      }      
      paintingService.update(painting, isPreviewModified);
      response.sendRedirect("/management?method=list");
    } catch (Exception e) {
      // TODO Auto-generated catch block
      e.printStackTrace();
    }
  
	}
	
	private void delete(HttpServletRequest request, HttpServletResponse response) throws IOException {
	  
	  Integer id = Integer.parseInt(request.getParameter("id"));
	  try {
	  paintingService.delete(paintingService.findById(id));
	  String result = "{\"result\":\"ok\"}";
	  JSONObject json = JSON.parseObject(result);
      response.getWriter().println(json);
    } catch (IOException e) {
      // TODO Auto-generated catch block
      e.printStackTrace();     
      String result = "{\"result\": "+  e.getMessage()+ "}";
      JSONObject json = JSON.parseObject(result);
      response.getWriter().println(json);
    }  

	  
	}
}

<%@page contentType="text/html;charset=utf-8"%>
<%@taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c"%>
<%@taglib uri="http://java.sun.com/jsp/jstl/fmt" prefix="fmt" %>

<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<title>油画列表</title>
<link rel="stylesheet" type="text/css" href="css\list.css">
<script type="text/javascript" src="js/jquery-3.4.1.min.js"></script>
<script type="text/javascript" src="js/sweetalert2.js"></script>
<script type="text/javascript">

function showPreview(previewObj){
	   var preview = $(previewObj).attr("data-preview");
	   var pname = $(previewObj).attr("data-pname");
	   swal.fire({
		  title: pname,
	      html: "<img src='"+preview+"' style='width:361px;height:240px'>",
	      showCloseButton: true,
	      showConfirmButton: false,
	   })
}

function del(obj){
	var id = $(obj).attr("data-id");
	var pname = $(obj).attr("data-pname");
	var preview = $(obj).attr("data-preview");
	
	swal.fire({
		   title: "确认要删除" + pname + "油画么",
		   html: "<img src='"+preview+"' style='width:361px;height:240px'>",
		   showCancelButton: true,
		   confirmButtonText:"是",
		   cancelButtonText:"否"	   		
	}).then(function(result){
		if(result.value == true){
			 $.ajax({
		           "url":"/management",
		            "type":"get",
		            "data":{"id":id,"method":"delete"},
		            "datatype":"json",
		            "success":function(data){
		            	var json = JSON.parse(data);

		                if(json.result == "ok"){
		                	console.log("1");
		                	window.location.reload();
		                }else{
		                	console.log("2");
		                	swal.fire({
	                             title: json.result       
		                	})
		                	
		                }
		            }
		    })			
		}
		
	})
	
	
	

	
}

</script>
</head>
<body>
	<div class="container">
		<fieldset>
			<legend>油画列表</legend>
			<div style="height: 40px">
				<a href="/management?method=show_create" class="btn-button">新增</a>
			</div>
			<!-- 油画列表 -->
			<table cellspacing="0px">
				<thead>
					<tr style="width: 150px;">
						<th style="width: 100px">分类</th>
						<th style="width: 150px;">名称</th>
						<th style="width: 100px;">价格</th>
						<th style="width: 400px">描述</th>
						<th style="width: 100px">操作</th>
					</tr>
				</thead>
				<c:forEach items="${pageModel.pageData }" var="painting">
					<tr>
					   <c:choose>
					       <c:when test="${painting.categroy ==1 }">
					       <td>现实主义</td>
					       </c:when>
						  <c:when test="${painting.categroy ==2 }">
						  <td>抽象主义</td>
						  </c:when>
						  <c:otherwise>
						  <td>未知的油画类型</td>
						  </c:otherwise>
					   </c:choose>
						<td>${painting.pname }</td>
						<td><fmt:formatNumber pattern="$,000.00" value="${painting.price }"></fmt:formatNumber>  </td>
						<td>${painting.descrption }</td>
						<td>
							<a class="oplink" data-preview="${painting.preview }"  data-pname="${painting.pname }" href="javascript:void(0)" onclick="showPreview(this)">预览</a>
							<a class="oplink" href="/management?method=show_update&id=${painting.id }">修改</a>
							<a class="oplink" href="javascript:void(0)"  data-id="${painting.id }"  data-pname="${painting.pname }" data-preview="${painting.preview }" onclick ="del(this)">删除</a>
						</td>
					</tr>
				</c:forEach>
			</table>
			<!-- 分页组件 -->
			<ul class="page">
				<li><a href="/management?method=list">首页</a></li>
				<c:if test="${pageModel.hasPreviousPage ==true}">
				<li><a href="/management?method=list&p=${pageModel.page -1}">上页</a></li>
				</c:if>
				<c:forEach begin="1" end="${pageModel.totalPages}" var="pno" step="1">
				<li ${pno==pageModel.page?"class='active'":""} ><a href="/management?method=list&p=${pno}">${pno}</a></li>
                </c:forEach>
                <c:if test="${pageModel.hasNextPage == true}">
				<li><a href="/management?method=list&p=${pageModel.page +1 }">下页</a></li>
				</c:if>
				<li><a href="/management?method=list&p=${pageModel.totalPages }">尾页</a></li>
			</ul>
		</fieldset>
	</div>

</body>
</html>