consejo para salir de una funcion

[doylelives]

Tengo 5 selects dependientes y dentro de un array alojo los id de estos 5 select.
Llamo a la funcion javascript que recibe el id del select actual, busca el id del select que le sigue y finalmente mediante ajax cargar ese select.
Pero si llegué el 5º select al buscar el siguiente se produce un error (advertencia en IE), porque no existen mas select.
Lo solucioné con un return.

Código PHP:

  function cargarSelect(idSelects){ 
   ...busca la posicion en el array 
   if (!listadoSelects[posicionSelectDestino]){ 
        return; 
   } 
   ...ejecuta ajax 
}  
   

Que opinan de ese escape de la funcion?
Porque en otros codigos que hice utilizaba esta condición

Código PHP:

  if(idSelectOrigen!=listadoSelects[listadoSelects.length-1]){  
   

pero me parece que el return es mas eficiente. o estoy equivocado?
Gracias

[Panino5001]

Sin mirar todo el contexto, ambas formas parecen igualmente efectivas. Para saber cuál es mejor podrías comparar tiempos de ejecución y consumo de memoria.

[doylelives]

Aha! y sino es mucho ¿como podria medir el tiempo de ejecucion?

[TonyChile]

Creo que return false; es mas adecuado pero si la que creaste te funciona no tiene nada de malo.
Disculpa pero te quiero pedir un favor.
podrias poner eso que hiciste con los select
es que necesito algo similar

Que estes muy bien

[doylelives]

Pues aqui va, aunque debo decirte que lo desarrolle yo y puede haber errores aunque funciona en IE y FF sin problemas. (lo estoy modificando borra lo que tira error)
Codigo js

Código PHP:

  function cargarSelect(idSelects){ 
    posicionSelectDestino = buscarEnArray(listadoSelects, idSelects)+1; 
    if (!listadoSelects[posicionSelectDestino]){ 
        return; 
    } 
    posicionVaciar=posicionSelectDestino+1; 
    objSelectVaciar=document.getElementById(listadoSelects[posicionSelectDestino]+'Span'); 
 
    objSelect=document.getElementById(listadoSelects[posicionSelectDestino]+'Span');//agrega Div al final, ej: select3Div 
    catPrevia =(document.getElementById(idSelects)).value; 
    var serverPage = 'ajaxCrearSelect.php?catPrevia='+catPrevia+'&select='+listadoSelects[posicionSelectDestino]; 
    processajax(serverPage, objSelect, "get", ""); 
}  
   

codigo html

Código PHP:

 <span id="select1Span"> 
<select name="filtrocat1" size="10" id="select1" onChange="cargarSelect(this.id)"> 
  <option value="altaCategory.php" selected>(ninguna)</option><? 
  while ( $row = mysql_fetch_array($result) ) { ?> 
   <option value="<?=$row[cat_id]; ?>"<? if ($_GET[cat1]==$row[cat_id]) echo 'selected'; ?>> 
   <?=$row['cat_name']; ?> 
  </option> <? 
} ?> 
    </select></span> 
<span id="select2Span"></span> 
<span id="select3Span"></span> 
<span id="select4Span"></span> 
<span id="select5Span"></span>  

[doylelives]

Me olvidaba codigo js

Código PHP:

  var listadoSelects=new Array(); 
listadoSelects[0]="select1"; 
listadoSelects[1]="select2"; 
listadoSelects[2]="select3"; 
listadoSelects[3]="select4"; 
listadoSelects[4]="select5";  
   

y mas php llamado por ajax

Código PHP:

 $listadoTablas=array( 
"select1"=>"category1", 
"select2"=>"category2", 
"select3"=>"category3", 
"select4"=>"category4", 
"select5"=>"category5" 
); 
 
 
$catPrevia = $_GET['catPrevia']; 
$select = $_GET['select']; 
if(validaSelect($select) && validaOpcion($catPrevia)){ 
    $tabla=$listadoTablas[$select]; 
    $respuesta=mysql_query("SELECT cat_id, cat_name FROM $tabla WHERE prev_cat='$catPrevia'") or die(mysql_error());?> 
    <select name="filtrocat1" size="10" id="<?=$select?>" onChange="cargarSelect(this.id)"> 
        <option value="altaCategory.php" selected>(ninguna)</option><? 
        while ( $registro = mysql_fetch_array($respuesta) ){ ?> 
            <option value="<?=$registro['cat_id'];?>"><?=$registro['cat_name'];?></option><? 
        }?> 
    </select><?     
}   

[TonyChile]

Gracias.
si lo quero ver y de modificarlo va a ser mucho
si veo que me puede ayudar si es para poder guiarme
ya que stoy usando asp

[Panino5001]

Para medir el tiempo esta es la idea más o menos:

Código PHP:

  <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> 
<html xmlns="http://www.w3.org/1999/xhtml"> 
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" /> 
<title>Documento sin t&iacute;tulo</title> 
<script> 
function t(id){return document.getElementById(id);} 
function aMedir(){ 
    var inicio=new Date().getTime(); 
    var i=0; 
    while(i<1000){ 
        i++; 
        t('log').innerHTML+='<br />'+i; 
    } 
    var fin=new Date().getTime(); 
    t('log').innerHTML+='<br />'+(fin-inicio)+' milisegundos'; 
     
} 
onload=aMedir; 
</script> 
</head> 
 
<body> 
<div id="log"></div> 
</body> 
</html>  
  

[doylelives]

ok gracias Panino5001!!