swiftでクロージャーが呼ばれない

Question

swiftでgithubからリポジトリを検索するプログラムを組んでいます。
データの検索、配列に挿入するまでが非同期処理で行われるので、それが終わった後に、tableViewを構築するためにセルの情報や配列の数を取得したいと思っています。
SearchViewControllerのtappedSearchButtonメソッドのクロージャーで呼べば非同期処理が終わった後にクロージャーの中身が呼ばれるかなと考え、以下のようにコードを書いたのですが、クロージャーの中身が呼ばれません。これはどうしてでしょうか？

また、SearchModel内で呼ぶことも考えましたが、Modelは完全に独立させたいのでその中にViewやPresenterの処理を書くことはしたくないです。MVPモデルではこのような考え方は正しいかもお聞きしたいです。

SearchViewController

import UIKit

class SearchViewController: UIViewController {

    
    @IBOutlet weak var searchTextField: UITextField!
    @IBOutlet weak var tableView: UITableView!
    @IBOutlet weak var searchButton: UIButton!
    var presenter:SearchViewInput!
    override func viewDidLoad() {
        super.viewDidLoad()
 
        initPresenter()
        

        

    }
    
    func initLayout(){
        
        
    }
    func initPresenter(){
         let model = SearchModel()
         let presenter = SearchPresenter(view:self,model:model)
        self.presenter = presenter
    }
    
    @IBAction func tappedSearchButton(_ sender: Any) {
        guard let text = searchTextField.text else{return}
        presenter.didTappedSearchButton(searchText: text) {
            let num = presenter.getRepositoryCount()
            print(num)
        }
    }
    
    


}

SearchPresenter

import Foundation

public protocol SearchViewInput{
    func didTappedSearchButton(searchText:String,complition:()->Void)
    func getRepositoryCount()->Int
    
}
public protocol SearchViewOutput{

}

class SearchPresenter:SearchViewInput{
    func getRepositoryCount() -> Int {
        let count = model.numberRepositories()
        return count
        
    }
    
    
    var view:SearchViewController!
    var model:SearchModelInput!
    func didTappedSearchButton(searchText: String, complition: () -> Void) {
        model.searchRepository(query:searchText)
        
    }
    init (view:SearchViewController,model:SearchModelInput){
           self.view = view
           self.model = model
       }
    func getRepositoryForRowAt(indexPath IndexPath:Int)->RepositoryModel.Repo{
        let repository = model.RepositoryForRowAt(IndexPath:IndexPath)
        return repository
    }
    
}

SearchModel

import Foundation

public struct RepositoryModel:Codable{
    var items:[Repo]
    struct Repo:Codable{
        var name:String!
        var fullName:String!
        var description:String!

        private enum CodingKeys:String, CodingKey{
            case fullName = "full_name"
            case name
            case description
        }
    }

}
protocol SearchModelInput{
    
    func searchRepository(query:String)
    func numberRepositories()->Int
    func RepositoryForRowAt(IndexPath:Int)->RepositoryModel.Repo
    
    
}



class SearchModel:SearchModelInput{
    
    var RepositoriesArray = [RepositoryModel.Repo]()
    

    func searchRepository(query:String) {
        let urlRequest = createURLRequest(query: query)
        print(urlRequest)
        let decoder = JSONDecoder()
        let task = URLSession.shared.dataTask(with: urlRequest){ (Data,URLResponse, Error) in
            do{
                let decodedStruct = try decoder.decode(RepositoryModel.self, from: Data!)
                for i in 0..<decodedStruct.items.count{
                    self.RepositoriesArray.append(decodedStruct.items[i]
                }
            }catch{
            }
        }
        task.resume()
    }
    
    func createURLRequest(query:String)->URLRequest{
        let baseURLString = "https://api.github.com/search/repositories"
        let parameter = query
        let urlString = baseURLString + "?q=(parameter)"
        let url = URL(string: urlString)!
        let urlRequest = URLRequest(url: url)
        return urlRequest
    }
    
    func numberRepositories() ->Int{
        return RepositoriesArray.count
    }
    func RepositoryForRowAt(IndexPath indexPath:Int) -> RepositoryModel.Repo {
        return RepositoriesArray[indexPath]
    }
}

Comments

[hoshi-takanori]

SearchPresenter の didTappedSearchButton メソッドの complition 引数が使われてませんね。
SearchModel の searchRepository メソッドに complition を渡して、for の後ろあたりで呼んであげればいいのでは。
(あと、MVP はよく分かりませんが、SearchViewController と SearchPresenter が循環参照してるのでメモリリークするのでは…。)

[oeiqgfodgfhps]

ありがとうございました！
hosgi-takanoriさんがおっしゃったようにやるとうまくいきました！
ありがとうございます！

Answer 1

hoshi-takanoriさんの助言をもとにコードを書き直したらうまくいきました