podría alguien revisarme esta función?

[eleyele]

buenas.
Yo se que ya hay información a cerca del tema en el foro, pero copié información de aquí y no se donde está el problema.. agradezco me ayuden ya que apenas me estoy conociendo con javascript...

tengo esto:

function Desplegar(TABLA_FORMA_PAGO) {
var Tabla = document.getElementById(TABLA_FORMA_PAGO);
var FORMA_PAGO = document.getElementById("RADIO_F_PAGO").value;
if (FORMA_PAGO = document.getElementById("RADIO_F_PAGO").value="EFE CTIVO")
{
Tabla.style.display= "none";
}
else
{
Tabla.style.display = "block";
}
}

en body tengo esto:

<input name="RADIO_F_PAGO" id="RADIO_F_PAGO" type="radio" value="EFECTIVO" onClick="Desplegar('TABLA_FORMA_PAGO')">

<input name="RADIO_F_PAGO" id="RADIO_F_PAGO" type="radio" value="CHEQUE" onClick="Desplegar('TABLA_FORMA_PAGO')">

<input name="RADIO_F_PAGO" id="RADIO_F_PAGO" type="radio" value="TARJETA" onClick="Desplegar('TABLA_FORMA_PAGO')">

y quiero que cuando le haga click en el radio CHEQUE o TARTEJA me aparezca una tabla:

<table width="100%" cellspacing="0" cellpadding="0" id="TABLA_FORMA_PAGO" name="TABLA_FORMA_PAGO" style="display:none"> ( ... )

alguien me podría decir qué es lo que hago mal o en su defecto que parte está bueno.... :(

[IsaBelM]

todo este código

var FORMA_PAGO = document.getElementById("RADIO_F_PAGO").value;
if (FORMA_PAGO = document.getElementById("RADIO_F_PAGO").value="EFE CTIVO")
{
Tabla.style.display= "none";
}
else
{
Tabla.style.display = "block";
}

se reduce a esto

Tabla.style.display= document.getElementById("RADIO_F_PAGO").value=="EF ECTIVO" ? "none" : "block";

uno de los errores que estás cometiendo es que estás asignando en vez de comparando
con = se asigna valor, por ejemplo a una variable, y con == se compara dos valores

[eleyele]

Iniciado por IsaBelM

todo este código

se reduce a esto

uno de los errores que estás cometiendo es que estás asignando en vez de comparando
con = se asigna valor, por ejemplo a una variable, y con == se compara dos valores

hola IsaBelM, gracias, por tu pronta respuesta y valiosa información.
Hice lo que me dices.. pero me sale este error: 'Tabla' no está definido
¿?

Edito..
Equivocación, no declaré la variable tabla, cuando te envié el mensaje... pero aún así no me funciona...

[malakian]

Iniciado por IsaBelM

Tabla.style.display= document.getElementById("RADIO_F_PAGO").value=="EF ECTIVO" ? "none" : "block";

Hola disculpa si me entrometo IsaBelM perome gustaria saber si esa forma de hacer una condicion tiene algun nombre en especial gracias!!

[_cronos2]

A eso se le suele llamar operador ternario (?:), porque recibe tres parámetros, la condición a evaluar, y qué hacer en caso de que sea true y en caso de que sea false. Se puede anidar pero particularmente prefiero usar if/else cuando lo que vayas a hacer dependa de varios factores. Ej:

Código Javascript:

var num=7;
alert(num>8?'mayor que 8':(num<7?'menor que 7':'es 7')); // es 7
 

Saludos (:

[IsaBelM]

quitale las id a los radios, están repetidas. por otro lado pasale a la función otro valor (this.value)

function Desplegar(rad, TABLA_FORMA_PAGO) {
var Tabla = document.getElementById(TABLA_FORMA_PAGO);
Tabla.style.display= rad=="EFECTIVO" ? "none" : "block";
}



<input name="RADIO_F_PAGO" type="radio" value="EFECTIVO" onclick="Desplegar(this.value, 'TABLA_FORMA_PAGO')">

<input name="RADIO_F_PAGO" type="radio" value="CHEQUE" onclick="Desplegar(this.value, 'TABLA_FORMA_PAGO')">

<input name="RADIO_F_PAGO" type="radio" value="TARJETA" onclick="Desplegar(this.value, 'TABLA_FORMA_PAGO')">

<table width="100%" cellspacing="0" cellpadding="0" id="TABLA_FORMA_PAGO" name="TABLA_FORMA_PAGO" style="display:none">

[eleyele]

Iniciado por IsaBelM

quitale las id a los radios, están repetidas. por otro lado pasale a la función otro valor (this.value)

hola IsaBelM... todo funcionó a la perfección, muchisimas gracias por tu tiempo y colaboración, aprendí un montón!!! gracias de nuevo!!