Discussion :

[Bovino]

@RV :

2) Heu... que veux-tu �viter en contraignant volontairement l'utilisation de param�tres � ce niveau ?

3) Nous sommes bien d'accord
Cependant je ne vois pas le lien (je veux dire : je ne le vois pas encore ) entre eval et ceux que tu d�signes comme ses d�riv�s, je pense notamment � setTimeOut ou setInterval...

Non, je ne cherche pas � contraindre l'utilisation de param�tres, mais les rendre possibles
Pour info, le premier argument de setTimeout et setInterval est :

• soit une cha�ne... si c'est le cas, la m�thode eval() sera appel�e pour interpr�ter le contenu ;
• soit un nom de fonction, mais aucun argument possible ;
• soit une fonction anonyme dans laquelle tu peux appeler une fonction avec des param�tres du contexte, c'est cette m�thode que je pr�conise !

A noter aussi qu'avec Firefox, il est possible d'ajouter des arguments � ces fonctions qui seront transmis en param�tres � la fonction appel�e :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

setTimeout(une_fonction, 1000, param1, param2);

est �quivalent � :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

setTimeout(function(){une_fonction(param1, param2)}, 1000);

malheureusement, n'�tant pas cross-browser, l'int�r�t reste relatif
Pour en revenir � :

1) Je mets le code Javascript en fin de document, pour une question de performance et d'am�lioration progressive.

Cette m�thode permet de modifier le contenu avant le d�clenchement du onload et peut � certains �gards �tre consid�r�e comme plus performante que le classique onload sur le body

2) Je privil�gie les notations litt�rales. C'est plus court et plus lisible � force.

+1

[SpaceFrog]

on ne peut malheureusement pas se passer compl�tement de l'eval sauf � passer par window[ ] masi cela revient au m�me
Je pense en particulier � un retour d'objet json (ou hash) par ajax, l'eval est obligatoire...

[Bovino]

on ne peut malheureusement pas se passer compl�tement de l'eval sauf � passer par window[ ] masi cela revient au m�me
Je pense en particulier � un retour d'objet json (ou hash) par ajax, l'eval est obligatoire...

il est possible de passer par un parseur JSON (ici par exemple). D'autre part, khayyam90 nous a propos� r�cemment une source qui devrait bient�t arriver sur les pages FAQ et Sources

Ceci dit, il faut cependant garder � l'esprit que ces bonnes pratiques sont plus un "id�al" � atteindre que des obligations grav�es dans le marbre !

[SpaceFrog]

oui � coup de split ou de regExp ou en parsant

mais bon c'est assez gourmand ...

[RomainVALERI]

Pour info, le premier argument de setTimeout et setInterval est :
* soit une cha�ne... si c'est le cas, la m�thode eval() sera appel�e pour interpr�ter le contenu ;
* soit un nom de fonction, mais aucun argument possible ;
* soit une fonction anonyme dans laquelle tu peux appeler une fonction avec des param�tres du contexte, c'est cette m�thode que je pr�conise !

Ok ^^ j'ignorais la possible utilisation de code "brut" en premier param�tre, je n'ai toujours employ� que les deux autres m�thodes, au d�but des noms de fonctions, et depuis quelques temps des fonctions anonymes...

[franculo_caoulene]

Au fait, quelle est votre fa�on de d�clarer des variables? Nous avons d�j� deux fa�ons de faire diff�rentes. Laquelle pr�f�rez-vous? Laquelle utilisez-vous?

[Auteur]

- Ne pas utiliser des expressions r�guli�res pour comparer / tester des dates. Utiliser... l'objet date exemple :
http://www.developpez.net/forums/d57...t/#post3415858 (oui je me fais de la pub )

* Pour les fonctions r�currentes et/ou non cross-browser, se faire une biblioth�que de fonctions utilitaires. L'exemple le plus courant �tant :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
function $(un_id){
    return document.getElementById(un_id);
}

Cette biblioth�que de fonction pourra par ailleurs �tre d�finie dans l'objet des variables globales !

