[SOLUCIONADO] PHP OO como bloquear sesion después de tres intentos?

[migsbaz]

Hola buenos tardes con todos; tengo este código para validar usuarios funciona bien; ahora tengo la necesidad de complementar este código para que después de tres intentos de ingreso fallidos se bloquee la cuenta del usuario.... Ojala puedan ayudarme y gracias de antemano...

formulario para ingresar usuario y contraseña: (login.html)

Código PHP:

<form action="login.php" method="POST">
Usuario: <input type="text" name="usuario" /><br />
Clave: <input type="password" name="clave" /><br />
<input type="submit" value="Iniciar" />
</form>
 

código para validar usuario y contraseña: (login.php)

Código PHP:

<?php
session_start();
require_once('funciones.php');
conectar('localhost', 'root', 'simplementemas', 'logistica_restaurant');
 
//Recibir
$usuario = strip_tags($_POST['usuario']);
$clave = strip_tags(sha1($_POST['clave']));
 
$query = @mysql_query('SELECT * FROM usuarios WHERE usuario="'.mysql_real_escape_string($usuario).'" AND clave="'.mysql_real_escape_string($clave).'"');
if($existe = @mysql_fetch_object($query))
{
    $_SESSION['logged'] = 'yes';
    $_SESSION['usuario'] = $usuario;
    echo '<script>window.location="logeado.php"</script>';
}else{
    echo 'El usuario y/o pass son incorrectos.';    
}
?>
 

confirmación de validación: (logueado.php)

Código PHP:

<?php
session_start();
if($_SESSION['logged'] == 'yes')
{
    echo 'Bienvenido '.$_SESSION['usuario'].'.';    
}else{
    echo 'No estas logeado.';   
}
?>
 

Función (funciones.php)

Código PHP:

<?php
function conectar($servidor, $user, $pass, $name)
{
    $con = @mysql_connect($servidor, $user, $pass);
    @mysql_select_db($name, $con);  
}
?>
 

[lauser]

Revisa este hilo ---> http://www.forosdelweb.com/f18/limitar-numero-intentos-para-acceder-566756/

o bien... con sesión.

Código PHP:

<?php
session_start();
if ( $_SESSION['intentos'] > 3 ) 
{ 
      $error_login = true;
}
?>
 

Código PHP:

<?php 
if( !empty( $error_login ) )
{
    echo "Limite de intentos de acceso sobrepasado";
}
?>
 

[migsbaz]

hola lauser disculpa si duplique el post realmente no era mi intencion esque solo modifique el prmer post y se genero otro ...no volvera a pasar.

[migsbaz]

amigo lauser soy estudiante de tercer ciclo de Informática y la verdad no tengo mucha experiencia en programación, el código que te estoy mostrando arriba no lo e creado yo ya que me lo paso mi profesor para acomodarlo a la tarea que estoy haciendo, bueno de alguna forma logre acomodarlo y ahora necesito bloquear la cuenta después de tres intentos fallidos y quisiera hacerlo con sesión como tu me sugieres, pero no se si me podrás explicar un poco mas sobre el código que me estas mostrando exactamente donde debería ubicarlo en las paginas que tengo ya creadas; o es que tengo que crear otra pagina. Ya de por si te agradezco bastante por tu intervención y disculpa la molestia.

[migsbaz]

aquí te muestro mi tabla USUARIOS de la base de datos:

Usuarios

id_usuarios.....INT (11)
usuarios.........varchar(20)
dni.................varchar(20)
clave..............varchar(40)
email.............varchar(100)
ip...................varchar(15)

debería agregar un campo intentos ?

[lauser]

Código PHP:

session_start
 

En el principio de tu archivo primario, en tu caso supongo que sera login.php.

El resto lo puedes colocar en el lugar que te venga mejor del código php.
http://php.net/manual/es/session.examples.basic.php

PD: Piensa que fdw es una gran comunidad forera, e igualmente que tu la visitas, tu profesor tambien.

[migsbaz]

bueno louser entonces estoy tratando de entender el ejemplo que me diste :

http://www.forosdelweb.com/f18/limitar-numero-intentos-para-acceder-566756/

