[SOLUCIONADO] Mostrar resultado o mensaje

[bandolera]

Hola chicos.
Tengo esta clase donde se ejecuta una búsqueda con LIKE.
Si hay registros, creo el arreglo $salida[], si no hay registros asigno a $salida el valor FALSE.

Código PHP:

  $conecta = new Data(); 
$stm = $conecta->prepare("SELECT codigo, nombre FROM tabla WHERE nombre LIKE ?"); 
$stm->bindValue(1, "%$valor%", PDO::PARAM_STR); 
$stm->execute(); 
if ($stm->rowCount() > 0) {  
    while ($fila = $stm->fetch(PDO::FETCH_ASSOC)) 
    { 
        $salida[] = new ClienteVO($fila['codigo'], $fila['nombre']); 
    } 
} else {  
    $salida = FALSE; 
} 
return $salida;  
   

En el controlador pregunto si $listas (que recibe datos) es verdadero. Si lo es asigna la plantilla si es falso imprime mensaje "No hay Registros".

Código PHP:

  $DAO = new ClienteDAO(); 
 
$listas = $DAO->Buscar_like($_POST['dato']); 
$tpl = new Plantilla(); 
if($listas) 
{ 
    $tpl->assign('lista', $listas); 
    $tpl->assign('NoRegistros', "");         
} else { 
    $tpl->assign('lista', ""); 
    $tpl->assign('NoRegistros', "No hay Registros"); 
} 
$tpl->display("mostrar.tpl.php");  
   

Mi vista usando Smarty:

Código PHP:

  <table id="miTabla"> 
    <tr> 
        <th>Código</th> 
        <th>Nombre Stud</th> 
    </tr> 
    {section name=i loop=$lista} 
    <tr> 
        <td>{$lista[i]->codigo}</td> 
        <td>{$lista[i]->nombre}</td> 
    </tr> 
    {/section} 
 
    {if $NoRegistros} 
    <h3>{$NoRegistros}</h3> 
    {/if} 
</table>  
   

Todo esto funciona pero tengo dos dudas:
1) Está bien que utilice if($listas) para saber si es verdadero o falso? o debo utilizar isset() ?
2) El uso de smarty al tener o no registros que mostrar se puede simplificar?
Gracias por su ayuda.
Mi sistema ya está casi caminando.

[hhs]

Bandoler sengun entiendo esta parte esta dentro de un método de tu clase

Código PHP:

if ($stm->rowCount() > 0) {  
    while ($fila = $stm->fetch(PDO::FETCH_ASSOC)) 
    { 
        $salida[] = new ClienteVO($fila['codigo'], $fila['nombre']); 
    } 
} else {  
    $salida = FALSE; 
} 
return $salida;
 

De ser así, no necesitas el if else, y puedes regresar false o un arreglo vació

Código PHP:

if ($stm->rowCount() > 0) {
    return false; // or return [];
}
 
while ($fila = $stm->fetch(PDO::FETCH_ASSOC)) { 
    $salida[] = new ClienteVO($fila['codigo'], $fila['nombre']); 
} 
 
return $salida;
 

En el if, debed de usar el not(!) ya que un arreglo vacío el if lo toma como false así que esto

Código PHP:

$tpl = new Plantilla(); 
if($listas) 
{ 
    $tpl->assign('lista', $listas); 
    $tpl->assign('NoRegistros', "");         
} else { 
    $tpl->assign('lista', ""); 
    $tpl->assign('NoRegistros', "No hay Registros"); 
} 
$tpl->display("mostrar.tpl.php");
 

debe ser algo como

Código PHP:

$tpl = new Plantilla(); 
if(!$listas) 
{ 
    $tpl->assign('lista', $listas); 
    $tpl->assign('NoRegistros', "");         
} else { 
    $tpl->assign('lista', ""); 
    $tpl->assign('NoRegistros', "No hay Registros"); 
} 
$tpl->display("mostrar.tpl.php");
 

