別クラスのクロージャの呼び出し

Question

別クラスに定義されたクロージャの呼び出しをしたいです。
下記のRootsMakerクラスのクロージャをViewContorollerのファンクション内で呼び出そうとしています。

self.VC.direction.calculation(completion: {[unowned self] route in
         guard let routes = route.routes as? [Routes] else {
            return
         }
         self.VC.mapView.addDirection(routes: routes, color: UIColor.red)
}, failuer: { error in
         print(error)
})

ViewControllerではインスタンスを生成してクロージャを実行するメソッドを呼んでいます。

let RM = RootsMaker()
RM.making(addressArray: self.addressArray, indexPathrow: indexPath.row ,mapView: self.mapView)

しかしこれを実行すると下記のようにエラーが出てしまいます。
エラー箇所は「self.VC.mapView.addDirection(routes: routes, color: UIColor.red)」になります。

Fatal error: Unexpectedly found nil while unwrapping an Optional value: file Direction_Sample/RootsMaker.swift, line 91
2021-04-05 13:16:46.659491+0900 Direction-Sample[1453:507980] Fatal error: Unexpectedly found nil while unwrapping an Optional value: file Direction_Sample/RootsMaker.swift, line 91

クロージャについて調べてみると強参照・弱参照という参照方法があり、循環参照という問題があるためそれに該当するのではないかと思い、「unowned self」を「weak self」に変えてみましたが結果は同じでした。
クロージャをViewControllerで直接実行した場合は問題なく動きます。

どうすればクロージャを呼び出しで実行できるでしょうか？？
ご教授よろしくお願い致します。

Comments

[tomato879241]

それは本来Main Threadで更新しなければいけないもの（mapView）をclosure内で更新しているからエラーが出るのではないですか？

[takk_014]

クラス分けする前は、ViewControllerで同じようにクロージャ内でmapViewを更新していましたが問題はありませんでした。

[tomato879241]

それだけのコードではわかりません。クラス同士でretain cycleになっているのでは？

[hoshi-takanori]

VC または mapView が nil ってことだと思いますが、コードが断片的なのでそれ以上のことは分かりません。(VC が怪しい気がしますが…。)

Answer 1

コンプライアンスの問題上コードが部分的となってすみません・・・
クロージャがselfをselfがクロージャを互いに強参照し合うことによって循環参照が発生していたのが原因のようです。
unownedかweakを使えば解決するはずなのですが解決しないため、mapViewを引数として別クラスで受け取ってselfを使わないようにして解決させました。