Tester si cookie existe

Version imprimable

28/07/2012, 16h47
Erudix

Tester si cookie existe
Bonjour,

J'aurais besoin de savoir si un cookie est déjà créé, et cela depuis différentes pages de mon site.

J'ai ce code mais il ne fonctionne pas, je ne vois pas l'erreur :?
Code:

1 2 3 4 5 6 7 <script type="text/javascript"> if (navigator.cookieEnabled) { if (document.cookie.indexOf("mycookie") == -1) { le cookie n'existe pas ! } } </script>
Quelqu'un a t-il une correction à apporter à ce code ou un code aussi simple qui fonctionne ?

Merci par avance

Salut,

Je sais pas si cela vient de là :
Code:

1 2 3 if (document.cookie.indexOf("mycookie") == -1) { le cookie n'existe pas ! }
mais tu n'as pas compris la philosophie de javascript:
Code:

1 2 3 if (document.cookie.indexOf("mycookie") == -1) { alert("le cookie n'existe pas !"); }

28/07/2012, 20h24
owenho
Bonjour,
normalement un coockie a un nom, donc on doit récuperer la valeur du coockie dans un variable, et testé si la variable est vide ou nom:
Code:

1 2 3 4 5 var contenucoockie=""+readCookie('nomcoockie'); if(contenucoockie='null') { // coockie existe sinon n'existe pas }
28/07/2012, 21h52
Erudix

Cela ne fonctionne pas, es-tu sûr que readCookie fonctionne bien?

readcoockie: fonction permet de lire un coockie:

Code:

1
2
3
4
5
6
7
8
9
10
function readCookie(name) {
	var nameEQ = name + "=";
	var ca = document.cookie.split(';');
	for(var i=0;i < ca.length;i++) {
		var c = ca[i];
		while (c.charAt(0)==' ') c = c.substring(1,c.length);
		if (c.indexOf(nameEQ) == 0) return c.substring(nameEQ.length,c.length);
	}
	return null;
}

pour créer un coockie:

Code:

1
2
3
4
5
6
7
8
9
10
function createCookie(name,value,days) {
 
	if (days) {
		var date = new Date();
		date.setTime(date.getTime()+(days*24*60*60*1000));
		var expires = "; expires="+date.toGMTString();
	}
	else var expires = "";
	document.cookie = name+"="+value+expires+"; path=/";
}

pour supprimer un coockie:

Code:

1
2
3
function eraseCookie(name) {
	createCookie(name,"",-1);
}

bonne chance

Comme indiqué dans mon premier message, il me faut quelque chose de court. Si je me permettais d'utiliser une fonction alors j'aurais déjà trouvé une solution depuis longtemps.
Code:

1 2 3 4 5 6 7 8 <script type="text/javascript"> if (navigator.cookieEnabled) { if (document.cookie.indexOf("mycookie") == -1) { // action 1 ... // action 2 : créer le cookie mycookie } } </script>
Dans mon code, si mycookie n'existe pas je fais des choses puis je finis par le créer. Le problème de ce code c'est la lecture du cookie : j'ai remarqué que document.cookie.indexOf("mycookie") lis le cookie uniquement si il a été créé par cette même page. Si mycookie est créé par page1.html seul page1.html pourra le savoir ! page1.html ne pourra pas savoir si page2.html à créé le cookie. Pareil pour l'inverse.

Bon essaye ça:

Code:

1
2
3
4
5
6
 
if(document.cookie.indexOf(cookieName+'=')!=-1){
	//le cookie existe
}else{
	//à l'instar de la cuillère le cookie n'existe pas
}

ou ça:

Code:

1
2
3
4
5
6
if(document.cookie == ""){
	    //Pas de cookie
	}
	else{
	   // la page  en cours possède de(s) cookie(s)
	}

29/07/2012, 16h59
Erudix

Dans ton premier code qu'est-ce que cookieName ?
Le deuxième code indique si il y a des cookies ou pas sans préciser lesquels, donc cela ne convient pas...
29/07/2012, 17h16
Watilin

Les fonctions viennent d'ici : http://ppk.developpez.com/tutoriels/...javascript/#LV :D

Erudix, ~~le code de ton test dans ton premier post était bon. Si ton script te dit que le cookie mycookie n'existe pas, c'est qu'il n'existe pas~~. Il faut te demander si la variable mycookie contient réellement ce que tu cherches, mais aussi si le cookie a été correctement écrit, notamment avec les bons paramètres domain ou path.

Je te conseille de lire l'article entier pour adopter une manière homogène de déclarer tes cookies sur tout ton site.

Pourtant il existe car il est visible dans les cookies de mon navigateurs.
Voici le code en entier.
Code:

1 2 3 4 5 6 7 8 9 10 11 <script type="text/javascript"> if (navigator.cookieEnabled) { if (document.cookie.indexOf("yes") == -1) { var today = new Date(), expires = new Date(); expires.setTime(today.getTime() + (24*60*60*1000)); document.cookie = "yes" + "=" + encodeURIComponent("true") + ";expires=" + expires.toGMTString(); window.location.replace('../users.html') } } } </script>
Ce code est le même sur toutes mes pages, et quand je change de page cela recréer un cookie "yes" preuve que le code n'arrive pas à savoir si un cookie existe ou pas.
Je vous serais infiniment reconnaissant si vous pouviez m'expliquer ce qui cloche :(

29/07/2012, 20h01
owenho

cookieName est bien le nom de votre coockie !!!!
toi tu veux testé l'existence d'un coockie précis donc avec son nom !
29/07/2012, 20h05
owenho

Citation:

Envoyé par Watilin

Les fonctions viennent d'ici : http://ppk.developpez.com/tutoriels/...javascript/#LV :D

Erudix, le code de ton test dans ton premier post était bon. Si ton script te dit que le cookie mycookie n'existe pas, c'est qu'il n'existe pas. Il faut te demander si la variable mycookie contient réellement ce que tu cherches, mais aussi si le cookie a été correctement écrit, notamment avec les bons paramètres domain ou path.

Je te conseille de lire l'article entier pour adopter une manière homogène de déclarer tes cookies sur tout ton site.

et oui , vous voulez que j'invente les fonctions de ma téte !
29/07/2012, 20h24
Watilin

Je crois savoir où est le problème : quand tu fais document.cookie.indexOf("yes"), JavaScript va te trouver la première occurence de "yes" dans la chaîne de cookies, que ce soit un nom ou une valeur. Autrement dit, si tu as un cookie de n'importe quel nom dont la valeur contient "yes", indexOf renverra une valeur différente de -1.

Donc en fait, j'ai parlé trop vite, le test du premier post n'était pas bon :aie:
29/07/2012, 21h59
Erudix
J'ai essayé
Code:

1 2 if(document.cookie.indexOf('lecookie=')!=-1){ }else{
ainsi que la recommandation de Watilin en changeant la valeur et nom du cookie mais rien n'y fait, j'ai toujours plusieurs cookies identiques de créés...
Pour info, ces cookies sont toujours les premiers avant les autres cookies donc c'est très surprenant que indexOf ne marche pas :?
30/07/2012, 01h59
Watilin

Et en essayant avec les fonctions de Peter-Paul Koch ?
Au fait, tu testes sous quels navigateurs ?
30/07/2012, 12h08
Erudix

Il ne faut pas que j'utilise de fonction pour conserver un code propre.
J'utilise Firefox
30/07/2012, 12h35
Willpower

Correction de ta phrase :

Citation:

Envoyé par Erudix

Il ne faut ~~pas~~ que j'utilise des fonctions pour conserver un code propre.

Et comme te l'a dit Watilin, la solution est de régler le path si le cookie apparait sur une page et pas une autre.

C'est expliqué dans l'article qu'il t'a d'ailleurs donné :

Citation:

Pour créer un cookie sur ce domaine avec une paire nom / valeur valant 'ppkcookie1=testcookie' et expirant le 28 février 2010, il faut faire :

Code:

document.cookie = 'ppkcookie1=testcookie; expires=Sun, 28 Feb 2010 00:00:00 UTC; path=/'

D'abord, la paire nom / valeur ('ppkcookie1=testcookie').
Puis un point-virgule et un espace.
La date d'expiration dans un format correct (expires=Sun, 28 Feb 2010 00:00:00 UTC).
De nouveau un point-virgule et un espace.
Le domaine (path=/).

Rajoute "path=/" à la fin de ton string lors de la création du cookie.

Ca fonctionne, merci à tous !! :zoubi:

Le code final :

Code:

1
2
3
4
5
6
7
8
9
10
11
	<script type="text/javascript">
	if (navigator.cookieEnabled) {
		if (document.cookie.indexOf("yes") == -1) {
			var today = new Date(), expires = new Date();
			expires.setTime(today.getTime() + (24*60*60*1000));
			document.cookie = "yes" + "=" + encodeURIComponent("true") + ";expires=" + expires.toGMTString() + ";path=/";		
			window.location.replace('../users.html')
			}
		}
	}
	</script>

30/07/2012, 15h38
Willpower
Je rajouterai qqes commentaires :

(1)
Code:

1 2 3 4 5 if(COND1){ if(COND2){ } }
Si il n'y a pas d'autre traitement en dehors du "if" intérieur (pas de else ni de code à l'extérieur) alors tu peux l'écrire plus simplement :
Code:

1 2 3 if(COND1 && COND2){ }
(2)

Code:

(navigator.cookieEnabled && document.cookie.indexOf("yes") == -1)

En javascript, (et probablement dans d'autres langages) il est bon de prendre pour habitude de simplement tester ce que tu comptes utiliser, même si ce que tu as fais est juste, prend pour habitude de simplement tester ce que tu utilises :

Code:

(document.cookie && document.cookie.indexOf("yes") == -1)

(3)

Code:

window.location.replace

La variable window est implicite pour toutes les variables de base, tu peux l'ajouter partout dans ton code, ex : window.document, window.history etc.. mais elle n'est pas nécessaire et tu peux simplement utiliser "location", "document", "history" seuls, sauf si tu les écrases en redéfinissant une variable du même nom ex "var location = ", ce qu'il ne faut surtout pas faire. Donc si tu respectes cette règle de ne pas redéfinir les variables existantes, tu peux te passer de "window" :

Code:

location.replace

Les espaces sont importants !

Citation:

Envoyé par Erudix

Code:

1
2
document.cookie = "yes=true; expires=" + expires.toGMTString() + "; path=/";
// ........................ ^ ici ................................ ^ et là