Как изменить UILabel при скролле UIScrollView внутри UITableViewCell?

Question

[zzasdee]

Суть вопроса: есть UITableView. Внутри неё кастомные ячейки класса MyTabbleViewCell. Внутри ячейки лежит UIScrollView (который скроллится горизонтально) и UILabel.
MyTabbleViewCell
|- ContentView
|--UIScrollView
|--UILabel

Мне надо при скролле скролл вью изменять значения в лейбле.
Я сделал свой MyScrollView. В нём:
@property (strong, nonatomic) id parent;

Во время создания ячеки

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath

я выставляю
cell.scrollView.parent = cell;

Теперь могу делать так:

- (void)scrollViewDidScroll:(UIScrollView *)scrollView
{
    if ([scrollView isKindOfClass:[MyScrollView class]])
    {
        if (scrollView.parent)
        {
            MyTableViewCell *cell = scrollView.parent;
            cell.myLabel.text = ...
        }
    }
}

Также можно сохранять в MyScrollView indexPath ячейки. Это не суть важно.
Вопрос: как-нибудь ещё более элегантно можно это сделать?

Answer 1

[Ан]

Странная ситуация, мой ответ исчез куда-то. Или же я по глупости нажал Предпросмотр вместо Опубликовать.

Ваш подход чертовски плох.
1) Циклическая ссылка вот тут cell.scrollView.parent = cell;
2) ScrollView лезит вверх по иерархии и чего-то меняет в дочернем элементе родителя. Очень плохая практика.

Что бы причесать все, нужно сделать примерно следующее.
1) Скорей всего прибить сабклас UIScrollView, если он ничего кроме ссылки на парента и обработчика делегата не делает.
2) ScrollView объявить как UIScrollView и убрать его из публичных свойств. Сделать его приватным, и проинициализировать где-то в awakeFromNib или init
self.scrollView.delegate = self;
Предварительно подтвердив интерфейс UIScrollViewDelegate для класса ячейки
3) Объявить метод scrollViewDidScroll внутри класса ячейки, выкинув эти тонны проверок.

- (void)scrollViewDidScroll:(DSScrollView *)scrollView
{
      //хотя может и добавить проверку на if (scrollView==self.scrollView)
      self.myLabel.text = ...
}

Answer 2

[zzasdee]

Что-то я погорячился со strong ссылкой. Week не увеличит retain count и цикла не будет.
@property (week, nonatomic) id parent;

А по сути то чем отличается установка делегата от моего варианта?

Comments

[Ан]

Я же написал. ScrollView лезит вверх по иерархии и чего-то меняет в дочернем элементе родителя. Очень плохая практика. Это как через жопу зубы вырывать. Ну и метод делегата. Сравинте своего монстра, с тем что в ответе.

[zzasdee]

- (void)scrollViewDidScroll:(DSScrollView *)scrollView
отработает на любой скролл, у кого установлен делегаетом self. В том числе и self.tableView, в котором у меня ячейки со скроллВьюхами.

Проверка if (scrollView==self.scrollView) не отработает, потому что я не создаю проперти для каждого скролла в каждой ячейке.

[Ан]

Я Вас все меньше понимаю.
>отработает на любой скролл, у кого установлен делегаетом self.
Понятие self слишком размазаное. В ответе выше подразумевается экземпляр ячейки. К ней self.tableView вообще не имеет никакого отношения и о своем перемещении не будет уведомлять.

>потому что я не создаю проперти для каждого скролла в каждой ячейке.
Вы сами же привели иерархию в вопросе
MyTabbleViewCell
|- ContentView
|--UIScrollView
|--UILabel

[zzasdee]

Я понял суть нашего спора. Мой
- (void)scrollViewDidScroll:(DSScrollView *)scrollView сейчас расположен в ViewController.m и я всё время думал, что вы говорите о том же, а вы рассказываете про MyTableViewCell.m. Понял вашу идею, она правда будет рациональнее в моём случае. Спасибо :)