shinonome-inc / mobile_sakai

0 stars 1 forks source link

Alamofireの共通化 #29

Closed yasyun closed 3 years ago

yasyun commented 3 years ago

現在はそれぞれのクラスのファイルにrequestメソッドを記述し、使用している。  ↓ 全てを共通化させて、一活で管理できるようにする。

yasyun commented 3 years ago

記事の取得はできたが、取得した記事を他のクラスに受け渡し、そちらのTableView等で表示することができない。

関数の返り値で取得した記事の配列を渡す → .responseから抜けてしまうと記事の中身が保持されないため✖ kaitoさんのコードを参考にして、completionを利用する → エラーは出ていないが処理順(?)的に記事の表示ができないため✖(取得前にforEachが回ってしまっている?)

yasyun commented 3 years ago

やはり処理順が悪さをしていたように感じたので、非同期処理についてもう少し調べた上で、kaitoさんと同じ方向性で実装した結果、上手くAlamofireを共通化させることができた。(CommonApi.swift)

Oauth認証のリクエスト以外を共通化した。