problema sencillo de eval (o talvez no sea necesario eval)

[jqcod]

Hola

Os planteo el siguiente problema:

Tengo un html con las siguientes etiquetas


<input type="text" name="cod_poblacion" onblur="ponerpoblacion();"/>
<input type="text" name="name_poblacion" disabled/>

El campo name_poblacion se actualiza tras el onblur del anterior campo y muestra la población que le corresponde a partir de:

<script type='text/javascript'>

c01 = new Array("Barcelona")
c02 = new Array("Valencia")
c03 = new Array("Bilbao")
c04 = new Array("Madrid")
c05 = new Array("resto")


function ponerpoblacion()
{
formulario.name_poblacion.value = eval("c" +formulario.cod_poblacion.value)[0];

if(eval("c" +datform.cnae.value)[0] == undefined){
alert ("No ha indicado un codigo válido.")
}
}
</script>

El código me funciona excepto cuando uno no indica un código válido (la parte del IF).
Me gustaría que si uno indicara como código 06 (que no esta en la lista de los posibles). Saltára el alert indicando que no es un código correcto.

Bueno, si podéis indicarme algo...Gracias!

[jqcod]

disculparme, la linea del if en realidad la tengo como:

if(eval("c" +formulario.cod_poblacion.value)[0] == undefined){

[Aijoona]

No hace falta usar eval. Podes acceder a las propiedades de un objeto mediante la notación de corchetes ([]):

Código Javascript:

var 
    c01 = 'WAKA',
    value = '01';
 
window['c' + value] ==  window.c01; // true
window['c' + value] ==  c01; // true
 

---

Te recomendaría que apliques un poco de buenas prácticas.

1. Estás declarando las variables como globales implícitas (no usas var)
2. Estás declarando las variables como globales cuando podrías agruparlas en un objeto (namespace).
3. Los nombres de las variables son horribles y no dicen nada sobre su contenido (semantica).
4. Estás mal usando el eval.

[jqcod]

Gracias Aijoona,

Tomo buena nota de todo lo que me indicas y voy a probar con el método que me propones.

Confío ir aprendiendo y no haceros daño a la vista con código tan feo como el que por ahora me sale.

Saludos.

[jqcod]

...el tema me esta haciendo sufrir, es lo que tiene la inexperiencia...
por ahora lo he arreglado con un try catch pero no he podido evitar el eval:

function ponerpoblacion()
{
try{
if(eval("c" +formulario.cod_poblacion.value)===undefined){
}
}catch(e){
e=e.description || e.message;
alert("Dato no correcto, intentelo de nuevo");
formulario.cod_poblacion.value="";
formulario.name_poblacion.value="";
formulario.cod_poblacion.focus();
}
formulario.name_poblacion.value = eval("c" +formulario.cod_poblacion.value)[0];
alert("Ha indicado un codigo correcto");


Bueno supongo que lioso pero igual alguien le sirve.
Saludos.

[_cronos2]

¿Leíste algo del post de @Aijonaa?

Iniciado por Aijonaa

No hace falta usar eval. Podes acceder a las propiedades de un objeto mediante la notación de corchetes ([]):

Código Javascript:

var
 c01 = 'WAKA',
 value = '01';
 
window['c' + value] == window.c01; // true
window['c' + value] == c01; // true
 

No uses eval!
Saludos (:

[jqcod]

pues si, lo leía pero no lo entiendo muy bien...si queréis darme alguna explicación adicional...de todas formas voy a seguir intentando entenderlo
saludos