Skip to main content
2 de 3
mejora del formato del código
DevMind
  • 5.6k
  • 11
  • 34
  • 65

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