Witaj Gościu! ( Zaloguj | Rejestruj )

Forum PHP.pl

 
 Reply to this topicStart new topic
> Optymalizacja kodu, ... wyslanie min. 100 maili
kr3t
post 16.12.2009, 10:53:36
Post #1





Grupa: Zarejestrowani
Postów: 15
Pomógł: 0
Dołączył: 7.12.2003

Ostrzeżenie: (0%)
-----

Przy wykorzystaniu klasy phpmailer probuje wyslac min 100 maili ....

teraz przy wysyłce osobnej do 10 adresatow osiagam czas wykonania skryptu 6-7 sekund...

a na serwerze jest limit 30 sekund i po tym czasie wywala error 500 :/

Czy ktos moze podpowiedziec co zrobic by skrypt wykonywal sie szybciej?

Z gory dziekuje!


[PHP] pobierz, plaintext 
  1.  
  2. <?
  3.  
  4. include "mysql_db.php";
  5. $baza = new MyDb;
  6.  
  7. require_once('class.phpmailer.php');
  8.  
  9. //pobieranie adresow maili z poprzedniego formularza
  10.  
  11. $size = sizeof($_POST["adres"])-1;
  12.     for($i=0;$i<=$size;$i++) { 
  13.       echo $_POST["adres"][$i];
  14.   }
  15.  
  16.  
  17. require_once('class.phpmailer.php');
  18.  
  19. $mail = new PHPMailer(true); // the true param means it will throw exceptions on errors, which we need to catch
  20.  
  21. $mail->Host       = ".."; // SMTP server
  22. $mail->SMTPDebug  = 2;                     // enables SMTP debug information (for testing)
  23. $mail->SMTPAuth   = true;                  // enable SMTP authentication
  24.  
  25. $mail->Host       = "..";     // sets  SMTP server
  26. $mail->Port       = 587;   
  27.  
  28. $mail->Username   = "...";  // username
  29. $mail->Password   = ">.";            //  password
  30.  
  31. for($i=0;$i<=$size;$i++)
  32.   {
  33.  
  34.     try {
  35.  
  36.        $mail->AddReplyTo('...', 'test');
  37.        $mail->SetFrom('mailing@...', 'test');
  38.        $mail->Subject  = $_POST["temat"];
  39.       $mail->AddAttachment('ll.jpg');      // attachment
  40.       $mail->AltBody = 'To view the message, please use an HTML compatible email viewer!'; // optional - MsgHTML will create an alternate automatically
  41.  
  42.       $tresc2=nl2br($_POST["tresc"]);
  43.      $tresc="<body style=\"margin: 10px;\"><div style=\"width: 640px; font-family: Arial, Helvetica, sans-serif; font-size: 11px;\">".$tresc2."
  44. <Br /><Br />";
  45.  
  46.  
  47.     $mail->MsgHTML($tresc);
  48.     $adres = $_POST["adres"][$i];
  49.     $mail->AddAddress($adres);
  50.  
  51.        if(!$mail->Send()) 
  52. 	   {
  53.  
  54.            //echo "Mailer Error (" . str_replace("@", "@", . $_POST["adres"][$i] . ') ' . $mail->ErrorInfo . '<br />';
  55.        } 
  56. 	   else 
  57. 	   {
  58.            echo "Message sent to :" . $_POST["adres"][$i] . ' (' . str_replace("@", "@", $row["email"]) . ')<br />';
  59.        }
  60.     echo "Message Sent OK</p>\n";
  61.  
  62.       } catch (phpmailerException $e) {
  63.         echo $e->errorMessage(); //Pretty error messages from PHPMailer
  64.       } catch (Exception $e) {
  65.         echo $e->getMessage(); //Boring error messages from anything else!
  66.       }
  67.    $mail->ClearAddresses(); 
  68.  
  69. }
  70. $mail->SmtpClose();
  71. ?>
[PHP] pobierz, plaintext


Ten post edytował kr3t 16.12.2009, 10:54:45


--------------------
KR3T
Go to the top of the page
+Quote Post
wry
post 16.12.2009, 12:35:44
Post #2





Grupa: Zarejestrowani
Postów: 32
Pomógł: 3
Dołączył: 9.06.2007

Ostrzeżenie: (0%)
-----

