Swift: Callback関数をさらに別の関数へ渡したい

Question

以下のように書いてみましたが、エラーが出ています。
解決方法がわかる方、教えていただけますと幸いです。

やりたいこととしては、
(1) MapViewControllerで、DataManagerのmapメソッドを呼び出し、
APIから情報を更新して、addAnnotationsメソッドを呼びたいと考えています。

// MapViewController.swift
class MapViewController: UIViewController, MKMapViewDelegate {
    override func viewDidLoad() {
        super.viewDidLoad()
        // Do any additional setup after loading the view.
        
        // AppDelegateのインスタンスを取得
        self.delegate = UIApplication.shared.delegate as? AppDelegate
        // DataManagerのインスタンスを取得
        self.dataManager = delegate?.dataManager
        // APIからデータの更新
        self.dataManager?.map(callback: self.addAnnotations) // (1)
    }
    func addAnnotations() {
        //
    }
}

(2) 成功すれば、callbackメソッドを呼び出しますが、
(3) 失敗した場合は、一旦loginメソッドを呼び出したあと、
再度mapメソッドをリトライしたいと考えています。

しかし、(3)の位置で、
Cannot convert value of type '()' to expected argument type '() -> Void'
というエラーが出てしまいました。

// DataManager.swift
class DataManager {
    func login(callback: @escaping () -> Void) {
        //
        providerAuthNone.request(.login) { result in
            switch result {
            case let .success(moyaResponse):
                do {
                    let filteredResponse = try moyaResponse.filterSuccessfulStatusCodes()
                    
                    callback()
                }
                catch {
                    // show an error to your user
                }
            case let .failure(error):
                //
                break
            }
        }
    }
    
    func map(callback: @escaping () -> Void) {
        providerAuthBearer.request(.map) { result in
            switch result {
            case let .success(moyaResponse):
                do {
                    let filteredResponse = try moyaResponse.filterSuccessfulStatusCodes()
                    
                    callback() // (2)
                }
                catch {
                    // show an error to your user
                    self.login(callback: self.map(callback: callback)) // (3)
                }
            case let .failure(error):
                //
                break
            }
        }
    }

以上です。
よろしくお願い申し上げます。

Answer 1

タイトルと内容がちょっとずれている気がしたのですが、おそらくやりたいこととしては「mapの引数にcallbackを渡した状態のものを、loginのcallbackとして呼ばせるようにしたい」というところでしょうか。

ご提示のエラーの通り関数の型が合わないので、以下のようにクロージャでラップした関数を定義してあげればよいと思います。

self.login { [weak self] in
    self?.map(callback: callback)
}

---

それと余談ですが、次の書き方には注意です。

self.dataManager?.map(callback: self.addAnnotations)

self.addAnnotations は暗黙的にselfを強参照でキャプチャしてしまうので、この書き方だと循環参照によるメモリリークが発生する可能性があります。メンバメソッドをcallbackとして渡したいのであれば、冗長には感じるかもしれませんがクロージャでselfのキャプチャの仕方を設定するようにしてください。

self.dataManager?.map { [weak self] in
    self?.addAnnotations()
}