0

Si unos registros cumplen la condición quiero mostrarlos en pantalla. Si ningún registro cumple la condición que muestre un mensaje. Tengo este código y muestro los datos usando plantilla Smarty. Pero me arroja un mensaje que la $matriz no está definida. La tabla tiene registros. El código es este:

$BD = new ConexionDB();
$sql = "SELECT COUNT(*) FROM tabla WHERE campo = 'X'";
if ($sth = $BD->query($sql)) {
    if ($sth->fetchColumn() > 0) {
        $sql = "SELECT cod, nom FROM tabla WHERE campo = 'X'";
        while ($fila = $sth->fetch(PDO::FETCH_ASSOC)) {
            $matriz[] = new DatosVO($fila['cod'], $fila['nom']);
        }
        $NoRegistros = "";
    } else {
        $NoRegistros = "NO EXISTEN REGISTROS";
        $matriz = '';
    }
    $exito = TRUE;
}
$tpl = new Plantilla();
if ($exito) {
    $tpl->assign('NoRegistros', $NoRegistros);
    $tpl->assign('lista', $matriz);
    $tpl->display('mostrar.tpl.php');
} else {
    $tpl->assign('mensaje', $mensaje);
    $tpl->display('fallo.tpl.php');
}

Yo utilizaba el rowcount() de esta manera:

$sql = "SELECT cod, nom FROM tabla WHERE condicion = 'X'";
$sth = $BD->prepare($sql);
$sth->execute();

if ($sth->rowCount() > 0) {

    while ($fila = $sth->fetch(PDO::FETCH_ASSOC)) {
        $matriz[] = new DatosVO($fila['cod'], $fila['nom']);
    }
    $NoRegistros = "";

} else {
    $NoRegistros = "NO EXISTEN REGISTROS";
    $prenonomstuds = "";
}

Pero en el manual de PHP http://php.net/manual/es/pdostatement.rowcount.php ejemplo #2 dice que rowcount no devuelve los registros de un select y recomienda usar query().

4
  • 1
    te marca error cuando existen o cuando no existen registros? Commented el 27 abr. 2017 a las 20:55
  • ¿y si en vez de un COUNT(*) haces esta consulta: SELECT cod, nom FROM tabla WHERE campo = 'X' y verificas si esa consulta trajo datos? Commented el 27 abr. 2017 a las 21:01
  • Sí hay registros que cumplen la condición.
    – Puntay
    Commented el 27 abr. 2017 a las 21:03
  • donde ejecutas tu segundo $sql? Commented el 27 abr. 2017 a las 21:04

1 respuesta 1

2

ya que nunca corres tu segundo Query, no encuentras ['cod'] ni ['nom'] y ya que tu matriz es creada dentro del while esta no se define cuando la llamas en el assign.

si lo defines antes igual siempre te marcaría una tabla vacía.

así que necesitarías hacer un array vació con $matriz=[]; y luego llenarla con tus datos, no tiene caso hacer dos llamadas a tu base de datos solo para conocer la longitud del resultado, solo llama tu segundo Query y si la matriz esta vacía tendrá un tamaño de 0.

$BD = new ConexionDB();
$sql = "SELECT cod, nom FROM tabla WHERE campo = 'X'";
$matriz=[];
if ($sth = $BD->query($sql)) {
    while ($fila = $sth->fetch(PDO::FETCH_ASSOC)) {
        $matriz[] = new DatosVO($fila['cod'], $fila['nom']);
    }
    if (count($matriz)==0){
        $NoRegistros = "NO EXISTEN REGISTROS";
    }
    $exito = TRUE;
}
$tpl = new Plantilla();
if ($exito) {
    $tpl->assign('NoRegistros', $NoRegistros);
    $tpl->assign('lista', $matriz);
    $tpl->display('mostrar.tpl.php');
} else {
    $tpl->assign('mensaje', $mensaje);
    $tpl->display('fallo.tpl.php');
}
12
  • El segundo query. Cierto!. Con respecto a tu código si en caso no hay registros igual va a procesar el while? Pregunto porque es después que preguntas si $matriz tiene o no registros.
    – Puntay
    Commented el 27 abr. 2017 a las 21:26
  • cuando no hay registros el código dentro del while no se ejecuta, por que en esencia le estas diciendo has esto mientras existan registros. Commented el 27 abr. 2017 a las 21:38
  • Una consulta más. El While para efectos de optimización se puede cambiar por un Foreach o no es necesario?
    – Puntay
    Commented el 27 abr. 2017 a las 21:45
  • Puedes cambiarlo no haría ninguna diferencia en la forma que se ejecuta el resto del código. Commented el 27 abr. 2017 a las 21:52
  • El código es redundante. Desde que ejecutas el query : if ($sth = $BD->query($sql)) { ya sabes si hay registros, pues en PDO el método query devuelve un array vacío si no hay registros. Por tanto todo el bloque if count($matriz)>0{... es inútil.
    – A. Cedano
    Commented el 27 abr. 2017 a las 22:05

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.