Правильные конструкторы в javascript

#javascript

Здравствуйте. Есть какой-нибудь конструктор.

var Animal = function (data) {
  this.name = data.name;
  this.age = data.age;
  this.kind = data.kind;
}


Надо произвести валидацию входных данных. То есть убедится, что typeof name === String,
typeof age === 'number' и 1+['cat', 'dog', 'mongoose'].indexOf(kind). Как это проверить
очевидно. Вопрос в том, где проверять. Прямо в конструкторе? Или так:

var validate = function () {
  if (typeof this.name !== 'string') throw new Error;
  if (typeof this.age !== 'number') throw new Error;
  if (-1 === ['cat', 'dog', 'mongoose'].indexOf(kind)) throw new Error;
},
Animal = function (data) {
  this.name = data.name;
  this.age = data.age;
  this.kind = data.kind;
  validate.call(this);
}


Как это вообще делать правильно?

@eicto,


  разница в моем и вашем коде ещё и такая - я сначала валидирую, а потом провожу
инициализацию


Animal = function (data) {
    validate.call(this);
    this.name = data.name;
    this.age = data.age;
    this.kind = data.kind;
}


Тот факт, что валидация происходит в конструкторе уже считается признаком гавнокода?

@Etki,


  Но если вы не стремитесь писать идеальный код сразу же - ну эээ это печально.


Ну с чего бы я вообще подымал бы этот вопрос, если бы не стремился писать нормальный
код изначально))?
    

Ответы

Ответ 1

Чтобы писать валидирущий код сразу, не задумываясь, но следуя неким правилам, лучше
использовать TypeScript или flow. По сути вашего вопроса: не так существенно, где проверять
в первый раз, однако разумно делать функцию this.valid(), чтобы в любой момент узнать
статус инстанса, а это бывает полезнее, чем бросать исключения. Но если задумка такова:

var Animal = function (data) {
  if(data === null || typeof data !== 'object') throw new Error;
  this.name = data.name;
  this.age = data.age;
  this.kind = data.kind;
  this.validate();
};

Animal.prototype.validate = function () {
  if (typeof this.name !== 'string') throw new Error;
  if (typeof this.age !== 'number') throw new Error;
  if (-1 === ['cat', 'dog', 'mongoose'].indexOf(kind)) throw new Error;
};