0

問題点・最終目的

java 初心者です。
jsp のフォームで受け取った画像ファイルを webサーバにアップロード・保存して、表示させる機能を作っています。
ネットやAPIを参考に、コードを書いていますが上手くアップロードファイルを作成することが出来ません。
現時点では、個人の開発環境なので自分のPC内のディレクトリからアップロードするファイルを選択する形ですが、本来は全く別のクライアントから画像を取得するプログラムを作成したいです。

開発環境

  • Eclipse(Version: 2019-03 (4.11.0), Tomcat9 , java11)
  • MacOS Mojave Version: 10.14.4

発生している例外

タイプ 例外報告

メッセージ /Applications/Eclipse_2019-03.app/Contents/workspace/.metadata/.plugins/org.eclipse.wst.server.core/tmp0/wtpwebapps/`upload_server/WEB-INF/upload/_1560801511375/img-landscape536x354.png`

例外

java.nio.file.`NoSuchFileException: `/Applications/Eclipse_2019-03.app/Contents/workspace/.metadata/.plugins/org.eclipse.wst.server.core/tmp0/wtpwebapps/upload_server/WEB-INF/upload/_1560801511375/img-landscape536x354.png
    java.base/sun.nio.fs.`UnixException.translateToIOException`(UnixException.java:92)
    java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)
    java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:116)
    java.base/sun.nio.fs.UnixFileSystemProvider.newByteChannel(UnixFileSystemProvider.java:215)
    java.base/java.nio.file.Files.newByteChannel(Files.java:370)
    java.base/java.nio.file.Files.createFile(Files.java:647)
    (以下略)

  

試した事・その他原因として考えた事 等

・パス・ファイル名の取得方法を変える。
(はじめ、Part.getSubmittedFileName()を使っていた。)
・保存先ディレクトリを移動させる。
(権限の問題?という事でWebContent外へ)
 

そもそもパス、シンボリックリンク、サーバの保存先、フォームから得られる情報と形式、など基本的な事を理解できていなかったので、(今も完璧には理解できていない)
APIなどを参考に、例外に書き出しながらコードを書き換えトライしてみましたが上手くいきません。
権限の問題なのかとも思い調べて見ましたが(FileSystem等)直接的な解決策になるのかわかりませんでした。

一番不明瞭なのは、

String uploadDir=getServletContext().getRealPath("/WEB-INF/upload");

で、得られたパスのディレクトリにファイルを作れるのでしょうか。
...そこに作成するというのがそもそも間違いなのでしょうか。

何か根本的に間違っている点があるのだと思います。

初心者すぎて、ややこしい質問の仕方で申し訳ありません。
相当時間を費やしていて先に進めないので、どなたかご教示いただければ幸いです。
どうぞ宜しくお願いします。

以下、コードになります。
サーブレット

package servlet;

import java.io.IOException;
import java.nio.file.FileAlreadyExistsException;
(略)

public class FileUpload extends HttpServlet {
    private static final long serialVersionUID = 1L;

    public FileUpload() {
        super();
    }

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

        String path="/WEB-INF/jsp/fileUpload.jsp";

        RequestDispatcher dsp=request.getRequestDispatcher(path);
        dsp.forward(request, response);

        response.getWriter().append("Served at: ").append(request.getContextPath());
    }


    protected void doPost(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException {
    String title=request.getParameter("title");
    Part part=request.getPart("data");
    String nameadd=(new StringBuilder(title).append("_").append(System.currentTimeMillis()).toString());//オリジナルの名前にする

    String filename = new GetFileNameLogic().getFileName(part);
    
    String uploadDir=getServletContext().getRealPath("/WEB-INF/upload");
    Path file=Paths.get(uploadDir,nameadd,filename);
    try {
        Files.createFile(file);
    } catch (UnsupportedOperationException  e) {
        e.printStackTrace();
        throw new UnsupportedOperationException  ("原子的に設定できない属性が配列に含まれます");
    } catch (FileAlreadyExistsException e) {
        e.printStackTrace();
        throw new FileAlreadyExistsException ("同名ファイルがすでに存在します");
    }

    FileUploadLogic flogic =new FileUploadLogic ();
    try {
        flogic.execute(part, file);
    } catch (IOException e) {
        e.printStackTrace();
        throw new IOException("ファイル書き込み入出力エラー");
    }

    String to="/WEB-INF/result.jsp";
    RequestDispatcher dsp=request.getRequestDispatcher(to);
    dsp.forward(request, response);

    doGet(request, response);
  }
}

 
ファイル名を得るBO

package model;

import javax.servlet.http.Part;

public class GetFileNameLogic{
    public String getFileName(Part part) {
        String name= null;

        for (String dispotion : part.getHeader("Content-Disposition").split(";")) {
            if (dispotion.trim().startsWith("filename")) {
                 name = dispotion.substring(dispotion.indexOf("=") + 1).replace("\"", "").trim();
                 name=name.substring(name.lastIndexOf("\\")+1);

                 break;
            }
        }
        return name;
    }
}

アップロードのBO

