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?