【Swift】Collection View CellでEXC_BAD_ACCESS (code=2, address=0x16b18bea0) エラーが出ます。

Question

前提・実現したいこと

画面遷移後に、保存した画像をコレクションビューに表示したい。

発生している問題・エラーメッセージ

画像を保存し、画面遷移すると以下のエラーメッセージが発生しました。

EXC_BAD_ACCESS (code=2, address=0x16b18bea0)

###詳細
AdditionImageでRealmを用いて画像を保存し、ViewControllerへ戻ると、以下の部分でエラーメッセージが発生します。

let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! CollectionViewCell

###わからないこと
処理が重くなりクラッシュしたと推測していますが、原因がわかっていません。
解消法についてご教授いただければと思います。よろしくお願いいたします。

該当のソースコード

・ViewController: コレクションビューを持ち、保存された画像を表示する。

import UIKit
import RealmSwift

class ViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDataSource, UICollectionViewDelegateFlowLayout {
    
    @IBOutlet var collectionView: UICollectionView!
    var list : UIImage?
    var idData : Int?
    var imageList: Results<Obj>?{
     do{
        let realm = try Realm()
        return realm.objects(Obj.self).sorted(byKeyPath: "id", ascending: false)

     }catch{
         print("エラー")
     }
     return nil
    }
    
    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
        return imageList?.count ?? 0
    }
    
    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
        let flag = imageList?[indexPath.row]
        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! CollectionViewCell
        let data = flag?.image
        let imageData = UIImage(data: data!)
        cell.imageVIew.image = imageData
        let layout = UICollectionViewFlowLayout()
        layout.sectionInset = UIEdgeInsets(top: 15, left: 15, bottom: 15, right: 15)
        collectionView.collectionViewLayout = layout
        return cell
    }
    
    func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, sizeForItemAt indexPath: IndexPath) -> CGSize {
        let horizontalSpace : CGFloat = 20
        let cellSize : CGFloat = self.view.bounds.width / 2 - horizontalSpace
        return CGSize(width: cellSize, height: cellSize)
    }
    
    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
        let detail = imageList?[indexPath.row]
        let data = detail?.image
        let id = detail?.id
        list = UIImage(data: data!)
        idData = id
        if data != nil {
            performSegue(withIdentifier: "toDetailImageView", sender: nil)
        }
    }
    
    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
        if (segue.identifier == "toDetailImageView") {
            let subVC: DetailImageView = (segue.destination as? DetailImageView)!
            subVC.detailImage = list
            subVC.id = idData
        }
    }

    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        
    }
    
    override func viewWillAppear(_ animated: Bool) {
        //print(collectionView)
        self.collectionView.reloadData()
        
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }

}

・AdditionImage: 画像をDBに保存する。

import UIKit
import RealmSwift

class AdditionImage: UIViewController, UIImagePickerControllerDelegate, UINavigationControllerDelegate {

    @IBAction func actionSheet(_ sender: Any) {
        let actionSheet = UIAlertController(title: "", message: "画像を選んでください", preferredStyle: UIAlertController.Style.actionSheet)
        
        let tappedCamera = UIAlertAction(title: "カメラで撮影する", style: UIAlertAction.Style.default, handler: {(action: UIAlertAction!) in self.takePhotos()
        })

        let tappedLibrary = UIAlertAction(title: "ライブラリから選択する", style: UIAlertAction.Style.default, handler: {(action: UIAlertAction!) in self.choosePhotos()
        })

        let cancel = UIAlertAction(title: "キャンセル", style: UIAlertAction.Style.cancel, handler: {(action: UIAlertAction!) in print("キャンセル")
        })
        
        actionSheet.addAction(tappedCamera)
        actionSheet.addAction(tappedLibrary)
        actionSheet.addAction(cancel)
        
        present(actionSheet, animated: true, completion: nil)
    }
    
    @IBOutlet weak var imageView: UIImageView!
    
    func takePhotos() {

        let camera = UIImagePickerController.SourceType.camera
        
        if UIImagePickerController.isSourceTypeAvailable(camera){
            let picker = UIImagePickerController()
            picker.sourceType = camera
            picker.delegate = self
            self.present(picker, animated: true, completion: nil)
        }
    }
    
    func choosePhotos() {
        let photoPick = UIImagePickerController()
        photoPick.sourceType = .photoLibrary
        photoPick.delegate = self
        self.present(photoPick, animated: true, completion: nil)
    }
    
