【Swift】カメラロールから画像を読み込む時に失敗する

Question

別の画面から遷移してきた時にカメラロールから画像を読み込んで、IMAGEVIEWに表示するアプリを作成しています。
以下のようにソースコードを書いたところ、ゆっくりと実行していくと特に問題なく動作するのですが、カメラロールで画像を選択する時に連打してしまうと、アプリ自体が初期の画面へ戻ってしまい、再度画像を読み込もうとすると全体が黒い画面になってしまいます。
この場合、何が原因として考えられるのでしょうか？
よろしくお願いします。

class MainViewController: UIViewController,UIPickerViewDelegate,UIPickerViewDataSource {
    @IBOutlet weak var imageView:UIImageView!
    var uiImage:UIImage!

    override func viewDidLoad() {
        super.viewDidLoad()
        
        //画像読込
        if UIImagePickerController.isSourceTypeAvailable(.photoLibrary) {
            // 写真を選ぶビュー
            let photoView = UIImagePickerController()
            // 写真の選択元をカメラロールにする
            photoView.sourceType = .photoLibrary
            // デリゲート
            photoView.delegate = self
            // ビューに表示
            self.present(photoView, animated: true)
        }
    }
}

extension MainViewController: UIImagePickerControllerDelegate, UINavigationControllerDelegate {
    // 写真を選んだ後に呼ばれる処理
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
        
        //画像サイズ
        let IMAGE_MAX_SIZE = 1000.0
        
        // 選択した写真を取得する
        let tmpImage = info[.originalImage] as! UIImage
        
        var tmpSize = Double(tmpImage.size.width)
        
        if(Double(tmpImage.size.width) > IMAGE_MAX_SIZE || Double(tmpImage.size.height) > IMAGE_MAX_SIZE){
            if(tmpImage.size.width >= tmpImage.size.height){
                tmpSize = IMAGE_MAX_SIZE
            }else{
                tmpSize = IMAGE_MAX_SIZE*Double((tmpImage.size.width/tmpImage.size.height))
            }
        }
        // ビューに表示する
        let resize_tmpImage:UIImage = resize(image: tmpImage,width: tmpSize)

        imageView.image = resize_tmpImage
        if(uiImage.size.width>0){
            image_flg = true
        }
        // 写真を選ぶビューを引っ込める
        self.dismiss(animated: true)
    }
}

Comments

[hameji]

コードは上下を「```」で囲んでください。
またスコープ({})は開く位置と閉じる位置を合わせてください。

現状では読みづらくて、コードが追いにくく、
読む気が出ません。

[hkbto]

すみません修正しました。よろしくお願いします。

Answer 1

何度もクリックすることにより、何度も同じ処理が重複して呼ばれて、
おそらくですが、メモリ不足に陥り落ちるのではないでしょうか？

なので、1回クリックして処理している時は、
重複して処理を行わないようにcheckしないといけないと思います。

よくあるものではHUD(head up display)を利用したものが見受けらると思います。
CocoaPodsに慣れている場合は、swiftだとPKHUDを利用してみるといいと思います。
公式ページ
チュートリアルっぽいページ

CocoaPodsに慣れてないということでしたら、
簡単なものではBool型のメンバ変数(例えば, isProcessing)などを作成し、
それにより条件分岐をすればいいと思います。
例)

class MainViewController: UIViewController,UIPickerViewDelegate,UIPickerViewDataSource {
    @IBOutlet weak var imageView:UIImageView!
    var uiImage:UIImage!
    var isProcessing: Bool = false

    // 間、省略

    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
        if isProcessing == false {
            isProcessing = true

            // ここに実行したい処理=画像の処理、viewのdismiss等

            isProcessing = false  // <-このように戻すの忘れないように
        } else {
            // 処理中であることを知らせたい場合はUIAlertControllerの表示等
        }
    }
}

Comments

[hkbto]

回答ありがとうございます。
ただ、内容の解決には至りませんでした...
下記のようなエラーが出ています。自分でも調べてみましたが、よくわからない状態です。

[discovery] errors encountered while discovering extensions: Error Domain=PlugInKit Code=13 "query cancelled" UserInfo={NSLocalizedDescription=query cancelled}

[hameji]

コードをコピペして、シュミレーターで動作確認を行って見ましたが、
特にエラーなく動きましたよ。

何がおかしいんでしょうね、、、
エラーが出るなら、示されているとこ以外に問題があるということになるのかな。