package model;
import java.io.BufferedInputStream;
import java.io.BufferedOutputStream;
import java.io.File;
import java.io.FileOutputStream;
import java.io.IOException;
import java.nio.file.Path;

import javax.servlet.http.Part;

public class FileUploadLogic {
        public void execute(Part part,Path path) throws IOException {
            File file=new File(path.toString());
            try(BufferedInputStream br=new BufferedInputStream(part.getInputStream());
                BufferedOutputStream bw=new BufferedOutputStream(new FileOutputStream(file))){
                int len=0;
                byte[] buff=new byte[(int)part.getSize()];
                while((len=br.read(buff)) != -1) {
                    bw.write(buff, 0, len);
                }
            }
        }
}



  

 

2 件の回答 2

0
String nameadd=(new StringBuilder(title).append("_").append(System.currentTimeMillis()).toString());//オリジナルの名前にする

これによって生成された名前(_1560801511375)のディレクトリーはOS上に存在しないはずです。存在しないディレクトリーにファイルを作成しようとして、NoSuchFileExceptionがスローされています。したがって、

Files.createFile(file);

の前に少なくとも以下のようにディレクトリーをつくっておかないと、正常に動作しないはずです。

Files.createDirectory(Paths.get(uploadDir,nameadd));
7
  • もし、String nameaddを無くして、uploadDirとfilenameのみでfileを作成するならば、ご教示のFiles.createDirectory(Paths.get(uploadDir,nameadd));はなくても作成されるt言う事でしょうか?@Kohei TAMURA 様
    – Se I.
    Commented 2019年6月18日 8:42
  • はい。そうですね。getServletContext().getRealPath("/WEB-INF/upload")が実在しないってことはないでしょうから。 Commented 2019年6月18日 8:52
  • @Kohei TAMURA, @yukihane 両名様がおっしゃられている事は大変参考になりよく分かりました。@Kohei TAMURA様のおっしゃる方法で試して見たのですが、(nameadd含む、含めない)やはり同じ例外が発生します。疑問なのは、@yukihane 様がおっしゃるように、毎回同じ場所にServletContext#getRealPathで取得するディレクトリが存在しないにしても、その時の一時ファイルは作成できると言う事はないのでしょうか。(間違った解釈であれば申し訳ありません。) try,cathch文により、Files.createFileでNoSuchFileExceptionが発生しているのは特定出来ているので、となればgetRealPathで取得するディレクトリが存在しないということなのでしょうか。何か理由がお分かりであれば教えて頂けますでしょうか。宜しくお願い致します。
    – Se I.
    Commented 2019年6月18日 21:11
  • 回答したとおり、存在しないディレクトリーにファイルを作成しようとすると、NoSuchFileExceptionがスローされます。/Applications/Eclipse_2019-03.app/Contents/workspace/.metadata/.plugins/org.eclipse.wst.server.core/tmp0/wtpwebapps/upload_server/WEB-INF/upload/というディレクトリは実在しませんか?ターミナル(かFinderなど)で確認してみて下さい。このディレクトリが無ければ、NoSuchFileExceptionがスローされます。※そもそも.metadata/.plugins以下にファイルを作成すること自体が間違ってはいますが、それはひとまず置いておいてと思います。 Commented 2019年6月18日 21:34
  • 申し訳ありませんでした!WebContent配下にdrug & drop した時に作られる場所と削除して作り直した場所が、ずれていました。現在はIOException: ファイル書き込み入出力エラーになっているので、ファイル自体は作成できているようです。失礼致しました。もう少しtryしてみます。
    – Se I.
    Commented 2019年6月18日 21:41
0

ServletContext#getRealPath について

一番不明瞭なのは、

String uploadDir=getServletContext().getRealPath("/WEB-INF/upload");

で、得られたパスのディレクトリにファイルを作れるのでしょうか。
...そこに作成するというのがそもそも間違いなのでしょうか。

シンプルな回答としては間違いです。
作れる場合もありますが作れない場合もあります。作れた場合でもその場所が恒久的に存在するとは限りません。


ServletContext#getRealPath については Servlet仕様書(Servlet 4.0 spec(pdf) - 3.6 Path Translation Methods)やAPI docで説明されています。

In situations where the servlet container cannot determine a valid file path for these methods, such as when the Web application is executed from an archive, on a remote file system not accessible locally, or in a database, these methods must return null.


具体例を挙げると、TomcatにはunpackWARsという設定項目がありこれをfalseに設定すると.warファイルは展開されずにデプロイされます。
この場合の ServletContext#getRealPath 戻り値は上記引用文の通り null です。

また別の例として、WildFlyでは.warファイルの自動展開先は一時ディレクトリであり、ServletContext#getRealPathの結果は同じアプリケーションでもデプロイのたびに変わる可能性があります。
アップロードしたファイルが、アプリケーションを再起動すると見つからなくなってしまう、ということです。


上の話を脇に置いておいたとしても、サーバリソースが配置されているWEB-INFディレクトリにユーザ生成コンテンツも一緒くたに保存するというのは、運用が煩雑になるというデメリット面が大きいかと思います。


発生している例外について

例外

