delegateが常にnilになって返される

Question

実現したいこと

現在QRコードを読み込んでその内容を表示するiOSアプリを製作しています。
QRコードの読み込みはViewControllerとは別にCameraViewControllerで実装しています。

直面している課題

CameraViewControllerのdelegateが常にnilになってしまい、処理がそれ以上実行できません。

処理を依頼する側

import UIKit
import Foundation
import AVFoundation


protocol CameraViewControllerDelegate{
    func ReadQRcode(data : String)
}

class CameraViewController: UIViewController, AVCaptureMetadataOutputObjectsDelegate {
    
    var delegate : CameraViewControllerDelegate!
    
    private var captureSession = AVCaptureSession()
    
    override func viewDidLoad() {
        super.viewDidLoad()
        
　　　　　//使用するカメラの設定
        captureSession.beginConfiguration()
        let videoDevice = AVCaptureDevice.default(.builtInWideAngleCamera,
                                                  for: .video, position: .back)
        guard
            let videoDeviceInput = try? AVCaptureDeviceInput(device: videoDevice!),
            captureSession.canAddInput(videoDeviceInput)
            else {return}
        captureSession.addInput(videoDeviceInput)
        
        let QRCodeOutput = AVCaptureMetadataOutput()
        guard captureSession.canAddOutput(QRCodeOutput) else {
            return
        }
        captureSession.sessionPreset = .photo
        captureSession.addOutput(QRCodeOutput)
        captureSession.commitConfiguration()
        
        QRCodeOutput.setMetadataObjectsDelegate(self, queue: DispatchQueue.main)
        QRCodeOutput.metadataObjectTypes = [.qr]
        
        // 背面カメラの映像を画面に表示するためのレイヤーを生成
        let previewLayer = AVCaptureVideoPreviewLayer(session: self.captureSession)
        previewLayer.frame = self.view.bounds
        previewLayer.videoGravity = .resizeAspectFill
        self.view.layer.addSublayer(previewLayer)
        
        self.captureSession.startRunning()
    }
    
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
        // Dispose of any resources that can be recreated.
    }
    
    internal func metadataOutput(_ output: AVCaptureMetadataOutput, didOutput metadataObjects: [AVMetadataObject], from connection: AVCaptureConnection) {
        for metadata in metadataObjects as! [AVMetadataMachineReadableCodeObject] {
            // QRコードのデータかどうかの確認と受け取ったデータの確認
            if(metadata.type == .qr && metadata.stringValue != nil){
                self.delegate?.ReadQRcode(data: metadata.stringValue ?? "failed")
                if delegate == nil{print("delegate is nil.")}
                self.captureSession.stopRunning()
            }
        }
    }
}

処理を実行する側

import UIKit

class ViewController: UIViewController,FUIAuthDelegate, CameraViewControllerDelegate{
    
    @IBOutlet weak var qrButton: UIButton!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        //...
        CameraViewController().delegate = self
    }
    
    @objc func QRButtonTaped(sender: AnyObject) {
        self.present(CameraViewController(), animated: true, completion: nil)
    }
    //...
}

よろしくお願いします。

Answer 1

    override func viewDidLoad() {
        super.viewDidLoad()
        //...
        CameraViewController().delegate = self
    }

    @objc func QRButtonTaped(sender: AnyObject) {
        self.present(CameraViewController(), animated: true, completion: nil)
    }

viewDidLoad()でインスタンス化した CameraViewController と QRButtonTapped(sender:)でインスタンス化した CameraViewController は異なっているのが原因かと思います。

なので、

        let vc = CameraViewController()
        vc.delegate = self
        self.present(vc, animated: true, completion: nil)

こんな感じにすれば期待された動作になるかとおもいます。

ちなみに、delegate を宣言するときには class を継承し、変数 delegate を弱参照させることで循環参照を防ぎますので、このような変更を加えた方が良いかと思います。

// class を継承させる
protocol CameraViewControllerDelegate: class {
    func ReadQRcode(data : String)
}

class CameraViewController: UIViewController, AVCaptureMetadataOutputObjectsDelegate {
    // weak キーワードをつけて循環参照を防ぐ（class に準拠させないと weak キーワードは付けられない）
    weak var delegate : CameraViewControllerDelegate!

Comments

[suzumushi]

ありがとうございます！原因をよく理解でき、無事動作させることができました。