wywa polaczenie z baza jesli go nie uzywasz
wypisanie do przegladarki moze dlugo trwac
wiec moze usun blok miedzy linia 1 a 16
wyciagnij z petli ustawianie tych atrybutow ktore sa niezmienne (subject, replayto itd)


--------------------
http://marcinwrobel.com.pl/
Go to the top of the page
+Quote Post
kr3t
post 16.12.2009, 14:28:10
Post #3





Grupa: Zarejestrowani
Postów: 15
Pomógł: 0
Dołączył: 7.12.2003

Ostrzeżenie: (0%)
-----

zeszlo tylko do 6 sekund.... :/


--------------------
KR3T
Go to the top of the page
+Quote Post
Blame
post 16.12.2009, 15:16:36
Post #4





Grupa: Zarejestrowani
Postów: 678
Pomógł: 124
Dołączył: 26.09.2009

Ostrzeżenie: (0%)
-----

Wiem, że to rozwiązanie to nie rozwiązanie ale może po prostu zwiększ limit i po sprawie.


--------------------
Napisane z palca! <- to tak dla lansu ;)
FancyStudio.pl - Tworzenie Stron WWW w Oparciu o XHTML, CSS, JS, PHP i MySQL.
Mój blog :: Portfolio Fotograficzne Igora Biedrawy
Go to the top of the page
+Quote Post
korro
post 16.12.2009, 15:31:31
Post #5





Grupa: Zarejestrowani
Postów: 259
Pomógł: 42
Dołączył: 8.04.2005
Skąd: Mława

Ostrzeżenie: (0%)
-----

Witam.
Ja takie rzeczy robię w tle.
Mała funkcja:
[PHP] pobierz, plaintext 
  1. function launchBackgroundProcess($call) {
  2.     if(is_windows()){
  3.         pclose(popen('start /low /b '.$call, 'r'));
  4.     }
  5.     else {
  6.         pclose(popen($call.' /dev/null &', 'r'));
  7.     }
  8.     return true;
  9. }
  10. function is_windows(){
  11.     if(PHP_OS == 'WINNT' || PHP_OS == 'WIN32') {
  12.         return true;
  13.     }
  14.     return false;
  15. }
[PHP] pobierz, plaintext

i wywołanie:
[PHP] pobierz, plaintext 
  1. launchBackgroundProcess('php -f plik.php');
[PHP] pobierz, plaintext

Użytkownik nie czeka na długie zadanie.


--------------------
Zapraszam na mojego devbloga

Używasz Firefoxa? Skracaj linki jednym kliknięciem!
Go to the top of the page
+Quote Post
kr3t
post 16.12.2009, 16:40:35
Post #6





Grupa: Zarejestrowani
Postów: 15
Pomógł: 0
Dołączył: 7.12.2003

Ostrzeżenie: (0%)
-----

to konto www jest na nazwa . pl

i tam jest limit 30 sek na wykonanie skryptu php :/

a w tle.. hmm to nadal skrypt bedzie wykonywany przez dluzej - to wtedy sie omija limity?


--------------------
KR3T
Go to the top of the page
+Quote Post
Sky_walker
post 16.12.2009, 16:48:24
Post #7





Grupa: Zarejestrowani
Postów: 214
Pomógł: 23
Dołączył: 26.09.2005

Ostrzeżenie: (0%)
-----

możesz też tak:
1) skorzystać z wewnętrznej funkcji php do maili i nie łączyć się z SMTP - nawiązanie połączenia jest czasochłonne
2) kolejnych odbiorców dodawaj w pole BCC zamiast w pole TO - wtedy wyśle do wszystkich za jednym połączeniem z usługą poczty (przydatne i dla php'owego mail i dla połączenia SMTP).


--------------------
Pomogłem? Kliknij poniżej.
Dzięki! :D
Go to the top of the page
+Quote Post
korro
post 16.12.2009, 18:09:01
Post #8





Grupa: Zarejestrowani
Postów: 259
Pomógł: 42
Dołączył: 8.04.2005
Skąd: Mława

Ostrzeżenie: (0%)
-----

Limit czasu nie obowiązuje w tym przypadku, ponieważ jest to zwykłe polecenie systemu operacyjnego.


--------------------
Zapraszam na mojego devbloga

Używasz Firefoxa? Skracaj linki jednym kliknięciem!
Go to the top of the page
+Quote Post
piotr94
post 16.12.2009, 18:25:59
Post #9





Grupa: Zarejestrowani
Postów: 331
Pomógł: 30
Dołączył: 11.11.2008
Skąd: Kraków

Ostrzeżenie: (0%)
-----

tylko że nie wszystkie serwery zezwalają na polecenia systemowe ;-)
ja bym to zrobił tak:
po wysłaniu każdego mejla sprawdzasz czas. jeśli zbliża się on do 30 s, to przerywasz działanie skryptu i dajesz 
[PHP] pobierz, plaintext 
  1. header('Location:http://www.aaa/plik.php?wznow_od_id='.$id_ostatnio wysłanego mejla);