Si no quieres usar la negación entonces usa empty pero no es necesario.
La parte que tienes en el if de listas lo puedes hacer un poco mas legible usando un helper y puedes hacer algo como esto.

Código PHP:

function view($template, Array $data) {
    
    $tpl = new Plantilla();
    
    foreach($data as $key => $value) {
        $tpl->assign($key, $value);
    }
    
    $tpl->display($template);
    
}
 

Y esto lo puedes incluir en código original

Código PHP:

if(!$listas) 
{ 
    view('mostrar.tpl.php', ['listas' => $listas, 'NoRegistros' => '']);
} else { 
    view('mostrar.tpl.php', ['listas' => '', 'NoRegistros' => 'No hay Registros']);
}
 

si ves como se simplifica ?
Otras mejoras que puedes hacer es usar la opción PDO::FETCH_CLASS de fetch y dejar que PDO llene tus objetos y evitas hacer la asignación que tienes en el while.

[bandolera]

Hola. Gracias por tu respuesta tan descriptiva. Pero tengo algunas dudas:
1) En el caso del if(), en qué caso podría haber un arreglo vacío? No se supone que con el rowcount() ya se validó eso?
1.b) Pusiste:

Código PHP:

  if ($stm->rowCount() > 0) { 
    return false; // or return []; 
}  
   

Pero siguiendo tu lógica creo que debería ser:

Código PHP:

  if ($stm->rowCount() = 0) { 
    return false; // or return []; 
}  
   

2) Lo último que dices que no es necesario la asignación y que deje que PDO llene los objetos. No es necesario usar el VO? A qué te refieres exactamente y cómo varía el proceso? No entendí.
Gracias.

[hhs]

Iniciado por bandolera

Hola. Gracias por tu respuesta tan descriptiva. Pero tengo algunas dudas:
1) En el caso del if(), en qué caso podría haber un arreglo vacío? No se supone que con el rowcount() ya se validó eso?
1.b) Pusiste:

Eso lo decides tu, es válido regresar false o un arreglo vacío.

2) Lo último que dices que no es necesario la asignación y que deje que PDO llene los objetos. No es necesario usar el VO? A qué te refieres exactamente y cómo varía el proceso? No entendí.
Gracias.

Los métodos fetch y fetchAll pueden regresar un objeto usando le opción PDO::FETCH_CLASS esto hace que estos métodos regresen un objeto de una clase que tu indiques. El llenado del objeto se realiza usando los nombres de propiedades que sean igual a las que regresa el resultado de la consulta.

Como ejemplo voy a usar una clase sencilla que solo tiene nombre y correo el nombre de estas propiedades corresponde con los mismo nombres que se tiene en la base de datos.

Código PHP:

class User {
 
    private $name;
 
    private $email;
 
    public function name()
    {
        return $this->name;
    }
 
    public function email()
    {
        return $this->email;
    }
}
 

El código que llena la clase puede ser algo como esto:

Código PHP:

try{
    $dbh = new PDO($dsn, $user, $pass, $options);
 
    $sqlResult = $dbh->query('select * from users');
    $rows = $sqlResult->fetchAll( PDO::FETCH_CLASS, User::class );
 
    foreach ($rows as $row) {
        echo "<p>{$row->name()}</p>";
        echo "<p>{$row->email()}</p>";
    }
}
    // Catch any errors
catch(PDOException $e){
    $error = $e->getMessage();
}
 

Si te das cuenta, obtuvimos un arreglo de objetos de la clase User y no tuve que asignar, llamar a métodos o crear el objeto para asignar los valores.
Te dejo como lo haces tu para que observes las diferencias:

Código PHP:

while ($fila = $stm->fetch(PDO::FETCH_ASSOC)) { 
    $salida[] = new ClienteVO($fila['codigo'], $fila['nombre']); 
}