me podrías explicar para que me sirve el campo nivel o que se almacena en el?

[Anonimo12]

Lo que te dice es que crees una variable dentro de "$_SESSION" que almacene los intentos inicializada a 0:

Código PHP:

  $_SESSION['intentos'] = 0; 
   

Cuando un usuario intenta entrar y comete un error le sumas 1:

Código PHP:

  $_SESSION['intentos'] += 1; 
   

Cuando llegue a 3 directamente le imprimes un mensaje de error al intentar logearse de nuevo:

Código PHP:

  if($_SESSION['intentos'] >= 3) {
   //Mensaje de error
}
else {
  //Si no ha fallado 3 veces, compruebas si la cuenta y contraseña son correctas
} 
   

Esto último lo tienes que meter al inicio del todo, inmediatamente después de "session_start()".

Aprovecho y te comento algunas cosas para la correcta programación:

Código PHP:

  conectar('localhost', 'root', 'simplementemas', 'logistica_restaurant'); 
   

Las cuentas siempre deben ser importadas de un fichero aparte, de hecho la modularidad se debe aplicar siempre, de manera que si tienes que acceder a la base de datos en 10 scripts distintos, el día que cambies la cuenta o la contraseña no tengas que modificarlo uno por uno. Lo ideal es crear un fichero llamado "config.php" con array con la información de acceso a la BD:

Código PHP:

  $bd = array('host' => 'localhost', 'BD' => 'mibd', 'usuario' => 'miusuario', 'pass' => 'mipass'); 
   

Otra cosilla, "mysql" está obsoleto, procura usar el módulo "mysqli" que es más eficiente e implementa nuevas funciones.

Un cordial saludo.

[migsbaz]

Grcias por la intervencion amigo Anonimo12 : es la primera vez que trabajo con sesiones y estoy bien liado; bueno vamos paso a paso aplicando lo que me recomendaste primero entonces mi login.php quedaria de esta forma:

Código PHP:

<?php
session_start();
 
require_once('funciones.php');
conectar('localhost', 'root', 'simplementemas', 'logistica_restaurant');
$usuario = strip_tags($_POST['usuario']);
$clave = strip_tags(sha1($_POST['clave']));
 
if($_SESSION['intentos'] >= 3) {
 echo 'El usuario y/o pass son incorrectos.';
}
 
$query = @mysql_query('SELECT * FROM usuarios WHERE usuario="'.mysql_real_escape_string($usuario).'" AND clave="'.mysql_real_escape_string($clave).'"');
else ($existe = @mysql_fetch_object($query))
 {
  $_SESSION['logged'] = 'yes';
    $_SESSION['usuario'] = $usuario;
    echo '<script>window.location="logeado.php"</script>';
} 
?>
 

estará bien lo que hice?

y donde colocaria lo siguiente es decier esto:

Código PHP:

$_SESSION['intentos'] = 0;  
 
 
 
$_SESSION['intentos'] += 1;
 

gracias de antemano.

[Anonimo12]

El script final te quedaría así:

Código PHP:

  <?php
session_start();
 
/* AQUÍ HACES LA COMPROBACIÓN */
if(!isset($_SESSION['intentos'])) //Si no se ha creado "intentos" es que aún no ha hecho ningún intento, por tanto la creamos.
   $_SESSION['intentos'] = 0;
else if($_SESSION['intentos'] >= 3) { //Si existe "intentos" y ya hecho 3 comprobaciones devolvemos el mensaje de error. Esta comprobación la hacemos aquí arriba porque si ya ha hecho 3 intentos ni siquiera hay que conectar a la BD
   echo 'El usuario y/o pass son incorrectos.';
}
/* FIN COMPROBACIÓN INTENTOS */
 
require_once('funciones.php');
conectar('localhost', 'root', 'simplementemas', 'logistica_restaurant');
$usuario = strip_tags($_POST['usuario']);
$clave = strip_tags(sha1($_POST['clave']));
 
