LocalStorage ne fonctionne pas

Version imprimable

Voilà, je suis bon développeur php, mais j'ai certaines difficultés en JS. J'ai trouvé un code pour stocker des données en local.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
<script type="text/javascript">
 	if(typeof localStorage != 'undefined'){
		//récup de la valeur dans le web store
		var nbvisites = localStorage.getItem('visites');
		//vérification de la présence du compteur
		if(nbvisites != null){
			//si oui, on convertit en nom entier la chaine de texte qiu fut stockée
			nbvisites = paseInt(nbvisite);
		}
		else{
			nbvisites = 1;
		}
		//incrémentation 
		nbvisites ++;
		//Stockage à nouveau en attendant la prochaine visites
		localStorage.setItem('visites', nbvisites);
		//affiche dans la page
		document.getElementById('visites').innerHTML = nbvisites;
	}
	else{
		alert("localstorage n'est pas supporté");
	}
	alert("localstorage n'est pas supporté");
  </script>

j'ai mis ça dans le body, mais rien ne fonctionne. Pourquoi ?

27/08/2012, 11h43
Bovino

Peut-être à cause d'une erreur de syntaxe :

Code:

nbvisites = paseInt(nbvisite);
27/08/2012, 11h46
Awery

Code:

nbvisites = parseInt(nbvisite);

corriger mais ça marche toujours pas
27/08/2012, 11h48
Bovino

Citation:

j'ai mis ça dans le body

Code:

document.getElementById('visites').innerHTML = nbvisites;

L'élément 'visites' existe bien lorsque le code est exécuté ?
27/08/2012, 11h58
Awery

Citation:

L'élément 'visites' existe bien lorsque le code est exécuté ?

Il sagit d'initialiser la variable ? si oui c'est fait, et, ça ne marche toujours pas.

Code:

nbvisite = 2;
27/08/2012, 15h57
sekaijin

je ne comprends pas nbvisite et nbvisites c'est quoi la différence et pourquoi il y en a deux et où sont-elle initialisées ?

A+JYT

Salut à tous, il y a un peu plus sort au niveau de ce que tu demande :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
<script>
if(typeof(localStorage) == "undefined") {
  alert("Ce navigateur ne supporte pas Web Storage");
} else {
  // Valeur par défaut si visites n'a pas encore été défini
  if(!localStorage.visites)  localStorage.setItem("visites","0");
  // Variable temporaire le temps d'un refresh...
  var nb = "";
 
 // Lecture
  // affectation de nb = visites convertit en nombre entier
  nb = parseInt(localStorage.getItem("visites"));
  // Incrémentation du compteur
  nb++;
  // Mémorisation de la nouvelle valeur incrémenté nb dans visites
  localStorage.setItem("visites",nb);
  // Affichage
  document.getElementById("visites").innerHTML = nb;
}
</script>

Bonne chance A+

Code:

1
2
3
4
5
6
7
8
9
  // Valeur par défaut si visites n'a pas encore été défini
  if(!localStorage.visites)  localStorage.setItem("visites","0");
  // Variable temporaire le temps d'un refresh...
  var nb = "";
 
 // Lecture
  // affectation de nb = visites convertit en nombre entier
  nb = parseInt(localStorage.getItem("visites"));
  // Incrémentation du compteur

Peut être remplacé par :

Code:

var nb = parseInt(localStorage.getItem("visites") || 0);

Il me semble.

28/08/2012, 07h48
Bovino

Dans tous les cas, invoquer

Code:

document.getElementById("visites").innerHTML = nb;

ne risque pas de fonctionner tant que l'élément dont l'id est 'visites' n'existe pas dans le DOM et au vu des explications et du code donné, il semble fort que ça soit le cas !

Salut à tous,

Citation:

Dans tous les cas, invoquer
Code :
Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById("visites").innerHTML = nb;

ne risque pas de fonctionner tant que l'élément dont l'id est 'visites' n'existe pas dans le DOM et au vu des explications et du code donné, il semble fort que ça soit le cas !

Il le met à la fin de body

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
 
<html>
<head>
</head>
<body>
<div id="visites">0</div>
<script>
if(typeof(localStorage) == "undefined") {
  alert("Ce navigateur ne supporte pas Web Storage");
} else {
  // Valeur par défaut si visites n'a pas encore été défini
  // merci à Willpower 
 var nb = parseInt(localStorage.getItem("visites") || 0);
  // Incrémentation du compteur
  nb++;
  // Mémorisation de la nouvelle valeur incrémenté nb dans visites
  localStorage.setItem("visites",nb);
  // Affichage
  document.getElementById("visites").innerHTML = nb;
}
</script>
</body>
</html>
</body>

ou si son SCRIPT et dans HEAD

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
 
<html>
<head>
<script>
window.onload = function(){
if(typeof(localStorage) == "undefined") {
  alert("Ce navigateur ne supporte pas Web Storage");
} else {
  // Valeur par défaut si visites n'a pas encore été défini
  // merci à Willpower 
 var nb = parseInt(localStorage.getItem("visites") || 0);
  // Incrémentation du compteur
  nb++;
  // Mémorisation de la nouvelle valeur incrémenté nb dans visites
  localStorage.setItem("visites",nb);
  // Affichage
  document.getElementById("visites").innerHTML = nb;
}
};
</script>
</head>
<body>
<div id="visites">0</div>
</body>
</html>

Sympa willpower, simple et sort. :ccool:

Bonne soirée

07/09/2012, 10h19
fabiochelly

Suis-je le seul à penser qu'utiliser le localStorage (propre à l'utilisateur donc) est une idée absurde pour implémenter un compteur de visites ?

Un compteur de visites et censé renvoyer le nombre de visites de tous les utilisateurs. Or, tel qu'il est implémenté, il renverra le nombre de visites de l'utilisateur et sera réinitialisé dès que l'utilisateur vide son cache.
07/09/2012, 10h50
Willpower
Citation:

Envoyé par headmax

Sympa willpower, simple et sort. :ccool:

En fait, y'a encore plus simple (et sort) :

Code:

var nb = Number(localStorage.getItem("visites"));

Car
Code:

1 2 parseInt(null) // NaN Number(null) // 0
;)
09/09/2012, 15h00
headmax

Merci pour l'info willporwer ;)