3

我正在尝试学习在 JavaScript 中进行面向对象的编程并严格违反 JSLint。我知道我在非全局环境中使用它（或类似的东西......），但我不知道如何正确地做到这一点。这是我的代码：

function piece(color, type, x, y, captured, hasMoved) {
    "use strict";
    this.color = color;
    this.type = type;
    this.x = x;
    this.y = y;
    this.captured = captured;
    this.hasMoved = hasMoved;

    this.movePiece = movePiece;
    function movePiece(x, y) {
        // if(isLegal(x, y, this.type){
            // this.x  =  x;
            // this.y  =  y;
        // }
         alert("you moved me!");
    }
}

var whitePawn1  =  piece("white", "pawn", 0, 1, false, false);
var blackBishop1  =  piece("black", "bishop", 8, 3, false, false);
4

4 回答 4

6

您需要将您的piece函数用作构造函数——换句话说，使用new关键字调用它。

就目前而言，this您的函数内部是全局对象。基本上，您不是创建一个新对象并向其添加属性，而是用垃圾来破坏全局对象。

由于您处于严格模式，this因此将未定义，因此您的代码将出错。

这就是你想要的：

function Piece(color, type, x, y, captured, hasMoved) {
    "use strict";
    this.color = color;
    this.type = type;
    //...

var whitePawn1  = new Piece("white", "pawn", 0, 1, false, false);
var blackBishop1  = new Piece("black", "bishop", 8, 3, false, false);

请注意，我重命名为piece以大写字母开头，因为按照惯例应该使用构造函数。

另外，假设像这样的构造函数确实是您想要的，与 Ian 的回答相比，您应该考虑将movePiece函数移出函数的原型，这样就不必每次创建函数时都重新创建一个新片。所以这

this.movePiece = movePiece;
function movePiece(x, y) {
   //...
}

会变成这样

//this goes **outside** of your function
Piece.prototype.movePiece = function movePiece(x, y) {
       //...
}
于 2013-04-12T19:47:44.923 回答
1

啊，谢谢亚当·拉基斯。做到了。作为参考，这是我的最终 JSLint 验证代码：

function Piece(color, type, x, y, captured, hasMoved) {
    "use strict";
    this.color = color;
    this.type = type;
    this.x = x;
    this.y = y;
    this.captured = captured;
    this.hasMoved = hasMoved;
}
Piece.prototype.movePiece = function movePiece(x, y) {
    "use strict";
    /*global alert */
    alert("you moved me to " + x + ", " + y + "!");
};
var whitePawn1  =  new Piece("white", "pawn", 0, 1, false, false);
var blackBishop1  =  new Piece("black", "bishop", 8, 3, false, false);
于 2013-04-12T20:16:54.957 回答
1

我不确定它在内部/功能上会有什么不同，但你可以使用：

function piece(color, type, x, y, captured, hasMoved) {
    "use strict";
    return {
        color: color,
        type: type,
        x: x,
        y: y,
        captured: captured,
        hasMoved: hasMoved
    };
}

var whitePawn1 = piece("white", "pawn", 0, 1, false, false);

不需要使用thisor new。

尽管我猜您不能使用 将.prototype共享属性/方法应用于所有实例。并且返回的对象初始化也是额外的。

于 2013-04-12T19:57:28.107 回答
0

在调用构造函数之前，您缺少new关键字。代码应如下所示：

var whitePawn1    = new piece("white", "pawn", 0, 1, false, false);
var blackBishop1  = new piece("black", "bishop", 8, 3, false, false);

在你的情况下，用大写字母命名构造函数也是一个很好的模式Piece

说明：没有new你的构造函数可能会破坏他们调用的环境。创建新环境并将其new绑定到构造函数。

于 2013-04-12T19:50:34.897 回答