miglioramento codice javascript

[pietro09]

Ho un oggetto activeX referenziato con la variabile map.

Se volessi lanciare il suo evento map.zoomScale, per non incorrere in errore devo prima verificare se una proprietà, Busy, è vera o falsa.

Ho pensato di ricorrere a setTimeout per rilanciare la funzione, e funziona.

Voglio però che setTimeout funzioni al massimo per 10 volte.
Per fare questo ho utilizzato una variabile contatore n, all'esterno della funzione.

Chiedo se non si possa migliorare facendo a meno di variabili esterne e demandando tutto alla funzione.

Ecco il mio codice:

var n = 0;
function _zoomScale(lat, lon, scale)
{
	
	n++;
	if(n > 10) {alert("troppe iterazioni"); return;}
	if(map.Busy) window.setTimeout(function(){_zoomScale(lat, lon, scale)}, 1);
	else {map.zoomScale(lat, lon, scale);alert(n);n=0;}
}

e la richiamo con:
_zoomScale(lat, lon, scale);

[Mich_]

No.
JS e` un linguaggio interpretato. Significa che viene letto ed eseguito subito.

Una funzione (e le sue variabili) esistono solo mentre viene eseguita, mentre vengono perse subito dopo.

Quindi hai mecessita` di una variabile globale, e questa e` creata mediante un var fuori dalla funzione.

[pietro09]

Ho optato per l'aggiunta dell parametro n

//----------------------------------------------------------------
// lancia il metodo map.zoomScale(lat, lon, scale) quando la mappa
// non è impegnata
//----------------------------------------------------------------
function _zoomScale(n, lat, lon, scale)
{
	if(n > 10) {alert("troppe iterazioni in _zoomScale"); return;}
	if(map.Busy) {n++; window.setTimeout(function(){_zoomScale(n, lat, lon, scale)}, 250)};
	else {map.zoomScale(lat, lon, scale);/*alert(n);*/}
}

e la richiamo con:
_zoomScale(0, lat, lon, scale);

Sembra che funzioni, nel senso che vedo, con alert(n), il numero di volte che lancio la funzione.

[pietro09]

chiedo ancora un parere:

dato che l'oggetto map ha diversi metodi che, per funzionare debbono controllare la proprietà booleana Busy, ho provato a fare una funzione generale del tipo:

function lancia_metodo_timeout(n, stringa_funzione)
{
	if(n > 10) {alert("troppe iterazioni in " + stringa_funzione); return;}
	if(map.Busy) {n++; window.setTimeout(function(){lancia_metodo_timeout(n, stringa_funzione)}, 250)};
	else { eval(stringa_funzione); /*alert(n);*/}
}

ossia, per esempio, al posto di richiamare:

map.zoomScale(lat, lon, scale);

chiamo:

lancia_metodo_timeout(0, "map.zoomScale(" + lat + ", " + lon + ", " + scale + ")"  );

cosa ne pensi? :master:

[Mich_]

Potrebbe anche funzionare, ma la funzione eval() e` deprecata, e potrebbe non funzionare in qualche browser presente o futuro.

Non e` fattibile una funzione per ogni metodo?
Tipo quella che hai gia` fatto, oppure:

function _zoomScale(lat, lon, scale, n) {
  if(n == undefined) n=10;
  if(--n <= 0) {alert("troppe iterazioni in _zoomScale"); return;}
  if(map.Busy) {
     window.setTimeout(function(){_zoomScale(lat, lon, scale, n)}, 250)};
  else {map.zoomScale(lat, lon, scale);/*alert(n);*/}
}

Nota che con questa variante, nella prima chiamata non devi specificare n.

[pietro09]

Dunque, mi hai proposto una cosa interessante.
In pratica simulo un parametro opzionale n, che, chiaramente deve essere messo alla fine.

Per l'altra cosa, una funzione wrapper per ogni metodo critico, era la prima strada scelta e, dopo un po' di prove, mi pare che sia la migliore.

Comunque, ho come la sensazione che la cosa sia interessante anche in altri contesti

Ciao