$query = @mysql_query('SELECT * FROM usuarios WHERE usuario="'.mysql_real_escape_string($usuario).'" AND clave="'.mysql_real_escape_string($clave).'"');
if ($existe = @mysql_fetch_object($query))
 {
  $_SESSION['logged'] = 'yes';
    $_SESSION['usuario'] = $usuario;
    echo '<script>window.location="logeado.php"</script>';
}
else { //Si la cuenta y/o contraseña es errónea sumamos 1 al número de intentos
   $_SESSION['intentos'] += 1;
}
?>   

Va comentado. Un cordial saludo.

[migsbaz]

hola de nuevo Anonimo12 ya corrí el script y no hay errores pero te cuento lo que sucede:

después de cada intento fallido me sale el mensaje 'El usuario y/o pass son incorrectos.'

esto es ta bien pero:

- lo que se me pide es que después de los tres intentos fallidos me salga el mensaje de que: "ya se supero el limite de intentos" y luego se bloque la cuenta del usuario.

es decir con el script que me diste hago tres intento fallidos y al hacer un cuarto intento ingreso la contraseña correcta y si se logra acceder a la pagina y esto es lo que no quiero que pase

como haría para lograr esto?

[migsbaz]

es decir en esta parte e cambiado el mensaje por 'limite de intentos superado pagina bloqueada.' pero el mensaje me esta saliendo después de cada uno de los intentos fallidos. Lo que quisiera es que este mensaje me salga después de tres intentos fallidos y luego bloquear la cuenta del usuario:

Código PHP:

if(!isset($_SESSION['intentos'])) 
   $_SESSION['intentos'] = 0; 
else if($_SESSION['intentos'] >= 3) { 
   echo 'limite de intentos superado pagina bloqueada.'; 
} 
/* FIN COMPROBACIÓN INTENTOS */
 

[Anonimo12]

Claro! Porque te falta añadirle la salida. El texto te lo he puesto de ejemplo, te comento el punto clave:

Código PHP:

if(!isset($_SESSION['intentos']))
   $_SESSION['intentos'] = 0; 
else if($_SESSION['intentos'] >= 3) 
   echo 'El usuario y/o pass son incorrectos.'; //ESTE TEXTO ES DE EJEMPLO, aquí deberías añadirle un "exit" o cualquier otra función para finalizar. Sino, te va a imprimir el mensaje pero continuará ejecutándose el script, por lo que vuelve a hacer las comprobaciones que hay a continuación
}
 

Utiliza "exit", o "return" si es una función.

[migsbaz]

ok amigo voy a probar

[migsbaz]

disculpa otra ves Anonimo12 puse de la siguene forma el return:

Código PHP:

<?php 
session_start(); 
 
/* AQUÍ HACES LA COMPROBACIÓN */ 
if(!isset($_SESSION['intentos'])) //Si no se ha creado "intentos" es que aún no ha hecho ningún intento, por tanto la creamos. 
   $_SESSION['intentos'] = 0; 
else if($_SESSION['intentos'] >= 3) { 
    
    echo 'limite de intentos superado pagina bloqueada.';
    return;
}
 

pero creo que esta mal ubicado ya que si hago un primer intento fallido me arroja el mensaje 'limite de intentos superado pagina bloqueada.' y si ingreso los los datos correctos tanbien me sale el mismo mensaje y a no logro acceder a la pagina.

[migsbaz]

por favor ayúdame Anonimo12 ya se me vence el plazo para presentar esta tarea y no logro encontrar la solución para bloquear la cuenta del usuario,

gracias y disculpa las molestias

[migsbaz]

hola Anonimo 12 te cuento que ya avance un poco y ya me esta corriendo el script lo canbie un poco para lograrlo:

Código PHP:

<?php 
session_start(); 
 
if(!isset($_SESSION['intentos'])) 
  $_SESSION['intentos'] = 0; 
  
else                                         
{     
$_SESSION['intentos'] ++ ; 
 ?> 
 <?php                              
if ($_SESSION['intentos'] < 3 )
{ 
require_once('funciones.php'); 
conectar('localhost', 'root', 'simplementemas', 'logistica_restaurant'); 
$usuario = strip_tags($_POST['usuario']); 
$clave = strip_tags(sha1($_POST['clave']));   
$query = @mysql_query('SELECT * FROM usuarios WHERE usuario="'.mysql_real_escape_string($usuario).'" AND clave="'.mysql_real_escape_string($clave).'"'); 
    
    if ($existe = @mysql_fetch_object($query)) 
     { 
     $_SESSION['logged'] = 'yes'; 
     $_SESSION['usuario'] = $usuario; 
      echo '<script>window.location="logeado.php"</script>'; 
     } 
    else 
      {  echo 'El usuario y/o pass son incorrectos.'.$_SESSION['intentos'];}
} ///fin del if
else
 {echo 'limite de intentos superado.' .$_SESSION['intentos'];}
}
?>
 

