OOP i problem z __construct Opcje

[spokoloko123]

Jako, że jestem nowy w OOP to za bardzo nie wiem dlaczego w tym kodzie (z książki) jest błąd. Wg. autora powinno być dobrze ale nie jest. Problem rozwiązują szybkie zmiany w kodzie ale nie o to mi chodzi. Raczej szukał bym jakiejś literówki ewentualnie wytłumaczenia dlaczego nie działa

[PHP] pobierz, plaintext 
<?php
	// class.emailer.php
class Emailer
{
   protected $sender;
   private $recipients;
   private $subject;
   private $body;
 
   function __construct($sender)
   {
      $this->sender = $sender;
      $this->recipients = array();
   }
 
   public function addRecipients($recipient)
   {
      array_push($this->recipients, $recipient);
   }
   public function setSubject($subject)
   {
      $this->subject = $subject;
   }
   public function setBody($body)
   {
      $this->body = $body;
   }
   public function sendEmail()
   {
      foreach ($this->recipients as $recipient)
      {
         $result = mail($recipient, $this->subject, $this->body,
            "From: {$this->sender}\r\n");
         if ($result) echo "Wiadomość została wysła do
            {$recipient}<br/>";
      }
   }
}
 
	class ExtendedEmailer extends Emailer
	{
 
 
		function __construct(){}
 
		public function setSender($sender)
		{
			$this->sender = $sender;
		}
	}
 
	$xemailer = new ExtendedEmailer();
	$xemailer->setSender("JA");
	$xemailer->addRecipients("user1@localhost");
	$xemailer->addRecipients("user2@localhost");
	$xemailer->setSubject("extended email");
	$xemailer->setBody("<h1>Extended</h1>");
	$xemailer->sendEmail();
 
?>
[PHP] pobierz, plaintext 

Wynik skryptu wygląda tak:

[PHP] pobierz, plaintext 
Warning: array_push() expects parameter 1 to be array, null given in C:\xampp\htdocs\OOP\ex2.1.php on line 18
 
Warning: array_push() expects parameter 1 to be array, null given in C:\xampp\htdocs\OOP\ex2.1.php on line 18
 
Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\OOP\ex2.1.php on line 30
[PHP] pobierz, plaintext 

Ten post edytował spokoloko123 8.05.2012, 14:42:15

[Psajkus]

nadpisujesz konstruktor w klasie ExtendedEmailer.

[redeemer]

Nie wywołujesz konstruktora klasy po której dziedziczysz.

Ten post edytował redeemer 8.05.2012, 14:53:45

[mortus]

Nie wywołujesz konstruktora klasy po której dziedziczysz.

Raczej jest tak, jak napisał Psajkus. spokoloko123 przeciąża konstruktor klasy, po której dziedziczy, wobec czego kod tego konstruktora nie jest wykonywany. Przeciążenia takiego powinniśmy dokonać wtedy, gdy klasa ExtendedEmailer będzie np. ustawiać dodatkowe właściwości/pola istotne dla tej właśnie klasy. Jeśli nie ma takiej potrzeby, to nie musimy przeciążać konstruktora i deklaracje funkcji __construct() w klasie dziedziczącej powinniśmy pominąć.

Edycja:
Wobec powyższego klasa ExtendedMailer mogłaby wyglądać po prostu tak:

[PHP] pobierz, plaintext 
class ExtendedMailer extends Mailer {
  public function setSender($sender) {
    $this->sender = $sender;
  }
}
// a jej wywołanie
$xmailer = new ExtendedMailer('sender');
// mogłaby wyglądać również tak
class ExtendedMailer extends Mailer {
  public function __construct($sender = null) {
    parent::construct($sender);
  }
  public function setSender($sender) {
    $this->sender = $sender;
  }
}
// a jej wywołanie
$xmailer = new ExtendedMailer();
$xmailer->setSender('sender');
[PHP] pobierz, plaintext 

Przy czym drugi sposób nie jest do końca prawidłowy, bo umożliwia utworzenie obiektu Mailera, który nie posiada "właściciela" (czyli osoby, która może z tego mailera korzystać (identyfikowanej po adresie e-mail)).

Ten post edytował mortus 8.05.2012, 15:12:39

[spokoloko123]

Okazało się to erratą w książce. Dla przyszłych pokoleń:

[PHP] pobierz, plaintext 
<?php
	// class.emailer.php
