Pull to refresh

Comments 5

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

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



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

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

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

addLabelsWithNumber — не очень. Во-первых, лучше всё-таки деражать массив со своими лейблами и удалять по нему. Во-вторых, он делает больше чем добавление и из названия этого не следует. Лучше разделить на два разных метода или подбрать более точное название
Вообще непонятно зачем отслеживать поворот экрана?
Если вы предполагаете изменение размера контрола, то отслеживать размер нужно всегда.
Лейблы надо конечно переиспользовать.
Only those users with full accounts are able to leave comments. Log in, please.