1

Есть массив допустим:

var points = [[52.0625,142.962494],[52.0625,143.129166],[52.054165,142.995834],[52.054165,143.079163],[52.054165,143.145828], ...]

Я пробегаю циклом и забираю только значения и кладу их в объект:

  $.each(points, function(k, v){
      var myArray = [];
      var myObject = {
        "lat": v[0],
        "lon": v[1]
      }
});

Есть так же пустой массив

var myArray = [];

при таком раскладе:

myArray.push(myObject);
  console.log(myArray)

Выводится:

[Object] [Object] [Object] [Object] [Object] [Object] [Object] ...

А мне надо положить все объекты в пусто массив myArray = []; чтобы consol.log выдавало вот так:

[Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, …]

Не могу понять где моя ошибка? Подскажите пожалуйста. Заранее благодарен!!!

2 ответа 2

3

Объявление myArray нужно вынести за пределы функции в each, ибо сейчас у вас на каждом шаге цикла создается новый массив, который в итоге содержит лишь один элемент, и доступ к которому за пределами цикла отсутсвует:

var points = [
    [52.0625, 142.962494],
    [52.0625, 143.129166],
    [52.054165, 142.995834],
    [52.054165, 143.079163],
    [52.054165, 143.145828]
  ],
  myArray = [];

$.each(points, function(k, v) {
  var myObject = {
    "lat": v[0],
    "lon": v[1]
  };
  
  myArray.push(myObject);
});

console.log(myArray);
<script src="https://ajax.googleapis.com/ajax/libs/jquery/2.1.1/jquery.min.js"></script>

2
  • «за пределами» функции, а не цикла. Можно в коде ТС просто убрать объявление массива внутри ф-ии, и код заработает.
    – Sergiks
    Commented 5 авг. 2016 в 10:13
  • Спасибо большое!!! Мелочь вроде а я всю голову сломал)))
    – mikhailir
    Commented 5 авг. 2016 в 23:37
2

Константин правильно ответил на непосредственный вопрос о том, где ошибка в вашем коде.

Добавлю, что вашу задачу элегантнее решить чуть иначе: использовать стандартный метод .map() – функция внутри применится к каждому элементу массива, а на выходе вы сразу получите новый массив с «переделанными» элементами:

var points = [[52.0625,142.962494],[52.0625,143.129166],[52.054165,142.995834],[52.054165,143.079163],[52.054165,143.145828],];
var myArray = points.map( function(el){
    return {
        lat: el[0],
        lon: el[1]
    };
});

document.body.innerHTML = JSON.stringify( myArray);

Это чистый JavaScript, не требует доп. библиотек вроде jQuery. В jQuery есть похожий метод, усложнять не нужно, но для общего развития, можно и так:

// изменения только в этой строке:
var myArray = $.map(points, function(el){
1
  • Круто!!! Спасибо большое!!!
    – mikhailir
    Commented 5 авг. 2016 в 23:39

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.