Develop.try catch #62
Conversation
samples/chat/js/messages.js
Outdated
There was a problem hiding this comment.
Это точно нужно оставлять?
js/qbUtils.js
Outdated
There was a problem hiding this comment.
Можешь пояснить эту строку? Можешь комментирий /** оставить, что ты тут делаешь?
There was a problem hiding this comment.
var listenerCall = Array.prototype.shift.apply(arguments);
Тут вырезаем из псевдомассива arguments первый элемент и сразу присваиваем в listenerCall, нам нужна функция (arguments[0] - это будет та самая функция, которая пришла в аргументах. Если её передать последним элементом, то аналогом было бы var listenerCall = Array.prototype.pop.apply(arguments);). В arguments теперь останутся только аргументы (без функции)
|
Пришли к тому, что лучше скопировать args в массив и с ним работать, т.к agrs и параметры ссылаються на одну ячейку памяти и могут быть не явные проблемы. |
… now the cycle does array from arguments
js/qbUtils.js
Outdated
There was a problem hiding this comment.
for (var i = 0; i < arguments.length; i++) {
argumentsArr.push(arguments[i]);
}
listenerCall = argumentsArr.shift();
вынести это из try - нам нужно только call проверить, так?
Может вместо argumentsArr назвать argumentsCopy,логичней звучит)
js/qbUtils.js
Outdated
There was a problem hiding this comment.
Мы разве оставляем console?
У нас вроде QBLog есть или что то такое. Которое следит за параметром debug конфига и пишит.
|
|
No description provided.