Wie kann die Code-Redundanz in Swift reduziert werden?

Question

Ich kodiere eine App in Swift, die das Auffüllen einer Tabelle basierend auf der Iteration von Informationen aus Arrays beinhaltet, die nach einer Datenbankabfrage angehängt werden. Ich habe die Abfrage, das Anfügen und die tableView.reloadData() alle in der viewDidLoad() Methode. Es funktioniert gut. Ich habe auch fast genau den gleichen Code in eine Refresh-Funktion geschrieben, die mit einem Zug auf dem Bildschirm reagiert. Es funktioniert auch gut. Das Problem ist jedoch, dass es sehr beschäftigt ist. Es gibt viel Code geschrieben zweimal und ich frage mich, was der beste Weg ist, das zu beheben. Hier ist der relevante Code:

im 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) 
} 

Die Refresh-Funktion:

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

Die Abfrage und Anfügen ist sehr lang, so würde ich einen besseren Weg, wie zu handhaben, dass als einfach den Code aus der viewdidload() kopieren und in die Refresh-Funktion einfügen und ein .endRefreshing() hinzufügen. Was ist der beste/effizienteste Weg, dies zu tun?

Dies ist alles in einem Tableviewcontroller getan.

Answer 1

Wie in meinem Kommentar besprochen, ziehen Sie den gemeinsamen Code in eine separate Funktion, die einen Abschluss-Handler hat. Ich bin etwas entlang der Linien des Denkens ...

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

In Protokoll orientiert:

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() 
    } 
} 

} 

einfach einen Entwurf Schnipsel, getestet nicht