2011-09-23 4 views
0

Voici le script qui est destiné à stocker l'heure, date à laquelle l'utilisateur a visité la dernière page Web. Mais rien ne se produit lorsque je cours le code HTML avec le script.impossible d'afficher la dernière date de visite

window.onload = init; 

function init() { 
var now = new Date(); 
    var last = new Date(); 
document.cookie = "username=" + ";path=/;expires=" + now.setMonth(now.getMonth() + 2).toGMTString() + ";lastVisit=" + last.toDateString(); 
var lastVisit = document.cookie.split("="); 
document.getElementById("lastVisitedOn").value = lastVisit[6]; 

} 

HTML

<html xmlns="http://www.w3.org/1999/xhtml"> 
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" /> 
<title>Untitled Document</title> 
<script type="text/javascript" src="lastVisitTester.js"> 
</script> 
</head> 

<body> 
<form> 
<label>Enter your name&nbsp;&nbsp;<input type="text" id="name_field" /></label> <br/> 
<input type="submit" value="submit" /> 
</form> 
<h1 id="lastVisitedOn"></h1> 
</body> 
</html> 

Pourquoi l'heure/date ne se définir pour la balise h? Quel est le problème avec le script?

Répondre

2
window.onload = function() { 
    var now   = new Date(), 
     expires  = now, 
     lastVisit = document.cookie.match(/lastVisit=([^;]+)/), 
     userName = 'somebody'; 
    // 1. You should set month in standalone way 
    expires.setMonth(now.getMonth() + 2); 
    // 2. For each cookie you set value individually: for username in 1st line, and for lastVisit in 2nd 
    document.cookie = "username=" + userName + ";path=/;expires=" + expires.toGMTString(); 
    document.cookie = "lastVisit=" + now.toDateString() + ";path=/;expires=" + expires.toGMTString(); 
    // 3. You should test and extract your cookie value BEFORE you set it (see above with cookie match) 
    // 4. You should test if it's not null also 
    if (null != lastVisit) { 
     // 5. You should use innerHTML property for set content 
     document.getElementById("lastVisitedOn").innerHTML = lastVisit[1]; 
    } 

    // 6. But in general you should RTFM more :) 
    // 7. ps: And also use some standard frameworks for this -- not manual raw JS 
} 
+0

@see http://www.quirksmode.org/js/cookies.html – gaRex

1

Bien, il y a quelques problèmes dans votre code.

Comme d'autres a déjà mentionné:

  1. La fonction "toGMTString()" est dépréciée.

    utilisation "toLocaleString()" ou "toUTCString()" au lieu de "toGMTString()" (voir aussi https://developer.mozilla.org/en/JavaScript/Reference/Global_Objects/Date#toGMTString)

  2. Vous devez utiliser innerHTML et vous avez eu votre index mal.
  3. Vous ne pouvez pas utiliser document.cookie de cette façon. Pas sûr.

    Exemple:

    var now = new Date(); 
    var last = new Date(); 
    var cookieText = "username=" + ";path=/;expires=" + now.setMonth(now.getMonth() + 2).toLocaleString() + ";lastVisit=" + last.toDateString(); 
    
    document.cookie = cookieText; 
    var lastVisit = cookieText .split("="); 
    document.getElementById("lastVisitedOn").innerHTML = lastVisit[4]; 
    
+0

il affiche undefined! –

Questions connexes