Реализация кастомного UI-элемента для выбора времени. Часть 1

[storoj]

* в этом NotificationCenter захват self ни к чему плохому не приводит
* неправильное отписывание от NotificationCenter. нужно отписывать то, что вернул addObserverForName:
* «блоки неявно захватывают self» – здесь как раз всё явно. неявно это когда например захватывается ivar
* addLabelsWithNumber слишком жёсткий. что если кто-то добавит в contentView какую-то более полезную UILabel? она удалится?

[haron1020]

> что если кто-то добавит в `contentView` какую-то более полезную `UILabel`? она удалится?

Нет, не удалится. На самом деле эта проверка избыточна. Этот метод не трогает `subviews` у `contentView`, поэтому ничего не случится с иерархией `contentView`.



А все, что связано с блоками действительно требует моей доработки, спасибо.

[Kirpa]

«в этом NotificationCenter захват self ни к чему плохому не приводит»

Разве? Блок содержит strong reference на объект, который является подписчиком. Он же не отпишится, пока его не освободят -> retain circle

[Kirpa]

UIScrollView — не лишний? Может лучше было обойтись распознованием жестов?

Не очень понял, зачем там преобразование weak reference в strong reference и как это вообще должно работать.

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

[K-o-D-e-N]

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