talvez puedas ayudarme a canbiar el mysql por Mysqli ...... gracias

[lauser]

talvez puedas ayudarme a canbiar el mysql por Mysqli ...... gracias

La tarea escolar la tienes que realizar tu, no esperes que te la hagan.
Revisa el manual... si no no aprenderás nada. (No engañas al profesor, te engañas a ti mismo)

http://ar2.php.net/manual/es/mysqli....onnections.php

[migsbaz]

Tranquilo lauser quizás esta sea mi forma de aprender atraves de una buena explicación, quizás todos no somos como tu; ni disponemos del tiempo que tu tienes. tiene derecho a criticar quien tiene voluntad para ayudar y tu lauser no me ayudaste en nada agradesco a Anonimo 12 por su buena voluntad el si me ayudo en ves de criticar, ya que era una urgencia porque el docente no me iba a espera hasta que me aprenda el manual. Gracias al ejemplo que me dio Anonimo12 ahora tengo mas claro es tema y puedo avanzar.
yo soy nuevo en programación y es difícil para mi ya que tengo una familia y tengo que trabajar y se hace mucho mas difícil si en el camino nos encontramos con tipos intolerantes y reprimidos y pedantes como tu que desde un principio me hiciste problemas. gracias Anonimo12

[HackID1]

Las funciones de mysql para cambiarlas a mysqli, no es complicado pero debes cambiar un poco las sintaxis que usabas anteriormente.

Aquí encontrarás las sintaxis de mysqli.

http://www.php.net/manual/es/mysqli.summary.php

PD: Si no requieres hacerlo aún con mysql, primero haz que funcione esta forma, y poco a poco ve integrando todo lo que hagas nuevo para realizar las consultas con mysqli.

Saludos

[lauser]

Tranquilo lauser quizás esta sea mi forma de aprender atraves de una buena explicación, quizás todos no somos como tu; ni disponemos del tiempo que tu tienes.

????---- me espías desde tu ventana? como calculas el tiempo libre del resto del universo? Me interesa saberlo.

y tu lauser no me ayudaste en nada

Si relees el hilo, observaras que te proporcione la solución correcta. Si en tercer curso no sabes aplicar una simple session..... realmente tienes un problema. Aparte de que cuando pediste aclaración Anonimo 12 te respondió exactamente en 6minutos.

agradesco a Anonimo 12 por su buena voluntad el si me ayudo en ves de criticar

Te lo explico de forma sencilla... a los niños se les enseña a comer solos por que si noooo tendrán 20 años y aun habra que darles con la cuchara en la boca (es un ejemplo). Y Anonimo lo que hizo fue hacer TU trabajo, que es lo que tu querías.... que te lo hicieran.
En ningun momento recibiste una critica por mi parte. Todo lo contrario.... fue un consejo. Y si te lo sigues tomando como critica, en vez de un problema tienes dos.

tengo una familia y tengo que trabajar

Pues como todos, o crees que se vive del aire?
Solo tienes familia tu?

tipos intolerantes y reprimidos y pedantes

Creo que me malinterpretaste, pero no tienes ningun derecho a decir todo tipo de improperios sobre mi, con los cuales ademas quedas retratado como alguien sin ningun tipo de educación, ni respeto hacia nada ni nadie.. y te exijo una rectificación de tus palabras. No juzgues y no seras juzgado.

Estamos en un foro de ayuda y aprendizaje, no un servicio técnico. El objetivo primordial es que los usuarios aprendan y entre todos se ayuda a que se resuelvan sus problemas. Los usuarios que te responden, lo hacen altruistamente y sin ánimo de lucro, con el único fin de ayudar. Piensa en ello.