Forum PHP.pl > [php] Klasa wysyłająca maila

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [php] Klasa wysyłająca maila

Lejto

2.07.2009, 11:49:52

Mam klase wysyłającą maila

[PHP] pobierz, plaintext 
<?
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 nag($email)
    {
               $header = "From:  <$email>\n"; 
              $header .= "MIME-Version: 1.0\n";
              $header .= "Content-type: text/html; charset=utf-8\n";
              $header .= "Return-Path: <$email>";
    }
    public function sendEmail()
    {
        
        foreach ($this->recipients as $recipient)
        {
            $result = mail($recipient,$this->subject,$this->body,$this->nag);
            if($result)
            {
            echo '<div id="good">';
            echo 'Wiadomość została wysłana';
            echo '</div>';
             }
        }
    }
    
}
?>
[PHP] pobierz, plaintext

Dodałem funkcje z nagłówkami nag();
Uruchamiam tak:

[PHP] pobierz, plaintext 
<?php
$email = new Emailer($email); //adres od 
            $email->addRecipients("agencik@gmail.com"); // adres do 
            $email->setSubject($temat);
            $email->setBody($wiadomosc);
            $email->nag($email);
            $email->sendEmail();
?>
[PHP] pobierz, plaintext

wszystko działało dobrze tylko jak dodałem te nagłówki to otrzymuje taki błąd

Kod

Catchable fatal error: Object of class Emailer could not be converted to string in /home/s/public_html/s-s.eu/mail.class.php on line 28

w lini 28 znajduje się

[PHP] pobierz, plaintext 
<?php
$header = "From:  <$email>\n";
?>
[PHP] pobierz, plaintext

osl

2.07.2009, 11:52:10

spójrz dobrze co tak na prawde robisz

nadpisujesz stringa podawanego do konstruktora...

Spawnm

2.07.2009, 11:52:17

[PHP] pobierz, plaintext 
<?php
public function nag($email)
   {
              $header = "From:  <$email>\n";
             $header .= "MIME-Version: 1.0\n";
             $header .= "Content-type: text/html; charset=utf-8\n";
             $header .= "Return-Path: <$email>";
   }
?>
[PHP] pobierz, plaintext

gdzie this-> lub return ?

Lejto

2.07.2009, 11:59:17

nie rozumiem za bardzo, mam dodać this w funkcji nag?

kbsucha

2.07.2009, 12:04:55

No nie, przy takiej konstrukcji nie, ale mozesz to zrobić bez parametru używając poprostu daną z konstruktora $this->sender.
No i w sendMail powinno być $this->nag(), a w metodzie jakis return. Ogolnie troche namieszales ;P

fander

2.07.2009, 12:20:14

Kod

<?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 nag($email)
   {
       $header = "From:  <$email>\n";
       $header .= "MIME-Version: 1.0\n";
       $header .= "Content-type: text/html; charset=utf-8\n";
       $header .= "Return-Path: <$email>";
       return $header;
   }
   public function sendEmail()
   {

       foreach ($this->recipients as $recipient)
       {
              $result = mail($recipient,$this->subject,$this->body,$this->nag());
           if($result)
           {
           echo '<div id="good">';
           echo 'Wiadomość została wysłana';
           echo '</div>';
            }
       }
   }

}
?>

Takie coś

osl

2.07.2009, 12:27:39

koleżanki i kolegi, wszystko ok co piszecie powyżej, ale ni jak to się ma do errora.
@autor - masz stringa w zmiennej $email, potem do tej zmiennej przypisujesz obiekt, a na koniec używasz jej jako argumentu funkcji, która potrzebuje stringa. I stąd ten error. I poczytaj co jeszcze Ci sugerują powyżej, dobrze prawią

fander

2.07.2009, 12:48:30

LOOOOL

Kod

<?php
[color="#ff0000"]$email [/color]= new Emailer($email); //adres od
            $email->addRecipients("agencik@gmail.com"); // adres do
            $email->setSubject($temat);
            $email->setBody($wiadomosc);
            [color="#ff0000"]$email->nag($email[/color][color="#ff0000"]);[/color]
            $email->sendEmail();
?>

$email->nag($email); <= nie może być czegoś takiego jak juz kolega powyżej zauważył zmień $email na $stringMail cokolwiek a dlatego to musisz zmienić ponieważ $email wyżej zadeklarowałeś jako Emailer a emaileraa nie da się wyświetlić jak stringa no chyba ze byśmy przeciązyli operatory ale w php nie da się ich przeciążyć.

Lejto

2.07.2009, 14:11:08

