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

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

新規登録して質問してみよう
ただいま回答率
85.48%
MVP

MVP(Minimum Viable Product)とは、「必要最低限の機能を兼ね備えた製品」を指します。企画書などを完成させる前に、とりあえず製品を形にする方法です。プロトタイプなどで一旦アウトプットさせることにより、無駄なコストや時間を削減できます。

iOS

iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

Q&A

解決済

1回答

1193閲覧

【Swift】MVPでのModel-Presenter間のデータ受け渡しがうまくいかない

duck015

総合スコア29

MVP

MVP(Minimum Viable Product)とは、「必要最低限の機能を兼ね備えた製品」を指します。企画書などを完成させる前に、とりあえず製品を形にする方法です。プロトタイプなどで一旦アウトプットさせることにより、無駄なコストや時間を削減できます。

iOS

iOSとは、Apple製のスマートフォンであるiPhoneやタブレット端末のiPadに搭載しているオペレーションシステム(OS)です。その他にもiPod touch・Apple TVにも搭載されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

0グッド

0クリップ

投稿2020/02/05 08:00

編集2020/02/05 08:02

前提・実現したいこと

teratailのAPIを叩き、質問一覧をTableViewに表示させたいです(Alamofire SwiftyJSONを使用)。
MVPの練習をしたく、アーキテクチャはMVPを採用しています。

QuestionModel内でAPIを叩き、配列questionsに追加しています。
そして、処理結果をQuestionListViewPresenterへ渡し、NewArrivalQuestionListViewControllerでTableViewに表示させるといった流れです。

ただ、このコードではTableViewにデータが表示できないでいます。
QuestionModelのfunc updateQuestions()内、print(self.questions)にてデータが出力されていることから、データは取得できているようです。
そのため、QuestionmodelからQuestionListViewPresenterへ処理結果ががうまく渡せていないものと推測しています。
実際にQuestionListViewPresenterのfetchQuestions()内、print(questionModel.questions)では[]が出力されています。
ただ、そこからいろいろ調べてみましたが原因が分からず、解決策をご教示いただけませんでしょうか?

該当のソースコード

QuestionEntity

1struct QuestionEntity { 2 var id: Int 3 var title: String 4 var tags: [String] 5 var displayName: String 6 var photo: String 7 var created: String 8 var isAccepted: Bool 9 10 init(id: Int, title: String, tags: [String], displayName: String, photo: String, created: String, isAccepted: Bool) { 11 self.id = id 12 self.title = title 13 self.tags = tags 14 self.displayName = displayName 15 self.photo = photo 16 self.created = created 17 self.isAccepted = isAccepted 18 } 19}

QuestionModel

1class QuestionModel { 2 3 var questions: [QuestionEntity] = [] 4 5 func fetchQuestions() { 6 print("fetch") 7 Alamofire.request("https://teratail.com/api/v1/questions").responseJSON { response in 8 guard let object = response.result.value else { return } 9 let json = JSON(object) 10 json["questions"].forEach { (_, json) in 11 let id = json["id"].intValue 12 let title = json["title"].stringValue 13 let tags = json["tags"].arrayObject 14 let displayName = json["user"]["display_name"].stringValue 15 let photo = json["user"]["photo"].stringValue 16 let created = json["created"].stringValue 17 let isAccepted = json["isAccepted"].boolValue 18 self.questions.append(QuestionEntity(id: id, title: title, tags: tags as! [String], displayName: displayName, photo: photo, created: created, isAccepted: isAccepted)) 19 } 20 print(self.questions) 21 } 22 }

QuestionListViewPresenter

1class QuestionListViewPresenter { 2 var view: NewArrivalQuestionListViewController! 3 var questionModel = QuestionModel() 4 var numberOfQuestions: Int { 5 return questionModel.questions.count 6 } 7 8 func updateQuestions() { 9 questionModel.refreshQuestions() 10 questionModel.fetchQuestions() 11 print(questionModel.questions) 12 } 13 14 func entity(at indexPath: IndexPath) -> QuestionEntity { 15 return questionModel.questions[indexPath.row] 16 } 17}

NewArrivalQuestionListViewController

1class NewArrivalQuestionListViewController: UIViewController { 2 3 @IBOutlet weak var newArrivalQuestionListTableView: UITableView! 4 var presenter = QuestionListViewPresenter() 5 var model: QuestionModel? 6 7 override func viewDidLoad() { 8 super.viewDidLoad() 9 initializeTableView() 10 presenter.updateQuestions() 11 }

参考記事

iOSをMVC,MVP,MVVM,Clean Architectureで実装してみた
【Swift】MVCから脱却したいのでMVPの勉強をした

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

非同期通信のためです。

Alamofireは非同期に結果を渡して来ます。
Alamofire.request関数に渡したクロージャ({}の中のコード)は、Alamofireが読み込み処理を終わらせた後で実行されます。
Alamofire.request関数自体はすぐに終了します。このため、fetchQuestions関数は、呼んだらすぐに戻って来ています。

読み込みが終わってから(つまりクロージャの中が呼ばれたら)、表示を開始するようにすると良いと思います。

投稿2020/02/05 08:14

eytyet

総合スコア803

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

duck015

2020/02/05 09:56

eytyet様 ご回答ありがとうございます。非同期通信の影響だったのですね、 私の理解不足で申し訳ないのですが、「読み込みが終わってから、表示を開始する」とは具体的にどのような実装になりますでしょうか? アドバイスを元に色々と調べてみましたが、コールバックを使用したりでしょうか?
eytyet

2020/02/05 14:23

そうですね。ですが、コールバックでQuestionListModelからNewArrivalQuestionListViewControllerを呼ぶのはMVPとしては御法度なので、デリゲートかNotificationを使ってQuestionListViewPresenterに知らせることになると思います。
duck015

2020/02/05 14:37

たしかにViewとModelは分離させなくてはいけないですもんね。 Notification等で実装してみたいと思います。 アドバイスありがとうございました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問