Script bannière aléatoire

[Invité]

Bonjour,

Je souhaite afficher une bannière aléatoirement (parmi 3) sur mon forum.

Sur la page gestion des codes javascripts, j'ai installé le script suivant :

Code:

<SCRIPT LANGUAGE="JavaScript">
var nbimage= 30;
var width;
var height;
var url;
var alte;
function bannH()
{
numimage= Math.round(Math.random()*(nbimage-1)+1);
if (numimage <= 10)
{
gotosite = "";
url = "http://i48.servimg.com/u/f48/15/18/79/87/728x9012.jpg";
alte = "runescapades générique";
height = "90";
width = "728";
}
if (numimage > 10 && numimage <= 20)
{
gotosite = "";
url = "http://i48.servimg.com/u/f48/15/18/79/87/728x9012.jpg";
alte = "runescapades generique";
height = "90";
width = "728";
}
if (numimage > 20)
{
gotosite = "";
url = "http://i48.servimg.com/u/f48/15/18/79/87/728x9012.jpg";
alte = "runescapades generique";
height = "90";
width = "728";
}
if(gotosite != "")
   {
   document.write ('<A HREF="' + gotosite + '">');   
   }
document.write('<IMG SRC="' + url + '" ALT="' + alte + '" HEIGHT="' + height + '" WIDTH="' + width + '" BORDER=0 / >')
if(gotosite != "")
   {
   document.write('</A>')
   }
}
</SCRIPT>

Et à l'endroit ou je souhaite afficher la bannière (template overall_header), le code suivant :

Code:

<SCRIPT language="JavaScript">
bannH();
</SCRIPT>

Rien ne s'affiche :no

Pourtant, si je créé une page html, et que j'insère ces codes respectivement entre les balises HEAD et BODY, cela fonctionne parfaitement.

Auriez vous une piste pour résoudre mon problème, je patauge depuis plusieurs heures...

Merci pour votre aide.

Olivier

[Invité]

Problème résolu après des ... heures de test :

Il semblerait que la compilation du script pause problème.
En effet, si je compile mon script à l'aide du closer-compiler le résultat donne ceci
(noms des variables modifiés depuis, ainsi que le nombre de bannières qui est passé à 4):

Code:


var bannHnbimage=40,bannHwidth,bannHheight,bannHurl,bannHalte;
function bannH(){numimage=Math.round(Math.random()*(bannHnbimage-1)+1);10>=numimage&&(gotosite="",bannHurl="http://i48.servimg.com/u/f48/15/18/79/87/728x9012.jpg",bannHalte="runescapades generique",bannHheight="90",bannHwidth="728");10<numimage&&20>=numimage&&(gotosite="",bannHurl="http://i48.servimg.com/u/f48/15/18/79/87/728x9012.jpg",bannHalte="runescapades generique",bannHheight="90",bannHwidth="728");20<numimage&&30>=numimage&&(gotosite="",bannHurl="http://i48.servimg.com/u/f48/15/18/79/87/728x9012.jpg",
bannHalte="runescapades generique",bannHheight="90",bannHwidth="728");30<numimage&&(gotosite="",bannHurl="http://i48.servimg.com/u/f48/15/18/79/87/728x9012.jpg",bannHalte="runescapades generique",bannHheight="90",bannHwidth="728");""!=gotosite&&document.write('<A HREF="'+gotosite+'">');document.write('<IMG SRC="'+bannHurl+'" ALT="'+bannHalte+'" HEIGHT="'+bannHheight+'" WIDTH="'+bannHwidth+'" BORDER=0 / >');""!=gotosite&&document.write("</A>")};

Ce n'est plus très "lisible" facilement mais en remplacant mon script original par ce script compilé, miracle, tout fonctionne à merveille....

L'avantage de ce script est qu'il permet de rediriger chaque bannière vers une page HTML (remplir alors l'adresse entre les parenthèses après le gotosite), et qu'il permet également de redimensionner les bannières.