Analizis de Clase

[alaintm]

He creado esta clase empleando mysqli para mostrar los usuarios activos en un sitio y necesito su sugerencias para optimizarlo lo más posible para su empleo en futuros proyectos de nuestro centro.
Antes de su creación comencé a crear una clase para el manejo de las BD pero como empleo mysqli me parecio una redundancia su creación, es correcta esta decisión?

Clase UsuariosEnLinea

Código PHP:

  <?php
 
  /**
  *
  * Conexion a db mysql
  *
  * @author Alain Turiño Mena
  * @link [email protected]
  * @copyright Copyleft GNU/GPL.
  *
  */
 
  $sevidor = "localhost";
  $usuario = "usuarios";
  $clave   = "clave";
  $bd      = "bd";
 
  class UsuariosEnLinea{
    
    public function __construct(){
      global $servidor, $usuario, $clave, $bd;
 
      $mysql          = new mysqli($servidor, $usuario, $clave, $bd);
      $ip             = ip2long($_SERVER["REMOTE_ADDR"]);
      $tiempo_limite  = 300;
      $tiempo_actual  = time();
 
      $mysql->query("DELETE FROM usuariosenlinea WHERE tiempo < $tiempo_actual - $tiempo_limite");
 
      if($mysql->query("SELECT ip FROM usuariosenlinea WHERE ip = $ip") == 0){
         $mysql->query("UPDATE usuariosenlinea SET tiempo = $tiempo_actual WHERE ip = $ip");
      }
      else{
         $mysql->query("INSERT INTO usuariosenlinea (ip, tiempo) VALUE ('$ip','$tiempo_actual')");
      }
 
      $usuarios_conectado = $mysql->query("SELECT ip FROM usuariosenlinea");
 
      echo $usuarios_conectado->num_rows;
 
      $usuarios_conectado->close();
 
      $mysql->close();
    }
  }
?>   

[GatorV]

1.- No uses variables globales cuando estas usando programación orientada a objetos, usa el patrón Registry o una clase de configuración.
2.- El calculo para hacer el "purge" lo puedes hacer en PHP, no se lo dejes completo al SQL.
3.- Seria buena idea que guardaras el valor de cuantos están conectados para poderlo usar después y no tener que estar re-calculando el valor para mostrarlo varias veces en la pagina.

Saludos.

[alaintm]

Iniciado por GatorV

1.- No uses variables globales cuando estas usando programación orientada a objetos, usa el patrón Registry o una clase de configuración.
2.- El calculo para hacer el "purge" lo puedes hacer en PHP, no se lo dejes completo al SQL.
3.- Seria buena idea que guardaras el valor de cuantos están conectados para poderlo usar después y no tener que estar re-calculando el valor para mostrarlo varias veces en la pagina.

Saludos.

No entiendo cuando te refieres al patron Registry y con respecto a una clase de configuración lo que entiendo es que cree una clase donde su objetivo se tener los valores de configuración

Y con respecto al purge no logro captar la idea.

Agradezco de antemano tu ayuda.

[GatorV]

Puedes buscar en Google por el patrón Registry, es básicamente una clase que se encarga de llevar un registro.

En tu Delete tienes:

Código:

DELETE FROM usuariosenlinea WHERE tiempo < $tiempo_actual - $tiempo_limite

Eso lo puedes calcular directo desde PHP:

Código PHP:

  $aborrar = $tiempo_actual - $tiempo_limite;
$mysql->query("DELETE FROM usuariosenlinea WHERE tiempo < $aborrar"); 
   

Saludos.

[alaintm]

Hice las primeras dos variaciones que me sugeriste, pero la tercera cuando te refieres a almacenar el resultado te refieres a crear una instancia del objeto y que este llame a una función que muestre el resultado n veces?

Aquí te muestro las variaciones.

Código PHP:

  class UsuariosEnLinea{
 
    public function __construct($servidor, $usuario, $clave, $bd){
 
      $mysql                 = new mysqli($servidor, $usuario, $clave, $bd);
      $ip                       = ip2long($_SERVER["REMOTE_ADDR"]);
      $tiempo_limite     = 300;
      $tiempo_actual    = time();
      $tiempo_conectado = $tiempo_actual - $tiempo_limite;
 
      $mysql->query("DELETE FROM usuariosenlinea WHERE tiempo < $tiempo_conectado");
 
      if($mysql->query("SELECT ip FROM usuariosenlinea WHERE ip = $ip") == 0){
         $mysql->query("UPDATE usuariosenlinea SET tiempo = $tiempo_actual WHERE ip = $ip");
      }
      else{
         $mysql->query("INSERT INTO usuariosenlinea (ip, tiempo) VALUE ('$ip','$tiempo_actual')");
      }
 
      $usuarios_conectado = $mysql->query("SELECT ip FROM usuariosenlinea");
 
      echo $usuarios_conectado->num_rows;
 
      $usuarios_conectado->close();
 
      $mysql->close();
    }
  }
 
  $ul = new UsuariosEnLinea($servidor, $usuario, $clave, $bd);