[PHP] pobierz, plaintext

i powinno być git ;-)


--------------------
http://www.piotr94.net21.pl/ - wykonanie stron i serwisów internetowych
Jeśli moje wypowiedzi były dla Ciebie pomocne, kliknij "Pomógł" i odwdzięcz się ;)
Go to the top of the page
+Quote Post
Thorang Hoog
post 16.12.2009, 21:52:01
Post #10





Grupa: Zarejestrowani
Postów: 41
Pomógł: 9
Dołączył: 8.02.2009

Ostrzeżenie: (0%)
-----

Nigdy nie używałem tej klasy ale temat mnie interesuje także poszukałem i znalazłem winksmiley.jpg funkcja 
[PHP] pobierz, plaintext 
  1. $mail->AddAddress($adres);
[PHP] pobierz, plaintext
dodaje adres do tablicy. Podczas wysyłania wiadomości wysyła pod wszystkie adresy jednocześnie.
Dlatego wystarczy w pętli umieścić: 
[PHP] pobierz, plaintext 
  1. $adres = $_POST["adres"][$i];
  2. $mail->AddAddress($adres);
[PHP] pobierz, plaintext

Ciekawe do ilu czas teraz zejdzie.
Go to the top of the page
+Quote Post
wry
post 17.12.2009, 10:39:06
Post #11





Grupa: Zarejestrowani
Postów: 32
Pomógł: 3
Dołączył: 9.06.2007

Ostrzeżenie: (0%)
-----

to nie jest dobre rozwiazanie bo wszyscy dostana maila z adresami innych do ktorych zostal on wyslany


--------------------
http://marcinwrobel.com.pl/
Go to the top of the page
+Quote Post
korro
post 17.12.2009, 10:45:41
Post #12





Grupa: Zarejestrowani
Postów: 259
Pomógł: 42
Dołączył: 8.04.2005
Skąd: Mława

Ostrzeżenie: (0%)
-----

Zostaje jeszcze kolejkowanie.
Tabela w bazie, do której dodajesz rekordy - jeden rekord to jeden adres mail, treść etc.
Do crona dodajesz zadanie które co jakiś czas pobiera dane, wysyła maila i zmienia status w kolejce na wysłano.


--------------------
Zapraszam na mojego devbloga

Używasz Firefoxa? Skracaj linki jednym kliknięciem!
Go to the top of the page
+Quote Post
Thorang Hoog
post 17.12.2009, 15:53:16
Post #13





Grupa: Zarejestrowani
Postów: 41
Pomógł: 9
Dołączył: 8.02.2009

Ostrzeżenie: (0%)
-----

Znalazłem w klasie phpmailer taką zmienną:
[PHP] pobierz, plaintext 
  1. /**
  2. * Provides the ability to have the TO field process individual
  3. * emails, instead of sending to entire TO addresses
  4. * @var bool
  5. */
  6.  
  7. public $SingleTo = false;
[PHP] pobierz, plaintext


Po zmianie wartości na true wyśle pojedyncze maile do użytkowników.
Go to the top of the page
+Quote Post
« Następny starszy · PHP · Następny nowszy »
 

Reply to this topicStart new topic
1 Użytkowników czyta ten temat (1 Gości i 0 Anonimowych użytkowników)
0 Zarejestrowanych:

 

Tryb wyświetlania: Standardowy · Przełącz na: Linearny+ · Przełącz na: Drzewo

Śledź ten temat · Wyślij temat na e-mail · Wydrukuj ten temat · Subskrybuj to forum

RSS Wersja Lo-Fi Aktualny czas: 14.08.2025 - 02:48
Powered By IP.Board © 2025  IPS, Inc.
All changes by PHP.pl Administrators
Hosting zapewnia NQ.pl hosting, trac, svn