Skip to main content
3 de 3
cambio los parentesis por corchetes, ya que de esa manera lo interpreta como una función en vez de como un array

El asunto esta en el foreach:

foreach ($this->getMaterias() as $unaMateria) {
  if($unaMateria==$pMateria){
     $unaMateria=NULL;
     return 'te has dado de baja de '.$pMateria;
  }
}

el objeto $unaMateria es una copia de cada uno de los elementos de getMaterias(). Cuando haces $unaMateria=NULL, realmente no has borrado la materia del array de materias del alumno.

Para poder hacer eso, puedes cambiar el for añadiendo el indice (key) del array:

foreach ($this->misMaterias as $key => $unaMateria) {
  if($unaMateria==$pMateria){
     unset($this->misMaterias[$key]);
     return 'te has dado de baja de '.$pMateria;
  }
}
Jakala
  • 4.1k
  • 1
  • 14
  • 20