индуизм

[scarbo]

Встала тут задача написать кастомный NumericStepper.
Открыл исходный код

/**
     *  Maximum value of the NumericStepper.
     *  The maximum can be any number, including a fractional value.
     *
     *  @default 10
     */
    public function get maximum():Number
    {
        return _maximum;
    }

    public function set maximum(value:Number):void
    {
        _maximum = value;
        
        // To validate the value as min/max/stepsize has changed.
        if (!valueChanged)
        {
            this.value = this.value;
            valueSet = false;
        }
        
        dispatchEvent(new Event("maximumChanged"));
    }

Это индуизм или все таки описка, а в swc все норм?Строка 693
Flex 3

[Psycho Tiger]

Отсюда не видно. value может быть сеттером и его захотели дёрнуть.

[scarbo]

сеттер конечно, но this.value = this.value, это берут данные из геттера и присваивают сеттеру

[Котяра]

сам так писал иногда.
например в сеттере у меня отсылалось оповещение об изменении.

public function someMethod(value)
{
// по каким то причинам надо присвоить именно аксцессору, например используется в дальнейших вычислениях
this._value=value;
doSmthWith_value();
// а тут вычисления закончились и можем спокойно вызвать диспатч изменения
this.value = this.value;
}

Криво немного, но зависит от контекста . Так что я бы не дал такому коду звание индусского.. не тянет.

[wvxvw]

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

[etc]

У меня такое тоже есть. Практическая цель — «передёрнуть» сеттер для валидации.

[fljot]

Сообщение от etc

У меня такое тоже есть. Практическая цель — «передёрнуть» сеттер для валидации.

А вы чего не делаете if (_prop == value) return; ?

[etc]

Сообщение от fljot

А вы чего не делаете if (_prop == value) return; ?

Делаю, конечно. История долгая, но суть примерна та же, что и в оп-посте: устанавливается максимум некоторого значения, потом происходит валидация этого значения путём «передёргивания» сеттера. В нём же стоит проверка:

if (value > this._maximum) value = this._maximum;
if (this._value == value) return;
...

Поэтому если изначально значение было больше, оно отвалидируется.

Неочевидный приём и за него надо давать по рукам, если есть кому
Ну и потом, иногда полезен для on-demand геттеров.

[alatar]

Неочевидный приём и за него надо давать по рукам, если есть кому

Видимо так и поступили, в четвертой ветке этот финт ушами переделали.
В третьей ветке это "решение" служило одной единственной цели, проверить, в конце-концов, value на валидность после изменения диапазона. Так что решение более чем неочевидное

[Silicium]

Я вот не так давно использовал в коде строчку:

simpleItemRenderer.data=simpleItemRenderer.data;

Сама по себе она выглядит странно, но в такой связке:

simpleItemRenderer.data.text="Text";
simpleItemRenderer.data=simpleItemRenderer.data;

смысл ее становится понятен)))))