JS meglio inline o esterno?

[fandangood]

In un piccolo Sito di circa 100 pagine .htm al fine di creare un menù con rollover delle immagini ho inserito in ciascuna pagina la seguente istruzione:

<script LANGUAGE="JavaScript">
<!--

a11=new Image(90,35)
a11.src="go.gif"
a12=new Image(90,35)
a12.src="go1.gif"
a21=new Image(90,35)
a21.src="home.gif"
a22=new Image(90,35)
a22.src="home1.gif"
a31=new Image(90,35)
a31.src="mail.gif"
a32=new Image(90,35)
a32.src="mail1.gif"
a51=new Image(90,35)
a51.src="car.gif"
a52=new Image(90,35)
a52.src="car1.gif"
a61=new Image(90,35)
a61.src="ord.gif"
a62=new Image(90,35)
a62.src="ord1.gif"
a71=new Image(90,35)
a71.src="spd.gif"
a72=new Image(90,35)
a72.src="spd1.gif"
a91=new Image(90,35)
a91.src="aiu.gif"
a92=new Image(90,35)
a92.src="aiu1.gif"

function filter(imagename,objectsrc){
if (document.images)
document.images[imagename].src=eval(objectsrc+".src")
}

//-->
</script>

Avrei alcune domande:
1) pensate sia meglio creare un .js esterno?
2) Una volta caricato la prima volta (il file .js esterno), il browser lo legge dalla cache come avviene per i file .css?
3) Ci sarebbero miglioramenti in termini di velocità di caricamento delle pagine .htm?

Grazie e buon lavoro

[fcaldera]

la risposta è si per tutte e tre le domande (l'ultima poi è diretta conseguenza delle prime due)

Ciao

[Mich_]

Altre precisazioni:

- filter e` parola chiave in qualche browser: converrebbe cambiare il nome della funzione.

- eval() e` funzione deprecata e molto lenta; inoltre nel tuo caso e` inutile:
document.images[imagename].src = objectsrc+".src";
oppure:
document.images[imagename].src = "./"+objectsrc+".src";

[fandangood]

- filter e` parola chiave in qualche browser: converrebbe cambiare il nome della funzione.
- eval() e` funzione deprecata e molto lenta; inoltre nel tuo caso e` inutile:
document.images[imagename].src = objectsrc+".src";
oppure:
document.images[imagename].src = "./"+objectsrc+".src";

Mi potresti postare come diventerebbe esattamente questa funzione:
function filter(imagename,objectsrc){
if (document.images)
document.images[imagename].src=eval(objectsrc+".src")
}

Grazie

[Mich_]

L'avevo gia` scritto, comunque:

function myfilter(imagename,objectsrc){
  if (document.images) {
    document.images[imagename].src = "./"+objectsrc+".src";
  }
}

Non capisco l'estensione .src alle immagini, ma c'era anche prima.

[fandangood]

function myfilter(imagename,objectsrc){
if (document.images) {
document.images[imagename].src = "./"+objectsrc+".src";
}
Mi da un errore...

Questo è un esempio del codice .htm che dovrebbe creare il rollover tramite la funzione JS:

[img]home.gif[/img]

Suggerimenti?

[Mich_]

Che errore ti da`? Su quale linea?

Esistono i file a22.src e a21.src ?

Tali file sono delle immagini?

[fandangood]

Questo è parte del codice attuale e funzionante

<script LANGUAGE="JavaScript">
<!--
b21=new Image(130,30)
b21.src="home.gif"
b22=new Image(130,30)
b22.src="home1.gif"

function filter(imagename,objectsrc){
if (document.images)
document.images[imagename].src=eval(objectsrc+".src")
}

//-->
</script>

e che viene richiamato all'interno della pagina .htm nel seguente modo:
[img]home.gif[/img]

Se modifico il js come da te suggerito in:

<script LANGUAGE="JavaScript">
<!--
b21=new Image(130,30)
b21.src="home.gif"
b22=new Image(130,30)
b22.src="home1.gif"

function filter(imagename,objectsrc){
if (document.images) {
document.images[imagename].src = "./"+objectsrc+".src";
}
//-->
</script>

Ottengo il seguente errore:
Linea: 108 (Fa riferimento alla riga <a href="index.htm" ecc...)
Carattere: 1
Errore: previsto oggetto
Codice: 0

Purtroppo non so praticamente nulla di JS...

[Mich_]

Ho capito: b22 non e` una stringa (come immaginavo), ma una variabile JS che rappresenta un oggetto.

Quindi e` chiaro che la sintassi che avevo suggerito non funziona. Prova cosi`:
onmouseover="filter('b20', b22);" // ho tolto gli apici al secondo parametro
Poi la funzione:

function myfilter(imagename,objectsrc){
  if (document.images) {
    document.images[imagename].src = objectsrc.src;
  }
}

[fandangood]

nel senso che non dà più nessun errore JS ma quando si effettua il rollover successivamente al posto della .gif rimane un'immagine "vuota".

Ti ringrazio ugualmente per l'interessamento.