Невозможно изменить переменную

Question

У меня есть код JavaScript (ниже). В каждом случае, в конце цикла, $ sid должен быть равен нулю, но «alert ($ sid)» всегда дает мне 200. Я не могу найти ошибку в моем коде. Не могли бы вы мне помочь?

Заранее спасибо.

$sid = 200; 
$canAppend = 0; 
$iPage = 1; 

while ($sid && $canAppend==0 && $iPage==1) { 
    alert($sid); 
    $.ajax({ 
    url: "tmp/result1.html", 
    success: function(html) 
    { 
     if(html) 
     { 
      $("#inzeraty").append(html); 
      $('div#loadmoreajaxloader').hide(); 
      $sid = 0; 
     }    
    }, 
    error: function() 
     { 
      $sid = 0; 
     } 
    }); 
} 

Answer 1

Проблема в том, что вызов ajax является асинхронным. JavaScript помещает это, делает запрос на сервер, а затем переходит к следующей строке кода. Ваши функции обратного вызова успеха и ошибки не вызываются до тех пор, пока JavaScript не завершит выполнение вашего цикла while.

Кроме того, структура кода не имеет никакого смысла (извините, не пытаясь быть грубой). Переменные $ canAppend и $ iPage не используются или не изменяются. Что этот код будет делать, это ввести цикл и никогда не выйти. Почему это? Это связано с тем, что вызов $ .ajax() не блокируется. Он не будет ждать там, пока запрос не будет завершен, он будет продолжен. Поскольку JavaScript (по существу) однопоточный, обратные вызовы для ошибки и успеха НЕ МОГУТ исполняться до тех пор, пока не завершится текущий процесс выполнения. Поскольку не удается запустить обработчики ошибок и ошибок, $ sid не может быть установлен. Поскольку $ sid не может быть отправлен, код не может выйти из цикла while.

Я не вижу, как вы на самом деле используете код while. Вместо этого просто вызовите функцию $ .ajax() и обработайте результаты в вашем обработчике успеха. Попробуйте это сделать, чтобы вы могли лучше понять, что происходит:

$sid = 200; 

alert("$sid is 200: " + $sid); // you'll see this first 

$.ajax({ 
    url: "tmp/result1.html", 
    success: function(html) 
    { 
     if(html) 
     { 
      $("#inzeraty").append(html); 
      $('div#loadmoreajaxloader').hide(); 
      $sid = 0; 
      alert("$sid is now 0: " + $sid); // you'll see this third if html is not false 
     } else { 
      alert("$sid is STILL 200: " + $sid); // you'll see this third if html is false 
     } 
    }, 
    error: function() 
    { 
     $sid = 0; 
     alert("you got an error, but $sid is now 0: " + $sid); // you'll see this third if there's an error 
    } 
}); 

alert("$sid is still 200: " + $sid); // you'll see this second 

Answer 2

По умолчанию вызовов Ajax является асинхронным и функция успеха или ошибок откладываются до тех пор, после того, как $ .ajax извлечет TMP/result1.html. В вашем случае они будут отложены навсегда, потому что цикл while будет держать руку, $ sid останется равной 200, и вы будете продолжать накапливать аякс-вызовы.

Быстрое исправление, чтобы сделать ваш АЯКС синхронным вызова:

$.ajax({ 
    url: "tmp/result1.html", 
    async: false, 
    success: function(html) 
// etc. 

Переписывая свой код, чтобы избежать сочетаний в то время как петля и Ajax могут быть лучшей идеей (но я не знаю ваш конкретный контекст) ,