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がそのような実装を行っているわけでもない)。
でまさに同じ状況での質問と回答がありました。
調べた結果「画像を直接DB保存」「画像は、別の場所に保存しパスのみをDB保存」 などの方法を知ることができたのですが、具体的にはどのようにするのがベストなのかもご教示いただけると幸いです。
こちらについては、JavaではなくファイルシステムやDBMSの機能性能の話になるかと思いますので、別に質問を立てられるのが良いかと思います。
また、システムに対する要求にも依存すると考えますので、どういう想定のシステム(例えば利用ユーザ数や取り扱うファイル数)に対する質問なのかも明記したほうが良いでしょう。