java.nio.file.`NoSuchFileException: `/Applications/Eclipse_2019-03.app/Contents/workspace/.metadata/.plugins/org.eclipse.wst.server.core/tmp0/wtpwebapps/upload_server/WEB-INF/upload/_1560801511375/img-landscape536x354.png
  java.base/sun.nio.fs.`UnixException.translateToIOException`(UnixException.java:92)
  java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:111)
  java.base/sun.nio.fs.UnixException.rethrowAsIOException(UnixException.java:116)
  java.base/sun.nio.fs.UnixFileSystemProvider.newByteChannel(UnixFileSystemProvider.java:215)
  java.base/java.nio.file.Files.newByteChannel(Files.java:370)
  java.base/java.nio.file.Files.createFile(Files.java:647)
  (以下略)

はスタックトレースからわかる通り

String nameadd=(new StringBuilder(title).append("_").append(System.currentTimeMillis()).toString());//オリジナルの名前にする

String filename = new GetFileNameLogic().getFileName(part);

String uploadDir=getServletContext().getRealPath("/WEB-INF/upload");
Path file=Paths.get(uploadDir,nameadd,filename);
try {
    Files.createFile(file);

File.createFile で発生しています。
例外の発生条件はAPI docに記述があり、今回発生している例外
発生している例外 NoSuchFileException について直接言及はされていませんが、クラス継承ツリーから NoSuchFileExceptionIOException を継承しているのがわかるため、次の記述が該当するであろうことが予想できます:

IOException - 入出力エラーが発生した場合または親ディレクトリが存在しない場合

要するに、ファイル img-landscape536x354.png を作成しようとしたが、このファイルを作成すべき場所(ディレクトリ) /Applications/Eclipse_2019-03.app/Contents/workspace/.metadata/.plugins/org.eclipse.wst.server.core/tmp0/wtpwebapps/upload_server/WEB-INF/upload/_1560801511375/ が存在しなかった(可能性が高い)、ということです。

従って、例外に対する直接的な対応としては、あらかじめディレクトリを作成しておく、というものが挙げられます:

    Files.createDirectories(file.getParent());
    Files.createFile(file);

ただし前述の通り、今回の場合ファイルを保存しようとしている場所自体が適切ではありませんので、例外が出なくなったことが根本的に解決されたことを意味するわけではありません。


コメントでの追加質問について

ネットで「画像アップデート・ダウンロード、保存」等で調べると、使用するクラスは違うものの、大体上記のような流れのソースコードしか見当たらなかったのですが、これらは全て、一時的なアップロードの作業と言う事でしょうか。だとすれば、「ここからこの画像を恒久的に保存する処理が必要」という解釈で間違い無いでしょうか。

ServletContext#getRealPath の戻り値をファイル出力場所として指定することはたとえそれが一時ファイルであろうと誤りである(更にいうとそんな場所にファイル出力する妥当な理由は無いだろう)、というのが回答になります。
つまりネットで検索して見つかるそれら全ての実装は(少なくともファイル出力場所指定処理について)間違っている、ということです。
おそらく誰かが最初に思いついたものを深慮なくコピペ実装した結果かと考えますが、真意は私にはわかりません(オフィシャルのtutorialがそのような実装を行っているわけでもない)。

でまさに同じ状況での質問と回答がありました。

またこちらの回答にはServletContext#getRealPathでない別の代替実装についても複数案記述があります。質問の内容からはやや逸れますが、今回の実装を完遂する上で必要な情報かと思います。
補足すると、現在ではMicroProfile configというJava EEで(も)利用可能なコンフィグ読み取りの仕様といくつかの実装が提供されいます(やることは結局前述リンク先回答と同じなのですが)。今回の件だけで用いるのはオーバースペックかもしれませんが、参考として。

調べた結果「画像を直接DB保存」「画像は、別の場所に保存しパスのみをDB保存」 などの方法を知ることができたのですが、具体的にはどのようにするのがベストなのかもご教示いただけると幸いです。

こちらについては、JavaではなくファイルシステムやDBMSの機能性能の話になるかと思いますので、別に質問を立てられるのが良いかと思います。
また、システムに対する要求にも依存すると考えますので、どういう想定のシステム(例えば利用ユーザ数や取り扱うファイル数)に対する質問なのかも明記したほうが良いでしょう。

1
  • ネットで「画像アップデート・ダウンロード、保存」等で調べると、使用するクラスは違うものの、大体上記のような流れのソースコードしか見当たらなかったのですが、これらは全て、一時的なアップロードの作業と言う事でしょうか。だとすれば、「ここからこの画像を恒久的に保存する処理が必要」という解釈で間違い無いでしょうか。又、調べた結果「画像を直接DB保存」「画像は、別の場所に保存しパスのみをDB保存」などの方法を知ることができたのですが、具体的にはどのようにするのがベストなのかもご教示いただけると幸いです。よろしくお願い致します。@yukihane 様
    – Se I.
    Commented 2019年6月18日 19:42

この質問に回答するには、ログインする必要があります。

求めていた回答ではありませんか? のタグが付いた他の質問を参照する。