Использование делегата для передачи var

Question

Я вытаскиваю свои волосы, пытаясь заставить эту делегацию работать в Swift для приложения, над которым я работаю.

У меня есть два файла: CreateEvent.swift и ContactSelection.swift, где первый звонит последнему.

CreateEvent 'содержание s являются:

class CreateEventViewController: UIViewController, ContactSelectionDelegate { 

    /... 

    var contactSelection: ContactSelectionViewController = ContactSelectionViewController() 

    override func viewDidLoad() { 
     super.viewDidLoad() 

     /... 

     contactSelection.delegate = self 
    } 


    func updateInvitedUsers() { 
     println("this finally worked") 
    } 

    func inviteButton(sender: AnyObject){ 
     invitedLabel.text = "Invite" 
     invitedLabel.hidden = false 
     toContactSelection() 
    } 

    /... 

    func toContactSelection() { 
     let contactSelection = self.storyboard?.instantiateViewControllerWithIdentifier("ContactSelectionViewController") as ContactSelectionViewController 
     contactSelection.delegate = self 
     self.navigationController?.pushViewController(contactSelection, animated: true) 
    } 

ContactSelection' s содержание:

protocol ContactSelectionDelegate { 
    func updateInvitedUsers() 
} 

class ContactSelectionViewController: UITableViewController { 

    var delegate: ContactSelectionDelegate? 

    override func viewDidLoad() { 
     super.viewDidLoad() 

     self.delegate?.updateInvitedUsers() 

    } 

    override func tableView(tableView: UITableView, didSelectRowAtIndexPath indexPath: NSIndexPath) { 

    // Stuff 

    self.delegate?.updateInvitedUsers() 

    } 
} 

Что я делаю неправильно? Я все еще новичок и не совсем понимаю эту тему, но после очистки Интернета я не могу найти ответа. Я использую кнопку Back, доступную в панели навигации, чтобы вернуться к моему CreateEvent.

Answer 1

var contactSelection: ContactSelectionViewController = ContactSelectionViewController() 

Это экземпляр контроллера представления напрямую, и значение никогда не используется. Поскольку похоже, что вы используете раскадровки, это не очень хорошая идея, так как ни одна из точек не будет подключена, и вы получите дополнительные отказоустойчивые сбои. Вы устанавливаете делегата этого контроллера вида, но это не имеет значения, поскольку оно не используется.

Это также не очень хорошая идея, потому что если вы выполняете несколько нажатий, вы будете повторно использовать один и тот же контроллер представления, и это в конечном итоге приведет к ошибкам, так как вы будете иметь оставшееся состояние из предыдущих применений, что может дать неожиданные результаты , Лучше каждый раз создавать новый контроллер просмотра.

В вашем коде вы создаете совершенно новый contactSelection из раскадровки и нажимаете его, не устанавливая делегата.

Вам необходимо установить делегат на экземпляр, который вы нажимаете на стек навигации.

Также полезно передать ссылку в методе делегата, которая может использоваться для извлечения значений, вместо того, чтобы полагаться на отдельную ссылку в var, как вы это делаете.

Итак, я хотел бы сделать следующее:

• Снимите var contactSelection
• Добавить делегат перед отправкой нового contactSelection объекта

• Изменить метод делегата подписи к этому:

  protocol ContactSelectionDelegate { 
      func updateInvitedUsers(contactSelection:ContactSelectionViewController) 
  } 
  

• Измените свои делегированные вызовы на это:

  self.delegate?.updateInvitedUsers(self)