Javascript - 原型函数数组

Question

我是一个javascript新手，所以到目前为止我写的丑陋代码有时候由于我缺乏经验以及它与我习惯的语言有多么不同，所以下面我要发布的代码有效，但我是想知道我是以正确的方式做到这一点，还是可行，但这是一种可怕的做法，或者有一种更好的方法。

基本上，我有一个在网格内移动的小家伙，他从服务器接收一个动作，他可以​​在8个方向移动（int）：0：向上，1：向上 - 向右，2：向右...... 7：向上-剩下。

服务器将向他发送此0 <= action <= 7值，并且他必须采取正确的操作...现在，而不是使用switch-case结构。 我创建了一个函数goUp（），goLeft（）等，并将它们加载到一个数组中，所以我有一个像这样的方法：

var getActionFunction = actions[action];
actionFunction();

但是，如何设置这一切是这样的：

1）创建构造函数：

function LittleDude(container) {
    this.element = container; //I will move a div around, i just save it in field here.
}

LittleDude.prototype.goUp() {
    //do go up
    this.element.animate(etc...);
}

LittleDude.prototype.actions = [LittleDude.prototype.goUp, LittleDude.prototype.goUpLeft, ...];
//In this array I can't use "this.goUp", because this points to the window object, as expected

LittleDude.prototype.doAction = function(action) {
    var actionFunction = this.actions[action];
    actionFunction(); //LOOK AT THIS LINE
}

现在，如果你注意，最后一行将无法工作..因为：当我使用索引访问数组时，它返回一个LittleDude.prototype.goUp例如...所以“this”关键字是未定义的..

goUp有一个语句“this.element”...但是“this”没有定义，所以我必须这样写：

actionFunction.call(this);

所以我的doAction将如下所示：

LittleDude.prototype.doAction = function(action) {
    var actionFunction = this.actions[action];
    actionFunction.call(this); //NOW IT WORKS
}

我需要知道这是否是hackish，或者我是否违反了某种“不要做这个”的规则。 或者也许它可以用更好的方式编写。 因为在我看来有点奇怪的是将它添加到原型中然后将其视为一个独立的功能。

Answer 1

你想要做的是一种可能的方法，但它可以使它更简单。 由于对象属性名称不是必需的字符串，因此可以直接在原型上使用动作索引。 你甚至不需要doAction功能。

LittleDude = function LittleDude(container) {
  this.container = container;
}

LittleDude.prototype[0] = LittleDude.prototype.goUp = function goUp() {
  console.log('goUp', this.container);
}

LittleDude.prototype[1] = LittleDude.prototype.goUpRight = function goUpRight() {
  console.log('goUpRight', this.container);
}

var littleDude = new LittleDude(123),
    action = 1;
littleDude[action](); // --> goUpRight 123
littleDude.goUp(); // --> goUp 123

Answer 2

 actionFunction.call(this); //NOW IT WORKS 

我需要知道这是否是hackish，或者我是否违反了某种“不要做这个”的规则。 或者也许它可以用更好的方式编写。

不，使用.call()完全可以绑定this关键字 - 这就是它的用途。

因为在我看来有点奇怪的是将它添加到原型中然后将其视为一个独立的功能。

如果你不直接使用它们，你不必在原型上定义它们:-)但是，如果你这样做，你可能不会将函数本身存储在数组中，但方法名称然后用括号表示法调用它们：

// or make that a local variable somewhere?
LittleDude.prototype.actions = ["goUp", "goUpLeft", …];

LittleDude.prototype.doAction = function(action) {
    var methodName = this.actions[action];
    this[methodName](); // calls the function in expected context as well
}