PHP + POO diferenciar objetos instanciados

[sebastianxjavier]

Estimados, junto con saludar, les cuento que estoy programando un sitio en php con poo.... Mi duda salta a la vista al realizar una query a la bd y la recorro para asi crear objetos dinamicamente con sus respectivos datos... pero luego al momento de mostrarlos, como los diferencio???

La idea es que los objetos se muestren de la forma:



Aplicando programación estructurada el código es así (Funciona Obviamente):

Código PHP:

  <?php
//el metodo "listarFactura" hace la query y la retorna en la variable $documentox
$documentox = $vendedor->listarFactura($nombre_cliente);
?>
//Se imprimen los registros resultantes
<table border="1" width="100%">
   <tr align="center">
    <td>N° Factura</td>
    <td>Cliente</td>
      <td>Fecha Emisión</td>
    <td>Fecha Vencimiento</td>
    <td>Monto Total</td>
    <td>Estado</td>
   </tr>
   <?php
   while ($registro = mysql_fetch_assoc($documentox))
   {
   ?>
   <tr align="center">
    <td><?php echo $registro['id_documento_pago']?></td>
    <td><?php echo $registro['cliente_id_cliente']?></td>
    <td><?php echo $registro['fecha_emision_documento_pago']?></td>
      <td><?php echo $registro['fecha_vencimiento_documento_pago']?></td>
    <td><?php echo $registro['total_documento_pago']?></td>
    <td><?php echo $registro['estado_documento_pago']?></td>
   </tr>
   <?php
   }
   ?>
  </table>  

Pero al llevarlo a objeto cambia a esto:

Código PHP:

  <?php
//el metodo "listarFactura" hace la query y la retorna en la variable $documentox
$documentox = $vendedor->listarFactura($nombre_cliente);
?>

/*
*se recorre la variable y se crea el objeto (solo funciona para un objeto)
*/
while ($registro = mysql_fetch_assoc($documentox))
{
     $documento = new Documento_Pago();
     $documento->setIdDocumentoPago($registro['id_documento_pago']);
     $documento->setFechaEmisionDocumentoPago($registro['fecha_emision_documento_pago']);
     $documento->setFechaVencimientoDocumentoPago($registro['fecha_vencimiento_documento_pago']);
     $documento->setTotalDocumentoPago($registro['total_documento_pago']);
     $documento->setEstadoDocumentoPago($registro['estado_documento_pago']);
}

/*
*Funciona solo para un objeto
*/
<table border="1" width="100%">
   <tr align="center">
    <td>N° Factura</td>
    <td>Cliente</td>
      <td>Fecha Emisión</td>
    <td>Fecha Vencimiento</td>
    <td>Monto Total</td>
    <td>Estado</td>
   </tr>
   <?php
   while ($registro = mysql_fetch_assoc($documentox))
   {
   ?>
   <tr align="center">
    <td><?php echo $documento->getIdDocumentoPago()?></td>
    <td><!--No se incluye en este objeto este dato--></td>
    <td><?php echo $documento->getFechaEmisionDocumentoPago()?></td>
      <td><?php echo $documento->getFechaVencimientoDocumentoPago()?></td>
    <td><?php echo$documento->getTotalDocumentoPago()?></td>
    <td><?php echo $documento->getEstadoDocumentoPago()?></td>
   </tr>
   <?php
   }
   ?>   

Ahora mis dudas... como lo hago para poder instancias dinamicas de objetos? poniendo un contador antes del while que crea y le hace set al objeto? hay una forma mas elegante???? pero lo mas importante si tengo ya creado 2 objetos de la misma clase... como los puedo diferenciar para trabajar con ellos? salu2!

[pateketrueke]

Yo veo que el primer bucle del segundo código está de sobra, pues ahí no haces nada con el objeto, solo lo estás creando y eso no sirve de nada.

En todo caso, ¿que tan difícil es usar sólo el segundo bucle para crear e imprimir todo ahí?

Además si sólo estás imprimiendo los datos, ¿que caso tiene crear el objeto y no imprimir directamente?

No veo ninguna ventaja de lo que haces, nada. ¡Justo como el primer código!

[sebastianxjavier]

Iniciado por pateketrueke

Yo veo que el primer bucle del segundo código está de sobra, pues ahí no haces nada con el objeto, solo lo estás creando y eso no sirve de nada.

En todo caso, ¿que tan difícil es usar sólo el segundo bucle para crear e imprimir todo ahí?

Además si sólo estás imprimiendo los datos, ¿que caso tiene crear el objeto y no imprimir directamente?

No veo ninguna ventaja de lo que haces, nada.

Creo el objeto para poder mostrarlo y trabajar con POO.. por lo otro no lo imprimo altiro porque omiti que estoy trabajando con MVC, por lo que de ahi me voy a la vista y ahi recien imprimo el objeto

[pateketrueke]

