Javascript：内部数组未重置为外部对象

Question

如何以正确的方式授予对对象内部属性的访问权限？这就是破坏我的应用程序的原因：

我有一个处理数组的对象（这里简化了）：

function ListManager() {
    var list = [],
        add = function (element) {
            list.push(element);
        },
        clear = function () {
            list = [];
        };

    return {
        add: add,
        clear: clear,
        list : list
    };
};

但是我在使用它时得到了这个：

var manager = new ListManager();
manager.add("something");
manager.clear();
console.log(manager.list.length); // <= outputs "1"!

单步执行代码显示，在 clear 方法中，list 变成了一个新数组。但是从ListManager 之外的列表没有被清除。

我做错了什么？

Answer 1

这是因为clear 设置了var list 的值，而不是ListManager() 返回的对象上的.list。您可以改用this：

function ListManager() {
    var list = [],
        add = function (element) {
            this.list.push(element);
        },
        clear = function () {
            this.list = [];
        };

    return {
        add: add,
        clear: clear,
        list : list
    };
}

Answer 2

使用您当前的结构，您可以：

function ListManager() {
    var list = [],
        add = function (element) {
            list.push(element);
        },
        clear = function () {
            list = [];
        };
        getList=function(){
            return list;
        }


    return {
        add: add,
        clear: clear,
        list : list,
        getList: getList
    };
};

var manager = new ListManager();
manager.add("something");
console.log(manager.getList()); // ["something"]
manager.clear();
console.log(manager.getList()); // []

Answer 3

    function ListManager() {
    var list = [],
        add = function (element) {
            this.list.push(element);
        },
        clear = function () {
            this.list = [];
        };

    return {
        add: add,
        clear: clear,
        list : list
    };
};

var manager = new ListManager();
manager.add("something");
manager.clear();
console.log(manager.list.length); // <= now outputs "0"!

Answer 4

正如已经解释的那样，您的问题是当您执行list = [] 时，您正在更改局部变量list，但您并没有更改this.list，因为它们是两个独立的变量。它们最初引用的是同一个数组，因此如果您修改数组而不是只为其中一个变量分配一个新数组，它们都会看到变化。

就个人而言，我认为您在创建此对象时使用了错误的设计模式，这只会使事情变得更加复杂，并且更有可能像您一样制造问题。如果您想维护外部世界无法访问的私有实例变量，该设计模式可能很有用，但如果所有内容都打算公开，它会创建更复杂的定义和维护。

我的一个编程目标是使用最简单、最简洁的方式来表达所需的功能。

因此，由于此对象中的所有内容都是公开的，并且可以从对象外部访问，因此这要简单得多，并且不会遇到您刚刚遇到的任何类型的问题：

function ListManager() {
    this.list = [];
    this.add = function(element) {
       this.list.push(element);
    }
    this.clear = function() {
        this.list = [];
    }    
}

或者，甚至可以使用原型：

function ListManager() {
    this.list = [];
}

ListManager.prototype = {
    add: function(element) {
       this.list.push(element);
    },
    clear: function() {
        this.list = [];
    }
};