[SOLUCIONADO] Tengo un problema con el listado

[ilu7ionHD]

Hola, si uso el siguiente código funciona a la perfección, pero quiero hacerlo mas prolijo y depender de 2 clases mas "lógica" y "categorias"

Código PHP:

$logica = new Logica();
$db = new Conexion();
$consulta="SELECT id, nombre, descripcion FROM categorias";
$res = $db->query($consulta);
//$listaCategorias = $logica->listarCategorias();
 
        echo "<table>";
                //header
                echo "<tr><td>ID</td>";
                echo "<td>Nombre</td>";
                echo "<td>Descripción</td></tr>";
                    //data  
                     while ($row = $db->recorrer($res))  {
                      echo "<tr><td>{$row['id']}</td>";
                      echo "<td>{$row['nombre']}</td>";
                      echo "<td>{$row['descripcion']}</td></tr>";
                    } 
 
        echo "</table>";
 

Me idea sería llamar a "listarCategorias()" que se encuentra en logica (solo un intermediario) y que a su vez esta llame a listar() que esta en "Categorias" que se encarga de hacer la consulta en la BD. Pero me sale error si lo hago de esta forma que quiero solo funciona de la forma que copie el código.
El error de que sale es:
Warning: mysqli_fetch_array() expects parameter 1 to be mysqli_result, object given in C:\blablablablalbla\SQL.php on line 17
ID Nombre Descripción

[pateketrueke]

Sin conocer cómo es el código exacto de tus clases es complicado entender el problema.

[ilu7ionHD]

Código PHP:

public function recorrer($x) 
      {
        return mysqli_fetch_array($x); 
      }
      
  public function rows($y)
      {
        return mysqli_num_rows($y);
      }
 
 
public function Listar()
    {
      $db = new SQL();
      $sentencia = "SELECT * FROM categorias;";
      $sql = $db->query($sentencia);
      $arrayAso = $db->recorrer($sql);
      return $arrayAso;
    }
 
 
class Logica
   {
    public function listarCategorias()
        {
          require 'Categoria.php';
          $categorias = new Categoria(null, null);
          $categorias->Listar(); 
          return $categorias;
        }
   }
 
require 'Logica.php';
require 'SQL.php';
$logica = new Logica();
$db = new SQL();
$listaCategorias = $logica->listarCategorias();
if(count($listaCategorias)>0)
    {
        echo "<table>";
                //header
                echo "<tr><td>ID</td>";
                echo "<td>Nombre</td>";
                echo "<td>Descripción</td></tr>";
                    //data  
                     while ($row = $db->recorrer($listaCategorias))  {
                      echo "<tr><td>{$row['id']}</td>";
                      echo "<td>{$row['nombre']}</td>";
                      echo "<td>{$row['descripcion']}</td></tr>";
                    } 
 
        echo "</table>";
    }
    
    else
        {
          echo 'No hay categorias';
 
        }
 

[pateketrueke]

Se supone que el método Listar() devuelve algo pero no lo usas, devuelves la misma instancia en lugar del valor de retorno.

Código PHP:

// MAL
$categorias->Listar(); 
return $categorias;
 
// BIEN
return $categorias->Listar();
 

[ilu7ionHD]

Hasta el día de hoy pensaba que era exactamente lo mismo ponerlo en una linea que en 2.

Gracias, voy a probar! luego te comento

[pateketrueke]

Iniciado por ilu7ionHD

Hasta el día de hoy pensaba que era exactamente lo mismo ponerlo en una linea que en 2.

Gracias, voy a probar! luego te comento

No, no es lo mismo.

El punto es que Listar() devuelve un array, y tu instancia $categorias es un objeto completamente diferente.

Poner las dos sentencias no arregla nada, sencillamente devuelves lo que va después del return, eso no implica que el resultado de Listar() se regrese por arte de magia.

Originalmente devuelves $categorias pero eso no tiene sentido: porque debes devolver el array que Listar() te entrega.

¿Se entiende mejor?