duda funciones

[Blashak]

Buenas,

Tengo una clase amigo que tiene dos funciones:
1. get_all() nos devuelve una array con todos los amigos de el
2. exists() // comprueba si existe ese amigo

mi duda esta en como hacer el exists, ya que tengo dos formas.

1.

Código PHP:

public function exists($id_usuario){
 
  foreach($this->amigos as $k => $v){
 
    if($this->amigos[$k]['id_usuario'] == $id_usuario){
       return true;
    }
 
  }
 
  return false;
 
}
 

2.

Código PHP:

public function exists($id_usuario){
 
  $sql = "...."// query que comprueba si ese amigo exist
 
  if(countr($sql) > 0){
  
     return true;
 
  }else{
   
    return false;
 
  }
 
}
 

La priimera funcion tiene que llamarse ante get_all() para que cargue la array en el objeto(amigo) y luego comprobar en la funcion exists

la segunda directamente consulta con una query y no hace falta cargar la funcion con todos los amigos

NOTA: la funcion get_all() aveces puede estar llamada ya que en algunas paginas carga todos los amigos

lo que quiero saber, que opcion es mas rapido???


Saludos

[alyciashape]

Puedes usar esto para calcular cuanto tiempo tarda en ejecutarse:


En el fichero en cuestión, donde quieras que empiece a contar pones:
$inicio = microtime(true);

Y al final:
$fin = microtime(true);
$diferencia = $fin - $inicio;
echo 'El script ha tardado '.number_format($diferencia, 2, '.', '').' segundos en ejecutarse.';

[Blashak]

Iniciado por alyciashape

Puedes usar esto para calcular cuanto tiempo tarda en ejecutarse:


En el fichero en cuestión, donde quieras que empiece a contar pones:
$inicio = microtime(true);

Y al final:
$fin = microtime(true);
$diferencia = $fin - $inicio;
echo 'El script ha tardado '.number_format($diferencia, 2, '.', '').' segundos en ejecutarse.';

Pero esta en local la web creo que eso iria bien cuando la suba al hosting. Ves valida las dos opciones??

[Nemutagk]

La ventaja del primero solo sería si ya tienes cargado los amigos, si vas a hacer una consulta SQL para luego buscar dentro del array arrojado por el metodo get_all() es mejor que directamente le preguntes a tu db si existe el amigo en tu tabla de relaciones, así evitas 2 cosas, 1 que tu db te retorne todos los amigos y 2 que después de obtener los amigos aparte debas de recorrer el array de amigos para buscar el amigo en concreto...

[lemyboy]

deberias preguntarle a tu DB directamente

Código PHP:

function exists($idUsuario)  {
       $query = mysql_query("SELECT.... etc.");
       return mysql_num_rows($query);
}
 

mysql_num_rows devuelve un entero que corresponde a la cantidad de filas devueltas por mysql (si no hay devuelve 0 o false)

[Nemutagk]

Iniciado por lemyboy

deberias preguntarle a tu DB directamente

Código PHP:

function exists($idUsuario)  {
       $query = mysql_query("SELECT.... etc.");
       return mysql_num_rows($query);
}
 

mysql_num_rows devuelve un entero que corresponde a la cantidad de filas devueltas por mysql (si no hay devuelve 0 o false)

Solo un dato, lo peor que puedes usar para solo saber el numero de registros (ojo, no si vas a usar los registros como tal) que coinciden con tu sentencia SQL es usar mysql_num_rows, lo mejor es usar COUNT(*) de SQL, es mucho mas optimo y rapido...

[lemyboy]

Iniciado por Nemutagk

Solo un dato, lo peor que puedes usar para solo saber el numero de registros (ojo, no si vas a usar los registros como tal) que coinciden con tu sentencia SQL es usar mysql_num_rows, lo mejor es usar COUNT(*) de SQL, es mucho mas optimo y rapido...

bueno si es cierto lo que decis, pero si la base de datos no llega a volumenes importantes no va a haber ningun problema de rendimiento, en caso de usar COUNT hay que usar la función mysql_result() para obtener el resultado del query, que en ese caso va a ser solo 1 entero

[mogurbon]

Iniciado por Nemutagk

Solo un dato, lo peor que puedes usar para solo saber el numero de registros (ojo, no si vas a usar los registros como tal) que coinciden con tu sentencia SQL es usar mysql_num_rows, lo mejor es usar COUNT(*) de SQL, es mucho mas optimo y rapido...

jejeje que pasa si usas un select pero con limit 1 e igual usas el numrows??

[lemyboy]

Iniciado por mogurbon

jejeje que pasa si usas un select pero con limit 1 e igual usas el numrows??

y que le pasa a COUNT si le pones limit?

[Nemutagk]

Iniciado por mogurbon

jejeje que pasa si usas un select pero con limit 1 e igual usas el numrows??

Que no tiene caso ni usar COUNT ni usar mysql_num_rows ya que SIEMPRE retornará un solo registro....

Al menos el sarcasmo debería tener logica >.<

[mogurbon]