class Emailer
{
   protected $sender;
   protected $recipients;
   private $subject;
   private $body;
 
   public function addRecipients($recipient)
   {
      array_push($this->recipients, $recipient);
   }
   public function setSubject($subject)
   {
      $this->subject = $subject;
   }
   public function setBody($body)
   {
      $this->body = $body;
   }
   public function sendEmail()
   {
      foreach ($this->recipients as $recipient)
      {
         $result = mail($recipient, $this->subject, $this->body,
            "From: {$this->sender}\r\n");
         if ($result) echo "Wiadomość została wysła do
            {$recipient}<br/>";
      }
   }
}
 
	class ExtendedEmailer extends Emailer
	{
 
		function __construct() 
		{
			$this->recipients = array();
		}
		public function setSender($sender) 
		{ 
			$this->sender = $sender; 
		}
	}
 
	$xemailer = new ExtendedEmailer();
	$xemailer->setSender("JA");
	$xemailer->addRecipients("user1@localhost");
	$xemailer->addRecipients("user2@localhost");
	$xemailer->setSubject("extended email");
	$xemailer->setBody("<h1>Extended</h1>");
	$xemailer->sendEmail();
 
?>
[PHP] pobierz, plaintext 

[Crozin]

Nadal nie jest to coś co można by nazwać poprawnym kodem. Klasa Emailer w metodzie addRecipients() zakłada, że właściwość recipients jest tablicą w momencie gdy tak na prawdę nie jest. Poniższy kod wywali błąd:

[PHP] pobierz, plaintext 
$emailer = new Emailer(); // Klasa ta nie jest abstrakcyjna, więc nic nie stoi na przeszkodzie by utworzyć jej istancję
$emailer->addRecipients("user1@localhost");
[PHP] pobierz, plaintext 

Tak więc jak już to:

[PHP] pobierz, plaintext 
class Emailer {
  ...
 
  public function __construct() {
    $this->recipients = array();
  }
}
 
class ExtendedEmailer {
  public function __construct() {
    parent::__construct();
 
    // Kod konstruktora specyficzny dla tej klasy
  }
}
[PHP] pobierz, plaintext 

PS. Skoro metoda addRecipients() dodaje tylko jednego odbiorcę, to czy nie powinna nazywać się addRecipient()?

Ten post edytował Crozin 8.05.2012, 15:19:36

[spokoloko123]

1. Dodaje paru (gdy dziedziczona)
2. Kod z przykładu w książce, nie mój
3. Tak, wywali ale nie jest to problem, bo jest używana tylko jako dziedziczenie (wiem głupie, patrz punkt 2) .
4. Dlaczego są te nawiasy klamrowe? np:

[PHP] pobierz, plaintext 
echo "Wiadomość została wysła do
            {$recipient}<br/>";
[PHP] pobierz, plaintext 

Dzięki za zainteresowanie.

Ten post edytował spokoloko123 8.05.2012, 15:27:33

[bastard13]

Abstrahując od Twojego problemu, który już został rozwiązany, to wydaje mi się, że hierarchia, którą tutaj przedstawiłeś jest totalnie bez sensu i nazywanie tego OOP jest 'lekkim' przekłamaniem. Jeżeli taki przykład jest w książce, to wrzuć tytuł, aby przyszłe pokolenia programistów po coś takiego nie sięgały:)

Dlaczego przykład jest zły?
Klasa ExtendedEmailer jest całkowicie nie potrzebna i tworzona chyba tylko po to, aby zrobić dziedziczenie. Wszystko co się w niej dzieje, to nadpisanie konstruktora i utworzenie metody setSender(), która poprzez swoją logikę wymusza zmianę widoczności atrybutu sender w klasie nadrzędnej na protected (czego również warto unikać.
Jeżeli istnieje sytuacja, że klasę tworzysz bez sendera i dodajesz go później (setSender()), to na co Ci nowa klasa? Wystarczy zmienić konstruktor w klasie Emailer tak, żeby parametr był opcjonalny i do tej klasy dodać metodę setSender(). I po sprawie.

Nadmierne dziedziczenie stwarza wiele problemów (pewnie nawet jest to jakiś antywzorzec:P).

Dodatkowa rada, to nie zmieniaj deklaracji metody jeżeli rozszerzasz klasę tzn. jeżeli deklaracja klonstruktora w klasie rodzica jest: __construct($sender), to w klasach podrzędnych również taka powinna być.