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
について直接言及はされていませんが、クラス継承ツリーから NoSuchFileException
は IOException
を継承しているのがわかるため、次の記述が該当するであろうことが予想できます:
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の機能性能の話になるかと思いますので、別に質問を立てられるのが良いかと思います。
また、システムに対する要求にも依存すると考えますので、どういう想定のシステム(例えば利用ユーザ数や取り扱うファイル数)に対する質問なのかも明記したほうが良いでしょう。