Здравствуйте. Есть какой-нибудь конструктор.
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,
Но если вы не стремитесь писать идеальный код сразу же - ну эээ это печально.
Ну с чего бы я вообще подымал бы этот вопрос, если бы не стремился писать нормальный код изначально))?