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;
}
}