Swiftのコード冗長性を減らすには？

Question

私はSwiftのアプリケーションをコーディングしています。これは、データベースクエリの後に追加された配列の情報を反復してテーブルを生成することです。私はviewDidLoad()メソッドのクエリ、追加、およびtableView.reloadData()を持っています。それはうまく動作します。私はまた、ほぼ同じコードを画面上のプルで応答するリフレッシュ機能に書き込んだ。それはうまくいく。しかし、問題はそれが非常に忙しいということです。 2回書かれたコードがたくさんあります。私はそれを修正する最良の方法が何であるか疑問に思っています。 viewDidLoad()で

：ここに関連するコードがある

override func viewDidLoad() { 
     super.viewDidLoad(){ 
//run query, append arrays 
self.tableView.reloadData() 
}refreshControl = UIRefreshControl() 
     refreshControl!.attributedTitle = NSAttributedString(string: "Pull to refresh") 
     refreshControl!.addTarget(self, action: #selector(DisplayEventsTVC.refresh(_:)), forControlEvents: UIControlEvents.ValueChanged) 
} 

リフレッシュ機能：

func refresh(sender: AnyObject){{ 
    //exact same query and appending 
} 
    self.tableView.reloadData() 
    self.refreshControl?.endRefreshing() 
    } 

クエリおよび添付は非常に長いので、私はそれを処理するためのより良い方法をしたいと思います単にviewdidload（）からリフレッシュ関数にコードをコピーして貼り付け、.endRefreshing（）を追加するよりも簡単です。これを行う最も効果的な方法は何ですか？

これはすべてtableviewcontrollerで行われます。

Answer 1

私のコメントで説明したように、共通コードを完成ハンドラを持つ別のfuncに引き出します。私は、プロトコル指向のように

override func viewDidLoad() { 
    super.viewDidLoad() 
    queryAndAppend(completion: { 
     self.tableView.reloadData() 
    }) 
    refreshControl = UIRefreshControl() 
    refreshControl!.attributedTitle = NSAttributedString(string: "Pull to refresh") 
    refreshControl!.addTarget(self, action: #selector(DisplayEventsTVC.refresh(_:)), forControlEvents: UIControlEvents.ValueChanged) 
} 

func refresh(sender: AnyObject) { 
    queryAndAppend(completion: { 
     self.tableView.reloadData() 
     self.refreshControl?.endRefreshing() 
    }) 
} 

func queryAndAppend(completion completion: (() -> Void)?) { 
    // your existing code goes here 
    if let completion = completion { 
     completion() 
    } 
} 

Answer 2

...の線に沿って何かを考えている：

class ViewController: UIViewController, Refreshable { 
    var refreshTitle: String = "Pull to refresh" 
    override func viewDidLoad() { 
     super.viewDidLoad() 
     queryAndAppend(completion: { 
      self.tableView.reloadData() 
     }) 
     self.setupRefresh()//I don't want to name it 
    } 
} 

//Somewhere else 
protocol Refreshable { 
    var refreshTitle: String {get} 
    var tableView: UITableView 
    var refreshControl: UIRefreshControl 
} 

extension Refreshable where Self: UIViewController { 
func setupRefresh(){ 
refreshControl = UIRefreshControl() 
    refreshControl!.attributedTitle = NSAttributedString(string: "Pull to refresh") 
    refreshControl!.addTarget(self, action: #selector(DisplayEventsTVC.refresh(_:)), forControlEvents: UIControlEvents.ValueChanged) 
} 

//copying from @Michael ans 
func refresh(sender: AnyObject) { 
    queryAndAppend(completion: { 
     self.tableView.reloadData() 
     self.refreshControl?.endRefreshing() 
    }) 
} 

func queryAndAppend(completion completion: (() -> Void)?) { 
    // your existing code goes here 
    if let completion = completion { 
     completion() 
    } 
} 

}