0

первый скрипт подключает файл с js и выводит кнопку и счетчик:

<script src="script.js"></script>
<input type="button" onclick="doWork" />
<div id="rate">0</div>

сам файл js:

function getHTTPObject(){
   if (window.ActiveXObject) return new ActiveXObject("Microsoft.XMLHTTP");
      else if (window.XMLHttpRequest) return new XMLHttpRequest();
           else {
           alert("Ваш браузер не поддерживает AJAX.");
           return null;
           }
   }

function setOutput(){
    if (httpObject.readyState == 4){
           document.getElementById('rate').innerHTML = httpObject.responseText;
    }

}

function doWork(){    
     httpObject = getHTTPObject();

     if (httpObject != null) {
        httpObject.open("GET", "2.php?rate="
            +document.getElementById('rate').innerHTML, true);
        httpObject.send(); 
    httpObject.onreadystatechange = setOutput();
      }
  }

 var httpObject = null;

и сам скрипт увеличения:

echo ($_GET['rate']+1);

кто разбирается в js скажите что не так, ибо ничего не работает.

ps брал отсюда: http://www.vr-online.ru/content/ajax-php-povyshaem-interaktivnost-1864

еще вопрос по улучшению скрипта: если я хочу в кнопке передать параметы, вот такие изменния корректы будут:

<input type="button" onclick="doWork('rate','rate','rate')" />

где 1 параметр - передаваемая переменная GET, 2й - откуда брать значение, 3й - куда записывать. И соотв. js код:

function doWork(p1,p2,p3){
...
httpObject.open("GET", "2.php?"+p1+"="+document.getElementById(p2).innerHTML, true); 
...
httpObject.onreadystatechange = setOutput(p3);
}
function setOutput(p){
   if (httpObject.readyState == 4){
       document.getElementById(p).innerHTML = httpObject.responseText;
}

Такой скрипт будет корректен?

2 ответа 2

1

ибо ничего не работает.

:D это очень информативное описание ошибки....

  1. Загляните в консоль javfascript проверьте ошибки.
  2. нужно писать так: httpObject.open("GET", "2.php", true); httpObject.send("rate="+document.getElementById('rate').innerHTML);

3.

httpObject.onreadystatechange = setOutput();

заменить на:

httpObject.onreadystatechange = setOutput;

  1. в функции setOutput httpobject должен приходить в параметрах.
1
  • 1)консоль ошибок оперы: ничего нет (: 2)исправил 3)убрал скобки А что за параметры должны приходить? можно подробнее? уже при выполнении пункта 3 заработало.
    – Antiless
    Commented 10 янв. 2012 в 13:50
1

1) Назначение функции обработчика "setOutput", необходимо указывать до отправки. Т.е. вот так: httpObject.onreadystatechange = setOutput;
httpObject.send();

2) httpObject.open("GET", "2.php", true); 3-ий параметр "true" устанавливает выполнение асинхронного AJAX запроса. Т.е, после отправки "httpObject.send();" скрипт продолжит свое выполнение, и не будет дожидаться AJAX ответов.

3) httpObject.onreadystatechange = setOutput; указывается без скобок, т.к. здесь в свойство "onreadystatechange" объекта "httpObject" передается хэш функции, что бы ее вызвать позже. Если указать "setOutput();" то сразу же будет вызвана эта функции и в свойство "onreadystatechange" запишется ее результат выполнения.

13
  • а вообще как-нибудь в setOutput можно передать параметр?
    – Antiless
    Commented 10 янв. 2012 в 17:17
  • можно. есть различные подходы для этого. Но это тема другого вопроса. Commented 10 янв. 2012 в 19:27
  • я просто создал целую функцию для выполнения всех дейсвий, и setOutput уже идет как подфункция - так и передаю параметры.
    – Antiless
    Commented 10 янв. 2012 в 19:30
  • 1
    Если вам нужно передавать параметры, вместо: =setOutput; // надо =function() { setOutput(p1,p2) }
    – user3545
    Commented 10 янв. 2012 в 19:47
  • это плохой вариант, связанный с глобальными переменными и ссылками на них, очень легко можно запутаться. Лучше использовать так называемый бинд аргументов. Commented 10 янв. 2012 в 21:19

Ваш ответ

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

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