Comments on: Динамическое создание обработчиков событий

By: dimzon541

dimzon541 — Wed, 21 Mar 2007 16:01:35 +0000

}}в общем, предлагаю сойтись на том, что eval() надо использовать аккуратно :)

Естественно :) Как любит говорить мой коллега про некоторые “фичи”:
Фича “какая-то фича” это острая бритва при прмощи которой при неправильном обращении можно отрезать себе яй#а

}}как я уже говорил в самом начале, все мои претензии к eval() связаны с тем, что я трепетно отношусь к security.

Это и понятно :) Я вот в своё время даже экзамены сдавал http://www.microsoft.com/learning/exams/70-340.mspx

By: Filosoff

Filosoff — Wed, 21 Mar 2007 15:54:03 +0000

отчасти согласен. но я очень против eval’а полученных данных. это потенциально несекьюрно. конечно, если это только для вывода, тогда ничего. но если потом данные в каком-то виде снова идут на сервер - это нехорошо.
в общем, предлагаю сойтись на том, что eval() надо использовать аккуратно :)

P.S. как я уже говорил в самом начале, все мои претензии к eval() связаны с тем, что я трепетно отношусь к security.

By: dimzon541

dimzon541 — Wed, 21 Mar 2007 15:48:34 +0000

Второе решение несколько замедляет ПЕРВЫЙ вызов но ускоряет последующие за счёт того что экономим на IF
Если вызов getSomeValue частый то общая экономия заметна. К тому-же избавляемся от глобальной переменной
Вообще в моей практике я достаточно часто использую eval чтобы генерировать некоторый код (причём бывает достаточно сложный) на лету.
Другое применение eval находит например для обыгрывания пользовательского интерфейса. К примеру есть таблица в которой надо отобразить записи, причём записи получаются с сервера (используем AJAX). Тогда например достаточно изящный способ - в заголовке таблицы доп-аттрибутами у ячейки я указываю выражение на JavaScript для форматирования. Например
[TD formatString="item.FirstName + ' ' + item.LastName + ' ' + item.MiddleName"]ФИО[/TD]
[TD formatString="item.Bithday"]День рождения[/TD]
Тогда при заполнении я использую универсальную функцию
function getTextForCell(item, formatString){return eval(formatString);}

By: Filosoff

Filosoff — Wed, 21 Mar 2007 15:21:56 +0000

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

By: dimzon541

dimzon541 — Wed, 21 Mar 2007 15:10:53 +0000

По поводу eval в JavaScript - если это работает на клиенте то в принципе дырок нет, тем более если ты знаешь что за строчку ты передаешь в eval
На самом деле eval бывает очень полезен. Он, к примеру, используется как основная фича в JSON
Иногда при помощи eval можно сделать очень элегантные вещи. Допустим есть некоторая операция getSomeValue() которая может выполняться долго но результат её выполнения можно “закэшировать”.

классический способ:
var getSomeValue_Cahce = 0;
function getSomeValue()
{
if(getSomeValue_Cahce==0)
getSomeValue_Cahce = getSomeValue_Actual();
return getSomeValue_Cahce;
}

использование eval()
function getSomeValue()
{ var n=getSomeValue_Actual();
eval(’function getSomeValue(){return ‘ + n + ‘;}’)
return n;
}

By: Filosoff

Filosoff — Wed, 21 Mar 2007 15:00:24 +0000

сначала по поводу приведенного варианта. он очень занятный. только понял я его со второго раза :). в общем-то, он тоже вполне жизнеспособен.

а про eval() - это, в общем, большая дырка. изначально это у меня пошло от PHP и Perl, где он медленный и может привести к различным проблемам с security. да и всегда можно обойтись без него.
в JavaScript он, в принципе, также может привести к проблемам. поэтому я не использую его вообще. просто перестраховка.

By: dimzon541

dimzon541 — Wed, 21 Mar 2007 14:48:00 +0000

Поясни если не сложно

Кстати, опять-таки если не нравится eval можно использовать такой код

function someFunction1(){alert(’someFunction1′);}
function someFunction2(){alert(’someFunction2′);}

var photo_larr_OnClick_PROXY = someFunction1;

function photo_larr_OnClick(){photo_larr_OnClick_PROXY();}

И соответственно когда её надо изменить можно позвать
photo_larr_OnClick_PROXY = someFunction2;

By: Filosoff

Filosoff — Wed, 21 Mar 2007 11:18:31 +0000

я испытываю большую неприязнь к eval() и стараюсь нигде и никогда его не использовать. если надо, могу пояснить почему.

By: dimzon541

dimzon541 — Wed, 21 Mar 2007 11:11:50 +0000

Есть более универсальный способ.
Изначально на этот photo_larr вешается в качетве обработчика функция назовём её photo_larr_OnClick()

потом когда нужно изменить реакцию делаем
eval(’function photo_larr_OnClick(){alert(”ПРЕВЕД!”);}’);

Таким образом мы глобально замещаем одну реализацию другой, причём всё в рамках стандарта javaScript безо всяких там обыгрываний DOM

By: Filosoff

Filosoff — Wed, 21 Mar 2007 07:49:56 +0000

я уже несколько раз с этим разбирался. каждый раз успешно забывал, к чему привели разбирательства :). сейчас вот решил, наконец, записать.