mostrar resultado de function{}

[donangelomix]

la sgte funcion subtrae los datos ingresados de cada input text, funciona bien con el alert(thePass); pero. el resultado lo quiero mostrar en otro input text readonly="readonly" pero nada, ayuda porfavor donde esta mi error.
nombre del formulario es form_a
nombre del input text donde se mostrara el resultado es password

Código Javascript:

function prueba(form,element){
    var a=form.nombres.value,b=form.ciudad.value,c=form.cedula.value,thePass;
        thePass=a.slice(0,1)+b.slice(0,1)+c.slice(0,4);
        //alert(thePass);
        eval('document.'+form+'.'+element+'.value = thePass');
    }
 

Código PHP:

<tr>
        <td ><b>Generador Clave<b></td>
        <td><input name="password" id="password" type="text"   readonly="readonly"></td>
        <td >
                   <input type="button" onClick="prueba('form_a','password');" value="Generar" ></td>
</tr>
 

[djaevi]

cambia esta linea

Código Javascript:

eval('document.'+form+'.'+element+'.value = thePass');
 

por:

Código Javascript:

document.getElementById("password").value = thePass;
 

Saludos!

[IsaBelM]

desde cuando esta es una referencia correcta a un elemento??

var a=form.nombres.value,b=form.ciudad.value,c=form.ce dula.value

por complicarlo, puedes hacer referencia así

document.forms[form].nombres.value

en lugar de usar slice() usa substr() o substring(). lee sobre la diferencia de estos métodos

por alguna razón usas eval(). hay que evitar usarlo siempre que sea posible. en el caso que ocupa no hay problema pero hay que saber cuando usarlo
una alternativa a su uso en tu caso sería

document.forms[form].elements[element].value = thePass;

[furoya]

Iniciado por IsaBelM

desde cuando esta es una referencia correcta a un elemento??

var a=form.nombres.value,b=form.ciudad.value,c=form.ce dula.value

...en lugar de usar slice() usa substr() o substring(). ...

Eso es una ocurrencia de Panino5001. Economiza var's.

Iniciado por IsaBelM

por complicarlo, puedes hacer referencia así

document.forms[form].nombres.value

Lo de usar id fue sugerencia mía. Es más seguro. Además que algún programa de validación suele recomendarlo, y después que lo ven vienen acá llorando que "su código está mal, porque lo dice el validador" y resulta que es solamente un aviso, ...pero ya posteó.

Te dejo el enlace de donde viene esto (y que pudo seguir ahí, donangelomix).
Lo tengo presente porque fue uno de los dos que hoy me hizo reir mucho.

Como hacer un generador de contraseñas

[IsaBelM]

Iniciado por furoya

Eso es una ocurrencia de Panino5001. Economiza var's.
no me refería a declarar varias variables con un solo var. me refería a form.nombres.value. si tenemos que el parámetro form es form_a, resultará form_a.nombres.value
en cuanto a usar slide() en lugar de substr() ha de ser por costumbre



Lo de usar id fue sugerencia mía. Es más seguro. Además que algún programa de validación suele recomendarlo, y después que lo ven vienen acá llorando que "su código está mal, porque lo dice el validador" y resulta que es solamente un aviso, ...pero ya posteó.
totalmente de acuerdo. de ahí que dijera por complicarlo

un saludo

[donangelomix]

gracias totales por la colaboracion ,ya resolvi mi duda