Al programar es importante siempre tener en cuenta lo que devuelven los objetos que estamos utilizando, para evaluarlos adecuadamente y escribir un código controlado. El Manual de PHP esa información viene siempre detallada en el apartado llamado Valores devueltos.
El Manual de PHP en ese apartado dice que query()
:
Retorna FALSE
en caso de error. Si una consulta del tipo SELECT
,
SHOW
, DESCRIBE
o EXPLAIN
es exitosa, mysqli_query()
retornará un
objeto mysqli_result
. Para otras consultas exitosas de
mysqli_query()
retornará TRUE
.
En tu caso, al no evaluar el resultado de la consulta que intentas obtener mediante el método query
de mysqli
obtienes este aviso:
Trying to get property of non-object ...
Significa que $resultado
no es un objeto mysqli_result
, debido a que la consulta no fue exitosa y por tanto no puedes usarlo para llamar al método num_rows
.
¿Qué falló en la consulta? La consulta tiene al menos un error de sintaxis (una coma antes del FROM
. Pero ese no es el único fallo posible en una consulta. Por eso, cuando manejas este tipo de objetos debes escribir un código que evalúe los objetos y te diga si algo ha fallado y por qué ha fallado. Para eso, en el código que propondré habrá una variable que vaya recogiendo a lo largo del código los posibles errores.
Mysqli tiene sus limitaciones, y una de ellas es lo difícil que se hace recoger un array de datos en consultas preparadas si no tienes el driver nativo llamado mysqlnd
. Por eso aquí se usa un función adicional get_result
que te permitirá como suma facilidad almacenar tus resultados en un array organizado.
Por otra parte, tu código es inseguro. Siempre que intervengan datos dinámicos en la consulta debes usar consultas preparadas, ya que al permitir en la consulta datos que otros usuarios pueden escribir, podrías tener una peligrosa inyección de código. Es un riesgo de seguridad muy alto que se suele ignorar al escribir consultas SQL.
He aquí mi propuesta de código:
$query="SELECT
tb_login.Usuario,
tb_login.Contra,
tb_login.Id
FROM
tb_login
WHERE tb_login.Usuario=?"; //'".$this->Usu_."'"; NUNCA PONGAS DATOS DINÁMICOS DIRECTAMENTE EN UNA CONSULTA
//instancia de las clases
$confi=new Datos_conexion();
$mysql=new mysqli($confi->host(),$confi->usuario(),$confi->pasword(),$confi->DB());
if($mysql){
$stmt=$mysql->prepare($query);
if ($stmt){
$stmt->bind_param("s", $this->Usu_);
$stmt->execute();
$arrResultado=get_result($stmt);
if ($arrResultado){
/*En este bloque se leerían los datos obtenidos*/
print_r($arrResultado);
}else{
$arrMensaje=array("mensaje"=>"No hay datos que cumplan el criterio de la consulta");
}
$stmt->close();
}else{
$arrMensaje=array("mensaje"=>"Hubo un fallo en la consulta ".$mysql->error);
}
/*Cerramos la conexión si no la vamos a usar en otras consultas*/
$mysql->close();
}else{
$arrMensaje=array("mensaje"=>"La conexión es nula");
}
/*Evaluamos si hubo error*/
if ($arrMensaje){
echo $arrMensaje["mensaje"];
}
/*Función auxiliar*/
function get_result( $Statement ) {
$RESULT = array();
$Statement->store_result();
for ( $i = 0; $i < $Statement->num_rows; $i++ ) {
$Metadata = $Statement->result_metadata();
$PARAMS = array();
while ( $Field = $Metadata->fetch_field() ) {
$PARAMS[] = &$RESULT[ $i ][ $Field->name ];
}
call_user_func_array( array( $Statement, 'bind_result' ), $PARAMS );
$Statement->fetch();
}
return $RESULT;
}
$respuesta = $mysql->query($query);
, es posible que$respuesta
seafalse
porque algo ha fallado. Yo cambiaría esto:if($respuesta->num_rows>0){
por esto:if($respuesta){ $row = $respuesta->fetch_row(); ...
ya que si no hay datos en la consulta, o si la consulta falla,$respuesta
seráfalse
.