JSLint 严格违规。面向对象的 Javascript 挫折

Posted 2023-02-22

【中文标题】JSLint 严格违规。面向对象的 Javascript 挫折

【英文标题】：JSLint Strict Violation. Object Oriented Javascript frustrations

【发布时间】：2013-04-05 11:19:26

【问题描述】：

我正在尝试学习在 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);

【问题讨论】：

具体的jslint消息是什么？

【参考方案1】：

您需要将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) 
       //...

【讨论】：

我认为在严格模式下，this 是未定义的，除非您通过对象调用该函数。 （即：默认情况下它不再是全局对象。）

@cHao - 是的，但是，我想我说得对，函数必须在严格模式下定义。如果你只是在函数内部说“使用严格”，那么这仍然是像往常一样的窗口。

哦，它必须在严格模式脚本中？

@cHao - gah，刚刚又测试了一遍。我错了。更新我的答案。

【参考方案2】：

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

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);

不需要使用this 或new。

虽然我猜您不能使用.prototype 将共享属性/方法应用于所有实例。而且返回的对象初始化也是多余的。

【参考方案3】：

啊，谢谢亚当·拉基斯。做到了。作为参考，这是我最终的 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);

【参考方案4】：

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

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

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

说明：如果没有new，您的构造函数可能会破坏它们被调用的环境。 new 创建新环境并将其绑定到构造函数。