Tempi di reazione di uno script lenti

[Fox82]

Ciao a tutti!

Ho modificato in questo modo uno script tra gli esempi
della sezione Javascript facendo in modo che cambiasse immagine al click del mouse.

<SCRIPT LANGUAGE = "JavaScript">

<!--
    window.onerror=null;

    nome1 = new Image(); nome1.src ="blu.jpg";
    nome2 = new Image(); nome2.src ="verde.jpg";

    attiva = false;

    function inactive(imgName)
    {
        if (attiva == true)
        {
	    attiva = false;
            img = eval("nome2.src");
	    document [imgName].src = img;
	}
	else
	{
	    attiva = true;
            img = eval("nome1.src");
	    document [imgName].src = img;
	}
    }

// -->

</SCRIPT>

La pagina di prova: http://www3.csr.unibo.it/~panico/prova/prova.html

Funziona abbastanza bene ma se clicco in tempi ravvicinati non cambia immagine. C'è un modo per risolvere il problema?

Grazie!

[Mich_]

eval() e` meglio non usarlo (tra l'altro e` deprecato)

    window.onerror=null;

    nome1 = new Image(); nome1.src ="blu.jpg";
    nome2 = new Image(); nome2.src ="verde.jpg";

    attiva = false;

    function inactive(imgName)
    {
        if (attiva) {
	    attiva = false;
	    document.images[imgName].src = nome2.src;
	} else {
	    attiva = true;
	    document.images[imgName].src = nome1.src;
	}
    }

Cosi` dovrebeb fare la stessa cosa, ed essere molto piu` performante.

Nota: Il tuo codice funzionava solo in IE, ora dovrebbe essere crossbrowser.

Ciao
Michele

[Fox82]

Grazie!

Tra l'altro eval non ho ben capito a cosa serve, l'ho copiato pari pari dallo script che ho trovato nella sezione Javascript

[Mich_]

eval() esegue quanto scritto nella stringa del parametro, e serviva per trasformare una stringa nell'oggetto corrispondente. Era essenziale per i browser tipo NN4 e IE5.

Nel tuo caso specifico non ha molto senso.

Ciao
Michele

[Fox82]

Scusa ancora, ho provato a generalizzare lo script per usarlo su matrici di immagini, purtroppo ora non funge + niente

            <script language="JavaScript">
            <!--
            window.onerror = null;
            on = new Image();
            on.src = "verde.jpg";

            off = new Image();
            off.src = "blu.jpg";

            stato = new Array(2);
            for (i = 0; i < 2; i++)
            {
                stato[i] = new Array(5);
                for (j = 0; j < 5; j++)
                    stato[i][j] = false;
            }

            nome = new Array(2);
            for (i = 0; i < 2; i++)
                nome[i] = new Array(5);

            nome[0][0] = "A1";
            nome[0][1] = "A2";
            nome[0][2] = "A3";
            nome[0][3] = "A4";
            nome[0][4] = "A5";
            nome[1][0] = "B1";
            nome[1][1] = "B2";
            nome[1][2] = "B3";
            nome[1][3] = "B4";
            nome[1][4] = "B5";

            function switch(r, c)
            {
                if (stato[r][c] == false)
                {
	            document [nome[r][c]].src = on.src;
	        }
	        else
	        {
	            document [nome[r][c]].src = off.src;
	        }

	        stato[r][c] = !stato[r][c];
            }

C'è qualcosa che sbaglio? Sembra che non inizializzi i vettori...
Le chiamate a switch sono queste:

                <td>
                    [img]blu.jpg[/img]
                </td>
                <td>
                    [img]blu.jpg[/img]
                </td>
                <td>
                    [img]blu.jpg[/img]
                </td>

ecc...

Per provarlo...
http://www3.csr.unibo.it/~panico/prova/sala.html

Grazie!!!

[Mich_]

Sicuramente switch non puo` essere il nome di una funzione (e` una istruzione JS).
Avrei dei dubbi anche su on e off, che potrebbero essere parole chiave (ma forse nel tuo contesto vanno bene).

Poi vedi questa altra riformulazione della tua funzione switch:

function myswitch(r, c) {
  document [nome[r][c]].src = (stato[r][c]) ? off.src : on.src;
  stato[r][c] ^= true;
}

veramente sul XOR per invertire una variabile booleana ho qualche dubbio, ma sarebbe elegantissimo.

Ciao
Michele

[Mich_]

Correzione

C'e` ancora un errore: un oggetto non si puo` montare in codesto modo; occorre scrivere:

document.images[nome[r][c]].src = (stato[r][c]) ? off.src : on.src;

Ciao
Michele

[Fox82]

Funziona tutto ora!
Grazie di nuovo