@fander twoja poprawka nic nie dała

fander

3.07.2009, 08:30:22

Kod

<?php
$email = new Emailer($sender); //adres od
            $email->addRecipients("agencik@gmail.com"); // adres do
            $email->setSubject($temat);
            $email->setBody($wiadomosc);
            $email->nag($mail)
            $email->sendEmail();
?>

Sprawdź teraz. Ten kod który wcześniej podałem nie był poprawiony!!! poniżej dałem ci wskazówki co i jak poprawić wystarczyło przeczytać uważnie moje sugestie.

phpion

3.07.2009, 09:00:58

Cytat(fander @ 2.07.2009, 13:48:30 )

LOOOOL

...

zadeklarowałeś jako Emailer a emaileraa nie da się wyświetlić jak stringa no chyba ze byśmy przeciązyli operatory ale w php nie da się ich przeciążyć.

LOOOOL a o metodzie __toString kolega nie słyszał? Operatorów nie da się przeciążyć, to prawda, ale można utworzyć metodę zwracającą obiekt w postaci stringu.

@Lejto:
Wydaje mi się, że powinno wystarczyć dodać metodę __toString (co nie rozwiąże problemu z generowaniem nagłówków) i jako tako powinno działać:

[PHP] pobierz, plaintext 
<?php
public function __toString() {
        return $this->sender;
    }
?>
[PHP] pobierz, plaintext

Jednak lepiej by było jakbyś rozwiązał tą sprawę ciut inaczej. Sprawdź ten kod. Powinien działać ale nie sprawdzałem, pisałem na czuja.

[PHP] pobierz, plaintext 
<?php
class Emailer
{
   private $sender;
   private $recipients;
   private $subject;
   private $body;
     private $headers = array();
   
   function __construct($sender)
   {
       $this->sender = $sender;
       $this->recipients = array();
             
             $this->addHeader('MIME-Version', '1.0');
             $this->addHeader('Content-type', 'text/html; charset=utf-8');
   }
     
   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 addHeader($name, $value) {
        $this->headers[$name] = $value;
     }
     
     private function getHeadersAsString() {
        $return = '';
        
        foreach ($this->headers as $name => $value) {
            $return .= $name.': '.$value."\n";
        }
        
        return $return;
     }
 
   public function sendEmail()
   {
            $headersAsString = $this->getHeadersAsString();
       
       foreach ($this->recipients as $recipient)
       {
           $result = mail($recipient,$this->subject,$this->body, $headersAsString);
           if($result)
           {
           echo '<div id="good">';
           echo 'Wiadomość została wysłana';
           echo '</div>';
            }
       }
   }
   
}
 
$email = 'nadawca@host.pl'; // od kogo jest mail
 
$emailer = new Emailer($email); //adres od 
$emailer->addRecipients("agencik@gmail.com"); // adres do 
$emailer->setSubject($temat);
$emailer->setBody($wiadomosc);
 
// dodanie naglowkow
$emailer->addHeader('From', $email);
$emailer->addHeader('Return-Path', $email);
 
$emailer->sendEmail();
?>
[PHP] pobierz, plaintext

Lejto

3.07.2009, 10:07:26

nie działa ;/ taki sam błąd jak wtedy

phpion

3.07.2009, 10:40:53

Cytat(Lejto @ 3.07.2009, 11:07:26 )

nie działa ;/ taki sam błąd jak wtedy

Tak? No to dziwne bo przed chwilą sprawdziłem oba podane przez siebie rozwiązania i oba działają poprawnie. Albo wcale ich nie sprawdziłeś albo bezmyślnie mówisz "nie działa" po ujrzeniu "Notice: Undefined variable".

fander

3.07.2009, 11:16:34

coś ktoś ściemnia, wystarczy zajrzeć w kod

! przeczytać błędy podstawić wartości

Kod

$email = 'nadawca@host.pl'; // od kogo jest mail

$emailer = new Emailer($email); //adres od
$emailer->addRecipients("agencik@gmail.com"); // adres do
$emailer->setSubject("Temat wiadomosci");
$emailer->setBody("<b>Wiadomość</b>");

// dodanie naglowkow
$emailer->addHeader('From', $email);
$emailer->addHeader('Return-Path', $email);

$emailer->sendEmail();
?>

Lejto

3.07.2009, 11:31:13

nie, dostałem taki sam błąd jak wcześniej, teraz sprawdziłem jeszcze raz i jest ok, dzięki, może wtedy jak sprawdzałem to plik do końca się nie przesłał bo szybko to robiłem...

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.