Iniciado por lemyboy

y que le pasa a COUNT si le pones limit?

pues no lo veo necesario hace lo mesmo

[lemyboy]

Iniciado por Nemutagk

Que no tiene caso ni usar COUNT ni usar mysql_num_rows ya que SIEMPRE retornará un solo registro....

Al menos el sarcasmo debería tener logica >.<

ja, nobleza obliga lo acabo de probar y funciona de todas formas, pero igual quién va a usar LIMIT si necesita todos los registros?

[mogurbon]

Iniciado por lemyboy

ja, nobleza obliga lo acabo de probar y funciona de todas formas, pero igual quién va a usar LIMIT si necesita todos los registros?

ojo necesitas todos los registros si vas a buscar por medio de arrays si la busqueda la haces directamente en la bd como el compañeros pregunto

Código PHP:

  $sql = "...."// query que comprueba si ese amigo exist 
  
  if(countr($sql) > 0){ 
   
     return true; 
  
  }else{ 
    
    return false; 
  
  } 
  
}  
   

una opción seria buscar en el sql pero solo trayendo un registro , para detectar si existe , los counts en tablas muy enormes se llegan a tardar mucho , por eso puse sobre la mesa esta opción

[lemyboy]

bueno entonces a ver esta ultima versión, asi ayudamos al amigo...

Código PHP:

function exists($idUsuario)  {
           $query = mysql_query("SELECT COUNT(*) FORM tabla WHERE etc.");
           return mysql_result($query, 0);
    }
 

[Blashak]

mmm
no es mas rapido poner un limi1 y depsues en php esto,

Código PHP:

if(countr($sql) > 0){
 
     return true;
 
  }else{
   
    return false;
 
  }
 

creo que para volumen altos es lo mejo, no??

[lemyboy]

Iniciado por mogurbon

ojo necesitas todos los registros si vas a buscar por medio de arrays si la busqueda la haces directamente en la bd como el compañeros pregunto

Código PHP:

  $sql = "...."// query que comprueba si ese amigo exist
 
  if(countr($sql) > 0){
  
     return true;
 
  }else{
   
    return false;
 
  }
 
}  
   

una opción seria buscar en el sql pero solo trayendo un registro , para detectar si existe , los counts en tablas muy enormes se llegan a tardar mucho , por eso puse sobre la mesa esta opción

tenes razón es interesante lo que planteas, por ahi en el primero codigo que tire (el de la discusion) con mysql_nums_rows, si le ponemos LIMIT 0,1 al query, entonces la funcion va a devolver 0 o 1 segun exista o no, al final me parece la mejor opción, no se que opinan.

La versión de array no la recomiendo, si la base se pasa de tamaño el array segun como se llene puede llegar a colapsar la memoria de php

[mogurbon]

Iniciado por lemyboy

bueno entonces a ver esta ultima versión, asi ayudamos al amigo...

Código PHP:

function exists($idUsuario)  {
           $query = mysql_query("SELECT COUNT(*) FORM tabla WHERE etc.");
           return mysql_result($query, 0);
    }
 

mmm viendo ya tu código me late que si numrows puede ser mejor opción , el query que manejas ahí en el where siempre deberá llevar la comparación con el id de usuario , osea count+where userid, pero si se hace el query buscando el id se usuario directamente solo seria la búsqueda de este , es mas el limit sale sobrando por que dudo que halla ids repetidos

[lemyboy]

Iniciado por Blashak

mmm
no es mas rapido poner un limi1 y depsues en php esto,

Código PHP:

if(countr($sql) > 0){
 
     return true;
 
  }else{
   
    return false;
 
  }
 

creo que para volumen altos es lo mejo, no??

yo creo que si, pero la funcion countr no existe, y supongo que quisiste poner count pero solo sirve para array, tenes que usar una funcion como mysql_result o mysql_num_rows para obtener ese dato.
Saludos.

[mogurbon]

Iniciado por lemyboy

yo creo que si, pero la funcion countr no existe, y supongo que quisiste poner count pero solo sirve para array, tenes que usar una funcion como mysql_result o mysql_num_rows para obtener ese dato.
Saludos.

jajajaja yo no lo puse , lo puso el compañero pero se sobreentiende que es pseudocodigo , pero si a mi parecer lo mas optimo seria buscar por id de usuario y usar en numrows de php

[lemyboy]

Iniciado por mogurbon

jajajaja yo no lo puse , lo puso el compañero pero se sobreentiende que es pseudocodigo , pero si a mi parecer lo mas optimo seria buscar por id de usuario y usar en numrows de php

bueno entonces cerrando, a ver si estamos todos de acuerdo

Código PHP:

function exists($idUsuario)  {
           $query = mysql_query("SELECT * FROM tabla WHERE ... LIMIT 0,1");
           return mysql_num_rows($query);
    }
 

[mogurbon]

el limit 1 ya no tendria caso

Código SQL:

SELECT * FROM tabla WHERE user_id=$user_id
 

si existe regresara solo un registo , si no , no regresa ningun renglon