Duda con clase mysql, saber si se ejecuta 2 veces la consulta

[neodani]

Iniciado por GatorV

O usar mejor is_resource y en efecto es mejor si usas directamente "return mysql_num_rows($this->resource);".

PD. no es tan recomendable que uses @ es mejor que tengas una mejor forma de controlar tus errores que solamente suprimirlos.

Saludos.

He entendido bien...?
Para ese caso del mysql_num_rows no haría falta usar el is_resource y es mejor utilizar directamente el return mysql_num_rows($this->resource);

Pero para este caso si que lo ideal sería que quedase así?

Código PHP:

private function __construct($servidor, $user, $password, $db){
        $this->conexion = mysql_connect($servidor, $user, $password);
        if (!is_resource($this->conexion)) {
            die('Can\'t connect : ' . mysql_error());
        }
        mysql_select_db($db, $this->conexion);
        $this->queries = 0;
        $this->resource = null;
    }
 

Y en lugar de

Código PHP:

public function execute(){
        if (!$this->resource){
            if(!($this->resource = mysql_query($this->sql, $this->conexion))){
                return null;
            }
        }
        $this->queries++;
        return $this->resource;
    }
 

Quedaría así?

Código PHP:

public function execute(){
        if (!is_resource($this->resource)) {
            if(!($this->resource = mysql_query($this->sql, $this->conexion))){
                return null;
            }
        }
        $this->queries++;
        return $this->resource;
    }
 

Muchas gracias de antemano