Это плохое использование eval()

Question

Я немного застреваю, пытаясь разобраться, как обойтись с помощью eval(). В основном нужно использовать строку ниже в моем коде, однако наличие кавычек, кажется, гадость это и так мой путь вокруг него было, чтобы Eval мой оригинальный вар ...

$("input").click(function(event) { 
    layerClicked = event.target.value; 
    layerClicked = eval(layerClicked); 

     if (map.hasLayer(layerClicked)) { 
      map.removeLayer(layerClicked); 
     } 
     else{ 
      map.addLayer(layerClicked); 
     } ; 
}); 

Является ли это законное и приемлемое использование eval или я пропустил что-то более основное?

Заранее спасибо, функция Роуон

Answer 1

Eval() оценивает или выполняет аргумент.

Если аргумент является выражением, eval() оценивает выражение. Если аргумент является одним или несколькими операторами JavaScript, eval() выполняет инструкции.

Я не знаю, что вы пытаетесь достичь, но так, что layerclicked, кажется, не математическое выражение, ни ломоть Javascript заявление, вы делаете это неправильно ...

Answer 2

Я предполагаю, что вы пытаясь получить доступ к глобальному значению переменной при нажатии на ввод. Альтернативой этому было бы размещение всех этих значений, которые вы хотите получить в объекте. Таким образом, вы можете использовать обозначение скобки для доступа к ним, и eval не понадобится.

var layers = { 
    a: 'a value', 
    b: 'b value', 
    c: 'c value' 
}; 

$("input").click(function(event) { 
    var layerClicked = event.target.value; 
    alert(layers[layerClicked]); 
}); 

FIDDLE:http://jsfiddle.net/rqo6tcy4/