Dziedziczenie i pętla foreach Opcje

[MicNeo]

Witam!

Raczkuje w OOP. Natknąlem się na taki błąd:

Warning: Invalid argument supplied for foreach() in /homez.108/micneo/www/test/class.emailer.php on line 43

class.emailer.php

[PHP] pobierz, plaintext 
<?
// class.emailer.php
class Emailer
{
   private $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 HtmlEmailer extends Emailer
{
   public function sendHTMLEmail()
   {
         foreach ($this->recipients as $recipient)
         {
            $headers  = 'MIME-Version: 1.0' . "r\n";
            $headers .= 'Content-type: text/html; charset=iso-8859-2' .
               "r\n";
            $headers .= 'From: {$this->sender}' . "r\n";
            $result = mail($recipient, $this->subject, $this->body,
               $headers);
            if ($result) echo "Wiadomość w formacie HTML została wysłana do
               {$recipient}<br/>";
      }
   }
}
?>
[PHP] pobierz, plaintext 

oraz plik emailer.php

[PHP] pobierz, plaintext 
<?php
include_once("class.emailer.php");
 
$mailHTML = new HtmlEmailer("adres@mail.pl");
$mailHTML->addRecipients("odbioraca@mail.pl");
$mailHTML->setSubject("temat");
$mailHTML->setBody("test");
$mailHTML->sendHTMLEmail();
 
?>
[PHP] pobierz, plaintext 

Nie wiem co jest nie tak. Jak się wczesniej bawiłem dziedziczeniem to śmigało (nie wykorzystalem pętli foreach). Nie wiem co źle robie. Prosze o pomoc.

Pozdrawiam!

[pyro]

[PHP] pobierz, plaintext 
<?php
private $recipients;
?>
[PHP] pobierz, plaintext 

Zamień na:

[PHP] pobierz, plaintext 
<?php
protected $recipients;
?>
[PHP] pobierz, plaintext 

// edit

i podobnie reszte pól klasy!

[PHP] pobierz, plaintext 
<?php
private $sender;
  private $recipients;
  private $subject;
  private $body;
?>
[PHP] pobierz, plaintext 

na:

[PHP] pobierz, plaintext 
<?php
protected $sender;
  protected $recipients;
  protected $subject;
  protected $body;
?>
[PHP] pobierz, plaintext 

// add

mały hint

[PHP] pobierz, plaintext 
<?php
function __construct($sender)
   {
      $this->sender = $sender;
      $this->recipients = array();
   }
?>
[PHP] pobierz, plaintext 

Nie musisz robić (tutaj) drugiej instrukcji w konstruktorze, możesz od razu dać

[PHP] pobierz, plaintext 
<?php
protected $recipients = array();
?>
[PHP] pobierz, plaintext 

Ten post edytował pyro 4.07.2009, 19:48:38

[MicNeo]

facepalm...

Cholera, nie zauważyłem tego (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Dzięki.