java_servlet 画像更新が反映されない

Question

前提・実現したいこと

結論：社員データ修正で、新規登録時の画像と異なる画像を選択すると、選択し直した画像が反映されず、最初に登録した画像が表示されてしまいます。修正時に選択した画像が反映されるようにしたいです。

ここに質問の内容を詳しく書いてください。
java servletで社員情報管理ツールを作っています。
駆け出しエンジニア（開発を初めて３か月目）になります。
そして、初のteratailへの質問になります。
何卒、よろしくお願いいたします。

発生している問題・エラーメッセージ

エラーメッセージ
上記三枚目の写真で、新規登録時の画像と異なる画像を選択し、
設定を押下してもエラーメッセージはでてこず、
再度、同ページを開いた際に、変更した画像が反映されない状況です。

//社員情報更新logic
package action;

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

import javax.servlet.ServletException;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
import javax.servlet.http.Part;

import dao.EmployeeDAO;
import dao.ImageDAO;

public class EmpUpdateLogic implements CommonLogic {
	@SuppressWarnings({ "null", "unlikely-arg-type" })
	@Override
	public String execute(HttpServletRequest request, HttpServletResponse response) {
		boolean empIDisNull = request.getParameter("empID").equals("");
		boolean empNameisNull = request.getParameter("empName").equals("");
		if(empIDisNull || empNameisNull) {
			request.setAttribute("errorMessage", "IDと名前は必須です。");
			return "error.jsp";
		}
		String empIDstr = request.getParameter("empID");
		if (!empIDstr.matches("[0-9]{1,5}")) {
			request.setAttribute("errorMessage", "社員IDは1~5桁の数字で入力してください。");
			return "error.jsp";
		}
		String age = request.getParameter("age");
		if (!(age.matches("[0-9]{1,3}") || age.equals(""))) {
			request.setAttribute("errorMessage", "年齢は1～2桁の数字で入力してください。(空欄も可)");
			return "error.jsp";
		}
		String zipcode = request.getParameter("zipcode");
		if (!(zipcode.matches("^[0-9]{3}-[0-9]{4}$") || zipcode.equals(""))) {
			request.setAttribute("errorMessage", "郵便番号はハイフンありの7桁の郵便番号で入力してください。<br>例：123-4567<br>(空欄も可)");
			return "error.jsp";
		}
		String dateEntering = request.getParameter("dateEntering");
		if (!(dateEntering.matches("^[0-9]{4}-[0-9]{2}-[0-9]{2}$") || dateEntering.equals(""))) {
			request.setAttribute("errorMessage", "入社日はハイフンで区切った年月日で入力してください。<br>例：2001-01-01<br>(空欄も可)");
			return "error.jsp";
		}
		String dateRetired = request.getParameter("dateRetired");
		if (!(dateRetired.matches("^[0-9]{4}-[0-9]{2}-[0-9]{2}$") || dateRetired.equals(""))) {
			request.setAttribute("errorMessage", "退社日はハイフンで区切った年月日で入力してください。<br>例：2001-01-01<br>(空欄も可)");
			return "error.jsp";
		}
		int pictID = Integer.parseInt(empIDstr);
		List<String> empParams = new ArrayList<String>();
		empParams.add(request.getParameter("empName"));
		empParams.add(age);
		empParams.add(request.getParameter("gender"));
		empParams.add(String.valueOf(pictID));
		empParams.add(zipcode);
		empParams.add(request.getParameter("prefecture"));
		empParams.add(request.getParameter("address"));
		empParams.add(request.getParameter("deptID"));
		empParams.add(dateEntering);
		empParams.add(dateRetired);
		empParams.add(empIDstr);
		EmployeeDAO empDao = new EmployeeDAO();
		if (empDao.updateEmp(empParams) == false) {
			request.setAttribute("errorMessage", "データベースへの登録に失敗しました。(EMP)");
			return "error.jsp";
		}
		Part pict = null;
		try {
			pict = request.getPart("picture");
		} catch (IOException e1) {
			e1.printStackTrace();
		} catch (ServletException e1) {
			e1.printStackTrace();
		}
		String pictureSTR = request.getParameter("pictureSTR");
		InputStream is = null;
		if(pict != null || !pict.equals("")) {
			try {
				is = pict.getInputStream();
			} catch (FileNotFoundException e) {
				e.printStackTrace();
			} catch (IOException e) {
				e.printStackTrace();
			} catch (Exception e) {
				e.printStackTrace();
			}
			ImageDAO imageDAO = new ImageDAO();
			if(pictureSTR == null) {
				if (imageDAO.addImage(pictID, is) == false) {
					request.setAttribute("errorMessage", "データベースへの登録に失敗しました。（画像データadd）");
					return "error.jsp";
				}
			} else if(!pictureSTR.equals("")) {
			} else {
				if (imageDAO.updateImage(pictID, is) == false) {
					request.setAttribute("errorMessage", "データベースへの登録に失敗しました。（画像データupdate）");
					return "error.jsp";
				}
			}
		}
		request.setAttribute("message", "データベースへの登録に成功しました。");
		return "success.jsp";
	}
}

