Fichier externe .js

jjaco Messages postés 575 Statut Membre -  
AssassinTourist Messages postés 409 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

1 réponse

  1. Groarh Messages postés 706 Statut Membre 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
    1. AssassinTourist Messages postés 409 Date d'inscription   Statut Contributeur Dernière intervention   1 313
       
      Ou plus simple document.getElementById =)
      0