Функция запускается после выполнения цикла?

Question

[Nikola24]

Почему то функция запускается после выполнения цикла, и в итоге получается что класс присваивается не каждому элементу, а только 79'ому. Или я неправильно написал функцию присваивания класса

for(var i=0;i<79;i++){
    long = (long + 1.26);
    var image = new AmCharts.MapImage();
    image.type = "rectangle";
    image.latitude = lat;
    image.longitude = long;
    image.width = 7;
    image.height = 7;
    image.fixedSize = 0;
    image.bringForwardOnHover = false;
    image.color = "#"+ i+ i+ i+ i+ i+ i;
    $(function() {
      $('svg g:nth-child(8) > g > g:nth-child(2) > g:nth-child(' + i + ')').addClass('picture' + i);
    });
    map.dataProvider.images.push( image );
    map.validateData();
  }

Answer 1

[Андрей Малиновский]

Убрать конструкцию
$(function() {});
Подобным образом вы откладываете выполнение кода до полной загрузки DOM, а на этот момент i уже равен 79.

Comments

[Nikola24]

Убрал, теперь вообще ничего не происходит.

[Андрей Малиновский]

В таком случае сам javascript код нужно расположить либо перед закрывающим тегом body, либо обернуть ВЕСЬ код в $(function() {/*ваш код*/});

Answer 2

[Tsimur_S]

$(function(){}) это сокращенный способ записи для $( document ).ready(function() {}). Уберите эту обертку или добавьте ещё одну прослойку с немедленно вызывающейся функцией / используйте forEach .

Comments

[Nikola24]

Когда убираю ничего не происходит

[Tsimur_S]

Nikola24: создайте внутри for дополнительный скоуп обернув весь код в iife (function(){})() либо внесите весь код внутрь domonload $(function(){}). вам лучше изучить что такое скоуп и замыкание в js что бы в будующем не бегать по таким граблям.