Как правильно инстанцировать и хранить ViewController-ы в iOS?

Question

[YellowCataclysm]

Инстанцирование ViewController-ов из storyboard осуществляется методом instantiateViewControllerWithIdentifier, после чего созданный VC инициализируют и "показывают".
Считается ли нормальной практикой хранить полученную strong-ссылку на VC в вызывающем классе, чтобы создаваемый VC не деаллоцировался сразу после его закрытия и его можно было отображать по необходимости, минуя стадию создания и инициализации? Или же правильным решением будет отпускать ссылку и при каждом обращении инстанцировать VC заново?

Answer 1

[Igor Cherny]

Не считается. Создание контроллера не такая уж затратная операция, а данные, которые ему необходимы, Вы конечно можете хранить. В противном случае есть опасность утечек. В вызывающем классе (или в NavigationController) есть системная ссылка на представленный контроллер, которая обнуляется конечно при удалении его с экрана.

Comments

[YellowCataclysm]

В таком случае становится возможным создание нескольких экземпляров одного и того же VC, ведь так? Как тогда поступить в ситуации, когда нужно гарантировать существование только одного VC (без дубликатов) ? Подойдет ли для этого хранение weak-ссылки?

[ManWithBear]

YellowCataclysm: если вам нужно гарантировать существование только 1 экземпляра конкретного контроллера, то вы делаете что-то не правильно.

[Igor Cherny]

Ни разу не встречал архитектуры, которая требовала бы какой либо ViewController как singleton , во вторых, если необходим синглтон, то это достигается другими средствами, уж ни как не strong ссылкой.

[Igor Cherny]

YellowCataclysm: если Вы хотите исключить создание более одного экземпляра класса, используйте паттерн Singleton, но это ни как не связано с weak или strong типами ссылок.

[YellowCataclysm]

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

Answer 2

[ManWithBear]

В более идеальном мире у вас для каждого контроллера будет свой xib, в котором будет собственно и происходить layout. Такой подход отлично решает проблему мержей сторибордов.
В итоге мы отказываемся от стринговых констант (viewControllerID). И тогда создание нового контроллера будет выглядеть вот так:
let vc = MyViewController()
Этот подход в свою очередь подталкивает нас к Dependency Injection.
В итоге получаем удобное:

let vc = MyViewController(dependency1: Dependency1, dependency2: Dependency2, ...)

Но если зависимостей больше 2-3, то их удобнее выводить в отдельный settings-класс/структуру:

let settings = MyViewController.Settings(<here init>)
let vc = MyViewController(with: settings)

В итоге смысла держать любую ссылку на контроллер нет.