//画像DAO
package dao;

import java.io.InputStream;
import java.sql.Blob;
import java.sql.Connection;
import java.sql.DriverManager;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;
import java.util.Base64;
import java.util.List;

import bean.Constants;
import bean.Picture;

public class ImageDAO {
	public List<Picture> findAllImage() {
		List<Picture> pictureAllList = new ArrayList<Picture>();
		try {
			Class.forName(Constants.DRIVER_NAME);
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		}
		try (
				Connection conn = DriverManager.getConnection(Constants.JDBC_URL, Constants.DB_USER, Constants.DB_PASS);
				PreparedStatement pstmt = conn.prepareStatement("SELECT pict_id,pict_image FROM PICTURE ORDER BY pict_id");
				ResultSet rs = pstmt.executeQuery();
			) {
			while (rs.next()) {
				int pict_id = rs.getInt("pict_id");
				Blob blob = rs.getBlob("pict_image");
				byte[] b = blob.getBytes(1, (int)blob.length());
				String imageSTR = Base64.getEncoder().encodeToString(b);
				Picture sw = new Picture(pict_id, imageSTR);
				pictureAllList.add(sw);
			}
		} catch (SQLException e) {
			e.printStackTrace();
			return null;
		}
		return pictureAllList;
	}

	public boolean updateImage(int pictID, InputStream is) {
		try {
			Class.forName(Constants.DRIVER_NAME);
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		}
		try (
				Connection conn = DriverManager.getConnection(Constants.JDBC_URL, Constants.DB_USER, Constants.DB_PASS);
				PreparedStatement pstmt = conn.prepareStatement("UPDATE PICTURE SET pict_image = ? WHERE pict_id = ?");
			) {
			pstmt.setBlob(1, is);
			pstmt.setInt(2, pictID);
			int result = pstmt.executeUpdate();
			if (result != 1) {
				return false;
			}
		} catch (SQLException e) {
			e.printStackTrace();
			return false;
		}
		return true;
	}

	public boolean addImage(int pictID, InputStream is) {
		try {
			Class.forName(Constants.DRIVER_NAME);
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		}

		try (
				Connection conn = DriverManager.getConnection(Constants.JDBC_URL, Constants.DB_USER, Constants.DB_PASS);
				PreparedStatement pstmt = conn
						.prepareStatement("INSERT INTO PICTURE (pict_id,pict_image) VALUES (?,?)");) {
			pstmt.setInt(1, pictID);
			pstmt.setBlob(2, is);
			int result = pstmt.executeUpdate();
			if (result != 1) {
				return false;
			}
		} catch (SQLException e) {
			e.printStackTrace();
			return false;
		}
		return true;
	}

	public boolean deleteImage(int pictID) {
		try {
			Class.forName(Constants.DRIVER_NAME);
		} catch (ClassNotFoundException e) {
			e.printStackTrace();
		}
		try (
				Connection conn = DriverManager.getConnection(Constants.JDBC_URL, Constants.DB_USER, Constants.DB_PASS);
				PreparedStatement pstmt = conn.prepareStatement("DELETE FROM PICTURE WHERE pict_id = ?");) {
			pstmt.setInt(1, pictID);
			int result = pstmt.executeUpdate();
			if (result != 1) {
				return false;
			}
		} catch (SQLException e) {
			e.printStackTrace();
			return false;
		}
		return true;
	}
}

補足情報（FW/ツールのバージョンなど）

tomcat8(java8)
windows 10
IDE:eclips4.7.3
DB:H2

何卒よろしくお願いいたします

Comments

[dodox86]

更新表示だけの問題ですか？　修正を指定した画像は、DBには正しく登録されているのでしょうか。
そこを確認することで、表示だけの問題か、そもそもDBに更新内容が保存されていないのか、あるいは別の問題なのか範囲を絞りやすくなります。

[A-pZ]

画像をアップロードしているform部分の実装(JSP)も記載すると回答が得られるかもしれません。

なお、画像アップロードする際にはサーブレットにアップロード用の定義が必要です。