Que opinan de esta clase que hice

[negro1985]

Código:

<?php
class Conexion
{
    private $cn;
    private $rs;

    public function __construct($serverName = "localhost", $user = "root", $pass =
        "220785", $db = "sce")
    {
        $this->cn = new mysqli($serverName, $user, $pass, $db);
    }

    public function dbExecute($query)
    {
        $this->rs = $this->cn->query($query);               
        return $this->rs;
        $this->cerrar();
    }

    public function getInsertedId()
    {
        return $this->cn->insert_id;
    }

    public function getAffectedRows()
    {
        return $this->cn->affected_rows;
    }
    private function cerrar(){
        return close($this->cn);
    }
}
?>

Que le mejorarian a esta clase que hice para conectarme a mysql

[GatorV]

Solo te puedo decir que tu método dbExecute es incorrecto ya que nunca va a llegar al $this->cerrar() porque esta despues del return.

Saludos.

[pateketrueke]

además la función close() parece ser que no existe...

[negro1985]

Entonces mi clase quedaria de la siguiente manera

Código:

<?php
class Conexion
{
	private $cn;
	private $rs;
		
	public function __construct($serverName = "localhost", $user = "root", $pass = "220785", $db = "sce")
	{
		$this->cn = new mysqli($serverName, $user, $pass, $db);
	}
	
	public function dbExecute($query) {
		$this->rs = $this->cn->query($query);	
		return $this->rs;
	}
	
	public function getInsertedId()
	{
		return $this->cn->insert_id;
	}
	
	public function getAffectedRows()
	{
		return $this->cn->affected_rows;
	}
    public function cerrar(){
            return $this->cn->close();
    }
}
?>

O sigo estando mal???

[pateketrueke]

pues ahora se ve mejor la clase... ;)

[negro1985]

Iniciado por pateketrueke

pues ahora se ve mejor la clase... ;)

Oye amigo pero no se si realmente se cierra la conexion que se hace a la base de datos, o si lo hace, es que tengo un poco de confusion en esa parte

[pateketrueke]

yo no me preocuparía, ya que el colector de basura de PHP cierra y elimina los recursos sin utilizar...

osea, que básicamente cerraría tu conexión en caso de ser necesario... ;)

[cesarpunk]

que son insert_id, affected_rows, close()... métodos? donde están? los puedes publicar?

Los datos de conexión los llamaría de algún fichero de configuración y el mysql_close() lo pondría como _destruct()

suerte