1. ActionScript / Говнокод #14539

    −122

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    protected override function getCurrentSkinState():String
{
	if (!this._currentState) this._currentState = "edit";
	else this._currentState = super.getCurrentSkinState() || "edit";
	return this._currentState;
}

    Еще грабли от Адоба.

    Запостил: wvxvw, 11 Февраля 2014

    Комментарии (20) RSS

    • ava kyzi007 11.02.2014 14:51 # 0

      Для полноты счастья надо сделать _currentState сеттером и чтобы обязательно в нем еще одна проверка с модификацией была.
      Ответить

      • ava Lure Of Chaos 12.02.2014 06:37 # 0

        в сеттере нужна модификация до super.getCurrentSkinState(), если это "edit"
        Ответить

    • ava Vasiliy 11.02.2014 15:41 # 0

      а АS в тенаринк не умеет ? к чему этот if
      Ответить

      • ava bormand 11.02.2014 15:46 # 0

        Да этот код вроде бы вообще можно упростить до:
        return super.getCurrentSkinState() || "edit"
        Ответить

        • ava bormand 11.02.2014 15:50 # 0

          Хотя нет, не эквивалентно. Вариант из топика первом вызове (если изначально в __currentState лежал null) возвращает тупо "edit", а при последующих - унаследованное значение, или "edit", если родитель вернул null...

          P.S. Что за хрень здесь вообще творится?!
          Ответить

          • ava Vasiliy 11.02.2014 15:55 # 0

            Автор не умеет в чистые функции.
            Ответить

            • ava roman-kashitsyn 11.02.2014 16:01 # +2

              Напомнило http://tinyurl.com/syadiw-tshort
              Ответить

            • ava wvxvw 11.02.2014 19:02 # +3

              На самом деле все гораздо хуже... стейты объявляются в МХМЛ, и время их инициализации неизвестно, поэтому:
              - конструктор недоступен.
              - неизвестно проинициализировался ли компонент, или нет.
              - на счет чистых функций, тут про чистоту нужно думать не в академическом, а в более приземленном смысле. Как ни открою адобовские исходники, так и хочется руки вымыть.
              Весч вообще никакой отладке не поддается, бряк поставить некуда, логгер - аналогично, негде. Сидишь вот перед монитором, и, практически, играешь в лотерею: скомпилировалось и... нет, сука, не работает, ну давай, еще тут че-та поменяем...
              Ответить

          • ava kegdan 11.02.2014 17:07 # 0

            присвоение нужно было в инициализацию вынести и делать как ты сказал

            На самом деле хуйня же - флаг храниться текстом, дефолтный флаг вшит в проверку исполнения. По идеи тут должен стоять жирный ексепшен.

            Если нет state - пшел нахуй из моего ооп
            Ответить

            • ava Vasiliy 11.02.2014 17:24 # 0

              зачем исключение. Ну нет значит не установили вернуть null, пусть кому надо разбирается. Эксепшен там или установить значение.

              может же быть вполне не винный код типа if (!this->getState()) this->setState('Дефолтное значение');

              я так в пхп делаю потому что в определении переменных класса нельзя выражения.
              Ответить

              • ava kegdan 11.02.2014 17:52 # 0

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

                Хотя с другой стороны всего говна в коде не видно, хз че там происходит
                Ответить

                • ava Vasiliy 11.02.2014 18:34 # 0

                  ну в кобобоксе должно же быть что то выбрано. вот и дефолтное значение.
                  Ответить

                  • ava kegdan 11.02.2014 19:13 # 0

                    Ну так там и так что то выбрано. Оно должно из комбобокса приходить, значение, а не синхронизироваться 1000 затычек.
                    Ответить

                    • ava Vasiliy 11.02.2014 20:16 # 0

                      Я не говор. про 1000 одной хватит. Но по хорошему надо в конструкторе.
                      Ответить

                  • ava 3Doomer 18.02.2014 15:40 # 0

                    "в кокобоксе"
                    >FIXED
                    Ответить

    • ava Konardo 12.02.2014 16:11 # −10

      показать все, что скрытоХуй.
      Ответить

    Добавить комментарий