Autor Tema: Problema con script en PHP (Leído 9681 veces)

Ale992 · « **en:** Miércoles 10 de Noviembre de 2010, 01:04 »

Bueno tengo el siguiente script:

Código: PHP

<?php
 
class db
{
 
private $servidor;
private $usuario;
private $pass;
private $base_datos;
private $descriptor;
 
function __construct($servidor = "mysql8.000webhost.com",$usuario = "a7863285_xasd",$pass = "xxxxxx",$base_datos = "a7863285_cManage")
{
$this->servidor = $servidor;
$this->usuario = $usuario;
$this->pass = $pass;
$this->base_datos = $base_datos;
$this->conectar_base_datos();
 
}
prívate function conectar_base_datos()   //ERROR EN ESTA LINEA
{
$this->descriptor = mysql__connect($this->servidor,$this-
>usuario,$this->pass) ;
mysql_select_db($this->base_datos,$this->descriptor);
}
public function consulta($consulta)
{
$this->resultado = mysql_query($consulta,$this->descriptor) ;
}
public function utf ()
{
mysql_query ("SET NAMES 'utf8'");
}
}
?>
 

Y el error es el siguiente:

Parse error: syntax error, unexpected T_STRING, expecting T_FUNCTION in /home/a7863285/public_html/mysql.php on line 21

Alguien tiene idea que es lo que hay mal o como solucionarlo?

Gracias.

JrcsDev

De momento veo que en la línea 21 tienes lo siguiente "prívate function" con acento... quíate el acento al "private".

En la línea 23 y 24 (deberia estar todo en uno sola línea) dice mysql__connect, cambialo por mysql_connect
$this->descriptor = mysql__connect($this->servidor,$this-
>usuario,$this->pass) ;

Ale992

Gracias, ya lo corregi.

Mi pregunta es si una vez creado un nuevo objeto con esa clase, y luego de haberme conectado a la base de datos y hecho la consulta a la misma usando las funciones de la clase puedo hacer mysql_close() directamente en el codigo o si la unica manera de cerrar la conexion es creando una nueva funcion en la clase que cierre la conexion. Porque lo cerre de la primer manera y me larga un error y no se si es que no se puede hacer o si volvi a hacer algo mal.

JrcsDev

La función mysql_close($link) no siempre es necesaria... pero cuando la colocas, si no le pasas un $link en el parámetro, se cierra la última conexion...

Ale992

Ok, ahora voy a probar, y otra cosa, disculpa que siga preguntando es que soy bastante nuevo en esto y todavia no logro comprender del todo el funcionamiento de la programacion orientada a objetos.

Este es el codigo de la clase:

Código: PHP

 
 
class db
{
 
private $servidor;
private $usuario;
private $pass;
private $base_datos;
public  $descriptor;
 
function __construct($servidor = "mysql8.000webhost.com",$usuario = "a7863285_root",$pass = "alexismoyano1992",$base_datos = "a7863285_cManage")
{
$this->servidor = $servidor;
$this->usuario = $usuario;
$this->pass = $pass;
$this->base_datos = $base_datos;
$this->conectar_base_datos();
 
}
private function conectar_base_datos()
{
$this->descriptor = mysql_connect($this->servidor,$this->usuario,$this->pass) ;
mysql_select_db($this->base_datos,$this->descriptor);
}
public function consulta($consulta)
{
$this->resultado = mysql_query($consulta,$this->descriptor) ;
}
public function extraer_registro ()
{
if ($fila = mysql_fetch_array($this->resultado,MYSQL_ASSOC)){
  return $fila;} 
else {
  return $fila[0]= "ERROR";
}
}
public function utf ()
{
mysql_query ("SET NAMES 'utf8'");
}
public function close (){
 mysql_close($this->descriptor);
}
}
 
 

Y este el del programa:

Código: PHP

 
          require_once("mysql.php");
          
          $usuario = new db();
          $query= "SELECT * FROM news ORDER BY newID DESC LIMIT 0,10";
          
          $usuario->utf();
          $usuario->consulta($query);
          $fila = $usuario->extraer_registro();
          
         
         // $resultado=mysql_query($query, $connect) or die ("Problema al acceder a los datos ".mysql_error());
         // while ($reg = mysql_fetch_array($resultado))
         // {
         //   $fecha= $reg['date'];
         //  echo "<div id="date" align="right">( ".cambiaf($fecha).")</div>";
         //  echo "<div id='met'>".stripslashes($reg["title"])."</div>";
         // echo "<div class='text'><p>".stripslashes($reg["article"])."<br/></p></div>";
         //  echo "<div id='hr2'></div>";
 
          //}
          
          echo $fila[0];
          echo $fila[1];
          echo "Hola";
 
          $usuario->close();
 
 

