Hay varios problemas en tu código:
- Es inseguro: no conviene mostrar mensajes de error internos y
mysqli_real_escape_string
no fue seguro en el pasado. Usaremos consultas preparadas.
- Es demasiado optimista: hay errores que no controlas. Vamos a controlarlo todo
- Hace evaluaciones fuera de sitio: evalúas la variable
$fetch
a posteriori. Vamos a evaluar las cosas en su sitio
- Mezclas estilos de programación, lo cual no es recomendable para la claridad del código: usaremos solamente el estilo orientado a objetos en todo, por ser más claro y menos verboso
Aplicando todo lo dicho, el código quedaría así:
$id=$_POST['users'] ?? NULL;
if ($id) {
if($link) {
$sql="SELECT rendiciones, montoporrendir, montorendido, apellidos, nombre FROM datos where ids=?";
if($stmt = $link->query($sql)) {
$stmt->bind_param("s", $id);
$stmt->execute();
$stmt -> store_result();
$stmt->bind_result($rendiciones, $montoporrendir, $montorendido, $apellidos, $nombre);
$stmt->fetch();
if ($stmt->num_rows > 0) {
$outPut=array(
"status"=>true,
"data"=>array($rendiciones, $montoporrendir, $montorendido, $apellidos, $nombre)
);
} else {
$outPut=array(
"status"=>false,
"msg"=>"No se encontraron filas"
);
}
} else {
$outPut=array(
"status"=>false,
"msg"=>"Error en la consulta"
);
}
} else {
$outPut=array(
"status"=>false,
"msg"=>"No hay conexión"
);
}
} else {
$outPut=array(
"status"=>false,
"msg"=>"No se postearon datos"
);
}
La práctica implementada consiste en ir controlando mediante una variable llamada $outPut
todos los posibles problemas del código. Ante cualquier error, el valor de status
será false
y tendrá una clave msg
con un mensaje descriptivo del error. En cambio, cuando haya datos, la clave status
será true
y los datos estarán en una clave data
.
Ahora, puedes consumir $outPut
de este modo:
if ($outPut["status"]) {
echo implode(",",$outPut["data"]);
} else {
echo $outPut["msg"];
}
¡Tu código es kilométrico! Gritará un programador optimista. Sí, pero es un código maduro, robusto y que hace todo lo que debería hacer un código: buscar los datos y si no los encuentra, informar de por qué no los encontró e informar al usuario sobre lo que debe hacer en ese caso.
Por eso hay tantas ramas en este código. Si falta una de ellas, el programa fallará gravemente, o dejará al usuario en una pantalla en blanco. Si en esa pantalla hay un botón por ejemplo que envía datos al servidor, el usuario le dará N
veces, invadiendo al servidor inútilmente, hasta hartarse. ¿El problema en el fondo? Un programa demasiado optimista que no pensó en todos los posibles problemas. La programación optimista es un lastre y la red está llena de esos malos ejemplos.