Как исправить memory leak in IE при удалении и повторном создании переменной?

Question

[Genome_X]

Собственно, вопрос.

Использую jQuery, есть код вида:

$(&quot;id&quot;).click(function() {<br/>
 if (&quot;perem1&quot; in window && &quot;perem2&quot; in window) {<br/>
 #someaction;<br/>
 delete perem1;<br/>
 delete perem2;<br/>
 } else {<br/>
 perem1= somevalue;<br/>
 perem2= somevalue;<br/>
 };<br/>

Т.е. смысл такой: скрипт проверяет наличие глобальных переменных perem1 и perem2, если не находит их, то выполняет условие после else, т.е. создает глобальные переменные и присваивает им некоторые значения. При повторном вызове функции условие дает TRUE, выполняется некий код #someaction где участвуют значения этих переменных, затем переменные удаляются.



В браузерах FF, Opera все работает корректно. Но вот в IE (8-й версии в частности), код выполняется ровно один раз, при повторной активации выдает ошибку memory leak и соответственно ничего не делает.

Как я понял, какая-то проблема с удалением переменных, в остальных браузерах она работает корректно, тогда как в IE вызывает какие-то проблемы, из-за чего код срабатывает только раз.



Вопрос — как победить?

Возможно стоит как-то переписать код, чтобы избежать этого delete, но пока сам не знаю как.

Answer 1

[monolithed]

Переменные объявленные без спецификатора var становятся свойствами глобального объекта window (в серверной реализации global).
К таким переменным допустимо применение инструкции delete:

property = 1;

top.property;    //1
self.property;   //1
window.property; //1

Т.е. инструкция delete работает только со свойствами объектов.
Например, допустимо удалять свойства встроенных объектов:

Math.abs;        // function abs() { [native code] }
delete Math.abs; // true
Math.abs;        // undefined

Соответственно, переменные объявленные со спецификатором var удалить нельзя:

var variable = 10;
delete variable; // false
variable;        // 10

PS: в консолях типа FF, Chrome, результат работы инструкции delete может быть не корректный.

Answer 2

[Олег Матрозов]

А как вы объявляете глобальные переменные?
Судя по справочникам, есть большая разница между объявлениями:

var a = 2;

и

a = 2;

В первом случае созданная переменная будет являться свойством класса window и напрямую такая переменная удалена быть не может (по крайней мере об этом твердят некоторые справочники). Т.е. вполне может быть, что многие современные браузеры игнорируют эту разницу и удаляют в обоих случаях, а более старые IE (еще видел упоминание про FF 1.5) получают проблемы.

Comments

[Genome_X]

Прошу прощения, некорректно исходник привел, сейчас он корректен и текст вопроса также исправил.
По поводу объявления, на сколько я знаю, выражение perem1= somevalue; и есть объявление глобальной переменной. Выражение var perem1= somevalue; есть уже объявление локальной переменной.
В моем выражении после первого клика выполняется код идущий после ELSE, т.к. переменных IF еще не видит. В коде после ELSE создаются эти перемнные и при повторном клике IF уже видит эти переменные и выполняет код идущий ДО ELSE. После выполнения этого кода он удаляет эти переменные, таким образом при третьем клике снова создаются эти переменные в коде после ELSE. И так по кругу.
Плюс, как я говорил, во всех браузерах кроме IE все работает корректно.

[Виталий Желтяков]

Браузеры IE и Chrome (даже последние версии) не игнорируют эту разницу. Mear, правильно указал на ошибку.

Здесь лучше не удалять переменную, а менять значение. Т.е. изначально предопределяете значение и потом производите сравнение с ним.

[Олег Матрозов]

Хм… у меня вот такой код работает как в последнем хроме, так и в IE 6 & 8:

    <script type="text/javascript">
      function bla()
      {
        if (typeof(myvar) == 'undefined')
        {
          myvar = "bla";
          document.getElementById('bla').innerHTML = 'set';
        }
        else
        {
          delete myvar;
          document.getElementById('bla').innerHTML = 'delete';
        }
      }
    </script>
    <div id="bla" onclick="bla();">click here</div>

Без всяких ошибок, корректно чередуя «set» и «delete»

[Виталий Желтяков]

Код может/будет работать, но память будет потихоньку течь. В Chrome незаметно, а IE достаточно сильно. Для этих двух браузеров необходимо использовать предопределение переменных.

[Олег Матрозов]

В данном случае мне было интересно самому натолкнуться на эту ошибку… я её не увидел. Т.е. браузер меня не остановил.
Хотя, конечно, я с Вами полностью согласен в вопросе «объявить глобальную переменную и хоть обnullять её». Опять же, в чем проблема использовать отдельную глобальную переменную для передачи состояния (true/false do_create/do_delete) (если уж мы допускаем, что в указанные переменные perem1 и perem2 может быть записано что угодно, включая undefined и null), я не понимаю.

Answer 3

[Genome_X]

Хм, спасибо за подсказку и новодки, попробую действитульео предопределить, а затем не удалять, а менять значения.
Но, тем не менее, если просто объявляется переменная выражением типа someval = 4; (к примеру), её уже не удалить? Для чего же существует тогда delete?

Comments

[Виталий Желтяков]

someval = 4; — это неправильное обявление переменной, хотя и допустимое. От сюда и все проблемы.

[Genome_X]

А как будет корректно её объявить в куске кода после ELSE, чтобы эта переменная была видна в куске кода ДО ELSE?

[Виталий Желтяков]

В вашем случаи предопределить её глобально, вне функции: var myvar = «bla»;

Answer 4

[Genome_X]

Так, только что проверил, в Хроме последней версии все также корректно работает. Т.е. работает во всех браузерах, кроме IE (на 9-ке не проверял кстати пока).
Как-то странно, что глобальную переменную нельзя удалить. Получается по логике все браузеры работают некорректно и только IE выдает ошибку потому что знает как правильно? Как-то это на сомнения наталкивает.

Comments

[Виталий Желтяков]

Хром просто не отображает ошибку, а память в нём всё равно течёт.

Правильно работают Опера и ФФ, т.к. у них есть автоматическая проверка существования переменной и поэтому в них не нужно использовать предопределение переменных. А Хром и ИЕ на этой проверке экономят.

Answer 5

[gaelpa]

Не считая того, что лучше такие переменные видимо не удалять а переводить в undefined (и не глобальные, а, например, в замыкании), попробуйте создавать их и удалять с полным указанием «пути», т.е. delete window.myvar1.

Comments

[Genome_X]

Пробовал, не работает. По всякому уже вертел.

Answer 6

[Genome_X]

Ааааааа, ну классс… :(
В общем, работает в IE9-й версии. не работает в IE более младших версиях (8-й и ниже).
Посмотрел под отладчиком, пишет «Object doesn't support this property or method». Ругается на строчку после «if (»perem1" in window && «perem2» in window)", где у меня есть некоторые манипуляции с переменными.
Блин. Засада.

Мысль, кстати, была неплохая, поиграться не удалением, а установлением значения переменной в null.
Работает, в принципе, только пока не могу понять как условие изменить условие («perem1» in window && «perem2» in window) для первого и последующих кликов.