Yo tengo que tomar los 10 ultimos registros que se encuentran en una base de datos, eso lo hago pero despues tengo que tomar los datos de cada fila y mostrarlos, eso antes lo hacia sin la programacion orientada a objetos, pero ahora quiero pasar todo y se me complico, la manera como la realizaba antes era lo que esta como comentario y de esa manera funcionaba perfecto, pero como deberia hacerlo en POO.

Y otra cosa, en la funcion extraer_registro() de la clase supuestamente deberia devolver una variable $fila que es un array con los datos, pero como los recibo a esos datos?
Lo hice de la siguiente manera como ven en el codigo:
$fila = $usuario->extraer_registro();
y despues intente ver que tenia la variable pero no tiene nada o por lo menos no muestra nada...

JrcsDev

En el método extraer_registro () de tu clase puedes colocar en el return lo siguiente:

Código: PHP

public function extraer_registro ()
{
return mysql_fetch_array($this->resultado,MYSQL_ASSOC);
}
 

Al retornar una estructura que usa el mysql para iterar los registros, luego puedes implementar tu clase como lo hacias con el mysql:

Código: PHP

while ($fila = $usuario->extraer_registro())
{
echo $fila[0];
echo $fila[1];
}
 

Debería funcionar. Mi hermano hizo una clase para mysql y creo que la implementó así.

RadicalEd

Te iba a envíar 2 clases hechas por mi en php pero el foro no me dejo, dame tú mail por MP y te las mando.

Ale992

Ahi esta, muchas gracias.

Otra pregunta y espero qeu la ultima asi no te sigo molestando, conviene agregar en un solo archivo todas las funciones MySQL que utilize en toda la web o conviene mas ir armando archivos separados para que no se arme un archivo que pese tanto?

Te molestaria agregarme al msn? alee.m (aroba) live.com.ar

Saludos y gracias denuevo.

Ale992

Cita de: "Edo"

Te iba a envíar 2 clases hechas por mi en php pero el foro no me dejo, dame tú mail por MP y te las mando.

En el mensaje anterior deje el mail porque aun no me deja enviar MP. Si quieren agregarme no hay drama, quizas algun dia yo pueda sacarlos de algun apuro (tendre que estudiar bastante para alcanzarlos pero quien sabe jaja ).

Saludos.

JrcsDev

Ya te agregué, te conviene tener mejor todo ordenado. No tengas miedo al tamaño del archivo. El texto plano representa una gran ventaja porque tendrás que escribir DEMASIADO para que un archivo de código te quede pesado. Lo mejor es que hagas tus clases, preferiblemente cada clase en un archivo para tener todo mejor ordenado (aunque nada impide que pongas todas las clases en uno solo). Lo mismo con las funciones, trata de centralizar los recursos.

RadicalEd

Cita de: "Ale992"

Ahi esta, muchas gracias.

Otra pregunta y espero qeu la ultima asi no te sigo molestando, conviene agregar en un solo archivo todas las funciones MySQL que utilize en toda la web o conviene mas ir armando archivos separados para que no se arme un archivo que pese tanto?

Te molestaria agregarme al msn? alee.m (aroba) live.com.ar

Saludos y gracias denuevo.

Yo no uso messenger, si quieres podemos por el GTalk mi usuario es willian.radicaled-gmail-com
ya se fueron las 2 clases.

JrcsDev

Te voy a agregar Edo ... saludos

DriverOp

Ya que al leer el código de la clase que has hecho me has dado una idea que me parece excelente (me refiero al método utf) devuelvo el favor con una mejora. Se trata de cómo recorrer los resultados de una consulta. He aquí el código:

Código: PHP

 
...
public $numrows = 0; // Nueva propiedad
...
public function consulta($consulta) {
  $this->resultado = mysql_query($consulta,$this->descriptor);
  $this->numrows = mysql_num_rows($this->resultado);
}
 
