Гость
Войти | Профиль | Очистить
Целевая тема:
Создать новую тему:
Автор:
Действия ...
Форумы / HTML, JavaScript, VBScript, CSS [игнор отключен] [закрыт для гостей] / has own property & typeof / 5 сообщений из 5, страница 1 из 1
16.01.2013, 19:35
    #38112892
scymaks
scymaks
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
has own property & typeof
Скажите, люди добрые есть ли смысл в проверке дополнительной?

Подход #1
Код: javascript
1.
2.
3.
4.
5.
6.
    function(data) {
        data.hasUserList = function() { return data.hasOwnProperty('users') && $.isArray(data.users); };
        data.hasDescription = function() { return data.hasOwnProperty('description') && (typeof(data.users) == 'string'); };

        ...
    }



Подход #2
Код: javascript
1.
2.
3.
4.
5.
6.
    function(data) {
        data.hasUserList = function() { return $.isArray(data.users); };
        data.hasDescription = function() { return typeof(data.users) == 'string'; };

        ...
    }



Безопасно ли всегда использовать такой подход?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
16.01.2013, 19:37
    #38112897
scymaks
scymaks
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
has own property & typeof
Скажите, люди добрые есть ли смысл в проверке дополнительной?

Подход #1
Код: javascript
1.
2.
3.
4.
5.
6.
    function(data) {
        data.hasUserList = function() { return data.hasOwnProperty('users') && $.isArray(data.users); };
        data.hasDescription = function() { return data.hasOwnProperty('description') && (typeof(data.description) == 'string'); };

        ...
    }



Подход #2
Код: javascript
1.
2.
3.
4.
5.
6.
    function(data) {
        data.hasUserList = function() { return $.isArray(data.users); };
        data.hasDescription = function() { return typeof(data.description) == 'string'; };

        ...
    }



Безопасно ли всегда использовать такой подход?
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
16.01.2013, 21:27
    #38112996
Яростный Меч
Яростный Меч
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
has own property & typeof
hasOwnProperty проверяет, есть ли свойство непосредственно в самом объекте (не заглядывая в прототип).

так что в #1 и #2 - разная логика.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
17.01.2013, 07:24
    #38113272
scymaks
scymaks
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
has own property & typeof
Яростный Меч,

ну вот здесь data это аргумент callback функции в ajax запросе:
Код: javascript
1.
2.
3.
$.get(someUrl, function(data) {
...
});
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
17.01.2013, 13:47
    #38113879
IDVsbruck
IDVsbruck
Участник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
has own property & typeof
Нет, смысла двойной проверки нет.
Что до первой строки, то $.isArray(obj), то код ее в jQuery следующий:
Код: javascript
1.
isArray:Array.isArray || function(obj){return jQuery.type(obj) === "array";}


то есть проверяется, не переопределен ли Array и равен ли тип объекта массиву. Так как подразумевается, что объект может быть или только объект (массив, другой объект или null) или undefined, то в любом случае будет сравнение типов и true будет только когда он будет массивом.

Со второй строкой тем более: если data.users == undefined, то будет и так false, равно как и при любом другом типе. Поэтому смысла делать проверку на наличие свойства в объекте.

Однако тут присутствует другой момент: в итоге и первая, и вторая строка используют функцию typeof, которая в JavaScript считается одной из самых "прожорливых" и медленных, а так как операции усливия в JavaScript лево-ассоциативны, то если не будет выполнена проверка первого условия - простой функции .hasOwnProperty(), то не будет затронута функция typeof(). Поэтому идеологически такая двойная проверка верна. Поэтому верны оба варианта.
...
Рейтинг: 0 / 0
| Ответить | Цитировать | Написать  
Форумы / HTML, JavaScript, VBScript, CSS [игнор отключен] [закрыт для гостей] / has own property & typeof / 5 сообщений из 5, страница 1 из 1
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение  
Полит. конфиден.  
созд. / загр.: 451ms
Закрыть
start [/forum/topic.php?fid=22&mobile=1&tid=1448841]:
 0ms
get settings:
 8ms
get forum list:
 20ms
check forum access:
 3ms
check topic access:
 3ms
track hit:
 148ms
get topic data:
 10ms
get forum data:
 3ms
get page messages:
 36ms
get tp. blocked users:
 1ms
others: 219ms
total:  451ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]