Teniendo en cuenta lo que dice el Manual de PHP sobre fetchAll
:
Valores devueltos
PDOStatement::fetchAll()
devuelve un array que contiene tadas las
filas restantes del conjunto de resultados. El array representa cada
fila como un array con valores de las columnas, o como un objeto con
propiedades correspondientes a cada nombre de columna. Es devuelto
un array vacío si hay cero resultados que obtener, o FALSE en caso de
fallo.
Puedes usar un operador ternario que evalúe el resultado de fetchAll
, devolviendo dicho resultado o 0
.
static public function mdlMostrarSaldo($tabla, $item, $valor){
$stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla WHERE $item = :$item");
$stmt -> bindParam(":".$item, $valor, PDO::PARAM_STR);
$stmt -> execute();
/*Almacenamos el resultado en $data*/
$data=$stmt -> fetchAll();
/*Esto debe ir antes del return. Ya tenemos los datos, podemos cerrarlo*/
$stmt = NULL; //Si quieres, para tranquilizar la conciencia
/*
*Aquí se evalúa $data:
* - si no es FALSE o NULL devuelve $data
* - si es FALSE o NULL devuelve 0
*/
return ( $data ) ? $data : 0;
}
Corrigiendo el error de lógica al recibir los datos
Dado que mdlMostrarSaldo
retornará 0
en caso de no haber resultados, debes adaptar el código donde lees el resultado de la llamada a esa función. Cuando retorne 0
no puedes usar un foreach
para leer los datos, porque 0
no es un array. Entonces tienes que hacer algo así:
$saldo = ControladorCuadros::ctrMostrarSaldo($item, $valor);
if ($saldo!=0) {
foreach ($saldo as $key => $value) {
$saldo = $value["saldo"];
}
}
ADVERTENCIA SOBRE UN ERROR DE LÓGICA: En tu código se aprecia un problema de lógica. En esta línea dentro del foreach
: $saldo=$value["saldo"];
se van a sobre-escribir las posibles filas que
retorne fetchAll
, y el valor de $saldo
se quedará únicamente con
el último registro. Esto no tiene ningún sentido. No sé que uso le das
a esos datos luego, pero lo correcto sería crear una variable de
retorno para devolverla a la vista o donde sea. Dentro del foreach
, se va concatenando cada fila (con .=
) a esa variable de retorno. Algo así:
$saldo = ControladorCuadros::ctrMostrarSaldo($item, $valor);
$output="No se encontraron registros";
if ($saldo!=0) {
$output="";
foreach ($saldo as $key => $value) {
$output .= $value["saldo"].PHP_EOL;
}
}
//retornar o imprimir $output
Otra cosa de la que debes asegurarte es que los datos que retorna fetchAll
tengan una clave llamado saldo
. Para verificar puedes hacer un var_dump($saldo)
antes del foreach
.
$data=$stmt -> fetchAll(); return ( $data ) ? $data : 0;
Por cierto, si haces$stmt -> close();
no necesitas usar también$stmt = null;
, y ponlo antes delreturn
. Todo lo que hay después de unreturn
no se ejecuta en una función.return
es como decir al código: no pases de aquí, retorna a la línea que e llamó.