Ver Mensaje Individual
  #3 (permalink)  
Antiguo 09/04/2011, 03:14
Avatar de masterpuppet
masterpuppet
Software Craftsman
 
Fecha de Ingreso: enero-2008
Ubicación: Montevideo, Uruguay
Mensajes: 3.550
Antigüedad: 16 años, 3 meses
Puntos: 845
Respuesta: Opinion acerca de codigo

Que tal jotaincubus,

En lo personal trato de evitar el uso de extract, suelo trabajar en equipo y no me agrade que el que esta al lado pierda tiempo buscando donde se define que, el código debería ser lo mas claro posible y extract a veces esconde esa claridad.

Estas trabajando con MVC ?, no me parece bien que Logueo acceda directamente al $_POST.

También esta lo que comenta Triby, el password mínimo debería estar encriptado con md5 + salt y en la query buscar la identidad correspondiente con el password ya encriptado.


Código PHP:
Ver original
  1. class Logueo
  2. {
  3.     const SALT = '%·s#!a/l)($t';
  4.     ...
  5.     public function loguear()
  6.     {  
  7.         ...
  8.         $query = "SELECT id_acceso, acc_correo, acc_seudonimo, acc_contrasena "
  9.                . ", id_roll, acc_confirmado, acc_estado "
  10.                . "FROM `prog_db`.`tbl_acceso` "
  11.                . "WHERE (acc_seudonimo = ? OR acc_correo = ?) AND passwd = ?"
  12.         $consulta = $basededatos->prepare($query);
  13.         $passwd   = md5(self::SALT . $passwd):
  14.         $consulta->execute(array($usuario, $usuario, $passwd));
  15.         ...
  16.     }
  17. }

y por último, estas utilizando PHP 4 ?, si no es el caso deberías declarar la visibilidad de las propiedades y métodos.

Saludos.
__________________
http://es.phptherightway.com/
thats us riders :)