Script Banniéres aléatoires

Fermé
Kaya - 17 mars 2012 à 15:40
Autumn`Tears Messages postés 1054 Date d'inscription samedi 14 mars 2009 Statut Membre Dernière intervention 23 octobre 2013 - 17 mars 2012 à 17:49
Bonjour,

Voilà n'y connaissant strictement rien en programmation javascript, j'ai essayé de modifier un script trouvé sur le net (script pour un affichage de banniére de manniéres aléatoire)

et comme prévu et bah ca marche pas :) quelqu'un pourrais m'aider ?

voici le code :


<script language="JavaScript"><![CDATA[

function ban_article()
{

var imagenumber = 5 ;
var randomnumber = Math.random() ;
var rand1 = Math.round( (imagenumber-1) * randomnumber) + 1 ;
images = new Array
images[1] = "IMAGE #1"
images[2] = "IMAGE #2"
images[3] = "IMAGE #3"
images[4] = "IMAGE #4"
images[5] = "IMAGE #5"
var image = images[rand1]

links = new Array
links[1] = "IMAGE #1"
links[2] = "IMAGE #2"
links[3] = "IMAGE #3"
links[4] = "IMAGE #4"
links[5] = "IMAGE #5"
var link = links[rand1]

document.write('<A HREF="' + link + '"><IMG SRC="' + image + '" border="0"></a>')
}
// ]]></script>

L'appel du script :

<script language="JavaScript">
document.write(ban_article);
</script>

1 réponse

Autumn`Tears Messages postés 1054 Date d'inscription samedi 14 mars 2009 Statut Membre Dernière intervention 23 octobre 2013 146
17 mars 2012 à 16:51
Salut,

Tu as oublié des ";" à la fin de tes instructions !
mages = new Array();
images[1] = "IMAGE #1";
et ainsi de suite !
1
ça marchais sans les ; avant ...

ça viens plutôt de la fin du script et/ou de l'appel au script qui merde...
0
Autumn`Tears Messages postés 1054 Date d'inscription samedi 14 mars 2009 Statut Membre Dernière intervention 23 octobre 2013 146
Modifié par Autumn`Tears le 17/03/2012 à 17:03
Essaie, pour l'appel, de faire uniquement : ban_article();
0
et cette ligne : document.write('<A HREF="' + link + '"><IMG SRC="' + image + '" border="0"></a>') tu parait bonne ?

je cours essayer ton idée... je te tiens informer
0
Autumn`Tears Messages postés 1054 Date d'inscription samedi 14 mars 2009 Statut Membre Dernière intervention 23 octobre 2013 146
17 mars 2012 à 17:11
Elle est bonne, oui
0
Bonne nouvelle ca a l'air de marcher :)

une derniere question pendant que j'y suis ou doit mettre un target blank (pour ouvrir le liens dans une nouvelle fenetre ?

et je suis un peu soupsonneux est-ce vraiment un affichage aléatoire ? j'ai l'impression qu'il reste bloqué sur seulement 2 - 3 bannieres... (bon j'avoue je galere un peu)
0