Javascript: получение личных переменных через анонимный объект возврата

Question

Этот образец достигает частных переменных прямоугольника. Переменные myLength и myWidth отличаются для разных экземпляров. Так почему же это не рекомендуется?

var rectangle = function() { 
    var myLength = 8; 
    var myWidth = 6; 

    var getMyLength = function() { 
     return myLength; 
    }; 

    var setMyLength = function (value) { 
     myLength = value; 
    }; 

    var getMyWidth = function() { 
     return myWidth; 
    }; 

    var setMyWidth = function (value) { 
     myWidth = value; 
    }; 

    var drawFigure = function() { 
     console.log("Draw invoked for figure: " + 
      getMyLength() + " * " + getMyWidth()); 
    }; 

    return { 
     getMyLength: getMyLength, 
     setMyLength: setMyLength, 
     getMyWidth: getMyWidth, 
     setMyWidth: setMyWidth, 
     drawFigure: drawFigure 
    } 
}; 

Затем мы используем его следующим образом:

var myRectangle = new rectangle(); 

myRectangle.drawFigure();   // Draw invoked for figure: 8 * 6 

myRectangle.setMyLength(3); 

myRectangle.setMyWidth(5); 

myRectangle.drawFigure();   // Draw invoked for figure: 3 * 5 

var myRectangle1 = new rectangle(); 

myRectangle1.drawFigure();   // Draw invoked for figure: 8 * 6 

Answer 1

По моему мнению, частные переменные переоценены. Вам действительно нужно скрывать вещи от программиста? Нет. Даже если вы сделали все ваши личные переменные общедоступными, какая разница? На самом деле я выступаю за то, чтобы сделать все публичным, потому что:

1. Это облегчило бы отладку. Если вы используете объект console.log, вы можете проверить его общедоступные свойства, что облегчает отладку, поскольку вы можете видеть состояние объекта.
2. Вам не нужно создавать ненужные затворы. Если вы хотите получить приватную переменную, а также хотите, чтобы она была доступна публичному методу, вам нужно создать закрытие (по одному для каждого экземпляра). Если вы публикуете свои свойства, вам не нужны закрытия. Вы можете поместить методы в прототип. Следовательно, у вас будет только один метод для каждого класса, который будет использоваться всеми экземплярами.

3. Вам не нужно создавать ненужные геттеры и сеттеры. Что нужно сделать переменной private, а затем разрешить кому-либо изменять ее с помощью функций getter и setter? Вы также можете сделать переменную открытой.

   Геттеры и сеттеры, по моему скромному мнению, только полезны для свойств фантомных:

   var Person = defclass({ 
       constructor: function (firstName, lastName) { 
        this.firstName = firstName; 
        this.lastName = lastName; 
       }, 
       getFullName: function() { 
        return this.firstName + " " + this.lastName; 
       }, 
       setFullName: function (fullName) { 
        var name = fullName.split(" "); 
        this.firstName = name[0]; 
        this.lastName = name[1]; 
       } 
   }); 
   

Таким образом, на мой взгляд, вы должны написали свой Rectangle класс следующим образом:

var Rectangle = defclass({ 
    constructor: function (width, height) { 
     this.width = width; 
     this.height = height; 
    } 
}); 

Затем мы используем его следующим образом:

var rectA = new Rectangle(8, 6); 

console.log(rectA);    // { width: 8, height: 6 } 

rectA.width = 3; 
rectA.height = 5; 

console.log(rectA);    // { width: 3, height: 5 } 

var rectB = new Rectangle(8, 6); 

console.log(rectB);    // { width: 8, height: 6 } 

Наконец, определение defclass:

function defclass(prototype) { 
    var constructor = prototype.constructor; 
    constructor.prototype = prototype; 
    return constructor; 
} 

Так что это только мои два цента на пути вы должны создавать объекты в JavaScript.