javascript - eval を使用して文字列を関数に変換するのは悪いことですか?

Question

eval() について多くの異なる意見を聞いたことがありますが、このコンテキストで eval() を使用してもよいかどうか少し確信が持てません:

次のようなオブジェクトがあるとします。

var bla = {
blubb: function (callback) {
       //GET some stuff via ajax
       //call the callback
    }
}

そして、次のような文字列：

 var someString = "bla.blubb";

関数 (およびコールバック) を呼び出すために文字列を評価するのは悪いことですか?

var callMe = eval(someString)
callMe(function(){
   alert('yay')!
});

score 5 · Accepted Answer

私は eval() について多くの異なる意見を聞いたことがあります

eval一般的に悪ではなく、用途があります。

このコンテキストで eval() を使用してもよいかどうか不明

いいえ、これは問題ありません。単純なブラケット表記メンバー operatorを使用した、はるかに優れたソリューションがあります。これは SO で毎日尋ねられるので、ここに何百もの重複をリストすることができます (最初のいくつかの Google 検索結果のみ):

score 1 · Accepted Answer

あなたのデザイン全体が完全に悪いわけではないとしましょう (しかし少しだけ)。

これは、にあるものを制限して指定することを意味しますsomeString。たとえば、オブジェクトへのパスとその関数を、引数なしで呼び出して (危険性を大幅に軽減します)、グローバルオブジェクトのコンテキストで呼び出すことができます。

次に、文字列を解析して関数を呼び出すのは簡単evalです。これはより安全です。例えば：

window.a = {b:{c:function(){console.log('here')}}};
var someString = "a.b.c";

var path = someString.split('.');
var f = window;
for (var i=0; i<path.length; i++) f = f[path[i]];
f.call(null);

改善は、ルートオブジェクト (ウィンドウではなく) を修正して、あらゆる種類の呼び出しを回避することです。

score 0 · Accepted Answer

デバッグが難しいので、これもまた悪です。より良いアプローチは、次のようなものを使用することです。

function getValue(namespace, parent) {
    var parts = namespace.split('.'),
        current = parent || window;
    for (var i = 0; i < parts.length; i += 1) {
        if (current[parts[i]]) {
            current = current[parts[i]];
        } else {
          if (i >= parts.length - 1)
            return undefined;
        }
    }
    return current;
}
var foo = {
    bar: {
        baz: function () { alert('baz'); }
    }
};
var f = getValue('foo.bar.baz'); //true
if (typeof f === 'function') {
    f();
}

コードは 1 つではありeval('foo.bar.baz')ませんが、実際に何が起こっているのかを把握し、ブレークポイントを使用してデバッグすることもできます。

ここでパフォーマンスについて話す場合、いくつかの jsperf の例を示します。

score 0 · Accepted Answer

オブジェクトがどのように見えるかがわかっている場合は、javascript で「安全な評価」を使用できることを忘れないでください。

スコープ内:

this.foo = { bar:function(){return "baz"}};

// some code.
var obj = "foo";
var param = "bar";

this[obj][param](); // same as this.foo.bar();

特定のもの (たとえば、テンプレート化など) を扱っていない限り、オブジェクトを呼び出すために eval を使用する必要はありません。

javascript - eval を使用して文字列を関数に変換するのは悪いことですか?

4 に答える 4

Related

Reference