    func imagePickerController(_ picker: UIImagePickerController, didFinishPickingMediaWithInfo info: [UIImagePickerController.InfoKey : Any]) {
        
        let image = info[UIImagePickerController.InfoKey.originalImage] as! UIImage
        self.imageView.image = image
        
        let imageData = image.jpegData(compressionQuality: 1.0)

        let realm = try! Realm()
        let firstImage: Obj = Obj.create(realm: realm)
        firstImage.image = imageData
        try! realm.write {
            realm.add(firstImage)
        }
        
        self.dismiss(animated: true, completion: nil)
    }
    
}

・コレクションビューセル内のUIImageView

import UIKit

class CollectionViewCell: UICollectionViewCell {
    
    @IBOutlet weak var imageVIew: UIImageView!
}

格納先のファイル

import UIKit
import RealmSwift

class Obj: Object{
   
   @objc dynamic var id: Int = 0
   @objc dynamic var image: Data?

   override static func primaryKey() -> String? {
           return "id"
    }
    
    static func newID(realm: Realm) -> Int {
        if let obj = realm.objects(Obj.self).sorted(byKeyPath: "id").last {
            return obj.id + 1
        } else {
            return 1
        }
    }
    
    static func create(realm: Realm) -> Obj {
        let obj: Obj = Obj()
        obj.id = newID(realm: realm)
        return obj
    }
}

試したこと

・エラー部分にprint("test")を記載したところ、211回出力されました。そのため、処理が重くなってクラッシュしたのではないかと思います。

・Swift次のコードでThread 1: EXC_BAD_ACCESS (code=2, address=0x110d75ea8)のエラーがでます。。を参考にし、Custom Classを確認しましたが、問題ないと判断しました。

補足情報（FW/ツールのバージョンなど）

Swift: version 5.1
Xcode: Version 11.1

Answer 1

原因がわかりませんが
メインスレッドで呼ばれていない場合は

DispatchQueue.main.async {
 self.collectionView.reloadData()
}

のようにreloadDataをのところに書いてみてはどうでしょうか？

メモリ不足の場合は
DBにバイナリを持っているのでローカルに画像ファイルを保存してそのパスをDBに持ってみてはどうでしょうか？
下記はPHPの話ですがDBの話なので当てはまると思います。ご参考にどうぞ。
データベースに画像を保存するのはありでしょうか？

また画像を取得してData型にする際に下記のように1.0を指定していますが0.9などにしてみてはどうでしょうか？

let imageData = image.jpegData(compressionQuality: 1.0)

ここの引数の1.0は1倍を意味するのではなく最高画質となり実際の画像よりもサイズをくうことがあります。下記ご参考ください。
UIImageJPEGRepresentationを使う際の注意点

Comments

[sain]

usagi001様

ご回答いただきありがとうございます。

＞メインスレッドで呼ばれていない場合は
→ご指摘の通り呼んでみましたが、変化なしでした。

＞また画像を取得してData型にする際に下記のように1.0を指定していますが0.9などにしてみてはどうでしょうか？
→いくつか取れる値を試しましたが、変化なしでした。

＞メモリ不足の場合は
DBにバイナリを持っているのでローカルに画像ファイルを保存してそのパスをDBに持ってみてはどうでしょうか？
→これを試そうと思います。参考URLまでご教示いただきありがとうございます。

[sain]

usagi001様

度々申し訳ありませんが、コメントさせていただきます。

＞メモリ不足の場合は
＞DBにバイナリを持っているのでローカルに画像ファイルを保存してそのパスをDBに持ってみてはどうでしょうか？
→パスで保存する方法をとりましたが、試したことの３枚目の画像の通り変わらずでした。
　　 また、画像左側をみると、メインスレッドでcellForItemAtが繰り返し呼ばれていました。もし、原因に心当たりがあればご教授いただけると幸いです。

[usagi001]

一度collectionView.collectionViewLayout = layoutのとこをコメントアウトして実行してみてください。ループしている可能性があります。

[sain]

usagi001様

ご指摘の通り、コメントアウトしたところリロードされました。
度々のご指導、ありがとうございました。

Answer 2

DispatchQueue.main.async {
  let flag = imageList?[indexPath.row]
  let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! CollectionViewCell
  let data = flag?.image
  let imageData = UIImage(data: data!)
  cell.imageVIew.image = imageData
  let layout = UICollectionViewFlowLayout()
  layout.sectionInset = UIEdgeInsets(top: 15, left: 15, bottom: 15, right: 15)
  collectionView.collectionViewLayout = layout
  return cell
}

このようにしても動きませんか?

Comments

[sain]

ren_ren様

ご回答いただきありがとうございます。
ご指摘の方法を試しましたが、エラーが発生しました。
質問内容に追加しましたので、お手数ですがご確認いただけますか。