i got my code working ,i want to know if i did it the right way :)

[canabatz]

i am not sure if this is the right way to use this AJAX script i did!

i want to know i f it will be compatible with IE 6 also!!

function getXMLHttp()
{
var xmlHttp;
  try
  {
    //Firefox, Opera 8.0+, Safari
    xmlHttp = new XMLHttpRequest();
  }
  catch(e)
  {
    //Internet Explorer
    try
    {
      xmlHttp = new ActiveXObject("Msxml2.XMLHTTP");
    }
    catch(e)
    {
      try
      {
        xmlHttp = new ActiveXObject("Microsoft.XMLHTTP");
      }
      catch(e)
      {
        alert("Your browser does not support AJAX!")
        return false;
      }
    }
  }
  return xmlHttp;
}
 var xmlHttp;
function MakeRequest2()
{
  var xmlHttp = getXMLHttp();
 
  xmlHttp.onreadystatechange = function()
  {
    if(xmlHttp.readyState == 4)
    {
      HandleResponse(xmlHttp.responseText);
    }
  }
nocache = Math.random();
  xmlHttp.open("GET", "check2.php?bid_id="+bid_id+'&nocache = '+nocache); 
  xmlHttp.send(null);
 setTimeout('MakeRequest2()', 4000);
}
 
function HandleResponse(res)
{
  if (document.getElementById('id').innerHTML == res)
    {}
    else if (document.getElementById('id').innerHTML = res)
    {
MakeRequest();MakeRequest4();MakeRequest5();Audio();
    }
    else
    {}
}
var xmlHttp1;
 
function MakeRequest()
{
  var xmlHttp1 = getXMLHttp();
  xmlHttp1.onreadystatechange = function()
  {
    if(xmlHttp1.readyState == 4)
    {
      HandleResponset(xmlHttp1.responseText);
    }
  }
nocache = Math.random();
  xmlHttp1.open("GET", "check.php?bid_id="+bid_id+'&nocache = '+nocache); 
  xmlHttp1.send(null);
}
function HandleResponset(response)
{
  if (document.getElementById('ResponseDiv').innerHTML == response)
    { }
else if (document.getElementById('ResponseDiv').innerHTML = response)
    {
  if (document.getElementById('ResponseDiv').innerHTML == "0")
    {
   nocache = Math.random();
window.location.href = ("GET", "closed.php?bid_id="+bid_id+'&nocache = '+nocache);
    }
    else {}
showPage(pageno);
    }
    else {}
}
var xmlHttp4;
 
function MakeRequest4()
{
  var xmlHttp4 = getXMLHttp();
  xmlHttp4.onreadystatechange = function()
  {
    if(xmlHttp4.readyState == 4)
    {
      HandleResponse4(xmlHttp4.responseText);
    }
  }
nocache = Math.random();
  xmlHttp4.open("GET", "over4.php?bid_id="+bid_id+'&nocache='+nocache); 
  xmlHttp4.send(null);
}
function HandleResponse4(over)
{
  if (document.getElementById('over').innerHTML == over)
    {}
    else if (document.getElementById('over').innerHTML = over)
    {}
    else
    {}
}
var xmlHttp5;
function MakeRequest5()
{
  var xmlHttp5 = getXMLHttp();
  xmlHttp5.onreadystatechange = function()
  {
    if(xmlHttp5.readyState == 4)
    {
      HandleResponse5(xmlHttp5.responseText);
    }
  }
nocache = Math.random();
  xmlHttp5.open("GET", "over9.php?bid_id="+bid_id+'&nocache = '+nocache); 
  xmlHttp5.send(null);
}
function HandleResponse5(over9)
{
  if (document.getElementById('over9').innerHTML == over9)
    {}
    else if (document.getElementById('over9').innerHTML = over9)
    {}
    else
    {
    }
}

thanx!

[acoder]

What exactly is the code supposed to be doing? Why do you have 5 or so Ajax calls?

[canabatz]

i got a table with results from the data base ,this table i s updating every 4
seconds ,the other calls are some limitations i put in the table ,there is two limitations ! ,this code i s working on my auction site ,there is one more call to check when to close the auction!

what i did there is to update the first call ,and if some thing as change, call the
other calls to check if everything is all right!

thats it :)

all the calls are working ,some times i got some call that is not working ,what im thinking is that the there is some problem with older browsers .

my question is if there is something wrong with my code! ,im not expert :)
thanx

[canabatz]

can someone look at the code and tell me please if it's the right way?

do i need to make new getXMLhttp request to each call ,or there is a way to use one xmlhttp for all the call requests?

i used xmlhttp1 , xmlhttp2.....

thanx

[Dormilich]

I think it shouldn’t be a problem to use only one request object, but how do you tell it, which response function to use?

[canabatz]

the other response i did is to just fire the php file , the first request is checking
for changes in the database ,and if there is a change ,it is fireing the others

function HandleResponse(res)
{
  if (document.getElementById('id').innerHTML == res)
    {}
    else if (document.getElementById('id').innerHTML = res)
    {
MakeRequest();MakeRequest4();MakeRequest5();Audio();
    }
    else
    {}
}

is this the right way :) ?

[Dormilich]

does look troublesome. you’ll probably get into an infinite loop.

further is if (element.innerHTML = res) intended? (if (res) seems better)

[canabatz]

from what you see ,there is only one code that is looping every 4 seconds no?

function MakeRequest2()
{
  var xmlHttp = getXMLHttp();
 
  xmlHttp.onreadystatechange = function()
  {
    if(xmlHttp.readyState == 4)
    {
      HandleResponse(xmlHttp.responseText);
    }
  }
nocache = Math.random();
  xmlHttp.open("GET", "check2.php?bid_id="+bid_id+'&nocache = '+nocache); 
  xmlHttp.send(null);
 setTimeout('MakeRequest2()', 4000);
}

this file check2.php is a small file 124 bytes ,and it's checking in the database for changes ,if there is a change ,then fireing the other requests!

[Dormilich]

@canabatz
if you have only one XHR object and you call within the responsehandler this object again to make a new request, then you will trigger new requests each time the previous request completed.

PS

setTimeout(MakeRequest2, 4000);

[canabatz]

so ,the way i have it ,it should be like that?