【swift】UIImageをcropした際に起こるエラー

###質問
UIImageをcropしたところ以下のようなエラーが発生します。

swift
1-[UIImage release]: message sent to deallocated instance 0x288ca6250

このエラーを追跡してみると以下のコードでエラ-が起きていることが判明しました。

swift
1extension UIImage {
2    func crop(rect to: CGRect) -> UIImage? {
3        var opaque = false
4        if let cgImage = cgImage {
5            switch cgImage.alphaInfo {
6            case .noneSkipLast, .noneSkipFirst:
7                opaque = true
8            default:
9                break
10            }
11        }
12
13        UIGraphicsBeginImageContextWithOptions(to.size, opaque, scale)
14        draw(at: CGPoint(x: -to.origin.x, y: -to.origin.y)) //ここで発生
15        let result = UIGraphicsGetImageFromCurrentImageContext()
16        UIGraphicsEndImageContext()
17        return result
18    }
19}

具体的には上のコードのこの部分です。

swift
1draw(at: CGPoint(x: -to.origin.x, y: -to.origin.y))

開発環境は
xcode 11.4です。
もし解決策をご教授いただけるのであれば大変助かります。
よろしくお願いいたします。

###追加情報
この関数を毎フレームに対して動かしていて、基本的に問題ないのですが、極たまに上記のエラーが急に発生します。
アプリを立ち上げた瞬間に起こることもあれば、長く動かした後に起こることもあります。
質問を投稿した後に判明しましたが、厳密にいうとdrawがエラーを起こしているわけではなくて、エラーが発生して、breakするタイミングが必ずdrawのタイミングになります。

この質問では私が投稿した前の質問にリンクしています。
https://teratail.com/questions/251948

私が取り組んでいるプロジェクトではベストアンサーのものを採用しました。理由は単純に今のアプリとのcompatibilityが高かったからです。他の方々の提示してくれた方法もcropできますが、アプリと最もよくマッチしたのはこちらのコードでしたので、このような形となっております。
前問に協力していただいた皆様には深く感謝を申し上げます。

後から追加ばかりですみません。
元々はEXE_BAD_ACCESSというiOSでは有名はエラーにぶち当たってしまったのが発端です。これに遭遇した人はわかると思うのですが、このエラーが発生すると、コードのどの部分がいけないのかを教えてくれません。そこでZombieを追跡することにしました。すると、まず最初に上記のエラー文がターミナル上に表示され、何回も実行してよくよく探してみるとどの実行時のエラーでも同じdrawで止まっている、という共通点を発見した次第です。

MasakiHori

2020/04/16 06:21

このエラーは止まった(crashした）ところが問題であることは決してありません。この問題はオーナーシップの設定ミスで本来保持（強参照）しなければならないオブジェクトを保持しなかった（弱参照）ために実際に利用しようとした時にオブジェクトが破棄されているため起こります。ですので止まったところのソースコードだけでは問題は見つけることはできません。オーナーシップを見直してください。問題が起こっているオブジェクトはUIImageのインスタンスですのでそこをヒントにひたすら探すしかありません。ローカス変数として保持したものを非同期な別スレッドに弱参照で渡したなどもありえますのでとにかくオーナーシップをすべて見直してください。

chestnutforest

2020/04/16 06:55 編集

ありがとうございます。おそらく私の問題は以下のリンクと同じ問題ですよね。１０年前ですが。 http://shitappaprogramer.seesaa.net/article/151284495.html 「ローカス変数として保持したものを非同期な別スレッドに弱参照で渡したなどもありえますのでとにかくオーナーシップをすべて見直してください。」自分でも身に覚えがあります。「弱参照」についてご教授いただけますか。やむなくグローバル変数で関数間の画像の受け渡しを行ってしまっているのですが.....

hoshi-takanori

2020/04/16 11:21

カメラの画像を毎フレーム UIImage にして、最終的にどうしたいのか (そもそも何をするアプリなのか) まで書いた方が適切なアドバイスが得られると思います。

行動規範の内容に同意します

回答1件

ベストアンサー

質問に書かれたcrop処理は切り取った画像の再描画処理が行われるため、かなり負荷がかかります。画像のサイズにもよりますが、これを毎フレーム毎に動かしていたのでは、処理が追いつかなくなったり、メモリ不足が発生したりしそうな気がします。基本的に問題ないのに、極たまにエラーが発生するということは、そういうシステム性能の問題が
起きているような気がします(単なる勘ですが)。

改善する保証はありませんが、
https://qiita.com/YutoMizutani/items/bca10568a875ed5ac3d3
で説明されている

swift
1extension UIImage {
2    func cropping(to rect: CGRect) -> UIImage? {
3        let croppingRect: CGRect = imageOrientation.isLandscape ? rect.switched : rect
4        guard let cgImage: CGImage = self.cgImage?.cropping(to: croppingRect) else { return nil }
5        let cropped: UIImage = UIImage(cgImage: cgImage, scale: scale, orientation: imageOrientation)
6        return cropped
7    }
8}