質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.20%

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 592

salud

score 178

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

やりたいこととしては、
(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
            }
        }
    }

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

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

checkベストアンサー

+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()
}

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.20%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる