openly-jp / voiscribe

1 stars 0 forks source link

Remove model garbage files #306

Closed shibukazu closed 1 year ago

shibukazu commented 1 year ago

Related Issue

close #297

Context

What

Memo

ooyamatakehisa commented 1 year ago

tmpディレクトリだしこっちが削除わざわざしなくてもiOSが自動でしてくれるだろうと思ったけど、できるだけ削除しろってAppleForumにかいてた https://developer.apple.com/forums/thread/680224

関連

ooyamatakehisa commented 1 year ago

@shibukazu これダウンロード先の一時ファイルがFileManager.default.temporaryDirectoryにあるというのは実験的に確かめた感じ?

shibukazu commented 1 year ago

Simulatorでアプリをキルしたあととかだと全然残っちゃってたんですよね 僕らの場合じゃ特にファイルサイズがでかいのでクリーンアップはしたほうがいいかと思いました

shibukazu commented 1 year ago

@shibukazu これダウンロード先の一時ファイルがFileManager.default.temporaryDirectoryにあるというのは実験的に確かめた感じ?

実験的にですね

shibukazu commented 1 year ago

https://developer.apple.com/documentation/foundation/urlsession/1411608-downloadtask

The location of a temporary file where the server’s response is stored. You must move this file or open it for reading before your completion handler returns. Otherwise, the file is deleted, and the data is lost.

あーでもダウンロード途中で終わった一時ファイルは削除されるっていう記述もありますね 現状一番大きなmediumでも1GB程度と考えると別に自分で削除しなくてもいいのかな

shibukazu commented 1 year ago

一時ファイルがCoreDataへの書き込みとか他の処理で使われない保証がないことと、起動時に毎回するのは無駄であることを踏まえるとこのPRはなくてもいいかもですね

ooyamatakehisa commented 1 year ago

いやでも、この

Otherwise, the file is deleted, and the data is lost.

は、tmpDirectoryとしていつか削除されるよって意味な気もするし、そうならやっぱり上のAppleForumで言ってるように自分で早めに削除したほうがいいきもする。からLGTM!(動作確認はしてないです)