Hay varios problemas en tu código:
- La variable no está definida para todos los ámbitos, de modo que si el
if
no se cumple la variable no existirá. Ese es el motivo del error actual. Pero hay más cosas ...
- No necesitas usar
mysqli_num_rows()
para saber si hay filas, de hecho, en algunos escenarios esta llamada no te garantizará tal comprobación. Es mejor verificar los datos en sí mismos como veremos en la solución propuesta.
- No necesitas llevar un contador (
$i
) para ir llenando el eventual array con los datos.
- No necesitas usar
MYSQLI_BOTH
el cual organiza los datos de dos formas, como array asociativo y como array numérico, basta con tenerlos como array asociativo, todo lo demás es desperdiciar memoria.
Propongo esta solución aplicando todas las mejoras señaladas más arriba:
function listar() {
/*
Al declararla aquí, la variable estará en
todo el ámbito de la función
*/
$arregloUnidadMediciones = array();
$comandoSql = "SELECT * FROM unidad_de_medicion";
$objControlConexion = new ControlConexion();
$objControlConexion->abrirBd($GLOBALS['serv'],$GLOBALS['usua'],$GLOBALS['pass'],$GLOBALS['bdat'],$GLOBALS['port']);
$recordSet = $objControlConexion->ejecutarSelect($comandoSql);
while($row = $recordSet->fetch_array(MYSQLI_ASSOC)){
$objUnidadMedicion=new Unidad_Medicion("","");
$objUnidadMedicion->setId($row['id']);
$objUnidadMedicion->setDescripcion($row['descr']);
$arregloUnidadMediciones[]=$objUnidadMedicion;
}
$objControlConexion->cerrarBd();
/*
Si no hay datos, $arregloUnidadMediciones será false
esto sustituye a mysqli_num_rows()
y permite una verificación más segura
basada en los mismos datos
*/
if (!$arregloUnidadMediciones) {
$arregloUnidadMediciones['error']='No hay datos';
}
return $arregloUnidadMediciones;
}
Donde recibes la respuesta, $arregloUnidadMediciones
tendrá los datos, o una clave error
describiendo que no hay datos, esto es útil si quieres (deberías) contemplar manejo de errores en tu código.
Otra cosa es que si el constructor de Unidad_Medicion
recibe en parámetro los datos luego pasas en los dos setter, los puedes obviar, pasando los datos al constructor, algo así:
$objUnidadMedicion=new Unidad_Medicion($row['id'],$row['descr']);
Pero no veo tu clase, para saber qué recibe el constructor de la misma.
if (mysqli_num_rows($recordSet) > 0)
, entonces$arregloUnidadMediciones
no estará definido, y abajo se está haciendoreturn
a una variable no definida. Dos opciones: (1) Coloca$arregloUnidadMediciones = array();
fuera delif
, o sea, en la línea anterior alif
. (2), Coloca unelse
para darle un valor a esa variable si no logró entrar alif
.