link achter roterende banners

[spongebob]

vrienden, ik probeer met een javascriptje een 23-tal banners te laten roteren op dezelfde pagina (want telkens wanneer de pagina opnieuw wordt ingeladen, moet er at random een andere banner verschijnen). Dat is tot nu toe gelukt en werkt perfect.

Maar nu komt het: iedere banner zou uiteraard een link moeten krijgen naar de desbetreffende website en hier stopt mijn kennis van dat javascript. Kan iemand me vertellen hoe ik codegewijs de linken kan toevoegen aan het scritpje?

Ik heb al vanalles geprobeerd, maar dan komen de foutmeldingen er aan?

hier is de desbetreffende website http://admin.appelsien.be/test.html

in de volgende posting zal ik het scriptje posten. Alvast bedankt voor het meezoeken.

[spongebob]

<script> function create() { this.width = '' this.height = '' this.src = '' this.border = '' this.text = '' } ads = new Array() for(var i=1; i<=23; i++) { ads[i] = new create() } ads[1].width = "468"; ads[1].height = "60"; ads[1].src = "http://banners.appelsien.be/aalter.jpg"; ads[1].border = "0";

ads[2].width = “468”;
ads[2].height = “60”;
ads[2].src = “http://banners.appelsien.be/atlascopy.jpg”;
ads[2].border = “0”;

ads[3].width = “468”;
ads[3].height = “60”;
ads[3].src = “http://banners.appelsien.be/belgacom.jpg”;
ads[3].border = “0”;

ads[4].width = “468”;
ads[4].height = “60”;
ads[4].src = “http://banners.appelsien.be/bevrijdewereld.jpg”;
ads[4].border = “0”;

ads[5].width = “468”;
ads[5].height = “60”;
ads[5].src = “http://banners.appelsien.be/deinze.jpg”;
ads[5].border = “0”;

ads[6].width = “468”;
ads[6].height = “60”;
ads[6].src = “http://banners.appelsien.be/dwagulu.jpg”;
ads[6].border = “0”;

ads[7].width = “468”;
ads[7].height = “60”;
ads[7].src = “http://banners.appelsien.be/eglantier.jpg”;
ads[7].border = “0”;

ads[8].width = “468”;
ads[8].height = “60”;
ads[8].src = “http://banners.appelsien.be/jeugdpastoraal.jpg”;
ads[8].border = “0”;

ads[9].width = “468”;
ads[9].height = “60”;
ads[9].src = “http://banners.appelsien.be/jva.jpg”;
ads[9].border = “0”;

ads[10].width = “468”;
ads[10].height = “60”;
ads[10].src = “http://banners.appelsien.be/lochristi.jpg”;
ads[10].border = “0”;

ads[11].width = “468”;
ads[11].height = “60”;
ads[11].src = “http://banners.appelsien.be/maldegem.jpg”;
ads[11].border = “0”;

ads[12].width = “468”;
ads[12].height = “60”;
ads[12].src = “http://banners.appelsien.be/medtronic.jpg”;
ads[12].border = “0”;

ads[13].width = “468”;
ads[13].height = “60”;
ads[13].src = “http://banners.appelsien.be/melle.jpg”;
ads[13].border = “0”;

ads[14].width = “468”;
ads[14].height = “60”;
ads[14].src = “http://banners.appelsien.be/ninove.jpg”;
ads[14].border = “0”;

ads[15].width = “468”;
ads[15].height = “60”;
ads[15].src = “http://banners.appelsien.be/oostvlaanderen.jpg”;
ads[15].border = “0”;

ads[16].width = “468”;
ads[16].height = “60”;
ads[16].src = “http://banners.appelsien.be/oxfam.jpg”;
ads[16].border = “0”;

ads[17].width = “468”;
ads[17].height = “60”;
ads[17].src = “http://banners.appelsien.be/pioneer.jpg”;
ads[17].border = “0”;

ads[18].width = “468”;
ads[18].height = “60”;
ads[18].src = “http://banners.appelsien.be/rotarylokeren.jpg”;
ads[18].border = “0”;

ads[19].width = “468”;
ads[19].height = “60”;
ads[19].src = “http://banners.appelsien.be/sintniklaas.jpg”;
ads[19].border = “0”;

ads[20].width = “468”;
ads[20].height = “60”;
ads[20].src = “http://banners.appelsien.be/villapace.jpg”;
ads[20].border = “0”;

ads[21].width = “468”;
ads[21].height = “60”;
ads[21].src = “http://banners.appelsien.be/weba.jpg”;
ads[21].border = “0”;

ads[22].width = “468”;
ads[22].height = “60”;
ads[22].src = “http://banners.appelsien.be/wichelen.jpg”;
ads[22].border = “0”;

ads[23].width = “468”;
ads[23].height = “60”;
ads[23].src = “http://banners.appelsien.be/zulte.jpg”;
ads[23].border = “0”;

var n = (Math.random()*23);
n = parseInt(n);
if(n >23) {n = n - 23;}
else if(n==0) {n = n + 23;}
n += “”;

var image = ads[n];
var ad = “”;
ad += ‘<img src="’ + image.src + ‘" width=’ + image.width;
ad += ‘n height=’ + image.height + ’ border=’ + image.border;
ad += ‘n><br>’ + image.text + ‘</a>’;
</script>
<script language=“JavaScript”>
<!-- hiding this script
document.write(ad);
// →
</script>

[webj]

Er moet dan sowieso een variabele bij die het website adres draagt. Hieraan voeg je dan toe een regel met .adres [code:1:70eb0fafb2]ads[23].width = "468"; ads[23].height = "60"; ads[23].src = "http://banners.appelsien.be/zulte.jpg"; ads[23].border = "0"; ads[23].adres = "http://www.hierjeadres.nl";[/code:1:70eb0fafb2]

En die zet je hierin terug. Er vanuit gaan de dat iedere banner even breed is en ze allemaal geen border nodig hebben zou je al die variabelen kunnen vervangen door vaste waarden in onderstaand script, maar dat terzijde:

[code:1:70eb0fafb2]ad += ‘<a href="’ + image.adres + ‘">’;
ad += ‘<img src="’ + image.src + ‘" width=’ + image.width;
ad += ‘n height=’ + image.height + ’ border=’ + image.border;
ad += ‘n><br>’ + image.text + ‘</a>’;
[/code:1:70eb0fafb2]

[spongebob]

ja ze hebben inderdaad allemaal dezelfde breedte en hoogte! Heel erg bedankt voor de snelle tussenkomst. Maar moet ik bovenaan het script niet een bijkomend regeltje schrijven?

[webj]

Forum is er om elkaar te helpen :) Geen bijkomend regeltje.

[spongebob]

nog een vraagje:

hoe zorg ik ervoor dat de linken opengaan op een nieuwe, blanco bladzijde en niet op dezelfde waar mijn website al opstaat?

[webj]

Je hebt deze regel: [code:1:f6190d0c3a]ad += '<a href="' + image.adres + '">'; [/code:1:f6190d0c3a]

Je zou daar dan dit van kunnen maken:

[code:1:f6190d0c3a]ad += ‘<a href="’ + image.adres + ‘" target="_blank">’; [/code:1:f6190d0c3a]

Maar let op; ik raad je het niet aan. Als mensen op banners klikken betekent dat waarschijnlijk dat ze je website toch wel wilden verlaten. Maargoed, het is een keuze die je maakt. Ik geef je slechts het scriptje dat je ervoor kan gebruiken

[spongebob]

nee die wijsneuzen moeten op mijn website blijven :wink: heheheh

nogmaals heel erg bedantk voor de snelle interventie