Consejo en uso de Classes, por favor.

[LordMOY]

Saludos colegas. Tengo años de programar en PHP pero no tanto de basar mi desarrollos en el uso de Objetos, se me viene encima un proyecto grande y quiero su consejo para ver si esta es la forma correcta de hacerlo.
Voy a simplificarlo con el siguiente ejemplo.

Código PHP:

  class Auto {
  public $TanqueGasolina = new TanqueGasolina();
  public $Motor = new Motor();
 
  // Funciones del Auto
}
 
class TanqueGasolina extends Auto {
  public $Gasolina = 100;
 
  // Funciones del Tanque de gas
}
 
class Motor extends Auto {
  public $Estado = "Apagado";
  
  public function Encender() {
    // AQUI EMPIEZAN MIS DUDAS
  } 
   

Supongamos que para "Encender()" el motor, necesito comprobar que el tanque tenga gasolina, puedo acceder a ese valor con algo como:

Código PHP:

   $this->TanqueGasolina->Gasolina; 
   

(Creo... la verdad es que ya ise tantas pruebas que traigo un nido de ideas en la cabeza)

¿Esta es la forma correcta de hacerlo?
Por que, si hago un print_r() de Auto, las dos sub-classes cargan de nuevo los valores del padre, y no se si eso sea bueno para el rendimiento, tampoco me queda claro que pasa si modifico un valor "padre" desde un "hijo", ¿repercute en todas las sub-classes?

La otra opción es manejar cada cosa como una Clase individual, pero si hago eso, no puedo acceder a los valores de otra si no es pasandolos en cada función. Tendria que ser algo como:

Código PHP:

  $this->Motor->Enceder($this->TanqueGasolina->Gasolina); 
   

El problema de esto, es que cada clase tendra tantas funciones y atributos que dependen unos de otros, que esto lo veo muy complicado a la hora de mantener (o tratar de) "orden" en el código.

Les voy a agradecer infinitamente su ayuda o consejos, si saben de alguna literatura que deba de leer sobre un modelo de código, o algún consejo basado en experiencia... cualquier cosa es bienvenida. No quisiera iniciar el desarrollo y toparme en seco por estructurar mal desde un inicio.

Perdón por la biblia de explicación, pero es lo mas simple que se me ocurre. Saludos y de antemano, gracias.

[alvaro_trewhela]

Primero, no se que tan necesario serían esos extends.
Segundo, en tu clase Auto no se puede instanciar clases de ese modo te arrojará error, la correcta forma sería:

Código PHP:

class Auto {
public $tanque;
public $motor;
    function Auto(){
    $this->tanque = new TanqueGasolina();
    $this->motor = new Motor();
    }
}
 

Tercero, de la forma en que estás trabajando te encajonas tu mismo, y claro no queda otra que pasar por parámetro y ver....

Código PSEUDO:

encender(gas){
    if(gas > 0){
    encender
    }
}
 

Entonces:

Código PHP:

$auto = new Auto();
$auto->motor->encender($auto->tanque->gasolina);
 

dejame darle una vuelta....

Por mientras te dejo mi código:

Código PHP:

<?php
 
class Auto {
public $tanque;
public $motor;
    function Auto(){
    $this->tanque = new TanqueGasolina();
    $this->motor = new Motor();
    }
}
 
class TanqueGasolina{
public $gasolina = 100;
}
 
class Motor{
public $encendido = "Apagado";
 
    public function encender($gas){
        if($gas > 0){
        $this->encendido = "Encendido";
        return "Encendido!!";
        }
    }
}
 
$auto = new Auto();
echo $auto->motor->encender($auto->tanque->gasolina);
 
 
?>
 

Falta encapsulamiento de todas maneras!!! jajaja

Edito, porque lo otro sería usar la funcion encender derechamente en auto:

Código PHP:

class Auto {
public $tanque;
public $motor;
    function Auto(){
    $this->tanque = new TanqueGasolina();
    $this->motor = new Motor();
    }
    
    public function encender(){
        if($this->tanque->gasolina > 0){
        $this->motor->encendido = "Prendido";
        }
    }
}
 

Pero no es la idea cierto?

[LordMOY]

Antes que nada, te agradezco el tiempo y el interes en ayudarme. Tienes razón, usando los extends me encajono solo, y no, no es que los considere necesarios, solo que estube leyendo sobre "subclases", y todo lo que encontre indicaba ese camino. Pero me da igual si son clases normales o con extends.

Realmente, creo que mis problemas se solucionarian si desde la clase "$motor" pudiera hacer algo como:

Código PHP:

  $parent->tanquegasolina->gasolina 
   

Pero hasta donde he leido, eso no es posible, si sabes de alguna forma de hacerlo, me cambiarias la vida.

Si no, no me queda mas que, sentarme, pensar bien como va a funcionar todo el sistema y ni hablar, a pasar variables.

Tengo tiempo, voy a seguir haciendo pruebas un par de dias mas y a darle mas vueltas en la cabeza, si se te ocurre en estos dias, te voy a agradecer mucho me lo compartas.

Nuevante, muchisimas gracias, sin tu aclaracion con los extends probablemente acabaria enredandome mas y mas.

[alvaro_trewhela]

Claro lo que sucede con extends, es que esto haces:



Con las flechitas me refiero a que le entregas a cada cual las propiedades de auto.

Edito, más aun, cuando extiendes y haces estas instancias de las clases queda asì:



No es que si haces esa mezcla entre extender y llamar haga que sea posible la conexión entre las clases

[hhs]

Código PHP:

  class Auto {
  public $TanqueGasolina = new TanqueGasolina();
  public $Motor = new Motor();
 
  // Funciones del Auto
} 
   

El primer problema que tienes es que la clase carro crea las instancias de tanque y motor esto genera un poco de acoplamiento para ciertos casos.

Código PHP:

  class TanqueGasolina extends Auto {
  public $Gasolina = 100;
 
  // Funciones del Tanque de gas
}
 
class Motor extends Auto {
  public $Estado = "Apagado";
  
  public function Encender() {
    // AQUI EMPIEZAN MIS DUDAS
  } 
   

El problema en esta parte es que estas intentando usar la herencia como forma de reutilización en clases que no tiene nada en común, a esto se le conoce como Refused bequest para resolver esto puedes usar composición o delegar la responsabilidad en otras clases. Por otro lado, algo que pasaste por alto es que estas explicando el problema que tienes pero no lo que realmente quieres hacer, pero con la información que proporcionaste y viendo el código puedes hacer algo como lo siguiente.

Código PHP:

class Tank {
    
    private $level = 0;
    
    public function level() {
        return $this->level;
    }
    
    public function fill($quantity) {
        $this->level = $quantity;
    }
}
 
class Engine {
    
    private $state = 'off';
    
    private $tanke = null;
    
    public function __construct(Tank $tank) {
        $this->tank = $tank;
    }
    
    public function turnOn() {
        
        if ($this->tank->level()) {
            $this->state = 'on';
        } else {
            $this->state = 'off';
        }
        
        return $this->state;
    }
    
    public function turnOff() {
        $this->state = 'off';
    }
    
}
 
class Car {
    
    private $tank;
    
    private $engine;
    
    public function __construct(Tank $tank, Engine $engine) {
        $this->tank = $tank;
        $this->engine = $engine;
    }
    
    public function turnOn() {
        return $this->engine->turnOn();
    }
    
    public function turnOff() {
        return $this->engine->turnOff();
    }
    
    public function fillGas($quantity) {
        $this->tank->fill($quantity);
    }
    
}
//ejemplo
$tank = new Tank();
$tank->fill(100);
$engine = new Engine($tank);
$car = new Car($tank, $engine);
var_dump($car->turnOn());
$car->turnOff();
$car->fillGas(0);
var_dump($car->turnOn());
 

Este ejemplo tiene mucho que mejorar pero la idea de utiliza delegación te simplifica el problema y obtienes un api mas sencilla de usar ademas cada objeto ya tiene su propia responsabilidad.
Si estas comenzando con la POO te aconsejo que leas sobre los principios GRASP para que aprendas como signar responsabilidades entre clases, un buen libro que habla sobre el tema es el de craig larman

[LordMOY]

Hola hhs. Antes que nada, al igual que a alvaro_trewhela, agradecer tu ayuda y tiempo.
Ya leí sobre los principios GRASP, por lo pronto de la Wikipedia, por que desconocía el termino (es lo que pasa cuando aprendes a programar sin universidad), voy a buscar el libro que mencionas para comprarlo en la primera oportunidad.

La idea final es/era tener algo como:

Código PHP:

  $Auto1 = new Auto();
$Auto2 = new Auto();
.... 
   

Y que cada "Auto" tenga todos sus componentes, "Tanque, Motor, etc.."
Probablemente lo pudiese resolver con solo una clase, la clase Auto, y ahi toooodas las funciones y atributos.
Pensé en utilizar varias clases para tener "orden" pero veo que lo plantee mal desde un inicio y de ahi todos los problemas. Pero sus codigos y consejos ya me dejan mas en claro cual es la forma correcta de hacerlo.

Una ultima pregunta, veo que en los constructores de las classes utilizas algo como:

Código PHP:

   __constructor(Tank $tank) 
   

Nunca lo habia usado en PHP, es igual que en C? para forzar que la variable enviada sea una instancia de Tank o tiene otra funcion?

[hhs]

Iniciado por LordMOY

Hola hhs. Antes que nada, al igual que a alvaro_trewhela, agradecer tu ayuda y tiempo.
Ya leí sobre los principios GRASP, por lo pronto de la Wikipedia, por que desconocía el termino (es lo que pasa cuando aprendes a programar sin universidad), voy a buscar el libro que mencionas para comprarlo en la primera oportunidad.

En realidad la universidad solo te da algunas bases la mayor parte lo aprendes en el camino, así que no te preocupes si no tienes estudios formales.

La idea final es/era tener algo como:

Código PHP:

  $Auto1 = new Auto();
$Auto2 = new Auto();
.... 
   

Y que cada "Auto" tenga todos sus componentes, "Tanque, Motor, etc.."
Probablemente lo pudiese resolver con solo una clase, la clase Auto, y ahi toooodas las funciones y atributos.
Pensé en utilizar varias clases para tener "orden" pero veo que lo plantee mal desde un inicio y de ahi todos los problemas. Pero sus codigos y consejos ya me dejan mas en claro cual es la forma correcta de hacerlo.

Lo facil seria hacer una sola clase como comentas, pero en realidad debes de crear un modelo de objetos que resuelva el problema que tienes, si vas a tener autos con diferentes motores , tipo de tanques, etc. entonces tienes que buscar la forma de que tus objetos puedan hacer eso y es probable que no lo resuelvas con una sola clase. Al punto al que voy es que tu tienes que resolver el problema de acuerdo a los requisitos de tu proyecto si en esos requisitos una sola clase resuelve el problema que bien por tí, pero si no es así, entonces vas a tener que crear todo lo que se requiera para resolver esos requisitos.

Una ultima pregunta, veo que en los constructores de las classes utilizas algo como:

Código PHP:

   __constructor(Tank $tank) 
   

Nunca lo habia usado en PHP, es igual que en C? para forzar que la variable enviada sea una instancia de Tank o tiene otra funcion?

Si, es para forzar que los métodos o funciones solo acepten ciertos tipos en sus argumento. Puedes ver mas en el manual: http://php.net/manual/es/functions.a...pe-declaration

[jhonnyalexander_91]

Mi sugerencia es que use un "Framework" actualizado para realizar su desarrollo entendiendo primero las recomendaciones y buenas prácticas del mismo, este le guiará para que desarrolle aplicaciones robustas sin que tenga muchos problemas de seguridad y a la vez pueda tener un desarrollo escalable.

Le sugiero también que use el método de desarrollo orientado a pruebas y que entienda el concepto de Inyección de Dependencias.