2

我有这样的代码：

var ws2812 = {};


ws2812.set = function(r,g,b){
    $.get( "/light?r="+r+"&g="+g+"&b="+b, function( data ) {
        console.log("Light set to:"+"/light?r="+r+"&g="+g+"&b="+b);
    })
};

ws2812.speech = function(word){
    switch(word){
        case "czerwone":
            this.set(255,0,0);
        break;
        case "zielone":
            this.set(0,255,0);
            break;
        case "niebieskie":
            this.set(0,0,255);
            break;
        case "białe":
            this.set(255,255,255);
            break;
        default:
            this.set(0,0,0);
            break;

    }
}

在控制台内运行时ws2812.speech("");，一切正常。但是，当与库配对时Annyang，我得到了这个：

Uncaught TypeError: this.set is not a function 
ws2812.speech @ script.js:29  
b.annyang.init.d.onresult @ annyang.min.js:6

怎么了？

[编辑]

命令是这样添加的：

annyang.addCommands({"ustaw *term światło":ws2812.speech});

具体来说，在 annyang 内部，这条线失败了：

f[h].callback.apply(this,m)

替换this为ws2812解决此问题的唯一方法吗？

4

1 回答 1

2

ws2812.speec 被定义为一个静态函数。所以this它里面的关键字是指它自己（函数范围），而不是你想要的对象ws2812 。

要修复它，可以做出以下任一快速选择：

选择#1 > 正确调用静态函数 ws2812.set

所以你的代码变成：

ws2812.speech = function(word){
    switch(word){
    case "czerwone":
        ws2812.set(255,0,0);
    break;
    case "zielone":
        ws2812.set(0,255,0);
        break;
    case "niebieskie":
        ws2812.set(0,0,255);
        break;
    case "białe":
        ws2812.set(255,255,255);
        break;
    default:
        ws2812.set(0,0,0);
        break;

    }
}

但是，this在代码的其余部分中其他部分引用的关键字也可能会遇到此问题。你可能需要去检查一下。

选择#2 > 转换为原型函数

这样您可以保留this关键字，但功能不再是静态的。您需要实例化一个ws2812要使用的实例对象。

所以你的声明变成：

var ws2812 = function(){};


ws2812.prototype.set = function(r,g,b){
    $.get( "/light?r="+r+"&g="+g+"&b="+b, function( data ) {
        console.log("Light set to:"+"/light?r="+r+"&g="+g+"&b="+b);
    })
};

ws2812.prototype.speech = function(word){
    switch(word){
    case "czerwone":
        this.set(255,0,0);
    break;
    case "zielone":
        this.set(0,255,0);
        break;
    case "niebieskie":
        this.set(0,0,255);
        break;
    case "białe":
        this.set(255,255,255);
        break;
    default:
        this.set(0,0,0);
        break;

    }
}

然后通过对象实例来使用它：

var myWs2812 = new ws2812();
myWs2812.speech('hello world!'); // inside it, should call this.set properly

选择#3 > 调用时绑定“this”对象

如果您坚持不想修改 this 的实现ws2812。使用它时可以留下它并绑定这个对象。

所以当你调用时ws2812.speech，你需要使用function.prototype.call并传入ws2812.set这个。

ws2812.call( ws2812.set, 'hello world!' );

但是，这看起来没有太多语义，并且可能会在将来维护此代码的人的使用中引起混淆。

我会留给您决定哪种方式最适合您。

于 2015-06-14T17:07:34.237 回答