22.04.2011, 16:31
|
|
Модератор Всея Форума
|
|
Регистрация: 14.05.2009
Сообщений: 4,021
|
|
Sweet, прости, я норм к тебе отношусь.
Сообщение от Sweet
|
И вообще, твой пример никак не связан с циклом for..in!
|
Ок, вот другой пример, когда есть дефолтные значения:
var defaultOptions = {};
function Constructor(options){
for(var property in defaultOptions){
if(defaultOptions.hasOwnProperty(property)){ // Проверяем только нужные свойства. Вот здесь hasOwnProperty обязательно.
// Делаем нужные действия
}
}
}
Когда нет дефолтных значений, проверять нужно так:
function Constructor(options){
// Допустим, нам нужно проверить конкретное свойство
// Если тип не важен:
if('prop' in options){}
// Если важен:
if('prop' in options && (/* тут проверка типа */)){} // prop in obj - тоже обязательно
}
Но никак не hasOwnProperty! Его нужно использовать только для "своих" объектов и то, если они не имеют кастомных прототипов.
Из этого видно, что значением может оказаться ваше кастомное свойство из прототипа.
Сообщение от Kolyaj
|
тебя спрашивают, почему плохо расширять Function.prototype, а ты объясняешь, почему плохо расширять Object.prototype. Так в Object.prototype никто и не лезет.
|
Откуда ты можешь знать, из какого прототипа берутся значения? Если объект приходит извне и тебе нужны только значения свойств, нельзя смотреть на тип объекта. В примерах выше я могу в options передать и функцию, если мне будет так удобно.
Что я этим хотел сказать:
Если везде применять hasOwnProperty и проверять тип объекта - создаются ограничения, в частности, нельзя уже использовать наследование. Если расширять прототипы нативных конструкторов - они начинают везде обрабатываться, что может привести назад к использованию hasOwnProperty или даже к багу. Замкнутый круг. В каком месте его разрывать - каждый решает сам для себя, о чем я писал выше. Все полюсы и минусы я показал.
__________________
Болтовня ничего не стоит. Покажите мне код. — Linus Torvalds
влад.куркин.рф
Последний раз редактировалось B~Vladi, 22.04.2011 в 16:38.
|
|
22.04.2011, 17:19
|
Профессор
|
|
Регистрация: 16.03.2010
Сообщений: 1,618
|
|
Насколько я понял, это выглядет примерно так:
function SomeConstructor( object ){
this.property1 = object.property1 || 'default';
this.property2 = object.property2 || 'default';
};
var func = function(){};
func.property1 = 'value';
Function.prototype.property2 = 'zapadlo';
new SomeConstructor( func ); // Тысяча чертей, почему property2 не 'default'???
Но для себя я решил, что имею полное право дополнять нативные конструкторы там, где считаю нужным. Что плохого в том, что я, например, добавляю в Array.prototype методы forEach, filter и пр.? Я, конечно, понимаю про for..in, но мы же знаем, как зовут тех, кто так переберает массивы?
|
|
22.04.2011, 17:49
|
|
Модератор Всея Форума
|
|
Регистрация: 14.05.2009
Сообщений: 4,021
|
|
Расширение Array.prototype, Function.prototype и т.д. менее опасно только из-за того, что их экземпляры используются в основном по назначению, а не для хранения/передачи свойств. forEach и filter это, конечно же, хорошо, но только потому что они есть в спецификации и в реализациях, поэтому безопасно.
__________________
Болтовня ничего не стоит. Покажите мне код. — Linus Torvalds
влад.куркин.рф
|
|
23.04.2011, 06:09
|
|
Рассеянный профессор
|
|
Регистрация: 06.04.2009
Сообщений: 2,379
|
|
Вот как-то так получилось:
Function.prototype.callAsync = function() {
var self = this, args = $A(arguments), thisObj = args.shift(), elem = document.createElement('div'), done = false;
elem.onclick = function() {
if (!done) {
done = true;
self.apply(thisObj, args);
}
};
if (document.createEvent) {
var e = document.createEvent('HTMLEvents');
e.initEvent('click', false, false);
elem.dispatchEvent(e);
} else document.createDocumentFragment().appendChild(elem).fireEvent('onclick', document.createEventObject());
if (!done) setTimeout(elem.onclick, 15);
};
var $A = function(iterable) {
if (iterable) {
var result = [], i = iterable.length;
while (i) result[--i] = iterable[i];
return result;
}
return [];
};
var testFunc = function() {
alert('testFunc');
throw new Error('testFunc');
};
testFunc.method();
alert('global');
testFunc.method();
alert('global');
Последний раз редактировалось Riim, 24.04.2011 в 16:16.
|
|
23.04.2011, 10:52
|
Новичок на форуме
|
|
Регистрация: 19.02.2008
Сообщений: 9,177
|
|
Сообщение от B~Vladi
|
forEach и filter это, конечно же, хорошо, но только потому что они есть в спецификации и в реализациях, поэтому безопасно.
|
Почему безопасно? Они же тоже будут попадать в for-in.
|
|
23.04.2011, 11:30
|
|
Модератор Всея Форума
|
|
Регистрация: 14.05.2009
Сообщений: 4,021
|
|
Сообщение от Kolyaj
|
Они же тоже будут попадать в for-in.
|
Ну да, не совсем уж так и безопасно.
__________________
Болтовня ничего не стоит. Покажите мне код. — Linus Torvalds
влад.куркин.рф
|
|
23.04.2011, 11:35
|
Новичок на форуме
|
|
Регистрация: 19.02.2008
Сообщений: 9,177
|
|
Но при этом мы знаем, что не надо перебирать массивы с помощью for-in, поэтому не боимся дописывать forEach/map/filter/... .
|
|
28.04.2011, 07:49
|
|
|
|
Регистрация: 27.12.2008
Сообщений: 4,201
|
|
B~Vladi, ты прав, если неадекватные пользователи входят в целевую аудиторию решения (jquery way?), но все может быть проще
ссылка по смежной теме: What’s wrong with extending the DOM
|
|
28.04.2011, 21:02
|
Профессор
|
|
Регистрация: 20.03.2008
Сообщений: 1,183
|
|
__________________
.ня
|
|
28.04.2011, 22:43
|
sinistral
|
|
Регистрация: 28.03.2011
Сообщений: 5,418
|
|
намекаешь на дефолтную реализацию исполнения ф-й в отдельных потоках?
|
|
|
|