Sistema de login en PDO

[Strings]

Estoy aprendiendo PDO y he realizado este sistema de login y quiero compartirlo con vosotros. Acepto criticas.

Base de dato:

Código SQL:

CREATE TABLE IF NOT EXISTS `user` (
  `id` mediumint(9) NOT NULL AUTO_INCREMENT,
  `username` VARCHAR(20) NOT NULL,
  `password` VARCHAR(60) NOT NULL,
  `date` VARCHAR(19) NOT NULL,
  `ip` VARCHAR(10) NOT NULL,
  PRIMARY KEY (`id`)
)
 

/config/index.php

Código PHP:

  <?php 
    $connection = new PDO("mysql:host=hosting;dbname=base de dato","usuario","contraseña"); 
    $connection->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
?>   

Index.php

Código PHP:

  <?php 
session_start(); 
include('/config/index.php'); 
 
if(isset($_SESSION['username'])): 
    echo 'Estas logeado <a href="/logout.php">Logout</a>'; 
else: 
    if(isset($_POST['login'])): 
        if(empty($_POST['username']) || empty($_POST['password'])): 
            echo 'No dejes campos en blanco'; 
        elseif(strlen($_POST['username']) > 20): 
            echo 'El usuario no puede tener mas de 20 caracteres'; 
        elseif(strlen($_POST['password']) > 20): 
            echo 'La contraseña no puede tener mas de 20 caracteres'; 
        else: 
            $login = $connection->prepare("SELECT username FROM user WHERE username = :username AND password = :password"); 
            $login->bindParam(':username',$_POST['username']); 
            $login->bindParam(':password',crypt($_POST['password'], '$2a$07$rieh3693fjarjeuf38cw27fg2$')); 
            $login->execute(); 
            if($login = $login->fetch(PDO::FETCH_ASSOC)): 
                $_SESSION['username'] = $_POST['username']; 
                header('Location: /'); 
            else: 
                echo 'Datos incorrectos'; 
            endif; 
        endif; 
    endif; 
    echo '<form action="" method="post"> 
        <input name="username" placeholder="Username"><br> 
        <input name="password" placeholder="Password"><br> 
        <input name="login" type="submit"> 
    </form> 
     
    <a href="/register.php">Registrate</a>'; 
endif; 
?>   

Register.php

Código PHP:

  <?php 
session_start(); 
include('/config/index.php'); 
 
if(isset($_SESSION['username'])): 
    header('Location: /'); 
else: 
    if(isset($_POST['register'])): 
        if(empty($_POST['username']) || empty($_POST['password'])): 
            echo 'No dejes campos en blanco'; 
        elseif(strlen($_POST['username']) > 20): 
            echo 'El usuario no puede tener mas de 20 caracteres'; 
        else: 
            $user = $connection->prepare("SELECT username FROM user WHERE username = :username"); 
            $user->bindParam(':username',$_POST['username']); 
            $user->execute(); 
            if($user->fetch(PDO::FETCH_ASSOC)): 
                echo 'El usuario ya existe'; 
            elseif(strlen($_POST['password']) > 20): 
                echo 'La contraseña no puede tener mas de 20 caracteres'; 
            elseif($_POST['password'] <> $_POST['password2']): 
                echo 'Las contraseñas no coinciden'; 
            else: 
                $register = $connection->prepare("INSERT INTO user(id,username,password,date,ip) VALUES ('',:username,:password,'".date('H:i:s d/m/Y')."','".$_SERVER['REMOTE_ADDR']."')"); 
                $register->bindParam(':username',$_POST['username']); 
                $register->bindParam(':password',crypt($_POST['password'], '$2a$07$rieh3693fjarjeuf38cw27fg2$')); 
                $register->execute(); 
                if($register->rowCount() > 0): 
                    $_SESSION['username'] = $_POST['username']; 
                    header('Location: /'); 
                else: 
                    echo 'Ha ocurrido un error'; 
                endif; 
            endif; 
        endif; 
    endif; 
    echo '<form action="" method="post"> 
        <input name="username" placeholder="Username"><br> 
        <input name="password" placeholder="Password"><br> 
        <input name="password2" placeholder="Vuelve a ingresar la contraseña"><br> 
        <input name="register" type="submit"> 
    </form>'; 
endif; 
?>   

Logout.php

Código PHP:

  <?php 
    session_start(); 
    session_destroy(); 
    header('Location: /'); 
?>   

Prefiero que me digais en que puedo mejorar

[jonni09lo]

Hola, acostumbrate despues de un header colocar exit() de esa manera evitas que se siga ejecutando codigo.

Saludos

[Strings]

Muchisimas gracias, lo tendre en cuenta.

[loncho_rojas]

Gracias por compartirlo con nosotros... solo una cosa.. Edita el post y cambia el tipo de campo que recibe la fecha llamado date al tipo date y no como varchar... saludos

[Strings]

No me sale la opcion de editar el tema principal, yo utilizo varchar(19) porque cuando utilizo date se guarda como 000-000-000

[loncho_rojas]

Podrías dejarlo en DATE y cuando haces el insert lo haces con NOW() en vez de date('H:i:s d/m/Y') ...

en realidad da igual, pero es lo que se acostumbra ya que no es tan estricto el tema de la fecha en que se logueo y luego será más facil de formatear.

Saludos

[Strings]

Lo probare, haber si me funciona y comento.

Me sigue saliendo 000-000-000

[Strings]

¿Podria mejorar el codigo de alguna manera?

[loncho_rojas]

En el registro.

Creo que si tu intención es saber si ya existe un usuario logueado y evitar duplicaciones deberías usar la funcion exec de PDO que te permite saber la cantidad de registros que devuelve una consulta SQL.. en vez de:

Código PHP:

   
$user = $connection->prepare("SELECT username FROM user WHERE username = :username");  
            $user->bindParam(':username',$_POST['username']);  
            $user->execute();  
            if($user->fetch(PDO::FETCH_ASSOC)):  
                echo 'El usuario ya existe';  
   

podrías intentar algo como:

Código PHP:

  $count = $connection->exec("SELECT username FROM user WHERE username = ' ".$_POST['username']." ' ");  
 
/* Devuelve el número de filas existentes */ 
if($count > 0) 
echo 'El usuario ya existe';  
   

y si no existe, recien lo insertas con el PREPARE...

[Strings]

Si eso lo pense, pero entonces, como filtro las variables?