[SOLUCIONADO] PHP OO usar metodo estatico o no?

[xSkArx]

Hola a todos, estoy programando una clase y tengo un metodo que obtiene todos los registros de una tabla, ya que siempre va a ser igual, me conviene dejarlo como.estatico o no? pongo codigo

dataBaseConnection.php

Código PHP:

abstract class dataBaseConnection{
/*
*propiedades
*/
protected function open_connection(){
    
          $this->pdo= new PDO('mysql:host='.self::$host.';dbname='.$this->dbname , self::$user, self::$pass);
          
    }
    
protected function close_connection(){
                  $this->pdo=null;
    }
}
 

newsModel.php

Código PHP:

require_once 'dataBaseConnection.php';
class newsModel extends datBaseConnection{
 
/*
*propiedades
*/
 
function __construct(){
            $this->dbname="database";
    }
          
    public function getAllNews(){
           $this->query= "SELECT * FROM noticias ORDER BY id DESC";
           $this->open_connection();
           $this->rows=$this->pdo->query($this->query)->fetchAll();
           return $this->rows;
    }
}
 

me podrian aconsejar, si lo dejo como lo tengo o como estatico, gracias.

[pateketrueke]

Estás extendiendo e instanciando, si dicho método hace uso de variables de instancia propias es obvio que no puedes dejar dicho método estático.

Eso de "ya que siempre va a ser igual" no es un argumento muy inteligente que digamos.

[xSkArx]

Iniciado por pateketrueke

Estás extendiendo e instanciando, si dicho método hace uso de variables de instancia propias es obvio que no puedes dejar dicho método estático.

Eso de "ya que siempre va a ser igual" no es un argumento muy inteligente que digamos.

Gracias, ese "argumento" lo di porque recuerdo haber.leido aqui en el.foro que si un metodo no cambia y no recibe parametros se "podria" dejar como metodo estatico, me imagino que habre leido mal o no lo entendi bien, pero gracias, alguna otra recomendacion?

[pateketrueke]

alguna otra recomendacion?

Podrías consultar una referencia de diseño OOP, supongo que ahí se deben explicar dichos detalles.

[xSkArx]

Gracias, solucionado.

[dashtrash]

Que un método "siempre vaya a ser igual", o que use o no variables miembro de una clase, es una consecuencia de que el método sea estático, no la causa.
Un método es estático cuando es un método de clase, no de instancia.
En este caso:
Puedo pedirle a una instancia de newsModel que me dé todas las news que existen en la BD? Es algo que se le pide a una *instancia*, a una cierta noticia?Una noticia puede ser publicada, comentada, desactivada, etc..Pero, que devuelva instancias (o datos de instancias) que no son ella misma?
Puedo pedirle a la clase newsModel que me dé todas las news que existen en la BD? Esto tiene mejor pinta.La clase newsModel no representa a ninguna "news" en particular, y puede hacerse cargo de estas cosas, a falta de una factoría.

Por eso, es muy raro que un método del tipo "getAll-----" necesite variables de instancia.Así he visto que estás derivando modelos de la clase databaseConnection.Aqui hay un error y un comentario:
- El error: la conexión a la bd es una variable miembro *de instancia*.Esto significa que cada instancia, de cada modelo, está abriendo su conexión (!!)..Seguro que no quieres eso.

- El comentario: sé que es una práctica muuuy habitual en ORMs de todo tipo, pero derivar modelos de la clase de DB me parece muy discutible.
Desde el punto de vista de la OOP, es una derivación muy forzada.No tengo tan claro que un objeto News *sea* una conexión a una base de datos (derivación).Ni siquiera que un objeto News *tenga* una conexión a una base de datos (composición).Veo más limpio que un objeto base de datos sepa persistir objetos que deriven de Model.Es decir, el método loadModel() y saveModel() pertenece al objeto "BD", y no a Model (repito, es *mi* punto de vista, sé que en mucha literatura respetable Model deriva de BD).

Y lo que esto significa, es que un objeto "News" es independiente de cómo se *guarda* o se obtiene el objeto.Supón que tuvieras que crear objetos News a partir de un XML, o un json.O, más habitual, que tengas una conexión de lectura (esclavo sql), y otras de escritura (maestro sql). O modelos almacenados en distintas bases de datos.