Bien, pues entonces primero debes almacenar los objetos creados en un array, y luego iterar ese array en tu vista.

Código PHP:

  $out = array(); 
 
while ($row = fetch($algo)) { 
  $doc = new Doc; 
  // ... 
 
  $out []= $doc; 
} 
 
// vista 
 
foreach ($out as $doc) { 
  echo $doc->getBar(); 
}  
   

[sebastianxjavier]

Iniciado por pateketrueke

Bien, pues entonces primero debes almacenar los objetos creados en un array, y luego iterar ese array en tu vista.

Código PHP:

  $out = array(); 
 
while ($row = fetch($algo)) { 
  $doc = new Doc; 
  // ... 
 
  $out []= $doc; 
} 
 
// vista 
 
foreach ($out as $doc) { 
  echo $doc->getBar(); 
}  
   

Muchas gracias compadre! me ha servido de maravilla :D!

[oscartt67]

Hola Sebastian. Queria hacerte una observación:

En lugar de crear un método GetPropiedad y un SetPropiedad para cada propiedad de la clase, puedes plantearte hacer un Get(propiedad) y un Set(propiedad,valor), y definir en la clase además un preproceso y postproceso para los casos que lo necesiten.

Saludos!

[sebastianxjavier]

Iniciado por pateketrueke

Bien, pues entonces primero debes almacenar los objetos creados en un array, y luego iterar ese array en tu vista.

Código PHP:

  $out = array(); 
 
while ($row = fetch($algo)) { 
  $doc = new Doc; 
  // ... 
 
  $out []= $doc; 
} 
 
// vista 
 
foreach ($out as $doc) { 
  echo $doc->getBar(); 
}  
   

Compadre tengo una ultima consulta : Como te mencione con anterioridad me resulto perfecto guardando el objeto en un arreglo y luego recorrerlo con foreach... pero ahora me salto un pequeño problemilla que creo que me puedes asesorar....

Código PHP:

  $out1 = array(); 
$out2 = array(); 
 
while ($row = fetch($algo)) { 
  //1er objeto 
  $doc = new Doc(); 
  //2do objeto 
  $cliente = new Cliente(); 
  // ... 
 
  $out1 []= $doc; 
  $out2 []= $cliente; 
} 
 
// vista 
 
foreach ($out as $doc) { 
  echo $doc->getBar(); 
}  
   

Como veras en la vista no modifique nada, mi pregunta es la siguiente.. como puedo hacer que imprima los objetos de documento y a su vez los del cliente asociado a ese documento, que se encuentra en otro objeto y a su vez en otro arreglo? trate de utilizarlo en el mismo arreglo pero me cambiaba el objeto con los metodos del otro xD! Atento a tus comentarios

Iniciado por oscartt67

Hola Sebastian. Queria hacerte una observación:

En lugar de crear un método GetPropiedad y un SetPropiedad para cada propiedad de la clase, puedes plantearte hacer un Get(propiedad) y un Set(propiedad,valor), y definir en la clase además un preproceso y postproceso para los casos que lo necesiten.

Saludos!

Se agradece el comentario y optimizare las clases... gracias

[pateketrueke]

Podrías guardar ambos objetos en otro array, y luego listarlos.

Código PHP:

  $out []= array($foo, $bar); 
 
// ... 
while (list($foo, $bar) = each($out)) { 
  $foo->candy(); 
  $bar->does(); 
}  
   

[sebastianxjavier]

Iniciado por pateketrueke

Podrías guardar ambos objetos en otro array, y luego listarlos.

Código PHP:

  $out []= array($foo, $bar); 
 
// ... 
while (list($foo, $bar) = each($out)) { 
  $foo->candy(); 
  $bar->does(); 
}  
   

pucha compadre no me resulto.... me sale este error: "Fatal error: Call to a member function getIdCliente() on a non-object in C:\wamp\www\svcb\_vista\clienteMoroso.php on line 26". En esa linea tengo el codigo "echo $cliente->getIdCliente()"

El error no me hace referencia al objeto... tengo el codigo de esta manera:

Código PHP:

  $out1 = array(); 
$out2 = array(); 
 
while ($registro = mysql_fetch_assoc($documentox)) 
{ 
   $cliente = new Cliente(); 
   $cliente->setIdCliente($registro['id_cliente']); 
   // ... 
  $documento = new Documento_Pago(); 
  $documento->setIdDocumentoPago($registro['id_documento_pago']); 
  // ... 
  $out1[] = $documento; 
  $out2[] = $cliente; 
} 
$gua[] = array($out1,$out2); 
 
// vista 
 
while (list($documento, $cliente) = each($gua)) 
{ 
   echo $cliente->getIdCliente(); 
   echo $documento->getIdDocumentoPago(); 
}  
   

Ojala me puedas ayudar, ya que los objetos ya me tienen mariado :(... saludos compadre

[oscartt67]

...no se localizar el error en tu código, pero teniendo en cuenta que hay el mismo num de elementos en ambos arrays, otra manera seria:

Código PHP:

  $out1 = array(); 
$out2 = array(); 

while ($registro = mysql_fetch_assoc($documentox)) 
{ 
   $cliente = new Cliente(); 
   $cliente->setIdCliente($registro['id_cliente']); 
   // ... 
  $documento = new Documento_Pago(); 
  $documento->setIdDocumentoPago($registro['id_documento_pago']); 
  // ... 
  $out1[] = $documento; 
  $out2[] = $cliente; 
} 

// vista 
for($i=0;$i<count($out1);$i++){
   echo $out1[i]->getIdCliente(); 
   echo $out2[i]->getIdDocumentoPago(); 
}  
   

saludos.

[masterpuppet]

Que tal sebastianxjavier,

No tienes una asociación entre las clases ? cual es tu lógica ?, un cliente tiene varios documentos y un documento pertenece a un cliente ?, no deberias tener algo asi:

Código PHP:

$client = new Client();
$doc->setClient($client);
$docs[] = $doc;
 
foreach($docs as $doc) {
   //doc
    echo $doc->getId();
    echo $doc->getName();
    //client
    echo $doc->getClient()->getName();
   //or proxy method
    echo $doc->getClientName();
}
 

Con respecto a los getters & setters, la tendencia por suerte es como lo tienes ahora, menos magia, te sugiero que lo dejes tal cual, y ten en cuenta que si haces tal modificación los IDE's o editores no van a poder hacer una correcta introspección de la clase.

Saludos.

[oscartt67]

Con respecto a los getters & setters, la tendencia por suerte es como lo tienes ahora, menos magia, te sugiero que lo dejes tal cual, y ten en cuenta que si haces tal modificación los IDE's o editores no van a poder hacer una correcta introspección de la clase.

Es cierto que limita a los editores, pero definiendo una sola clase de datos CRUD y extendiéndola, solucionas todas las clases que se relacionan con la db de tu web. La otra opción, es escribir gets y sets de todos los campos de todas las tablas.

Saludos.

[sebastianxjavier]

Iniciado por oscartt67

...no se localizar el error en tu código, pero teniendo en cuenta que hay el mismo num de elementos en ambos arrays, otra manera seria:

Código PHP:

  $out1 = array();  
$out2 = array();  
 
while ($registro = mysql_fetch_assoc($documentox))  
{  
   $cliente = new Cliente();  
   $cliente->setIdCliente($registro['id_cliente']);  
   // ...  
  $documento = new Documento_Pago();  
  $documento->setIdDocumentoPago($registro['id_documento_pago']);  
  // ...  
  $out1[] = $documento;  
  $out2[] = $cliente;  
}  
 
// vista  
for($i=0;$i<count($out1);$i++){ 
   echo $out1[i]->getIdCliente();  
   echo $out2[i]->getIdDocumentoPago();  
}  
   

saludos.

Muchas gracias compadre... asi mismo me ha funcionado :).. lo raro es que me quedare con la duda de porque no me sirvio pasar los array a una lista.. pero bueno.. funciona que es lo importante... te pasaste! salu2!

Iniciado por masterpuppet

Que tal sebastianxjavier,

No tienes una asociación entre las clases ? cual es tu lógica ?, un cliente tiene varios documentos y un documento pertenece a un cliente ?, no deberias tener algo asi:

Código PHP:

$client = new Client();
$doc->setClient($client);
$docs[] = $doc;
 
foreach($docs as $doc) {
   //doc
    echo $doc->getId();
    echo $doc->getName();
    //client
    echo $doc->getClient()->getName();
   //or proxy method
    echo $doc->getClientName();
}
 

Con respecto a los getters & setters, la tendencia por suerte es como lo tienes ahora, menos magia, te sugiero que lo dejes tal cual, y ten en cuenta que si haces tal modificación los IDE's o editores no van a poder hacer una correcta introspección de la clase.

Saludos.

Bueno, básicamente como se trabaja en objetos, el id del cliente y los atributos que siguen de él solo pertenecen a la clase cliente que hereda de persona, por lo cual no son atributos de la clase documento, por lo que es esa la razón para no haber puesto los datos del cliente en esta ultima clase, pero obviamente se ve reflejado en el modelo físico de datos con la asociación pertinente 1->n. Saludos y gracias

[masterpuppet]

Desde el momento que el documento tiene la clave foránea de cliente a nivel de domain model existe la asociación, luego te toca a ti decidir si es uni o bidireccional y en el ejemplo que muestras es mas que claro que tiene que ser bidireccional..., tu clase documento debería tener como propiedad un cliente y el cliente como propiedad una colección de documentos, es un one-to-many clásico, te sugiero que revises tu domain model.

Saludos.