[PHP]OOP na życiowym przykładzie? Opcje

[alex19]

Nadszedł najwyższy czas żeby nauczyć się programowania obiektowego. Przeczytałem wiele tutoriali w sieci, ale to niestety dla mnie za mało. Wszędzie piszą co to klasa i tak dalej, ale nigdzie nie znalazłem wykorzystania w praktyce. Nie ma co ukrywać, że na jednej klasie żadnej projekt inny niż "hello world" nie powstanie.

Czy ktoś może polecić jakiś dobry tutorial gdzie były by jakieś zaaansowane przykłady programowania obiektowego?

Czego szukam? Na przykład sposobu na rozwiązanie takiego problemu:

Dajmy na to że mamy 3 klasy. Jedną swoją i dwie gotowe.

Klasa gotowa to np mPDF, która dla uproszczenia będzie wyglądała tak:

[PHP] pobierz, plaintext 
class mPDF{
	function WriteHTML(){
 
	}
	function Output(){
		/*jaks magia */
	}
}
[PHP] pobierz, plaintext 

Kolejną gotową klasą będzie PHPmailer, który w uproszczeniu będzie taki:

[PHP] pobierz, plaintext 
class PHPmailer{
	public $Host;
	public $SMTPAuth;
	public $Username;
	public $Password;
	public $SMTPSecure;
	public $From;
	public $FromName;
	public $addAddress;
	public $addAddress;
	public $addReplyTo;
	public $WordWrap;
	public $Subject;
	public $Body;
 
	function addAttachment($att){
		/* cos sie dzieje */
	}
 
	function isSMTP(){
	}
 
	function isHTML(true){
	}
 
	function send() {
	/*wysylka */
	}
}
[PHP] pobierz, plaintext 

Teraz mamy sobie naszą własną klasę, która powiedzmy odpowiada za stworzenie faktury. Coś się dzieje i w efekcie otrzymamy HTML.

[PHP] pobierz, plaintext 
class faktura{
	function generujDokument(){
		/* nie ważne co i jak, ważny efekt ;) */
		return $html;
	}
}
[PHP] pobierz, plaintext 

Co bym chciał zrobić? Chciałbym wygenerować html z fakturą, następnie zrobić z tego pdf'a i wysłać go mailem do klienta. Pojawia się pytanie jak?

Oczywiście możemy potworzyć po koleji obiekty i "proceduralnie" wysłać naszą fkaturę, ale przecież nie o to chodzi.

[PHP] pobierz, plaintext 
$fak = new faktura;
$html = $fak->generujDokument();
 
$mpdf = new mPDF;
$mpdf->WriteHTML($html);
$pdf_file = 'filename.pdf';
$mpdf->Output($pdf_file, 'F');
 
$mail = new PHPMailer;
$mail->isSMTP();
$mail->Host = 'smtp1.example.com;smtp2.example.com'; 
$mail->SMTPAuth = true;
$mail->Username = 'user@example.com';
$mail->Password = 'secret'; 
$mail->SMTPSecure = 'tls';
$mail->From = 'from@example.com';
$mail->FromName = 'Mailer';
$mail->addAddress('joe@example.net', 'Joe User');
$mail->WordWrap = 50;
$mail->addAttachment($file_name);
$mail->isHTML(true);
$mail->Subject = 'Here is the subject';
$mail->Body    = 'This is the HTML message body <b>in bold!</b>';
$mail->AltBody = 'This is the body in plain text for non-HTML mail clients';
$mail->send();
[PHP] pobierz, plaintext 

To nie jest ładne rozwiązanie i najlepiej było by dodać do naszej klasy metodę

[PHP] pobierz, plaintext 
function wyslijFakture(){
  /*pdf*/
  /*wyslanie maila*/
}
[PHP] pobierz, plaintext 

Tylko pojawia się pytanie jak to zrobić?

Ten post edytował alex19 9.08.2014, 07:09:41

[golabow]

Wykorzystaj kompozycje i dziedziczenie.
Mogłbyś też wykorzystać wzorzec projektowy "metoda szablonowa". Dzięki temu możesz zmieniać kroki twojego i robić różne "szablony" zależności od potrzeb.
W kodzie wyglądało by to tak:

[PHP] pobierz, plaintext 
<?php
	abstract class Template {
		protected $mailWorker;
		protected $pdfWorker;
		protected $fakturaWorker;
 
		public function prepareWokers() {
			$this->mailWorker = new PHPmailer;
			$this->pdfWorker = new mPDF;
			$this->fakturaWorker = new faktura;
		}
 
		public function template() {
			$html = $this->generateFaktura();
			$file = $this->generatePDF( $html );
			$this->sendMail();
		}
 
		abstract public function generateFaktura();
		abstract public function generatePDF( $html );
		abstract public function sendMail( $file );
	}
 
	class Worker extends Template {
 
		public function generateFaktura() {
			// tutaj kod 
		}
 
		public function generatePDF( $html ) {
			// tutaj kod
		}
 
		public function sendMail( $file ) {
			// tutaj kod
		}
 
	}
 
	$woker = new Worker();
	$worker->prepareWokers();
	$worker->template(); 
?>
[PHP] pobierz, plaintext 

[Pyton_000]

@up Twój kod generuje zbędne obiekty.

Alex to co zrobiłeś jest prawie dobre Zrób na zasadzie:

