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

Ответ

Чтобы писать валидирущий код сразу, не задумываясь, но следуя неким правилам, лучше использовать 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; };

Как лучше оптимизировать код?

Здравствуйте! Подскажите, как лучше сделать. Есть экран LinearLayout, внутри него еще LinearLayout, и в нём TextView. По сути: и тд. С сервера приходят данные, и некоторым поля присваивается значение null, соответственно, отображать на экране не стоит, и нужно спрятать не сам TextView, а контейнер. Как сделать, чтобы не приходилось очень много кода вручную писать? Мой код String type = types.getType(); Long value = values.getValues(); AQuery aq = new AQuery(rootView);//библиотека чтобы кода меньше было
aq.id(R.id.tv_type).text(type); // это вместо findViewById и setText type!= null ? type: "");
И теперь, инициализируем контейнер ll_type = (LinearLayout) rootView.findViewById(R.id.ll_type); if (type==null) ll_type.setVisibility(View.INVISIBLE);
Одно поле отобразили.

Ответ

В таких случаях неплохой вариант динамически добавлять элементы в зависимости от данных, которые приехали с сервера. Например, у нас есть некий ArrayList