public function primero() {
  if ($this->numrows > 0) {
    mysql_data_seek($this->resultado,0);
   return mysql_fetch_assoc($this->resultado);
  }
 else { return false; }
}
 
public function siguiente() {
  if ($this->numrows > 0) {
    return mysql_fetch_assoc($this->resultado);
  }
  else { return false; }
}
...
 
 

Y se puede usar así (usando tu ejemplo):

Código: PHP

 
if ($fila = $usuario->primero()) {
  do {
// Hacer lo apropiado con $fila
  } while ($fila = $usuario->siguiente());
}
 
 

O bién, en caso que quieras saber si la consulta regresó 1 o más registros:

Código: PHP

 
if ($usuario->numrows > 0) {
  echo "La consulta generó: ".$usuarios->numrows." registros.";
}

Enjoy

JrcsDev

Excelente aporte DriverOp... lo cual me da una idea para mejorar una clase que ya tengo. Esta se puede mejorar más si se crean 2 clases... una que permita especificar los datos de conexion y el query, y que una vez se haya hecho eso, retorne otro objeto de tipo BASE DE DATOS, de manera que se pueda usar de la siguiente forma:

Código: PHP

$objDB=connect($host,$user,$pass);
$rs=$objDB->query("select dato from tabla where activo=1");
 

Y lo funcional estaría en que con el mismo objeto puedes abrir otra consulta a otra tabla, incluso otra base de datos:

Código: PHP

$rs2=$objDB->query("select otros from tabla2");
 

Y sería muy útil poder recorrerlo así

Código: PHP

while ($rs->fila())
{      echo $rs[0];
       echo $rs['dato'];
}
 

Y de la misma forma con el $rs2... y tener funciones como:

Código: PHP

$rs->count();
$rs2->vacio();
$rs2->error();
 

Es una idea, sería bastante útil...

Ale992

Gracias a los dos, ahora voy a ver como optimizar las clases.

Y sobre el UTF es una clase que cree ya que estoy haciendo un manejador de noticias y de no poner todo en UTF no me reconocia los acentos en las palabras, asique me sirvio para solucionar ese problema y me alegro que te haya servido

.

RadicalEd

Si vas a usar paginador te funcionará, es más es algo que les falta a mis clases.

JrcsDev

Aquí les dejo la clase que hice para mysql:

Código: PHP

class FMysql
{   var $host;
    var $pass;
    var $user;
    var $link;
    var $numQuerys=0;
    var $db;
    function __construct($host,$user,$pass,$db)
    {   $this->user=$user;
        $this->pass=$pass;
        $this->host=$host;
        $this->db=$db;
        $this->link=mysql_connect($this->host,$this->user,$this->pass);
        mysql_select_db($this->db,$this->link);         
    }
    function query($sql)
    {   $this->numQuerys++;
        return new FMysqlObjectDB($sql,$this->link);
    }
}
 
class FMysqlObjectDB
{   var $regs;
    var $res;
    var $pos;
    var $data;
    function __construct($sql,$link)
    {   @$this->res=mysql_query($sql,$link);
        $this->regs=mysql_num_rows($this->res);
        if (mysql_error())
        {   echo "<div style="border:1px solid gray;font-family:Helvetica;font-size:11px;width:200px;background-color:#FFFF99;">
                    <b>Error (".mysql_errno()."):</b> ".mysql_error().
                    "</div>";
        
            exit();
        }
        $this->pos=0;
    }       
    function numRegistros()
    {   return $this->regs;
    }
    function getFila()
    {  
        if ($this->regs>0 && $this->pos<$this->regs)
        {   mysql_data_seek($this->res,$this->pos);
            $this->data=mysql_fetch_array($this->res);
            $this->pos++;
            
            return $this->data;
        }
        else
        {   $this->pos=0;
            return false;               
        }
    }
    function __get($nombre)
    {   return $this->data[$nombre];
    }
}
 
 

Y la forma de implementarlo sería:

Código: PHP

$SQL=new FMysql("localhost","root","","datos");    
//Forma 1
$rs1=$SQL->query("select id,nombre from vt_clientes order by id");
while ($rs1->getFila())
{   echo "Id: ".$rs1->id;
    echo "<br>Nombre: ".$rs1->nombre;       
    echo "<br>";
 
}
 
