Понимание ООП JavaScript: паттерн модуля с наследованием

Gentleman.prototype = Object.create(People.Human.prototype);

Gentleman.prototype = {
    constructor = Gentleman,
    toString = toString
};

Должно быть constructor: Gentleman, .... Более того, вы назначаете prototype дважды и поэтому перезаписываете его. Это имеет побочный эффект, заключающийся в том, что Gentleman больше не наследуется от Human. Вы должны добавить к нему:

Gentleman.prototype.toString = toString;

Или вы добавляете свойства с помощью вызова Object.create(). см. ссылку

Что касается this:

Вы можете использовать _peopledirectly, потому что это переменная. Все экземпляры People используют один и тот же список людей.

Когда вы вызываете функцию как метод объекта, this относится к самому объекту. Поскольку у каждого Human должно быть свое имя, this._name относится к имени this ^^ человека. Таким образом, aGentleman.toString() вернет имя ровно this джентльмена.

People.Human = (function () {
  this._name; //<- does not do anything.

Я думаю, что основная проблема заключается в прототипе Джентльменов. Для одного вы перезаписываете прототип, унаследованный от человека, для другого вы используете = вместо : для назначения функций :)

Попробуй это:

    var People = People || {};

People = (function() {
    var _people = [];

    var People = function(){};

    var addGentleman = function (name) {
        _people.push(new People.Gentleman(name));
    };

    var getList = function () {
        var temp = [];

        for (var i = 0; i < _people.length; i++) {
            temp.push(_people[i].toString());
        }

        return temp;
    };

    People.prototype = {
        constructor: People,
        addGentleman: addGentleman,
        getList: getList
    };

    return People;
})();

People.Human = (function () {
    this._name;

    var Human = function (name) {
        this._name = name;
    };



    Human.prototype = {
        constructor: Human,
    };

    return Human;
})();

People.Gentleman = (function () {

    var Gentleman = function (name) {
        People.Human.call(this, name);     
    }

    var toString = function () {
        return 'Mr. ' + this._name;
    };

   Gentleman.prototype = Object.create(People.Human.prototype);


    Gentleman.prototype.constructor = Gentleman;
    Gentleman.prototype.toString = toString;

    return Gentleman;
})();

$(function () {
    var people = new People();
    people.addGentleman('Viktor'); // this is me
    people.addGentleman('Joakim'); // and my friend!
    var list = people.getList();
    var $ul = $('#people');

    for (var i = 0; i < list.length; i++) {
        $ul.append('<li>' + list[i] + '</li>');
    }
});

Вы можете видеть, что я добавил новый метод toString к объекту-прототипу, а не перезаписал то, что уже есть. Я не знаю, есть ли у шаблона более приятный способ сделать это (я не знаком с ним).

Вы можете сделать это, если это кажется более подходящим:

Gentleman.prototype = Object.create(People.Human.prototype, 
{ 
    constructor : { configurable: true, get : function() { return Gentleman } }, 
    toString : { configurable: true, get : function() { return toString } }
});