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

    −118

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    override public function onRemove() : void {
    super.onRemove();
    var status : String = this.status;
    var index : int = questList.indexOf(this);
    var questList : Vector.<Quest> = Quest[status];

    questList.splice(index, 1);
}

    Судя по тому что проект не падает квесты не удаляются...

    Запостил: kyzi007, 04 Марта 2013

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

    • ava wvxvw 04.03.2013 22:41 # 0

      Может они в super.onRemove() сначала добавляются?
      Ответить

      • ava kyzi007 04.03.2013 23:07 # 0

        А, забыла сказать в классе нету поля questList. Поэтому когда берется индекс...

        Но в состояние злости меня привело Quest[status] и лишняя переменная.
        Как мне сказали я залезла в самые древние части кода которые бояться трогать уже.
        Ответить

        • ava wvxvw 04.03.2013 23:23 # 0

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

          • ava kyzi007 04.03.2013 23:35 # 0

            Что значит может оказаться? Все объявления переменных (в тч и в условиях и циклах) выставляются при компиляции в начало метода.
            А вот присваивание остается где было и переменная в 4 строчке - андефайнед .
            То что компилируется вполне себе логично.
            Ответить

            • ava wvxvw 04.03.2013 23:38 # 0

              К сожалению Адоби похерили старую базу багрепортов, так бы нашел. У меня получалось так, что когда объявлялись две зависимые друг от друга переменные, то одна из них, вне зависимости от того, которая физически объявлялась первой, становилась первой. И плеер падал с VerifyError. Хз, может найду...
              Ответить

            • ava wvxvw 04.03.2013 23:48 # 0

              Хех, нашел, все-таки: 
              https://bugs.adobe.com/jira/browse/ASC-4234
              Ответить

              • ava kyzi007 04.03.2013 23:50 # 0

                А, с super там тонко, и это тоже не баг насколько я знаю.
                Ответить

                • ava wvxvw 04.03.2013 23:57 # 0

                  А, сейчас вспомнил: В тикете по ссылке - баг о том, как забыли дописать опкод с информацией о строке, где была объявлена переменная, и поэтому когда ошибка случалась, то сообщение об обшибке показывало на неправильную строчку. Да... это не совсем то :) Но был где-то и случай с инициализацией в произвольном порядке... хм :|
                  Ответить

                  • ava kyzi007 05.03.2013 00:04 # 0

                    У меня когда то падало с сообщением о битой swf без указания строк естессно, из-за трайкетча в анонимной функции листнере. Нашла методом поиска льва в пустыне.
                    Ответить

              • ava bot 04.03.2013 23:57 # 0

                Лур придёт, я ему всё расскажу.
                Ответить

    • ava bot 04.03.2013 23:54 # 0

      А какая функция сильнее "final" или "override"?
      Ответить

    • ava Lure Of Chaos 05.03.2013 00:59 # 0

      судя по тому, что квесты прыгают от статуса к статусу, там еще полно багов и неочевидного поведения
      Ответить

      • ava kyzi007 05.03.2013 17:06 # 0

        Да.
        Цитата из моей истерики

        Для того чтобы узнать для какой локации квест

        берем все теги квеста
        для каждого пытаемся по айдишника взять локацию
        если получилось - эта локация и есть целевая для квеста
        если не получилось - целевая для квеста это дефолтная локация

        Для того чтобы перейти на локацию мы выдаем себе награду
        если в имени ресурса есть кусок строки GOTO_WORLD
        то надо разбить строку на две части, вторая - имя локации, на нее то нам и надо перейти
        Ответить

    • ava nonamez 05.03.2013 10:02 # +5

      FLASH не нужен.
      Ответить

      • ava Xom94ok 05.03.2013 18:40 # +5 

        ┌─────────────┐
│    FLASH    │
│   УБИВАЕТ   │
└─────────────┘

        один из popup-баннеров
        Ответить

    • ava guest 02.06.2017 08:50 # 0

      wh0cd679282 <a href=http://sildenafilcitrate24.us.org/>extra resources</a>
      Ответить

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