//Forma 2
$rs2=$SQL->query("select id,nombre from vt_clientes order by id");
while ($dat=$rs2->getFila())
{   echo "Id: ".$dat['id'];
    echo "<br>Nombre: ".$dat['nombre'];     
    echo "<br>";
 
}
 
 

La forma 1 es implementando métodos mágicos, y la forma 2 es similar a la normal de mysql...

Para ingresar y eliminar:

Código: PHP

$SQL->query("insert into personas (nombre,apellido) values ('pedro','perez')");
$SQL->query("delete from personas where id=1");
 

De igual forma se hace para actualizar. Le faltan otras cosas como un método que devuelva el último id...

DriverOp

Bien pues, para controlar errores se me ha ocurrido lo siguiente.

Código: PHP

 
/* Nuevas propiedades */
  public $errmsg = "";
  public $error = false;
  public $errno = 0;
 
  private function VerifError() { // No es necesario que sea accesible desde afuera
   $this->errno = mysql_errno();
   $this->error = $this->errno != 0;
   $this->errmsg = $this->errno.": ".mysql_error();
   return $this->error;
}
 
 

Luego en cada método suceptible de experimentar un error agregar este código, por ejemplo al conectar...

Código: PHP

 
private function conectar_base_datos() {
  $this->descriptor = mysql_connect($this->servidor,$this->usuario,$this->pass);
  if (!$this->VerifError()) {
    mysql_select_db($this->base_datos,$this->descriptor);
    $this->VerifError();
  }
}
 
 

Y se usaría (usando el ejemplo de ale992):

Código: PHP

 
$usuario = new db();
if (!$usuario->error) {
  // trajabar con la base de datos
} else {
  echo $usuario->$errmsg; // Mostrar el mensaje de error.
}
 
 

Y así con el resto de los métodos.
También se me ha ocurrido agregar métodos específicos para hacer Insert, Update y Delete sin tener que escribir toda la sentencia completa sino pasándole los parámetros propios, obligatorios y opcionales a través de métodos de la clase; pero quedará para un próximo aporte

RadicalEd

Lo maluco de los metodos propios para INSERT, DELETE Y UPDATE es que al pasarle los parámetros solo estarías ahorrando muy poco de código, por que debes pasar un array con los campos y un parámetro extra con la tabla, yo para eso solo hicé un metodo llamado query de esta forma:

Código: PHP

  /*
   * Este m&eacute;todo mostrar&aacute; datos de una consulta de varias tablas
   * @param string $query    sql query
   * @param integer $tipo    El tipo de query que se ejecutar&aacute;
   * @param string $error    Para mostrar un error diferente al default de MySQL
   * @return $arrayRetorno   Tiene el resultado, la cantidad de registros, un array con los registros y el &uacute;ltimo indexado
   */
  public function query($query, $tipo=0, $trace=0, $error='<br /><b>Query ejecutado incorrectamente<br />')
  {
    $this->result = mysql_query($query) or die(mysql_error().$error.$query.'</b>');
 
    //Dependiendo del tipo de consulta SQL escojida
    //extraemos las variables
    switch($tipo)
    {
      case 0:  //SELECT
           $this->numrows = mysql_num_rows($this->result);
           $this->arrayRegistros = mysql_fetch_array($this->result);
           $this->ultimo = '';
           break;
      case 1:  //INSERT
           $this->ultimo = mysql_insert_id();
           $this->numrows = '';
           $this->arrayRegistros = '';
           break;
      case 2:  //UPDATE
      case 3:  //DELETE
      case 4:  //ALTER
           $this->numrows = '';
           $this->arrayRegistros = '';
           $this->ultimo = '';
           break;
      case 5:  //SHOW TABLES
      case 6:  //OPTIMIZE TABLE
           $this->numrows = '';
           break;
    }
 
    //Estos son todos los datos que se pueden extraer de un query
    //Aca lo que se hace es devolverlo todo en un array
    $arrayRetorno = array($this->result, $this->numrows, $this->arrayRegistros, $this->ultimo);
 
    if ($trace==1)
    {
      debug_print_backtrace();
    }
 
    return $arrayRetorno;
  }
 
 

DriverOp

Edo:
La tuya es una buena idea pero mi intención no era ahorrar código sino que no tener que lidiar con la sintaxis de insert, update y delete toda las veces. Es cierto que tienes que pasar un array pero es "más fácil" recordar la sintaxis de un array que de un insert o update. Al menos para mí es así.
Por ejemplo:

