Skip to main content
2 de 3
se añadieron 24 caracteres en el cuerpo
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

Hay varias cosas que corregir:

  • Declara los miembros como private para encapsular. var es muy antiguo y en versiones modernas de PHP ya levanta un warning, por tanto evita usarlo
  • Agrega una propiedad salario en la clase
  • Usa $this en el cálculo del salario
  • Usa una convención de nombre que sea uniforme
  • Se agregó un método get_alldata que satisface el requerimiento "Actualizar los métodos para que el sueldo del profesor se muestre junto con sus demás datos"
  • Se simplificó la lógica, asignando un salario mínimo de 2000

La clase quedaría así:

Class Profesor{
    #Usamos private para encapsular
    private $nombre;
    private $direccion;
    private $experiencia;
    private $asignatura;
    private $anhos;
    private $salario;

    public function __construct($nombre, $direccion, $experiencia, $asignatura, $anhos){
        $this->nombre = $nombre;
        $this->direccion = $direccion;
        $this->experiencia = $experiencia;
        $this->asignatura = $asignatura;
        $this->anhos = $anhos;
        #Salario mínimo
        $this->salario=2000;
    }
    
    public function get_nombre(){
        return $this->nombre;
    }
    
    public function set_nombre($x){
        $this->nombre = $x;
    }
    
    public function get_direccion(){
        return $this->direccion;
    }
    
    public function set_direccion($x){
        $this->direccion = $x;
    }
    
    public function get_experiencia(){
        return $this->experiencia;
    }
    
    public function set_experiencia($x){
        $this->experiencia = $x;
    }
    
    public function get_asignatura(){
        return $this->asignatura;
    }
    
    public function set_asignatura($x){
        $this->asignatura = $x;
    }
    
    public function get_anhos(){
        return $this->anhos;
    }
    
    public function set_anhos($x){
        $this->anhos = $x;
    }
    
    public function get_salario(){
        #Usamos $this y simplificamos el salario sólo cambia en estas dos condiciones
        if($this->anhos > 10){
            $this->salario = 3000;
        }else if ($this->anhos >= 5 && $this->anhos <= 10){
            $this->salario = 2500;
        }
        return $this->salario;
    }
    
    public function get_alldata(){
        $data="Nombre: $this->nombre\nApellido: $this->apellido\nSalario:$this->get_salario()"; //Agregar los otros datos
        return $data;
    }

}
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257