Nu e neapărat ceva în neregulă, dar cred că are mai mult sens să trimiți o instanță - și astfel să separi responsabilitățile unei clase - decât să o instanțiezi în acea clasă.
Ajută la:
- separarea responsabilităților (o clasă ar trebui să facă un singur lucru, nu? Tu faci ceva și instanțiezi o altă clasă);
- testare (probabil acum nu testezi, dar în cazul în care decizi să o faci, va ajuta);
- decuplarea componentelor;
- în cazul în care decizi că vrei o altă clasă în
$this->_property
va trebui să editezi o clasă care o folosește (în loc de clasa care o instanțiază)
- în cazul în care decizi că vrei o altă clasă în