cosa sbaglio(js esterno)

[cepi]

ho in un js esterno questa funzione di banner a rotazione

function choices()
{
this[0] = 3;
this[1] = "<a href=\"http://tod.altervista.org"><img src=\"images/todbann.gif\" width=450 height=73 border=1 alt=\"Tod Official Web Site - TOD Black Metal Band\"></a>";
this[2] = "<a href=\"http://cepi.altervista.org/todfc"><img src=\"images/todclub.gif\" width=377 height=46 border=1 alt=\"Tod Club - The Official TOD's Fan CLub\"></a>";
this[3] = "<a href=\"http://cepi.altervista.org"><img src=\"images/cepi.gif\" width=468 height=60 border=1 alt=\"Cepi - My WebRing\"></a>";
}
function popUpBanner(list)
{
var today = new Date();
var choiceInstance = new choices();
var Banner = choiceInstance[(today.getSeconds() % choiceInstance[0]) + 1];
}
function grabBanner()
{
var today = new Date();
var choiceInstance = new choices();
var Banner = choiceInstance[(today.getSeconds() % choiceInstance[0]) + 1];
return Banner;
}
document.writeln(grabBanner());



ma mi da errore ,che cose?

[bigmaul]

Posso dirti che 1 errore c'è nella 1à funzione.
Ad ogni elemento di "this" (escluso l'elemento 0) hai omesso 1 '\\' alla fine di ogni link

(x capirci: this[1] = "<a href=\"http://tod.altervista.org\"><img src=\"images/todbann.gif\" width=450 height=73 border=1 alt=\"Tod Official Web Site - TOD Black Metal Band\"></a>"; )

X le altre 2 funzioni non posso dirti molto poikè non ho la + pallida idea di cosa significa "new choise()"

1 curiosità...serve passare list nella funzione popUpBanner ?

[cristiano_longo]

se magari scrivessi l'errore ... nel dettaglio ...

[bigmaul]

In rosso ho segnato lo slash (\) che hai omesso.

This[1] così com'è conterrà "<a href=\"http://tod.altervista.org"
tutto il resto è da considerarsi "errore"

Così è + corretto !!!
this[1] = "<a href=\"http://tod.altervista.org\"><img src=\"images/todbann.gif\" width=450 height=73 border=1 alt=\"Tod Official Web Site - TOD Black Metal Band\"></a>";

[Francis87]

saresti un po' OT qui.. meglio scripting!

ma ormai hai ottenuto una risp!

[cepi]

grazie ,ho risolto il probblema mettendo la / dove mancava.


X bigmaul

1 curiosità...serve passare list nella funzione popUpBanner ?

non ne ho la minima idea


ciao