Esto, en el fondo es así a nivel conceptual.Un modelo es un objeto que pertenece al modelo de negocio de la aplicación.Una tabla de la BD es dónde se almacena, aunque también se almacena, incluso al mismo tiempo, en otros sitios, como memcache, una sesión, una variable..Y no por eso las clases Model derivan de Memcache o Session.
A nivel práctico, es muy sencillo de implementar a nivel de modelo.Pero ser independiente del sistema de almacenamiento para operaciones más allá de manipular un simple modelo, la cosa se complica.

[xSkArx]

Iniciado por dashtrash

Que un método "siempre vaya a ser igual", o que use o no variables miembro de una clase, es una consecuencia de que el método sea estático, no la causa.
Un método es estático cuando es un método de clase, no de instancia.
En este caso:
Puedo pedirle a una instancia de newsModel que me dé todas las news que existen en la BD? Es algo que se le pide a una *instancia*, a una cierta noticia?Una noticia puede ser publicada, comentada, desactivada, etc..Pero, que devuelva instancias (o datos de instancias) que no son ella misma?
Puedo pedirle a la clase newsModel que me dé todas las news que existen en la BD? Esto tiene mejor pinta.La clase newsModel no representa a ninguna "news" en particular, y puede hacerse cargo de estas cosas, a falta de una factoría.

Por eso, es muy raro que un método del tipo "getAll-----" necesite variables de instancia.Así he visto que estás derivando modelos de la clase databaseConnection.Aqui hay un error y un comentario:
- El error: la conexión a la bd es una variable miembro *de instancia*.Esto significa que cada instancia, de cada modelo, está abriendo su conexión (!!)..Seguro que no quieres eso.

- El comentario: sé que es una práctica muuuy habitual en ORMs de todo tipo, pero derivar modelos de la clase de DB me parece muy discutible.
Desde el punto de vista de la OOP, es una derivación muy forzada.No tengo tan claro que un objeto News *sea* una conexión a una base de datos (derivación).Ni siquiera que un objeto News *tenga* una conexión a una base de datos (composición).Veo más limpio que un objeto base de datos sepa persistir objetos que deriven de Model.Es decir, el método loadModel() y saveModel() pertenece al objeto "BD", y no a Model (repito, es *mi* punto de vista, sé que en mucha literatura respetable Model deriva de BD).

Y lo que esto significa, es que un objeto "News" es independiente de cómo se *guarda* o se obtiene el objeto.Supón que tuvieras que crear objetos News a partir de un XML, o un json.O, más habitual, que tengas una conexión de lectura (esclavo sql), y otras de escritura (maestro sql). O modelos almacenados en distintas bases de datos.

Esto, en el fondo es así a nivel conceptual.Un modelo es un objeto que pertenece al modelo de negocio de la aplicación.Una tabla de la BD es dónde se almacena, aunque también se almacena, incluso al mismo tiempo, en otros sitios, como memcache, una sesión, una variable..Y no por eso las clases Model derivan de Memcache o Session.
A nivel práctico, es muy sencillo de implementar a nivel de modelo.Pero ser independiente del sistema de almacenamiento para operaciones más allá de manipular un simple modelo, la cosa se complica.

Creo entender lo que dices (a lo mejor no), la clase para obtener las news, no debe extender la clase bd, si.no, instanciarla?
Pretendo hacer una clase para obtener las news y otra para guardarlas, esta mal asi? Todo lo hago guiandome de un libro que encontre.

[dashtrash]

Iniciado por xSkArx

Creo entender lo que dices (a lo mejor no), la clase para obtener las news, no debe extender la clase bd, si.no, instanciarla?
Pretendo hacer una clase para obtener las news y otra para guardarlas, esta mal asi? Todo lo hago guiandome de un libro que encontre.

Instanciarla, nunca, ya que entonces crearías 1 conexión por cada objeto que crees.Eso es lo que pasa ahora con el código.
Hay 3 alternativas: lo que te está diciendo que hagas el libro que encontraste (y, si no, debería decirtelo), es lo siguiente, en la clase base:

Código PHP:

<?php
    class databaseConnection
    {
            static $connection=null;
            private __construct(){}
            function getConnection()
            {
                    if(databaseConnection::$connection==null){
                        /* abrir la conexion y asignarla a $connection
                    }
                  return databaseConnection::$connection;
            }
 
     }
?>
 

Basicamente, un singleton.En la clase derivada, debes llamar a getConnection() para obtener la conexión, y no almacenarla en una variable de instancia (no estática).
Como singleton que es, no puedes llamar desde la derivada al constructor de la base, lo cual puede complicar algo las cosas.

A partir de ahi, si quieres derivar esta clase, instanciarla, o pasarla como parámetro, al menos, no estarás creando una conexión por objeto.