3

我对 JavaScript 还是很陌生（虽然不是编码），所以请随意为我挑选和白痴映射一些东西。

我试图创建一些可以接受用户输入的东西。如果第一个字符是感叹号，它将尝试使用该名称创建一个对象并运行该对象的“action”方法。否则它会像普通文本一样对待它（现在是一个警报）

<script type="text/javascript">
function GetInput(input){

    // Trim the spaces off the beginning and end
    input = input.trim();

    if(input.charAt(0) != "!"){
        // Just some normal text
        alert(input);

        return true;
    }

/* Cut off the exclamation point, replace multiple spaces with one,
 * and get the arguments if any. args[0] = the command. */

    var args = input.substr(1).replace(/\s{2,}/g, " ").split(" ");

// Make sure the function is an object with a method named "action"
    if(eval("typeof "+args[0]+";") === "function"
        && eval("typeof "+args[0]+".prototype.action;") === "function"){

        eval("var command = new "+args[0]+"();");

        command.action(args);
    }else{
        alert('"'+args[0]+'" is not a command.');
    }

    return true;
}
</script>

到目前为止，我注意到的唯一问题是 eval 语句。我知道我可以同时使用 switch/case 并抛弃 eval，甚至可以创建一个包含允许函数名称的数组，并在 eval 之前将输入与该数组进行比较，但我确信一定有更好的方法.

我只想能够制作对象和方法而不更新任何东西（我相信这是鸭子打字的主要用途之一？）。没有评估这可能吗？如果没有，是否有一种简单的方法来清理字符串的输入以避免诸如“！eval(alert('u b haxed'))”或“！a;alert('u b haxed')”之类的事情？

提前致谢

4

1 回答 1

3

您应该eval只使用一次来获取函数，然后在变量中使用它执行所有操作。

var args = input.substr(1).split(/\s+/);
var fn = eval(args[0]);
if (typeof fn == 'function' && typeof fn.prototype.action == 'function') {
    var command = new fn();
    command.action(args);
} else {
    alert('"'+args[0]+'" could not be evaluated to a valid command.');
}

return true;

如果这些构造函数是全局变量，您也可以将它们作为window对象的属性进行访问：

var fn = window[ args[0] ];
于 2012-12-03T01:23:31.250 回答