Condicional que no funciona

astut · #1 (**permalink**) 07/08/2007, 07:58

Hola a todos,

Código:

	function Mover(campo,arriba,abajo,izquierda,derecha,direccion) 
			{
				if (document.getElementById(arriba).value= "np" || document.getElementById(abajo).value= "np" ||
						document.getElementById(izquierda).value= "np" || document.getElementById(derecha).value= "np")
				{
					window.open("panel.php", "_blank", 'width=300, height=400');
					break;
				}}

Pues el condicional no funciona, para llamar a la funcion ago:

Código HTML:

<body onkeyup="Mover('coordenada_nueva','coordenada_arriba','coordenada_abajo','coordenada_izquierda','coordenada_derecha',event)">

Lo que falla es el condicional, ya que quitandolo todo funciona bien

a ver si me pueden ayudar =), saludos!

mount · #2 (**permalink**) 07/08/2007, 08:52

Cita:

Iniciado por astut

Código:

	function Mover(campo,arriba,abajo,izquierda,derecha,direccion) 
			{
				if (document.getElementById(arriba).value= "np" || document.getElementById(abajo).value= "np" ||
						document.getElementById(izquierda).value= "np" || document.getElementById(derecha).value= "np")
				{
					window.open("panel.php", "_blank", 'width=300, height=400');
					break;
				}}

Pues el condicional no funciona

Sustituye

Código:

value= "np"

por

Código:

value == "np"

astut · #3 (**permalink**) 07/08/2007, 09:06

Si eso lo probe pero no funciona.. dejo todo el javascript mejor:

Código:

<script language="javascript" type="text/javascript">
			<!--Funcion que utilizaremos para mover el guerrero por el mapa-->
			function Mover(campo,arriba,abajo,izquierda,derecha,direccion) 
			{
				if (document.getElementById(arriba).value== "np" || document.getElementById(abajo).value== "np" ||
						document.getElementById(izquierda).value== "np" || document.getElementById(derecha).value== "np")
				{
					window.open("panel.php", "_blank", 'width=300, height=400');
					break;
				}
				valor= parseInt(document.getElementById(campo).value);
  			switch (direccion.keyCode) 
				{
					case 37: valor-=1; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();          break;
    			case 38: valor-=12; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();         break;
    			case 39: valor+=1; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();          break;
    			case 40: valor+=12; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();          break;
				}
			}
		</script>

A ver ahora si se encuentra solucion, saludos!

astut · #4 (**permalink**) 07/08/2007, 13:13

¿Que puede estar mal?

MaBoRaK · #5 (**permalink**) 07/08/2007, 13:50

loading.............

Quitale el brak del IF.. usa un return

Código PHP:

  function Mover(campo,arriba,abajo,izquierda,derecha,direccion) 
            {
                if (document.getElementById(arriba).value== "np" || document.getElementById(abajo).value== "np" ||
                        document.getElementById(izquierda).value== "np" || document.getElementById(derecha).value== "np")
                {
                    window.open("panel.php", "_blank", 'width=300, height=400');
                    return false;
                }
                valor= parseInt(document.getElementById(campo).value);
              switch (direccion.keyCode) 
                {
                    case 37: valor-=1; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();          break;
                case 38: valor-=12; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();         break;
                case 39: valor+=1; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();          break;
                case 40: valor+=12; document.getElementById(campo).value= String(valor); document.formulario_guerrero_coordenada.submit();          break;
                }
            }

connection closed.

astut · #6 (**permalink**) 07/08/2007, 14:32

Vale era un problema externo, argumentos de la funcion erroneos xD, sorry