Erreur de syntaxe Javascript

Question

Quelqu'un peut-il comprendre pourquoi cela génère une erreur de syntaxe? Tout le code me semble correct.

<script type="text/javascript"> 

var rootdomain="http://"+window.location.hostname; 

function ajaxinclude(url) 
{ 
    var pagerequest = false; 
    if (window.XMLHttpRequest) // if Mozilla, Safari etc 
     pagerequest = new XMLHttpRequest() 
    else if (window.ActiveXObject){ // if IE 
     try { 
      pagerequest = new ActiveXObject("Msxml2.XMLHTTP") 
     } 
     catch (e){ 
      try{ 
       pagerequest = new ActiveXObject("Microsoft.XMLHTTP") 
      } 
     catch (e){} 
     } 
    } 
    else 
     return false 

    pagerequest.open('GET', url, false) //get page synchronously 
    pagerequest.send(null) 
    writecontent(pagerequest) 
} 

function writecontent(page_request){ 
    if (window.location.href.indexOf("http")==-1 || pagerequest.status==200) 
    document.getElementById("page1").innerHTML = pagerequest.responseText; 
} 

Il est lancer une erreur sur la ligne 7 - var pageSolliciter = false;

Si vous le commentez, une erreur se produit sur la ligne suivante. Des idées?

Merci d'avance pour votre aide !!

Answer 1

Pour commencer - utiliser http://jslint.com/

Il n'aime pas votre fonction writecontent.

Et pagerequest = new XMLHttpRequest() est manquant un point-virgule.

Aussi, j'aime aussi "ripper" mon javascript à travers YUI Compressor pour aider à révéler les erreurs de syntaxe.

http://developer.yahoo.com/yui/compressor/

Quelques points-virgules manquants: plus

pagerequest = new ActiveXObject("Msxml2.XMLHTTP") 
pagerequest = new ActiveXObject("Microsoft.XMLHTTP") 

une chose. Même si le javascript vous permet de faire quelque chose, cela ne veut pas dire que vous devriez le faire. Déclarer pagerequest comme un booléen, puis le définir sur un ActiveXObject est un peu déroutant. Je l'initialiserais probablement à null. Puis "test" pour null plus tard dans le code.

Answer 2

yopur WriteContent est wronge (de nommage des arguments) essayer:

function writecontent(page_request){ 
    if (window.location.href.indexOf("http")==-1 || page_request.status==200) 
    document.getElementById("page1").innerHTML = page_request.responseText; 
} 

De plus, il n'y a pas de valeur réelle à ceci:

var pagerequest = false; 

Puisque vous revenez jamais sans le mettre quelque part ailleurs dans votre code pourrait tout aussi bien être:

var pagerequest; 

Answer 3

Il vous manque des points-virgules sur presque toutes les lignes.

Nettoyé code:

function ajaxinclude(url) { 
    var pagerequest; 

    if (window.XMLHttpRequest) { // if Mozilla, Safari etc 
     pagerequest = new XMLHttpRequest(); 
    } else if (window.ActiveXObject){ // if IE 
     try { 
      pagerequest = new ActiveXObject("Msxml2.XMLHTTP"); 
     } catch (e) { 
      try { 
       pagerequest = new ActiveXObject("Microsoft.XMLHTTP"); 
      } catch (ec) {} 
     } 
    } 
    else { 
     return false; 
    } 

    pagerequest.open('GET', url, false); // get page synchronously 
    pagerequest.send(); 
    writecontent(pagerequest); 
} 

function writecontent (page_request) { 
    if (window.location.href.indexOf("http") == -1 || page_request.status == 200) { 
     document.getElementById("page1").innerHTML = page_request.responseText; 
    } 
} 

Answer 4

Dans votre fonction writecontent, vous appelez l'argument page_request, mais se réfèrent à lui dans le corps de la fonction comme pagerequest (sans le trait de soulignement).

Sinon, votre code devrait fonctionner - voir http://jsfiddle.net/2eynH/ pour un exemple.

Answer 5

Votre code n'est pas valide. Les points-virgules sont ajoutés à votre code quand ils ne l'ont pas ou pensent qu'il devrait l'avoir.

Ainsi, dans

function ajaxinclude(url) 
{ 
    var pagerequest = false; 
    if (window.XMLHttpRequest) // if Mozilla, Safari etc 
     pagerequest = new XMLHttpRequest() 
    else if (window.ActiveXObject){ // if IE 
     try { 

le compilateur javascript va effectuer les opérations suivantes

function ajaxinclude(url); // note the semi-color meaning the { starts floating in the middle of nowhere 
{ 
    var pagerequest = false; 
    if (window.XMLHttpRequest) // if Mozilla, Safari etc 
     pagerequest = new XMLHttpRequest(); 
    else if (window.ActiveXObject){ // if IE 
     try { //and so on 

Comme la plupart des gens ont suggéré courir JSLint dessus pour voir les erreurs.

Modifier de commentaire

Vous pouvez voir les détails d'insertion-virgule dans le blog