【Swift】UIRefreshControl使用時のエラー

Question

前提・実現したいこと

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

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

発生している問題・エラーメッセージ

データ自体はTableViewに表示されて問題ないのですが、TableViewを下に引っ張った時にUIRefreshControlでデータ更新をしたいです。
現状だと下に引っ張ったときに下記エラーが出ています。

UIRefreshControlのインスタンスには関数updateQuestions()をaddTargetで指定しています。
このupdateQuestions()はNewArrivalQuestionListViewControllerのviewDidLoadでも使用しています。
viewDidLoadの時には問題ないのに、なぜ下に引っ張った時にエラーになるのか疑問です。

QuestionListViewPresenterのfunc entity()でエラーが発生していますが、データを格納している配列questionsを見るとデータが入っていないため、エラーになっているようです。
そのため、恐らくAlamofireの非同期通信が影響しているのではと思っています。

しかし、QuestionModel内の関数fetchQuestions()ではNotificationCenterを使用し、QuestionListViewPresenterへ通知を行っているため、その影響は無いはずです（実際にviewDidLoadで表示できているため）。

色々と考えてみましたが、原因が分からずお手上げの状況のため、ご教示いただけませんでしょうか？
補足ですが、fetchQuestions()の中のprint("通知")は下に引っ張った時に出力されませんでした。
（つまり通知する前にエラーになっている？）

エラー内容：Fatal error: Index out of range

QuestionListViewPresenterのこの箇所で。
エラー発生箇所 ： func entity(at indexPath: IndexPath) -> QuestionEntity {
                   return questionModel.questions[indexPath.row]
               }

該当のソースコード

struct QuestionEntity {
    var id: Int
    var title: String
    var tags: [String]
    var displayName: String
    var photo: String
    var created: String
    var isAccepted: Bool
    
    init(id: Int, title: String, tags: [String], displayName: String, photo: String, created: String, isAccepted: Bool) {
        self.id = id
        self.title = title
        self.tags = tags
        self.displayName = displayName
        self.photo = photo
        self.created = created
        self.isAccepted = isAccepted
    }
}

class QuestionModel {
    
    var questions: [QuestionEntity] = []
    var notificationName: Notification.Name {
        return Notification.Name(rawValue: "questions")
    }
    
    func fetchQuestions() {
        print("fetch")
        Alamofire.request("https://teratail.com/api/v1/questions").responseJSON { response in
            guard let object = response.result.value else { return }
            let json = JSON(object)
            json["questions"].forEach { (_, json) in
                let id = json["id"].intValue
                let title = json["title"].stringValue
                let tags = json["tags"].arrayObject
                let displayName = json["user"]["display_name"].stringValue
                let photo = json["user"]["photo"].stringValue
                let created = json["created"].stringValue
                let isAccepted = json["isAccepted"].boolValue
                self.questions.append(QuestionEntity(id: id, title: title, tags: tags as! [String], displayName: displayName, photo: photo, created: created, isAccepted: isAccepted))
            }
            self.notify()
            print("通知")
        }
    }
    
    func refreshQuestions() {
        questions = []
        print("refresh")
     }
    
    func addObserver(_ observer: Any, selector: Selector) {
        NotificationCenter.default.addObserver(observer, selector: selector, name: notificationName, object: nil)
    }
    
    func removeObserver(_ observer: Any) {
        NotificationCenter.default.removeObserver(observer)
    }
    
    func notify() {
        NotificationCenter.default.post(name: notificationName, object: nil)
    }
}

class QuestionListViewPresenter {

    private weak var view: ListViewInterface!
    
    var questionModel: QuestionModel!
   
    var numberOfQuestions: Int {
        return questionModel?.questions.count ?? 10
    }

    init(view: ListViewInterface) {
        self.view = view
        self.questionModel = QuestionModel()
        self.questionModel.addObserver(self, selector: #selector(self.updated))
    }
//
//    deinit {
//        questionModel.removeObserver(self)
//    }
//
    @objc func updateQuestions() {
        questionModel.refreshQuestions()
        questionModel.fetchQuestions()
        print("presenter")
    }
    
    func entity(at indexPath: IndexPath) -> QuestionEntity {
        return questionModel.questions[indexPath.row]  //Fatal error: Index out of range
    }

    @objc func updated() {
        view?.reloadData()
    }
}

protocol ListViewInterface: class {
    func reloadData()
}

class NewArrivalQuestionListViewController: UIViewController, ListViewInterface {
    
    @IBOutlet weak var newArrivalQuestionListTableView: UITableView!
    
    var presenter: QuestionListViewPresenter!
    
    var itemInfo: IndicatorInfo = "新着"
    
    override func viewDidLoad() {
        super.viewDidLoad()
        initializePresenter()
        initializeTableView()
        newArrivalQuestionListTableView.refreshControl = UIRefreshControl()
        newArrivalQuestionListTableView.refreshControl?.addTarget(self, action: #selector(self.updateQuestions), for: .valueChanged)
        updateQuestions()
    }

    func initializeTableView() {
        newArrivalQuestionListTableView.delegate = self
        newArrivalQuestionListTableView.dataSource = self
        newArrivalQuestionListTableView.register(UINib(nibName: "QuestionListTableViewCell", bundle: nil), forCellReuseIdentifier: "QuestionListTableViewCell")
    }
    
    func initializePresenter() {
        presenter = QuestionListViewPresenter(view: self)
    }
    
    func reloadData() {
        newArrivalQuestionListTableView.refreshControl?.endRefreshing()
        newArrivalQuestionListTableView.reloadData()
        print("リロード")
    }
    
    @objc func updateQuestions() {
        presenter.updateQuestions()
        newArrivalQuestionListTableView.refreshControl?.beginRefreshing()
    }
}

extension NewArrivalQuestionListViewController: UITableViewDelegate, UITableViewDataSource {
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return presenter.numberOfQuestions
    }
    
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = newArrivalQuestionListTableView.dequeueReusableCell(withIdentifier: "QuestionListTableViewCell", for: indexPath) as! QuestionListTableViewCell
        cell.setQuestion(entity: presenter.entity(at: indexPath))
        return cell
    }
    
    func tableView(_ tableView: UITableView, didSelectRowAt indexPath: IndexPath) {
        newArrivalQuestionListTableView.deselectRow(at: indexPath, animated: true)
    }
}

参考記事

【Swift】MVCから脱却したいのでMVPの勉強をした

Answer 1

QuestionListViewPresenterのupdateQuestionsでrefreshQuestionsとfetchQuestionsを呼び出してますが、これらはそれぞれ
・refreshQuestions: questionsを空にする
・fetchQuestions: APIを叩き、結果が返ったらquestionsに値を入れてnotifyを呼び、表示を更新する
という処理をしています。ここで、「結果が返ったら〜する」の部分は非同期で実行されるので、updateQuestions自体はnotify（reloadData）が呼ばれる前に終了します。このため、引っ張った時はquestionsが変更されたのにreloadDataされてない状態になり、TableViewが存在すると思い込んでいるデータを参照しようとしてエラーになっています。なお、初回はもともとquestionsが空なので問題ありません。

修正方法としては、
・refreshQuestionsでquestionsを空にしたらnotifyを呼ぶ、または
・updateQuestionsではrefreshQuestionsせずに、APIの結果が返ってきた時にquestionsをいったん空にしてから結果を格納する
などでしょうか。

Comments

[duck015]

hiroshi-takanori様
ご回答ありがとうございます。なるほど、非同期通信の影響だったのですね...。
頂いたアドバイスの通り実装したところ、問題なく動作しました。
ありがとうございました！