Volanie funkcie z ineho repository

No můžeš si tam to repository samozřejmě normálně injectnout, ale otázka zní, jestli je to vůbec dobře.

Ty vlastně chceš docílit toho, aby CategoriesRepository fušovalo do práce PhotosRepository a osobně si nemyslím, že je to příliš dobré. Problém totiž je, že ve chvíli, kdy to uděláš tak jak teď chceš, už daná metoda CategoriesRepository::delete() nedělá to, co by se od ní na první pohled očekávalo.

Pokud někde zavolám:

$categoriesRepository->delete($category);

tak předpokládám, že to udělá jen to o co jsem opravdu požádal, tedy smaže se kategorie. Jenže ono to tak není, protože repository „zcela bez mého vědomí“ začne mazat i fotky z dané kategorie, což navenek nikde nemám šanci poznat. Vznikne tím i další „problém“, že ne vždy musíš chtít dané fotky opravdu smazat. Například chceš jen smazat kategorii a kategorii u fotek nastavit na NULL nebo chceš udělat merge kategorií.

Pro takovéhle věci bych si spíše udělal fasádu a až tam to řešil, protože je pak jasně vidět, co se opravdu stane a eliminuje to WTF faktor.

class PhotosService {

	private $categoriesRepository;

	private $photosRepository;

	public function __construct(CategoriesRepository $categoriesRepository, PhotosRepository $photosRepository){
		list($this->categoriesRepository, $this->photosRepository) = func_get_args();
	}

	public function deleteCategory($category){
		$this->photosRepository->mergeCategories($category, NULL);
		$this->categoriesRepository->delete($category);
	}

	public function deleteCategoryAndPhotos($category){
		$this->photosRepository->deleteByCategory($category);
		$this->categoriesRepository->delete($category);
	}
}

Pokud teď zavolám:

$photosService->deleteCategoryAndPhotos($category);
// nebo
$photosService->deleteCategory($category);

Tak to udělá přesně to co by člověk očekával. První prostě smaže kategorii včetně fotek. Druhé nastaví u fotek z dané kategorie kategorii na NULL a následně kategorii smaže. Je ovšem hned na první pohled vidět, co se bude v kterém případě opravdu dít a není to schované pod „nicneříkajícím“ $categoriesRepository->delete($category);.

Editoval Etch (10. 2. 2014 3:35)

Etch napsal(a):

.....

class PhotosService {

	private $categoriesRepository;

	private $photosRepository;

	public function __construct(CategoriesRepository $categoriesRepository, PhotosRepository $photosRepository){
		list($this->categoriesRepository, $this->photosRepository) = func_get_args();
	}
    .....

Tak to udělá přesně to co by člověk očekával. První prostě smaže kategorii včetně fotek. Druhé nastaví u fotek z dané kategorie kategorii na NULL a následně kategorii smaže. Je ovšem hned na první pohled vidět, co se bude v kterém případě opravdu dít a není to schované pod „nicneříkajícím“ $categoriesRepository->delete($category);.

Proč toto? To mě nepřijde jako moc přehlednej (a čistej) zápis. Vypsat to po jednom nezabere tolik času.

list($this->categoriesRepository, $this->photosRepository) = func_get_args();