[PHP] pobierz, plaintext 
class faktura{
	function generujDokument(){
		/* nie ważne co i jak, ważny efekt ;) */
		$this->dok = $html;
	}
    function wyslijFakture(PHPMailer $mailer) {
        $mailer-> ....
 
}
}
 
$faktura = new Faktura();
$faktura->geerujDokument();
$faktura->wyslijFakture(new phpMailer);
[PHP] pobierz, plaintext 

To co podałem nie jest jakies super ale zawsze możesz rozdzielić zadania i część wykonać, a część nie.

Ten post edytował Pyton_000 9.08.2014, 13:05:20

[golabow]

Warto jednak pamiętać o zasadzie jednej odpowiedzialności...

[alex19]

Wykorzystaj kompozycje i dziedziczenie.
Mogłbyś też wykorzystać wzorzec projektowy "metoda szablonowa". Dzięki temu możesz zmieniać kroki twojego i robić różne "szablony" zależności od potrzeb.
W kodzie wyglądało by to tak:

Masz na myśli dziedziczenie po klasach PHPmailer i mPDF czy coś takiego jak w kodzie?
Kod jest fajny, ale to niestety moim zdaniem kod z serii "tych magicznych", które nie wiadomo co robią a działają.

Warto jednak pamiętać o zasadzie jednej odpowiedzialności...

O czym??

Alex to co zrobiłeś jest prawie dobre Zrób na zasadzie:

[PHP] pobierz, plaintext 
 
    class faktura{
    function generujDokument(){
    /* nie ważne co i jak, ważny efekt ;) */
    $this->dok = $html;
    }
    function wyslijFakture(PHPMailer $mailer) {
    $mailer-> ....
 
    }
    }
 
    $faktura = new Faktura();
    $faktura->geerujDokument();
    $faktura->wyslijFakture(new phpMailer);
 
[PHP] pobierz, plaintext 

Czyli brak tu jakiejkolwiek filozofii... po prostu w obiekcie tworzony jest kolejny i tak dalej i tak dalej?

[PHP] pobierz, plaintext 
function generujDokument(){
    /* nie ważne co i jak, ważny efekt ;) */
    $this->dok = $html;
    }
[PHP] pobierz, plaintext 

Rozumiem, że to uproszczenie i chodzi o podstawienie $html do zmiennej klasy, która tu nie została zdefiniowana?

[PHP] pobierz, plaintext 
function wyslijFakture(PHPMailer $mailer)
[PHP] pobierz, plaintext 

Dlaczego tak, skoro dalej musimy zrobić

[PHP] pobierz, plaintext 
$faktura->wyslijFakture(new phpMailer);
[PHP] pobierz, plaintext 

A co z "konfiguracja", tak jak w przypadku mailera? Czy można ją zdefiniować w jakiś inny sposób niż w metodzie? Tu akurat przykład jest taki sobie, ale myślę np pod kątem bazy danych - coś na zasadzie otwarcia połączenia/uchwytu na początku skryptu i korzystania z niego aż do zakończenia, tak żeby móc już tylko wykonywać zapytania do bazy i nie robić nic innego.

[PHP] pobierz, plaintext 
mysql_connect();
...
mysql_query();
...
mysql_query();
...
mysql_close();
[PHP] pobierz, plaintext 

Jak by coś takiego wyglądało obiektowo, żeby nie powielać kodu, a korzystać bez ograniczeń z bazy (tej samej) z różnych obiektów w stylu:

[PHP] pobierz, plaintext 
$faktura->zapisz();
$klient->pobierz();
$towar->usun();
$dostawca->aktualizuj();
[PHP] pobierz, plaintext 

[Wazniak96]

Zobacz co kolega @pyton_000 dał jako argument metody wyslijFakture. Znajduje się tam taka fraza jak "PHPmailer". Jest to rzutowanie danych. To znaczy, że metoda ta przyjmie jako argument tylko obiekt klasy PHPMailer.

Ogólnie poczytaj o takim wzorce jak wstrzykiwanie zależności (Dependency Injection) (IMG:style_emoticons/default/wink.gif)

Ten post edytował Wazniak96 9.08.2014, 18:41:51

[alex19]

Zobacz co kolega @pyton_000 dał jako argument metody wyslijFakture. Znajduje się tam taka fraza jak "PHPmailer". Jest to rzutowanie danych. To znaczy, że metoda ta przyjmie jako argument tylko obiekt klasy PHPMailer.

Ogólnie poczytaj o takim wzorce jak wstrzykiwanie zależności (Dependency Injection) (IMG:style_emoticons/default/wink.gif)

Poczytam, ale zanim to zrobię to zapytam (IMG:style_emoticons/default/wink.gif)

Czy to jest obowiązkowe czy też jako "zabezpieczenie"i dobra praktyka, czy można by było dać jako argument metody tylko $mail.

[Wazniak96]

Jest to po prostu wymuszenie przekazania tego właśnie obiektu do metody. Dlaczego nie tworzyć bezpośrednio obiektu w metodzie? Bo zakładamy, że w skrypcie używasz już mailera to po co kolejny raz pobierać konfigurację skoro możesz przekazać ten sam obiekt, którego używałeś już do czegoś innego? (IMG:style_emoticons/default/wink.gif)

PS: Jest to moje rozumowanie. Sam dopiero uczś się OOP. Jeśli wprowadzam w błąd proszę bardziej zaawansowanych o poprawę. (IMG:style_emoticons/default/smile.gif)

Ten post edytował Wazniak96 9.08.2014, 19:09:39