Juste une pr�cision : pas la peine de t�l�charger les 100-200ko d'une biblioth�que JS pour le plaisir d'utiliser uniquement la fonction $() (si, si j'ai vu �a )
Donc utilisez une biblioth�que de fonctions... pour utiliser les fonctions de la biblioth�que, ou cr�ez votre biblioth�que.

[RomainVALERI]

Tiens au fait : je me demandais autre chose :

>>> est-ce que vous vous autorisez de longs chainages de fonctions ? ou est-ce que vous pr�f�rez "atomiser" un peu les instructions, pour une meilleure lisibilit� ?
un exemple (assez simple) de ce que je veux dire (mais qui pourrait bien s�r �tre bien plus long ou tordu) :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
 
// version chainée, compacte mais pas très lisible sans un rapide examen pour quelqu'un reprenant le code...
var truc = tableauMachin[new Date().getFullYear()].substr((condition)? 0 : n);
// où tableauMachin ne contient que des string, condition est un booléen, et n un number
// au final, truc contiendra une sous-chaine de l'une des chaines contenues dans le tableau

___________________________________

M�me chose pour l'op�rateur ternaire : "(cond)?a:b;"

>>> quand on en chaine ou imbrique plusieurs, �a fait un peu mal les yeux
Mais j'avoue que je le fais quand m�me de temps en temps... et vous autres ? ^^

[le_chomeur]

tout d�pend de l'utilit� et des commentaires qui sont fait sur la source

de m�me qu'un var b = (a) ? c:b;
est parfois plus pratique si l'on n'est pas dans une optique de performance

[franculo_caoulene]

�a d�pend de la lisibilit� dans les deux cas. Si c'est trop long, ou si le code contient d�j� pas mal d'indentations � ce niveau l�, j'ai tendance � �crire sur plusieurs lignes.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
        if (bien.Description) {
            s = (bien.Description.length > 150)
                ? bien.Description.substring(0, 149)
                : bien.Description;
        }

En suivant ton exemple, plut�t tordu, j'�crirais

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
var truc = tableauMachin[
  new Date()
  .getFullYear()
].substr((condition)? 0 : n);

je suis le m�me principe pour les crochets que pour les accolades. L'op�rateur ternaire est court et intimement li� au substring(), le mettre � la ligne rendrait ce bout de code moins lisible.

[franculo_caoulene]

tout d�pend de l'utilit� et des commentaires qui sont fait sur la source

Je ne fais pas de commentaire � ce niveau l�, personnellement. J'essaie d'utiliser des noms assez lisibles et de factoriser au maximum de fa�on � ce qu'on comprenne sans commentaire.

de m�me qu'un var b = (a) ? c:b;
est parfois plus pratique si l'on n'est pas dans une optique de performance

Des performances? C'est moins performant qu'un if?

[DoubleU]

il est possible de passer par un parseur JSON (ici par exemple). D'autre part, khayyam90 nous a propos� r�cemment une source qui devrait bient�t arriver sur les pages FAQ et Sources

C'est quoi l'int�r�t de ce parseur ?

Le seul cas ou le eval se jusifie, c'est sur un retour json d'une requete Ajax. Tant que l'ajax reste non cross-browser, ca signifie que tu maitrises ce qui se passe cot� serveur, donc que tu n'as pas besoin de te m�fier de ce que tu renvoies toi m�me.

[le_chomeur]

oui un test conditionnel est plus lent qu'un if , car il doit �valuer l'instruction pour pouvoir tester la condition.

[franculo_caoulene]

Je veux bien te croire mais j'aimerais une source, je n'en trouve pas.

[le_chomeur]

un exemple simplifi� :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<html>
	<head>
		<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
		<title>Untitled Document</title>
	</head>
	<body>
		<input type="button" onclick="contional()" value="test conditionnel" />
		<input type="button" onclick="normal()" value="test normal" />
		<script type="text/javascript">
			function contional(){
				var Debut = new Date();
 
				var a = true;
 
				for(var i = 0 ; i<500000 ; i++){
					var lol = (a) ? "ok":"ko";
				}
				var Fin = new Date();
				alert((Fin-Debut)+ " ms"); 
			}
 
			function normal(){
				var Debut = new Date();
 
				var a = true;
				for(var i = 0 ; i<500000 ; i++){
					if(a){
						var lol = "ok";
					}
					else{
						var lol = "ko";
					}
				}
				var Fin = new Date();
				alert((Fin-Debut)+ " ms"); 
			}
 
		</script>
	</body>
</html>

sur mon poste , j'ai 10 ms de diff�rences, avec un test ULTRA simple , donc je te laisse imaginer le r�sultat avec des boucles plus complexe ...

[DoubleU]

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
26
27
28
function time(fn, n)
{
 var start = new Date(); 
 for(var v=0; v<n; v++)
 {
  fn();
 } 
 var stop = new Date();
 return (stop.getTime() - start.getTime());
}
 
function test1()
{
  return (Math.random() > Math.random() ? "aaaa" : "bbbb");
}
 
function test2()
{
 if( Math.random() > Math.random() )
  return "aaaa";
 else
  return "bbbb";
}
 
var n = 1000000;
 
alert("test 1:" + time(test1, n));
alert("test 2:" + time(test2, n));

Quelques dizaines de millisecondes pour un million d'it�rations. Bref, du vent...

M�mes tests, conclusions diff�rentes mdr

[franculo_caoulene]

Chez moi le test de doubleU confirme l'hypoth�se, mais pas le test de le_chomeur!

[le_chomeur]

Exp�rience personnel :

http://easynews.free.fr/calendrier/agenda/

optimis� au maximum ( � l'�poque, ce code date d'il y a un an ^^ )

et le "scan" �tant utilis� a chaque changement de cellule, en rempla�ant les ? : par des if/else j'ai une perte cons�quente des performances ...

d'ou ma pr�cision dans un cas pratique et non pas sur des benchs simples

[DoubleU]

Quel "scan"?

[le_chomeur]

pardon, "scan" = mouseover permettant de r�initialiser l'�tat des cellules
( spaffy avait suivit le dev de cette appli ^^ )