Fichier externe .js

jjaco Messages postés 534 Date d'inscription   Statut Membre Dernière intervention   -  
AssassinTourist Messages postés 6029 Date d'inscription   Statut Contributeur Dernière intervention   -
Bonjour,

Bonjour,

Voici le code javascript pour un compte à rebours qui fonctionne très bien si je le mets dans le body

<script language="javascript">
// PLF - https://twitter.com/ceosammassey
datedujour = new Date();
hdj=datedujour.getHours();
mdj=datedujour.getMinutes();
sdj=datedujour.getSeconds();
date_even = new Date(2012, 5, 30, hdj, mdj, sdj); //janv =0 > decemb =11
date1=datedujour.getTime();
date2=date_even.getTime();
nbrej= date2-date1;
nbrej= Math.round((Math.round(nbrej)/1000)/60/60/24);
document.write("<font size=\"4\"color=\"#FFFFFF\" face=\"Verdana, Arial, Helvetica, sans-serif\"><b>plus que "+nbrej+" jours avant les vacances</b></font>");
</script>


Si je le mets dans un fichier exetrne comme ceci :
fichier.js
// JavaScript Document

function decompte()
{
("datedujour = new Date();
hdj=datedujour.getHours();
mdj=datedujour.getMinutes();
sdj=datedujour.getSeconds();
date_even = new Date(2012, 5, 30, hdj, mdj, sdj); //janv =0 > decemb =11
date1=datedujour.getTime();
date2=date_even.getTime();
nbrej= date2-date1;
nbrej= Math.round((Math.round(nbrej)/1000)/60/60/24);
document.write("<font size=\"4\" color=\"#FFFFFF\" face=\"Verdana"><b>plus que "+nbrej+" jours avant les vacances</b></font>");"
);
}

Dans le head :
<script type="text/javascript" src="fichier.js" language="javascript"></script>

Et dans le body :
<script type="text/javascript">decompte();</script>

Il ne se passe rien

Merci de vos conseils





A voir également:

1 réponse

Réponse 1 / 1
Groarh Messages postés 682 Date d'inscription   Statut Membre Dernière intervention   185
 
Salut,

vérifications élémentaires :
- assure-toi que le fichier .js est dans le même dossier que ta page html ;
- vérifie que tu n'as pas fait de fautes de frappe en recopiant le script.

Ensuite, j'ai quelques recommandations :
- Tu as un double Math.round dans la même formule, celui qui est à l'intérieur est inutile ;
- document.write est à éviter absolument, utilise les méthodes du DOM comme appendChild et createElement à la place (une rapide recherche te donnera de nombreux tutos).

Sinon, je ne vois pas d'erreurs a priori.
1
AssassinTourist Messages postés 6029 Date d'inscription   Statut Contributeur Dernière intervention   1 312
 
Ou plus simple document.getElementById =)
0