JavaScript
- 35 ответов
- 0 вопросов
30
Вклад в тег
var call= $.getJSON('https://api.vk.com/method/users.get?user_ids=' + id + '&callback=?', function(resp){
for (var i = 0, len = arr.length; i < len; i += 1)
. Так как у вас инкримент происходит каждый раз на единицу, то вместо i += 1
лучше написать i++
. var deleteUser;
и deleteUser = i;
смысла нет, так как область видимости переменной var i
уже будет внутри функции, а не цикла (в ECMAScript 2015 есть чудесная вещь let
).if (deleteUser === undefined) return;
. Это лишние проверки того, что уже известно. Делайте действие сразу.for (var i = 0, len = arr.length; i < len; i++) {
if (arr[i].name === name) arr.splice(i, 1);
}
for (var i = 0, len = arr.length; i < len; i++) {
if (arr[i].name === name){
arr.splice(i, 1);
break; /* или сразу return; */
}
}
<a href="tel:+380991234567">Позвонить</a>