Не работает сравнение в цикле?

Question

[Sergey750il]

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

var number = $('.ofice .ofice_status').map(function() {
        return $(this).html();
    });
    var number_js = [];

    for (var p = 0, len = number.length; p <= len; p++) {
        if (number[p] == 'Свободен') {
            number_js[p] = $('.ofice:eq(' + p + ')>.ofice_number>a').map(function() {
                    return $(this).html();
                })[0];
               
              
                for (var i = 0, le = rsrGroups.length; i <= le; i++) {
                     // console.log(number_js[p]);
                     // console.log(rsrGroups[i].data("room"));
                     if (number_js[p]==rsrGroups[i].data("room")) {
                          rsrGroups[i].attr('fill','red');
                     }
                }
   
        }
    }

Comments

[Sergey750il]

Ответил ниже.

[Sergey750il]

Я склоняюсь, что вся проблема в первом массиве number_js[p], хотя в консоль вывод правильный.

[Sergey750il]

Все сделал как Вы написали, но не работает, в чем я ошибаюсь... в консоли ошибок нет...

[Алексей Зуев]

Sergey750il: киньте ссылку рабочую, можно в скайп если скрытно всё

[Sergey750il]

Сейчас скину файл

[Sergey750il]

Можно почту?

[Sergey750il]

Отправил paths.js. Все сделал как в примере, но понять не могу почему массив во втором цикле не перебирается.

[Sergey750il]

Не пугайтесь большого объема кода. Это просто SVG карты, мой код в самом низу.

Answer 1

[Алексей Зуев]

Проблема в условии цикла

for (var i = 0, le = rsrGroups.length; i <= le; i++) {

i <= le - нужно оставить только меньше, иначе на i, равном rsrGroups.length, свалится с ошибкой
Ну и верхний цикл тоже лучше только меньше, итерация с 0 идет
В итоге получится

for (var i = 0, len = rsrGroups.length; i < len; i++) {
  if (number_js[p] == rsrGroups[i].data("room")) {
    rsrGroups[i].attr('fill', 'red');
  }
}

Comments

[Sergey750il]

Заработало!!!Но честно говоря я так и не понял в чем было дело, кроме <=

[Алексей Зуев]

Проблема в том, что при переборе во внутреннем цикле если оставить равно, то на последней итерации будет rsrGroups[12].data("room"), элементов всего 12, отчет от нуля. rsrGroups[12] - будет равен undefined. И мы на этом undefined пытаемся еще вызвать data. В итоге получим ошибку rsrGroups[i] is undefined и дальше js отвалится

[Алексей Зуев]

Sergey750il: TypeError: rsrGroups[i] is undefined

[Sergey750il]

Теперь понятно. На будущее всегда лучше ставить < в таких случаях? И я не могу понять почему консоль не ругалась...

[Алексей Зуев]

Sergey750il: у меня ругалась, плохо смотрели наверно. Насчет меньше или меньше или равно - все зависит от того, с чего начинается шаг итерации. А эта ошибка обычно называется выход индекса за границы массива

[Sergey750il]

Спасибо. Теперь понятно. Я думаю если вы запустили файлы у себя то наверняка у Вас тоже ругается на событие click. Оно работает нормально, но смущает сама ошибка и я не могу понять в чем дело, хотя нареканий к работе нет.

[Sergey750il]

И как Вы увидели ошибку "выхода индекса" в консоли? Может я не там просто смотрел, но у меня в строке одна ошибка - click. Или Вы через отладку смотрели?

[Алексей Зуев]

Sergey750il: там такая же причина, на последней итерации ошибка. Уберите равно из цикла

[Sergey750il]

В FF консоль показывает, в Chrome не много по другому.

Answer 2

[Дмитрий Ковальский]

for (var i = 0, le = rsrGroups.length;
le - это опечатка?

Comments

[Sergey750il]

Опечатка. Подправил, все равно не работает.

[Дмитрий Ковальский]

Sergey750il: тогда берите DevTools ,ставьте точки останова и осваивайте технику пошаговой отладки.

[Sergey750il]

И без того уже часа 4 пытаюсь понять, что к чему.

Answer 3

[Sergey750il]

var rsrGroups = [path_ag, path_ah, path_aj, path_ak, path_al, path_am, path_ao, path_ap, path_aq, path_ar, path_ca, path_cb];

path_ag.data("room", "201");
    path_ah.data("room", "212");
    path_aj.data("room", "218");
    path_ak.data("room", "219");
    path_al.data("room", "202");
    path_am.data("room", "215");
    path_ao.data("room", "205");
    path_ap.data("room", "204");
    path_aq.data("room", "214");
    path_ar.data("room", "213");
    path_ca.data("room", "216");
    path_cb.data("room", "203");

<div id="ofices">
<div class="ofice">
<div class="ofice_number"><a href="#">201</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">212</a></div>
<div class="ofice_status" data-select-like-a-boss="1">Занят</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">218</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="sds">219</a></div>
<div class="ofice_status">Занят</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="sd">202</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="sdd">215</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">205</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">204</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">214</a></div>
<div class="ofice_status">Занят</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">213</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">216</a></div>
<div class="ofice_status">Свободен</div>
</div>
<div class="ofice">
<div class="ofice_number"><a href="#">203</a></div>
<div class="ofice_status">Свободен</div>
</div>
</div>

Comments

[Sergey750il]

Не могу понять, почему во втором цикле не происходит перебор массива number_js[p] в условии.