Duda sobre Programación Orientada a Objetos

[alaintm]

Estoy desarrollando un script para probar las ventajas del desarrollo en PHP orientado a objetos pero tengo dudas ya que estoy creando una clase para el manejo de BD con MySQL pero no logro que me funciones la parte de mostrar los registros o la forma que escoji no se si es la correcta, aquí les muestro el código. Es parte de un ejercicio así que vienen bien todas las sugerencias ya que me piden que sea lo más óptimo y funcional posible.

Código PHP:

  class phpMYSQL
  {
    private $conexion;
 
    function conectar($servidor, $usuario, $password, $bd)
    {
      $this->conexion = mysql_connect($servidor, $usuario, $password, $bd);
      mysql_select_db($bd, $this->conexion);
    }
 
    function desconectar()
    {
      mysql_free_result($this->consulta());
      mysql_close($this->conexion);
    }
 
    function consulta(){
      return mysql_query("SELECT * FROM militantes", $this->conexion);
    }
  }
  $mysql = new phpMYSQL();
  $mysql->conectar("localhost", "usuario", "clave", "bd");
 
  $resultado = $mysql->consulta();
 
  while($registro = mysql_fetch_object($resultado)){
    echo "$registro->ci<br />";
  }
 
  $mysql->desconectar(); 
   

Agradezco de antemano su ayuda

[DarkJ]

En el mysql_connect() se te ha colado un cuarto parametro.

[alaintm]

Gracias, además haz encontrado alguna deficiencia en el código?

[GatorV]

Trasladado a PHP Orientado a Objetos.

[pragone]

Hola alaintm,

Pues todo depende de lo que buscas.

Sin embargo, me parece que es un poco limitado y veo un error en tu función desconectar. invocas a un free result que a su vez invoca a un mysql_query... esto hace que cuando te desconectes realices una query sin necesidad.

Saludos,

[alaintm]

En que sentido te refieres a limitado?, que puede ser variado para mejorarlo

[pragone]

Hola alaintm,

Pues yo no colocaría los SQL directamente en la clase, sino que lo haría como un parámetro de consulta. Además, haría otros métodos que hagan cosas comunes como un fetchAll y un fetchOne que hagan la query y traigan todos o un solo registro respectivamente.

Podrías hacer también que los parámetros que debas pasar a las queries sean escapados por la clase, para ya estar seguro de evitar ataques de inyección de SQL

Si me permites la recomendación, mira la documentación de POD o de MDB2 y fíjate en los métodos que tienen y los parámetros que reciben.

Saludos