[POO] Bonnes pratiques href="javascript:fonction()"

**LhIaScZkTer** · 03/04/2009, 15h32

Bonjour à tous,

J'ai trouvé un document en furetant sur le net au fil de la lecture je me suis retrouvé sur cette page : Bonnes pratiques

Ma question est la suivante, dans le "ce qu'il ne faut pas faire" il y a :

Ne jamais utiliser le protocole javascript: href="javascript:function1()"

Qu'est-ce que cela veut-il dire ? Qu'il n'est pas bon de faire quelque chose dans le genre :

Code :

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

1
2
 
<a href="Javascript:alert('hello')">hello</a>

et pourquoi ? Il m'arrive de faire des appels à des fonctions comme ceci pour ne pas avoir un # dans l'url.

Quelqu'un pourrait-il m'éclairer sur ce sujet ?
Merci à tous pour votre aide.

**SpaceFrog** · 03/04/2009, 15h39

href attend une url ...
le fait de mettre du js dedans revient à taper ta commande js dans la barre d'adresse

De plus les moteurs de recherche recupèrent l'arborescence de navigation de ton site grace à cela...

si tu tiens à utiliser une balise href pour lancer une fonction $passe par le onclick

Code :

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

<a href=#" onclick="function1(); return false;)>

par contre la ligne du dessus je vois pas

Pas d'événements dans le HTML : onclick="function0()"

on est supposés lancer comment alors les fonctions ?

**LhIaScZkTer** · 03/04/2009, 15h47

Encore une fois merci pour tes lumières SpaceFrog

Je pense que pour le onclick="function0()" , il doit parler de la semantique, genre, les personnes qui foutent (mettent) des onclick dans des div par exemple

Mais alors dans le cas d'un popup modal ?

**RomainVALERI** · 03/04/2009, 15h48

Envoyé par SpaceFrog

on est supposés lancer comment alors les fonctions ?

Tout est dans un script attaché dans le head de la page : le script en question contient notamment ton "point d'entrée", c'est-à-dire la plupart du temps on attache un listener de "onload" sur le body et c'est là qu'on va aller placer les autres listeners (onclic, onmouseover etc.) de la page ^^ un peu laborieux mais "propre"... ^^

edit : décidément je ne refresh pas mes pages assez vite bande de ninjas des forums... ... 3 minutes 4 messages qui s'intercalent et me voilà dans les choux avec mon explication hu hu

**SpaceFrog** · 03/04/2009, 15h53

100% non intrusive et peupler les evénements sur le onload ?

c'est ce que j'avais compris ...

merci de cette confirmation...

Mais bon les paranos du js ...

**LhIaScZkTer** · 03/04/2009, 15h57

houaaa merci à tous pour votre intervention, j'ai vite été débordé par les évènement, je ne m'attendais pas à tant de réaction si vite.

Merci Bovino, pour le lien sur la FAQ il est vraiment très très bien.

En gros, de gérer les évènements de cette manière ne sert qu'à rendre les couches encore plus indépendantes. Cela me rappel un bon vieux MVC, évidemment très schématiquement parlant

**SpaceFrog** · 03/04/2009, 16h01

Code :

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

<noscript> t'es qu'une buse! depèche toi d'activer ton js</noscipt>

**RomainVALERI** · 03/04/2009, 15h43

Le principal défaut de cette pratique est de constituer une infraction au principe de l'unobstrusive javascript.
Comme on ne peut pas déterminer a priori que l'utilisateur final consultera tes pages avec un navigateur capable d'interpréter du javascript (et qui "l'accepte" en terme de sécurité), il faut le proposer sans l'imposer ^^