Código: PHP

 
// Nueva propiedad pública.
public $affectedrows = 0;
 
public function Update($tabla, $lista, $where) {
  $this->affectedrows = -1;
  if (!is_array($lista)) {
    $this->error = true;
    $this->errno = -1;
    $this->errmsg = "Segundo parámetro no es array ('campo'=>'valor')";
  return false; }
  else  {
    $aux = array();
    foreach ($lista as $key => $value) {
       $aux[] = sprintf("`%s` = `%s`", $key, $value);
    } // foreach
    $sql = sprintf("UPDATE `%s` SET %s", $tabla, implode(", ",$aux));
    if (!empty($where)) {
      $sql .= " WHERE ".trim($where);
    }
    $this->resultado = mysql_query($sql,$this->descriptor);
    if (!$this->VerifError()) {
      $this->affectedrows = mysql_affected_rows($this->descriptor);
      return true;
     } else { return false; }
  } // else
} // Update
 
 

Y se usa así:

Código: PHP

 
if ($usuario->Update("usuarios",array("nombre"=>"Juan", "apellido"=>"Perez"),"id = 1")) { echo "Se actualizaron ".$usuario->affectedrows." registros."; }

JrcsDev

Mi hermano hizo una clase que implementaba los métodos de ingresar, modificar, eliminar asi:

Para ingresar:

Código: PHP

$persona->nombre="jose";
$persona->apellido="perez";
$persona->guardar();
 

Para modificar:

Código: PHP

$persona->nombre="jose";
$persona->apellido="perez";
$persona->actualizar();
 

Para eliminar:

Código: PHP

$persona->id=1;
$persona->eliminar();
 

RadicalEd

Igual, debes decirle primero que todo cuando creas el objeto a que tabla va a apuntar y en vez de pasarle un array estás pasando parámetro por parámetro.

Todas son validas, lo que quise mostrar fue el hacerlo más corto y que quien lo vaya a leer después de mi no se trabe con tantos metodos, sino que encuentra uno solo y se guía con este.

DriverOp

Pues supongo que es una cuestión de costumbre. A mi me parece más intuitivo para el programador que venga después de mí, esto $objeto->update() que estar recordando que un 2 en el segundo parámetro de la función indica que se está actualizando la tabla.

RadicalEd

jejeje, sí, eso estaba viendo, pero el switch se puede pasar por parámetro string, ósea:

Código: PHP

  /*
   * Este método mostrará datos de una consulta de varias tablas
   * @param string $query    sql query
   * @param integer $tipo    El tipo de query que se ejecutará
   * @param string $error    Para mostrar un error diferente al default de MySQL
   * @return $arrayRetorno   Tiene el resultado, la cantidad de registros, un array con los registros y el último indexado
   */
  public function query($query, $tipo='SELECT', $trace=0, $error='<br /><b>Query ejecutado incorrectamente<br />')
  {
    $this->result = mysql_query($query) or die(mysql_error().$error.$query.'</b>');
 
    /* 
     * Dependiendo del tipo de consulta SQL escojida extraemos las variables
     */
    switch($tipo)
    {
      case 'SELECT':
           $this->numrows = mysql_num_rows($this->result);
           $this->arrayRegistros = mysql_fetch_array($this->result);
           $this->ultimo = '';
           break;
      case 'INSERT':
           $this->ultimo = mysql_insert_id();
           $this->numrows = '';
           $this->arrayRegistros = '';
           break;
      case 'UPDATE':
      case 'DELETE':
      case 'ALTER':
           $this->numrows = '';
           $this->arrayRegistros = '';
           $this->ultimo = '';
           break;
      case 'SHOW TABLES':
      case 'OPTIMIZE TABLE':
           $this->numrows = '';
           break;
    }
 
    /*
     * Estos son todos los datos que se pueden extraer de un query
     * Acá lo que se hace es devolverlo todo en un array
     */
    $arrayRetorno = array($this->result, $this->numrows, $this->arrayRegistros, $this->ultimo);
 
    if ($trace==1)
    {
      debug_print_backtrace();
    }
 
    return $arrayRetorno;
  }
 
 

Y se puede usar:

Código: PHP

$sql-query("UPDATE TABLA SET a=1 WHERE x=y", "UPDATE")
 