Solución Formulario php

Jozhxe · #1 (**permalink**) 18/03/2014, 09:37

Hola Buenas, la verdad es que esto hace un tiempo me funcionaba bien...creo que cambié algo pero no sé qué...Alguien me podría decir que estoy haciendo mal?

Es solo un formulario php que recibe un usuario y su pass, y dependiendo del cargo que tiene lo redirecciona hacia otra página.

Código HTML
<!DOCTYPE html>
<html>
<head>
<title>Página de Inicio</title>
<link rel="stylesheet" type="text/css" href="http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/Style.css" />
<link rel="icon" type="image/png" href="http://www.clubalfayomega.cl/PaginaWeb/Blog/Im%C3%A1genes/img-logo.png" />
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>

<body background ="http://www.clubalfayomega.cl/PaginaWeb/Imagenes/smoothy-gris-wallpapers_978_1280x1024%20copia.jpg">
<div id="form">
<form action = "http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/ConquiIngresado.php" method="post" name= "formulario">
<p align="right"><font color="#253491"><b>
Usuario: </b></font><input type="text" name="usuario"> </input></p>
<p align="right"><font color="#253491"><b>
Contraseña: </b></font><input type="password" name="clave"> </input></p>
<center>
<input type="submit" name= "botonGuardar" value="Ingresar">
</center>
</form>
</div>
</body>
</html>

Código PHP

<?php
if(isset($_POST ['usuario'])
&& isset($_POST['clave'])) {

$usuarioRegistrado = $_POST ['usuario'];
$contraRegistrada = $_POST ['clave'];

$db = "conquis";
$host = "localhost";
$pw = "j1324";
$user = "jozhxe";

$con = mysql_connect ($host, $user,$pw) or die ("No se pudo autenticar la BD.");
mysql_select_db ($db,$con) or die ("No se pudo conectar a la base de datos.");

$sql = "SELECT cargo FROM conquistador WHERE usuario='$usuarioRegistrado' AND clave='$contraRegistrada'";
$resultado = mysql_query($sql,$con);

$row = mysql_fetch_array($resultado);

if($row['cargo']=='SubDirector'){
echo"<script language='javascript'>window.location='http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/SubDirector.html'</script>";
}
else if($row['cargo']=='Director'){
}
else if($row['cargo']=='Secretario'){
echo"<script language='javascript'>window.location='http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/Secretario.html'</script>";
}
else if($row['cargo']=='Tesorero'){
}
else if($row['cargo']=='Capelln'){
}
else if($row['cargo']=='Consejero'){
}
else if($row['cargo']=='Conquistador'){
}
}else{
echo("Debes llenar todos los campos");
}
?>

El problema es que cada vez que intento ingresar me dice "Debe llenar todos los campos"

Creo que tiene problemas con el traspaso de variables.

Desde ya Muchas Gracias

Eleazan · #2 (**permalink**) 18/03/2014, 09:44

¿Que devuelve la consulta? :O

PD: En vez de tanto if / else if.. .podrías poner un switch, en plan:

Código PHP:

  
switch($row['cargo']) {
    case 'Subdirector':
        //blabla
    break;
    case 'Director':
       //blabla2
    break;
    //etc etc

}

PD: La próxima vez usa el highlight

alexisverano · #3 (**permalink**) 18/03/2014, 09:48

Deberia utilizar los Highlight para poder enterder mas el codigo.
El codigo PHP que muestras esta comprendido dentro del cigo HTML?, o estan en paginas separadas.

Si estas utilizando esto:

Código PHP:

Ver originalif($row['cargo']=='SubDirector'){
echo"<script language='javascript'>window.location='http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/SubDirector.html'</script>";
}

es para realizar acciones dependiendo del cargo de la persona, entyonces para que cierras el final con esto:

Código PHP:

Ver original}else{
echo("Debes llenar todos los campos");
}

A mi entender es innesesario.

Jozhxe · #4 (**permalink**) 18/03/2014, 10:06

Gracias, tomaré todas sus consideraciones y les avisaré :D

Jozhxe · #5 (**permalink**) 18/03/2014, 10:25

Ocupé el switch, pero el problema es que ni siquiera entra al primer if, adjunto código

Código HTML:

Ver originalCódigo HTML
<!DOCTYPE html>
<html>
<head>
<title>Página de Inicio</title> 
<link rel="stylesheet" type="text/css" href="http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/Style.css" />
<link rel="icon" type="image/png" href="http://www.clubalfayomega.cl/PaginaWeb/Blog/Im%C3%A1genes/img-logo.png" />
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
</head>
 
<body background ="http://www.clubalfayomega.cl/PaginaWeb/Imagenes/smoothy-gris-wallpapers_978_1280x1024%20copia.jpg">
<div id="form">
<form action = "http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/ConquiIngresado.php" method="post" name= "formulario">
<p align="right"><font color="#253491"><b>
Usuario: </b></font><input type="text" name="usuario"> </input></p>
<p align="right"><font color="#253491"><b>
Contraseña: </b></font><input type="password" name="clave"> </input></p>
<center>
<input type="submit" name= "botonGuardar" value="Ingresar">
</center> 
</form>
</div>
</body>
</html>

Este formulario le envía las variables a la siguiente página php

Código PHP:

Ver original<?php
if(isset($_POST ['usuario']) 
&& isset($_POST['clave'])) {
 
$usuarioRegistrado = $_POST ['usuario'];
$contraRegistrada = $_POST ['clave'];
 
$db = "conquis";
$host = "localhost";
$pw = "j1324";
$user = "jozhxe";
 
 
$con = mysql_connect ($host, $user,$pw) or die ("No se pudo autenticar la BD.");
mysql_select_db ($db,$con) or die ("No se pudo conectar a la base de datos.");
 
$sql = "SELECT cargo FROM conquistador WHERE usuario='$usuarioRegistrado' AND clave='$contraRegistrada'"; 
$resultado = mysql_query($sql,$con); 
 
$row = mysql_fetch_array($resultado);
 
switch($row['cargo']){
                
                case 'SubDirector':
                echo"<script language='javascript'>window.location='http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/SubDirector.html'</script>";
                break;
                case 'Director':
                break;
                case 'Secretario':
                echo"<script language='javascript'>window.location='http://www.clubalfayomega.cl/PaginaWeb/Portal_Club/Secretario.html'</script>";
                break;
                case 'Tesorero':
                break;
                case 'Conquistador':
                break;
            }
}else{
echo("Debes llenar todos los campos");
}
?>

Eleazan · #6 (**permalink**) 18/03/2014, 10:41

Sigo preguntando, ¿q devuelve la consulta?

Puedes verlo poniendo esto antes del switch, y despues del fetch..

Código PHP:

  echo '<pre>';
print_r($row);
echo '</pre>';

Tb te vendria bien asegurar que el query no tiene error:

Código PHP:

  $resultado = mysql_query($sql,$con) or die("Se produjo un error en la consulta: ".mysql_error());

Y el switch, revisa la doc: http://es1.php.net/manual/es/control...res.switch.php

El "else" te sobra, deberia ir en la condicion default:

Código PHP:

  switch($var) {
   case 'valor1':
      //asdasdads
   break;
   case 'valor2':
       //asdasdads
   break;
   default:
      echo "Valor no esperado";
   break;
}

andresgarciadev · #7 (**permalink**) 18/03/2014, 10:48

para que usas else en el switch?
switch lo que maneja es default
pero bueno... igual por todo lo que e visto creo que tu problema viene de mas atras... antes de hacer el switch
haz un var_dump de $row y $row['cargo'] seguido de un exit; para verificar que es lo que te esta imprimiendo realmente