Problema obteniendo el valor del botón de radio seleccionado

[KazerPowa]

Hola,

Tengo este código Javascript:

Código Javascript:

function redirect(){
    var i
    for (i=0;i<document.form_option.option.length;i++){
       if (document.form_option.option[i].checked)
          break;
    }
    option = document.form_option.option[i].value;
    location.href='index.php?o=' + option;
    }
 

Y esto en el HTML:

Código HTML:

<form action="" id="login_form" method="post" name="form_option" accept-charset="ISO-8859-1">
        <input style="margin-bottom: 5px" checked="checked" type="radio" name="option" value="1" /><a><?php echo $welcome_o0s2; ?></a> <br />
        <input style="margin-bottom: 5px" type="radio" name="option" value="2" /><a><?php echo $welcome_o0s3; ?></a> <br />
        <input style="margin-bottom: 5px" type="radio" name="option" value="3" /><a><?php echo $welcome_o0s4; ?></a> <br />
        <input onmouseover="change_color('1', 'button')" onmouseout="change_color('0', 'button')" style="background-color: #dadada" onclick="redirect()" id="button" class="button" value="<?php echo $welcome_next; ?>" type="button" />
</form>
 

Por algún motivo, a mí me funciona pero a algunos usuarios no :/
Debido a esto, no puedo comprobar cuándo está arreglado, así que no puedo hacer mucho...
El código se puede ver en http://tinyurl.com/389fraz
Si a alguien no le funciona el botón "siguiente", y sabe cómo solucionarlo, le agradecería que me dijera el fallo....
Saludos!

[zerokilled]

me sorprende que a ti te funcione cuando en realidad no deberia funcionar en ningun interprete. si analizas la consola de error veras que te marca error de sintaxis en el script.

[KazerPowa]

Iniciado por zerokilled

me sorprende que a ti te funcione cuando en realidad no deberia funcionar en ningun interprete. si analizas la consola de error veras que te marca error de sintaxis en el script.

Supongo que será por "var i", así que lo he cambiado a "var i=0;"
De todas formas, miré la consola JavaScript en Chrome y no me marcaba nada :/

Saludos!

[Adler]

Hola

Revisando la consola de errores FireBug, tienes errores en las funciones change_color y redirect. En la función change_color el primer valor que le pasas es un string, pero lo evalúas como numérico

if (num == '1')

En la función redirect has de usar getElementsByTagName para listar todos los elementos. Prueba con esto

Código Javascript:

function redirect(){
elementos = document.getElementById('login_form').getElementsByTagName('input');
    for (i=1;i<elementos.length;i++){
       if (elementos[i].checked)
option = elementos[i].value;
          break;
    }
    
    location.href='index.php?o=' + option;
    }
 

Suerte

[zerokilled]

@kazerpowa,
no, ahi no es donde esta el problema. var i; es perfectamente valido. por cierto, chrome a mi me marca error. el problema esta especificamente en estas lineas ";. fijate que se abren comillas para string pero no se cierran.

[KazerPowa]

Bueno, pues he quitado mucho código que no hacía falta en la página principal, y he cambiado redirect() por lo que me puso Adler (eso sí, algo modificado).

También he puesto comillas en todos los condicionales, y ahora FireBug no da ningún error!

Código Javascript:

function redirect(){
        var i = 0;
        elementos = document.getElementsByName('option');
        for (i=0;i<elementos.length;i++){
        if (elementos[i].checked){
        option = elementos[i].value;
        break;
        }
        }
        location.href='index.php?o=' + option;
    }
 

Código Javascript:

function change_color(num, id) {
        if (num == '1') {
            document.getElementById(id).style.backgroundColor = "#bfbfbf";
        }
        if (num == '0') {
            document.getElementById(id).style.backgroundColor = "#dadada";
        }
        
    }
 

Gracias a todos.
Saludos!

[Adler]

Hola

Esta es la parte que yo cambiaría, es mejor acotarlo lo máximo posible, nunca se sabe lo que se necesitar´´a en un futuro

elementos = document.getElementsByName('option');

¿Has puesto un radio mas o me lo parece? Juraría que esta mañana había un checkbox y dos radios: Lo digo por que si no es así no entiendo por qu´´e le di el valor 1 a i

Suerte

[KazerPowa]

Qué va, son 3 checkbox.
He probado la web en el ciber (para asegurarme) y funciona de maravilla, no te preocupes.