Předávání factory do vytvořeného objektu

chci se zeptat, zda-li je správné předávat samotnou factory do vytvořeného objektu, odkud těží závislosti

Správné asi to není to správné slovo :-), ale určitě to není doporučené. Možná ti to teď přijde elegantní, jak ušetřit ruční předávání 5 proměnných, ale zkušenost praví, že si tímto patternem v dlouhodobějším horizontu spíš zavaříš (např. tím, že ti vzniká cyklická závislost). My někdy předáváme i 8 závislostí a není to zas takový problém, jednou to napíšeš a je to.

Spíš bych se ale zamyslel nad tím abstract a base mail. Opravdu potřebuješ třídu per e-mail? Nestačí nějaký obecný mail builder, do kterého bys jen vložil šablonu a parametry? Např.:

final class MessageBuilder
{
	public function __construct(
		private LinkGenerator $linkGenerator,
		// ostatní závislosti...
	) {}

	public function build(mixed $data): Message
	{
		// tady vytvoříš komplet zprávu, na vstupu máš data jako template file a parametry

		$message = new Message(); // ...
		$this->templateFactory->create(); // ...
		// ...
		return $message;
	}
}

To bys pak použil buď z komponenty, nebo z nějaké další service takto:

final class Mailer
{
	public function __construct(
		private Mailer $sender,
		private MessageBuilder $messageBuilder,
	) {}

	public function send(mixed $data, bool $dump = false): void
	{
		if ($dump) {
			Debugger::dump($this->message);
			return;
		}

		$this->sender->send($this->messageBuilder->build($data));
	}
}

Editoval dakur (5. 5. 2022 8:18)

Nakonec jsem to vyřešil takto…

class Mailer
{
	public const TEMPLATE_EXT = '.latte';

	public array $config;
	public TemplateFactory $templateFactory;
	public LinkGenerator $linkGenerator;
	public SmtpSender $sender;
	public Request $request;
	public Message $message;
	public Template $template;
	public string $lang;

	public function __construct(
		array $config,
		TemplateFactory $templateFactory,
		LinkGenerator $linkGenerator,
		SmtpSender $sender,
		Request $request
	)
	{
		$this->config = $config;
		$this->templateFactory = $templateFactory;
		$this->linkGenerator = $linkGenerator;
		$this->sender = $sender;
		$this->request = $request;
	}

	private function createTemplate(string $file, string $lang): Template
	{
		$template = $this->templateFactory->createTemplate();
		$template->setTranslator(new Translator($lang));
		$template->getLatte()->addProvider('uiControl', $this->linkGenerator);
		$template->setFile($this->config['templateDir'] . $file . self::TEMPLATE_EXT);
		return $template;
	}

	private function getLang()
	{
		return preg_match('/^\\/([a-z]{2})\\//', $this->request->getUrl()->path, $matches) ? $matches[1] : Translator::DEFAULT_LANG;
	}

	public function create(string $templateFile, string $lang = null): self
	{
		$this->message = new Message;
		$this->message->setFrom($this->config['fromMail']);
		$this->template = $this->createTemplate($templateFile, $lang ?? $this->getLang());
		Debugger::dump($lang);
		return $this;
	}

	public function to(string|array $recipients): self
	{
		if(!is_array($recipients)) {
			$recipients = [$recipients];
		} else {
			if(!count($recipients))
				throw new InvalidArgumentException("Recipient is an empty array.");
		}

		foreach($recipients as $recipient) {
			if(!Validators::isEmail($recipient))
				throw new InvalidArgumentException ("$recipient is not valid email. All emails was not sent.");
		}

		foreach ($recipients as $recipient) {
			$this->message->addTo($recipient);
		}

		return $this;
	}

	public function body(mixed $data): self
	{
		if(is_array($data)) {
			foreach($data as $key => $value) {
				$this->template->$key = $value;
			}
		} else {
			$this->template->data = $data;
		}

		$this->message->setHtmlBody($this->template->renderToString());
		return $this;
	}

	public function send(bool $dump = false): self
	{
		if($dump)
			$this->dump();
		else
			$this->sender->send($this->message);

		return $this;
	}

	public function dump(): self
	{
		Debugger::dump($this->message);
		return $this;
	}
}

a používám to třeba takto

		$this->mailer
			->create('verification')
			->to('test@gmail.com')
			->body($verifyToken)
			->send();

je s tím, co se týče návrhu kódu, něco špatně, nebo co by bylo lepší řešit jinak? Chci se naučit psát kód pokud možno co nejlépe, nejsrozumitelněji, tak jak by ho psal zkušený programátor.

Editoval Michwuanquana (5. 5. 2022 12:32)

Ještě bych podotkl, že mi přijde riskantní hard-codovat do MessageBuilderu pravidlo, jak z url vytahovat jazyk. S tím by měl pracovat pouze router, ten ho předá presenteru a ten ho předá metodě MessageBuilder::create().

Teď ti to funguje, ale časem něco změníš v router a zapomeneš změnit tady, nebo přijde routa, na které to fungovat nebude… Navíc porušení DRY.

Služba UserManager by mohla mít závislost na Nette\Application\Request a z něj dostaneš parametr routy taky.