Заполнение и видимость значений массива в обработчике ajax и за его пределами

@N3stY · Регистрация: 05.06.2014

Студворк — интернет-сервис помощи студентам

Доброго времени суток.

Нужна помощь с этим скриптом:

Javascript

var markers = new Array();
 
 
$.ajax({
            url: "/index.php?action=getmap",
            type: 'POST',
            data: {"AJAX" : true}, 
            cache: false,
            success: function(html){
                var addmrk = $.parseJSON(html);
                var i=0;
                while (i < addmrk.array_num) {
                markers.push({
                  latitude: addmrk.content[i].lat,
                  longitude: addmrk.content[i].lng,
                  marker_content: '<div class="marker ' + addmrk.content[i].color + '"><i class="fa fa-'+addmrk.content[i].icon+'"></i></div>',
                  content: '<div class="infobox ' + addmrk.content[i].color + ' "><a class="infobox-image" href=""><img src="'+addmrk.content[i].image+'" alt=""></a><div class="infobox-content"><div class="infobox-content-title"><a href="#">'+addmrk.content[i].title+'</a></div><div class="infobox-content-body">'+addmrk.content[i].descr+'</div></div><div class="infobox-contact"><div class="infobox-contact-title"><a href="#">'+addmrk.content[i].location+'</a></div><div class="infobox-contact-body"><i class="fa fa-phone"></i>'+addmrk.content[i].phone+'<br><i class="fa fa-envelope"></i>'+addmrk.content[i].email+'<br>Site: '+addmrk.content[i].site+'</div><a href="#" class="close"><i class="fa fa-close"></i></a></div></div>'
        });
        i++;
    }
                
            }
        });

Дело в том, что массив markers заполняющийся внутри функции срабатывающей по событию success, заполненным доступен только там. Вне функции он пуст.

@Fedor Vlasenko · 11.03.2015, 02:06

1. Не правильна конструкция цикла
while (addmrk.length--)
или
for(var i= 0; i < addmrk.length; i++)
или each jQuery
2. А ведь можно и так

Javascript

1
2
3

$.post("/index.php?action=getmap", {AJAX : true}, function(result){
    console.log(result)
}, 'json');

@Lazy_Den · 11.03.2015, 03:41

Не по теме:

Сообщение от Poznakomlus

А ведь можно и так

Можно, но чем вас полная форма записи не устраивает? И зачем одни параметры прописывать в URL, а вторые в параметр data? Хотя это уже вопрос и к ТС.

Во-первых, по поводу кода:

Javascript

var markers = []; // в 21-ом веке живем ))
$.ajax({
    url: '/index.php',
    type: 'POST',
    dataType: 'json',
    data: {
        action: 'getmap',
        AJAX: true
    },
    cache: false,
    success: function (html) {
        // парсить ответ не нужно, т.к. мы указали тип ожидаемых данных
        // html - это уже готовый к использованию объект
    }
});

Сообщение от N3stY

массив markers ... Вне функции он пуст.

Это и понятно, запрос ведь выполняется асинхронно. Ответы по этому вопросу, вы . Для того, чтоб понять с какой лучше стороны подойти к решению вашей задачи, нужно хотя бы знать, зачем вам понадобилось использовать массив вне callback-функции success, как инициализируете ajax-запрос и куда хотите передавать полученный массив.

@N3stY · 11.03.2015, 15:27 **[ТС]**

Сообщение от Lazy_Den

И зачем одни параметры прописывать в URL, а вторые в параметр data

Параметр в УРЛ (GET) говорит о том, что вызывать нужно модуль карт, параметр в DATA (POST) отвечает за AJAX режим (т.е. ответ придет только JSON ответ модуля без основной страницы т.к. запрос посылается к index.php).

Сообщение от Lazy_Den

зачем вам понадобилось использовать массив вне callback-функции success

Это массив для установки маркеров на гуглокарту. Проблема в том, что шаблон писал не я(купил уже готовый) и весь JS этого шаблона очень запутанный. Получается, что инициализация карт идет в одном файле, а маркеры в другом, для этого и нужно его использовать вне функции.

Сообщение от Poznakomlus

1. Не правильна конструкция цикла
while (addmrk.length--)
или
for(var i= 0; i < addmrk.length; i++)

по началу использовал for, но while то же выполняет свою задачу.

Добавлено через 17 минут
Помогло

Javascript

1	async: false

@Lazy_Den · 11.03.2015, 15:39

Сообщение от N3stY

Параметр в УРЛ (GET) говорит о том, что вызывать нужно модуль карт, параметр в DATA (POST) отвечает за AJAX режим

Железно. А что мешает сделать без лишних извращений:

PHP

if( isset( $_SERVER['HTTP_X_REQUESTED_WITH'] ) && ( $_SERVER['HTTP_X_REQUESTED_WITH'] == 'XMLHttpRequest' ) ) {
    // в этой секции обрабатываются все ajax-запросы
    if(isset($_POST['getmap'])) {
        // подключаем модуль с картами
        // пара "AJAX: true" - вообще не нужна
    }
}

Сообщение от N3stY

инициализация карт идет в одном файле, а маркеры в другом

Если файлы уже подключены на странице, то я не вижу проблем с инициализацией в success. Как вы вызываете функцию, которая отвечает за расстановку маркеров? Вы же именно при вызове этой функции, хотите передавать массив данных. Я правильно понимаю схему?

Добавлено через 36 секунд

Сообщение от N3stY

Помогло async: false

Выбрали худший из возможных вариантов.

@N3stY · 11.03.2015, 18:54 **[ТС]**

Сообщение от Lazy_Den

Выбрали худший из возможных вариантов.

Я бы не сказал, маркеры размещаются при загрузке страницы, поэтому подходит.

Сообщение от Lazy_Den

if( isset( $_SERVER['HTTP_X_REQUESTED_WITH'] ) && ( $_SERVER['HTTP_X_REQUESTED_WITH'] == 'XMLHttpRequest' ) ) {
* * // в этой секции обрабатываются все ajax-запросы
* * if(isset($_POST['getmap'])) {
* * * * // подключаем модуль с картами
* * * * // пара "AJAX: true" - вообще не нужна
* * }
}

Лишние "если", да и мне нужен именно $_GET для определения модулей и $_POST для всего остального.

@N3stY (ノಠ益ಠ)ノ彡┻━┻ 151 / 151 / 78 Регистрация: 05.06.2014 Сообщений: 710 Записей в блоге: 2
	11.03.2015, 18:54 [ТС]	6
	Сообщение от Lazy_Den Выбрали худший из возможных вариантов. Я бы не сказал, маркеры размещаются при загрузке страницы, поэтому подходит. Сообщение от Lazy_Den if( isset( $_SERVER['HTTP_X_REQUESTED_WITH'] ) && ( $_SERVER['HTTP_X_REQUESTED_WITH'] == 'XMLHttpRequest' ) ) { * * // в этой секции обрабатываются все ajax-запросы * * if(isset($_POST['getmap'])) { * * * * // подключаем модуль с картами * * * * // пара "AJAX: true" - вообще не нужна * * } } Лишние "если", да и мне нужен именно $_GET для определения модулей и $_POST для всего остального. 0

Опции темы