Question

我想创建一个负责生成PlayerList的工厂，但是我在访问我在initialize函数中设置的变量时遇到了问题。代码是

app.factory("PlayerList", function(){

    // Define the PlayerList function
    var PlayerList = function() {

        this.initialize = function() {
            // create an array for our players
            var players = [];
        };

        this.add = function(player) {
            this.players.push(player);
        }

        this.remove = function(player) {
            if ( players.length > 0 )
            {
                this.players.splice(players.indexOf(player), 1);    
            }
        }

        this.initialize();
    };

    return (PlayerList);

});

我想在添加和删除方法中引用player数组，但我还没有定义。

Answer 1

var playerList =（function（）{

var playerLists = {};

playerList.playerList = function（）{

    this.initialize = function() {
        // create an array for our players
        var players = [];
    };

    this.add = function(player) {
        this.players.push(player);
    }

    this.remove = function(player) {
        if ( players.length > 0 )
        {
            this.players.splice(players.indexOf(player), 1);    
        }
    }

    this.initialize();
};

return playerLists;

}）（）;

app.factory（＆＃34; PlayerList＆＃34;，playerList.playerList）;

Answer 2

此处var players = [];是initialize的本地变量，但您期望this.players.push(player);表示players应位于PlayerList范围内。

所以你的工厂应该看起来像

app.factory("PlayerList", function () {

    // Define the PlayerList function
    var PlayerList = function () {

        var self = this;

        this.players = [];

        this.initialize = function () {
            self.players = [];
        };

        this.add = function (player) {
            self.players.push(player);
            console.log(self.players);
        }

        this.remove = function (player) {
            if (self.players.length > 0) {
                self.players.splice(self.players.indexOf(player), 1);
            }
        }

        this.initialize();
    };

    return (PlayerList);
});

参考Angular Factory中的Initialize变量

2 个答案: