PHP OO ¿ Estaría el código bien orientado a objetos ?

[rodrypaladin]

Buenas compis!!, ando intentando cambiar la forma de programar de estilo por procedimientos a estilo orientado a objetos, y me preguntaba si este sencillo ejemplo que he hecho estaría bien planteado y que cumple con todos los requisitos para que esté orientado a objetos correctamente

Código PHP:

<?php 
 
class Empleado {
    private $nombre;
    private $apellidos;
    private $dni;
    private $sueldoAnual;
 
    public function Empleado($nombre, $apellidos, $dni, $sueldoAnual) {
        $this->setNombre($nombre);
        $this->setApellidos($apellidos);
        $this->setDni($dni);
        $this->setSueldoAnual($sueldoAnual);
    }
 
    public function ascender($porcentaje) {
        $nuevoSueldo = $this->getSueldoAnual() + $this->porcentaje($porcentaje);
        $this->setSueldoAnual($nuevoSueldo);
    }
 
    public function descender($porcentaje) {
        $nuevoSueldo = $this->getSueldoAnual() - $this->porcentaje($porcentaje);
        $this->setSueldoAnual($nuevoSueldo);
    }
 
    public function porcentaje($porcentaje) {
        return $this->getSueldoAnual() * $porcentaje / 100;
    }
 
    public function getPrimerApellido() {
        $porcion = explode(" ", $this->getApellidos());
        return $porcion[0];
    }
 
    public function getSegundoApellido() {
        $porcion = explode(" ", $this->getApellidos());
        return $porcion[1];
    }
 
    public function getNombre() {
        return $this->nombre;
    }
 
    public function getApellidos() {
        return $this->apellidos;
    }
 
    public function getDni() {
        return $this->dni;
    }
 
    public function getSueldoAnual() {
        return $this->sueldoAnual;
    }
 
    public function setNombre($nombre) {
        $this->nombre = $nombre;
    }
 
    public function setApellidos($apellidos) {
        $this->apellidos = $apellidos;
    }
 
    public function setDni($dni) {
        $this->dni = $dni;
    }
 
    public function setSueldoAnual($sueldoAnual) {
        $this->sueldoAnual = $sueldoAnual;
    }
 
}
 
 ?>
 

Un saludo!!

[hhs]

Me puedes explicar cual es el propósito de tu clase y que es lo que hace ?

[rodrypaladin]

Guardar los datos de un empleado y su sueldo. Solo quiero saber si está bien orientado a objetos el código. No es una clase que vaya a usar para nada, es solo un ejercicio práctico

[marlanga]

En cuanto a tu clase, el método porcentaje tiene pinta de ser privado, y no público.

Como mejora, convierte el salario en una nueva clase Salario, que contenga esos métodos para incrementarla o decrementarla, y deja limpia tu clase Empleado limpia de métodos que solo sirven para el salario.

Clases pequeñas, y que cada clase resuelva sus mierdas.

[rodrypaladin]

Eso es loq ue estaba mirando para aprender ahora. Estoy creando la clase Sueldo, pero lo que no se es como comunicarlas, que es lo que me has recomendado que haga, ¿No?

[marlanga]

Código PHP:

<?php 
 
class Empleado {
    private $nombre;
    private $apellidos;
    private $dni;
    private $sueldoAnual;
 
    public function __construct($nombre, $apellidos, $dni, Salario $sueldoAnual) {
        $this->setNombre($nombre);
        $this->setApellidos($apellidos);
        $this->setDni($dni);
        $this->setSueldoAnual($sueldoAnual);
    }
 
    public function getPrimerApellido() {
        $porcion = explode(" ", $this->getApellidos());
        return $porcion[0];
    }
 
    public function getSegundoApellido() {
        $porcion = explode(" ", $this->getApellidos());
        return $porcion[1];
    }
 
    public function getNombre() {
        return $this->nombre;
    }
 
    public function getApellidos() {
        return $this->apellidos;
    }
 
    public function getDni() {
        return $this->dni;
    }
 
    public function getSueldoAnual() {
        return $this->sueldoAnual;
    }
 
    public function setNombre($nombre) {
        $this->nombre = $nombre;
    }
 
    public function setApellidos($apellidos) {
        $this->apellidos = $apellidos;
    }
 
    public function setDni($dni) {
        $this->dni = $dni;
    }
 
    public function setSueldoAnual(Salario $sueldoAnual) {
        $this->sueldoAnual = $sueldoAnual;
    }
 
}
 
class Salario {
 
    private $cantidad;
 
    public function __construct($cantidad) {
        $this->setCantidad($cantidad);
    }
 
    public function getCantidad() {
        return $this->cantidad;
    }
 
    public function setCantidad($cantidad) {
        $this->cantidad = $cantidad;
    }
 
    public function modificar($porcentaje) {
        $nuevaCantidad= $this->getCantidad() + $this->porcentaje($porcentaje);
        $this->setCantidad($nuevaCantidad);
    }
 
    public function porcentaje($porcentaje) {
        return $this->getSueldoAnual() * $porcentaje / 100;
    }
 
}
 
 ?>
 

Todo esto es muy bonito como introducción a la programación orientada a objetos, pero programar aplicaciones tochas es muchísmo más complejo.
KISS, DRY, principios SOLID, patrones de diseño, arquitectura de software, código limpio, refactorización...

[rodrypaladin]

Gracias por el ejemplo. Claro, esto es introductorio, supongo que hay mucho más que espero ver poco a poco.

En el ejemplo cuando pasas por parámetro: Salario $salarioAnual estás creando un nuevo objeto Salario llamado SalarioAnual de la misma manera que se hace con $salarioAnual = new Salario() ??

[hhs]

No, en realidad esta indicando que ese método puede recibir como argumento una instancia de Salario tendrías que crear Salario y pasarlo como argumento

[rodrypaladin]

De acuerdo, entonces supongo que en el constructor en el momento que le asigno el sueldo, es ahí donde tendría que crear la instancia de la clase Salario. rectificarme si me equivoco ;)

[hhs]

Claro que es posible hacerlo de esa forma, todo depende de lo que requieras

[hhs]

Tenia un poco de tiempo asì que hice un ejemplo a partir de tu clase original, la cual lo único que hace es guardar la información que querías y cree dos clases adicionales una que solo se encarga de calcular el sueldo a partir de un porcentaje como lo tenìas pensado y otra que se encarga de proporcionar el sueldo que requiere el Empleado cuando es contratado o promovido a un puesto, es algo sencillo que se puede refactorizar pero espero que te ayude.

Código PHP:

class Empleado
{
    private $nombre;
    private $apellidos;
    private $dni;
    private $sueldoAnual;
 
    private function __construct($nombre, $apellidos, $dni, $sueldoAnual) {
        $this->setNombre($nombre);
        $this->setApellidos($apellidos);
        $this->setDni($dni);
        $this->setSueldoAnual($sueldoAnual);
    }
 
    public static function contratar($nombre, $apellidos, $dni, $sueldoAnual)
    {
        return new static($nombre, $apellidos, $dni, $sueldoAnual);
    }
 
    public function promover($sueldo)
    {
        $this->setSueldoAnual($sueldo);
 
        return $this;
    }
 
    public function getPrimerApellido() {
        $porcion = explode(" ", $this->getApellidos());
        return $porcion[0];
    }
 
    public function getSegundoApellido() {
        $porcion = explode(" ", $this->getApellidos());
        if (!isset($porcion[1])) {
            return null;
        }
        return $porcion[1];
    }
 
    public function getNombre() {
        return $this->nombre;
    }
 
    public function getApellidos() {
        return $this->apellidos;
    }
 
    public function getDni() {
        return $this->dni;
    }
 
    public function getSueldoAnual() {
        return $this->sueldoAnual;
    }
 
    public function setNombre($nombre) {
        $this->nombre = $nombre;
    }
 
    public function setApellidos($apellidos) {
        $this->apellidos = $apellidos;
    }
 
    public function setDni($dni) {
        $this->dni = $dni;
    }
 
    private function setSueldoAnual($sueldoAnual) {
        $this->sueldoAnual = $sueldoAnual;
    }
}
 

Código PHP:

class Tabulador
{
 
    private $sueldoAnual;
 
    public function __construct($sueldoAnual)
    {
        $this->sueldoAnual = $sueldoAnual;
    }
 
    public function ascender($porcentaje = 0) {
        $nuevoSueldo = $this->sueldoAnual + $this->porcentaje($porcentaje);
        return $nuevoSueldo;
    }
 
    private function porcentaje($porcentaje) {
        return $this->sueldoAnual * $porcentaje / 100;
    }
}
 

Código PHP:

class RecursosHumanos
{
    private $tabulador;
 
    /**
     * @var Empleado
     */
    private $empleado;
 
    private $sueldo = null;
 
    public function __construct(Tabulador $tabulador)
    {
        $this->tabulador = $tabulador;
    }
 
    public function contratar(Array $persona = [])
    {
        $this->empleado = $persona;
 
        return $this;
    }
    public function promover(Empleado $empleado)
    {
        $this->empleado = $empleado;
 
        return $this;
    }
 
    public function para($puesto)
    {
        switch($puesto){
            case 'Programador':
                    $this->sueldo = $this->tabulador->ascender();
                    break;
            case 'Lider':
                    $this->sueldo = $this->tabulador->ascender(10);
                    break;
        }
 
        if (is_array($this->empleado)) {
            array_push($this->empleado, $this->sueldo);
            return Empleado::contratar(...$this->empleado);
        } else {
            return $this->empleado->promover($this->sueldo);
        }
    }
}
 

Ejempo de uso

Código PHP:

$sueldo_base = 40000.00;
 
        $persona = [
            'Juan Carlos',
            'Rodriguez',
            123456,
        ];
 
        $tabulador = new Tabulador($sueldo_base);
        $rh =  new RecursosHumanos($tabulador);
 
        $empleado = $rh->contratar($persona)->para('Programador');
 
        print_r($empleado->getSueldoAnual());
 
        $empleado = $rh->promover($empleado)->para('Lider');
 
         print_r($empleado->getSueldoAnual());