0

Swiftmailer を使用して、txt ファイルに保存されている受信者のリストに電子メールを送信するスクリプトに取り組んでいます。./receiversという名前のフォルダーからすべてのメールを取得することになっていますTH001.txt, TH002.txt... 。コードを実行すると、エラーが発生して無限ループになります。

"
PHP Notice:  Undefined index: 0
 in /var/www/sipnati/sendmail.php on line 37
PHP Notice:  Undefined index: 0
 in /var/www/sipnati/sendmail.php on line 37
"

./receivers フォルダー内のファイルの名前の解析で何かを台無しにしたようです。PHP5.3 で Ubuntu Server 12.04 を実行しています。私は完全に立ち往生しています。間違いを指摘する助けがあれば大歓迎です。

    <?php

require_once 'swift_required.php';

define("SMTP_SERVER", "xxx");
define("FROM_NAME", "xxx");
define("FROM_EMAIL", "xxx");
define("USERNAME", "xxx");
define("PASSWORD", "xxx");



define("LOG_PATH", "./");

class sendMail {

    public function sendMail() {

        $transport = Swift_SmtpTransport::newInstance(SMTP_SERVER, 25)
            ->setUsername(USERNAME)
            ->setPassword(PASSWORD)
            ;

        $mailer = Swift_Mailer::newInstance($transport);


 /*     $counter = file_get_contents("./variables.txt");
        $emails_to_send = $counter + 200;
        $file_counter = file_get_contents("./variables1.txt"); */

        $scan=array_values(array_diff(scandir('./receivers/'), array('..', '.','.htaccess')));
        $mails=array();
        $c=count($scan);
        if(isset($scan[0])){
            for($i=0;$i<$c;$i++){
                if(substr($scan[$i], strlen($scan[$i] - 3), 3) <> "txt"){
                    $tmp=file('./receivers/'.$scan[$i],FILE_IGNORE_NEW_LINES);
                    $mails=array_merge($mails,$tmp);
                }
            }
        }
 /*     file_put_contents("./variables1.txt", $count-1); */
        $regexp = '/^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$/';

        $count = 0;
        if($c>0){
            foreach ($mails as $key => $value) {
                while ($count < count($mails)) { /* replaced $counter with count($mails) */

                    if (preg_match($regexp, $value) != 0) {

                        $message = Swift_Message::newInstance('Wonderful Subject')
                            ->setFrom(array(FROM_EMAIL => FROM_NAME))
                            ->setTo(array('receiver@domain.org', 'other@domain.org' => 'A name'))
                            ->setSubject(file_get_contents('./thailand_subject.txt'))
                            ->setBody(file_get_contents('./thailand_body.txt'))
                            ;

                        if ($mailer->send($message)) {
                            $count++;
                        } else {
                            $not_sent[] = $value;
                        }
                    } else {
                        $not_sent[] = $value;
                    }
                    $count++;
                }
                /*file_put_contents("./variables.txt", $file_counter++);*/
                break;
            }
        }
        if (!isset($not_sent))
            $not_sent = null;
        $this->logToFile($count, count($mails), $not_sent);
    }

    private function logToFile($count, $on, $undelivered = null) {
        $toLog = "Sent " . $count . "/" . $on . " mails. Not sent: \n" . print_r($undelivered, true);
        file_put_contents(LOG_PATH . "MAIL_LOG_" . microtime(), $toLog);
    }

}

$p = new sendMail();
?>
4

1 に答える 1

2

新しいコード:

<?php

require_once 'swift_required.php';

define("SMTP_SERVER", "xxx");
define("FROM_NAME", "xxx");
define("FROM_EMAIL", "xxx");
define("USERNAME", "xxx");
define("PASSWORD", "xxx");

define("LOG_PATH", "./");

class sendMail {

    public function sendMail() {

        $transport = Swift_SmtpTransport::newInstance(SMTP_SERVER, 25)
            ->setUsername(USERNAME)
            ->setPassword(PASSWORD)
            ;

        $mailer = Swift_Mailer::newInstance($transport);

        $scan=array_values(array_diff(scandir('./receivers/'), array('..', '.','.htaccess')));
        $mails=array();
        $c=count($scan);
        if(isset($scan[0])){
            for($i=0;$i<$c;$i++){
                if(substr($scan[$i], strlen($scan[$i] - 3), 3) <> "txt"){
                    $tmp=file('./receivers/'.$scan[$i],FILE_IGNORE_NEW_LINES);
                    $mails=array_merge($mails,$tmp);
                }
            }
        }

        $regexp = '/^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$/';

        $c=count($mails);
        if($c>0){
            for($i=0;$i<$c;$i++){
                if (preg_match($regexp, $mails[$i]) != 0) {
                    $message = Swift_Message::newInstance('Wonderful Subject')
                        ->setFrom(array(FROM_EMAIL => FROM_NAME))
                        ->setTo($mails[$i])
                        ->setSubject(file_get_contents('./thailand_subject.txt'))
                        ->setBody(file_get_contents('./thailand_body.txt'))
                    ;
                    if (!$mailer->send($message))
                        $not_sent[] = $mails[$i];
                }
                else
                    $not_sent[] = $mails[$i];
            }
        }
        if (!isset($not_sent))
            $not_sent = array();

        $this->logToFile($c, $not_sent);
    }

    private function logToFile($on, $undelivered) {
        $sent=$on-count($undelivered);
        $toLog = "Sent " . $sent . "/" . $on . " mails. Not sent: \n" . print_r($undelivered, true);
        file_put_contents(LOG_PATH . "MAIL_LOG_" . microtime(), $toLog);
    }

}

$p = new sendMail();
?>
于 2013-09-02T09:58:05.523 に答える