Discussion :

[Awery]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 ?

[Bovino]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

nbvisites = paseInt(nbvisite);

[Awery]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

nbvisites = parseInt(nbvisite);

corriger mais ça marche toujours pas

[Bovino]

j'ai mis ça dans le body

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

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

[Awery]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

nbvisite = 2;

[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

[headmax]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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+

[Willpower]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Il me semble.

[Bovino]

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 !

[headmax]

Salut à tous,

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

Bonne soirée

[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.

[Willpower]

Sympa willpower, simple et sort.

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

Car

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
parseInt(null) // NaN
Number(null) // 0

[headmax]

Merci pour l'info willporwer