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

前提・実現したいこと

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

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

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

EXC_BAD_ACCESS (code=2, address=0x16b18bea0)

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

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

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

該当のソースコード

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

Swift
1import UIKit
2import RealmSwift
3
4class ViewController: UIViewController, UICollectionViewDelegate, UICollectionViewDataSource, UICollectionViewDelegateFlowLayout {
5    
6    @IBOutlet var collectionView: UICollectionView!
7    var list : UIImage?
8    var idData : Int?
9    var imageList: Results<Obj>?{
10     do{
11        let realm = try Realm()
12        return realm.objects(Obj.self).sorted(byKeyPath: "id", ascending: false)
13
14     }catch{
15         print("エラー")
16     }
17     return nil
18    }
19    
20    func collectionView(_ collectionView: UICollectionView, numberOfItemsInSection section: Int) -> Int {
21        return imageList?.count ?? 0
22    }
23    
24    func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
25        let flag = imageList?[indexPath.row]
26        let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! CollectionViewCell
27        let data = flag?.image
28        let imageData = UIImage(data: data!)
29        cell.imageVIew.image = imageData
30        let layout = UICollectionViewFlowLayout()
31        layout.sectionInset = UIEdgeInsets(top: 15, left: 15, bottom: 15, right: 15)
32        collectionView.collectionViewLayout = layout
33        return cell
34    }
35    
36    func collectionView(_ collectionView: UICollectionView, layout collectionViewLayout: UICollectionViewLayout, sizeForItemAt indexPath: IndexPath) -> CGSize {
37        let horizontalSpace : CGFloat = 20
38        let cellSize : CGFloat = self.view.bounds.width / 2 - horizontalSpace
39        return CGSize(width: cellSize, height: cellSize)
40    }
41    
42    func collectionView(_ collectionView: UICollectionView, didSelectItemAt indexPath: IndexPath) {
43        let detail = imageList?[indexPath.row]
44        let data = detail?.image
45        let id = detail?.id
46        list = UIImage(data: data!)
47        idData = id
48        if data != nil {
49            performSegue(withIdentifier: "toDetailImageView", sender: nil)
50        }
51    }
52    
53    override func prepare(for segue: UIStoryboardSegue, sender: Any?) {
54        if (segue.identifier == "toDetailImageView") {
55            let subVC: DetailImageView = (segue.destination as? DetailImageView)!
56            subVC.detailImage = list
57            subVC.id = idData
58        }
59    }
60
61    override func viewDidLoad() {
62        super.viewDidLoad()
63        // Do any additional setup after loading the view.
64        
65    }
66    
67    override func viewWillAppear(_ animated: Bool) {
68        //print(collectionView)
69        self.collectionView.reloadData()
70        
71    }
72    
73    override func didReceiveMemoryWarning() {
74        super.didReceiveMemoryWarning()
75    }
76
77}

・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

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

回答2件

ベストアンサー

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

swift
1DispatchQueue.main.async {
2 self.collectionView.reloadData()
3}

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

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

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

swift
1let imageData = image.jpegData(compressionQuality: 1.0)

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

投稿2019/11/04 04:43

usagi001

総合スコア208

sai

2019/11/04 07:27

usagi001様ご回答いただきありがとうございます。＞メインスレッドで呼ばれていない場合は →ご指摘の通り呼んでみましたが、変化なしでした。＞また画像を取得してData型にする際に下記のように1.0を指定していますが0.9などにしてみてはどうでしょうか？ →いくつか取れる値を試しましたが、変化なしでした。＞メモリ不足の場合は DBにバイナリを持っているのでローカルに画像ファイルを保存してそのパスをDBに持ってみてはどうでしょうか？ →これを試そうと思います。参考URLまでご教示いただきありがとうございます。

sai

2019/11/04 11:11

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

usagi001

2019/11/04 11:16

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

sai

2019/11/04 11:36

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

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

Swift
1DispatchQueue.main.async {
2  let flag = imageList?[indexPath.row]
3  let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "Cell", for: indexPath) as! CollectionViewCell
4  let data = flag?.image
5  let imageData = UIImage(data: data!)
6  cell.imageVIew.image = imageData
7  let layout = UICollectionViewFlowLayout()
8  layout.sectionInset = UIEdgeInsets(top: 15, left: 15, bottom: 15, right: 15)
9  collectionView.collectionViewLayout = layout
10  return cell
11}