カスタムPHP関数のPHPMailer

Question

PHPMailerに必要な基本情報を設定するカスタムクラス関数を作成しました（毎回入力する必要はありません）。ここに関数の正確なコードがあります。

<?php 

class PHPMailer { 

    public static function send() {// I will just add here the addAddress 
     require_once 'mail/PHPMailerAutoload.php'; 

     $mail = new PHPMailer; 

     $mail->isSMTP(); 
     $mail->SMTPDebug = 0; 
     $mail->Debugoutput = 'html'; 
     $mail->Host = "smtp.gmail.com"; 
     $mail->Port = 587; 
     $mail->SMTPSecure = 'tls'; 
     $mail->SMTPAuth = true; 
     $mail->Username = "validusername"; 
     $mail->Password = "validpassword"; 
     $mail->setFrom('validusername', 'Valid Username'); 
     $mail->addAddress('googol8080@gmail.com', 'Googol'); 

     $mail->Subject = "Subject"; 
     $mail->Body = "<a href=\"www.google.com\">www.google.com</a>"; 
     $mail->IsHTML(true); 

     if (!$mail->send()) { 
      return "Error sending message" . $mail->ErrorInfo; 
     } else { 
      return "Message sent!"; 
     } 
    } 
} 

は、これまでのところ、それは私のローカルホストに取り組んでいるが、私は疑問を持っている：

• は、これは良い習慣ですか？
• コードは大丈夫ですか？
• これには何らかの欠点がありますか？
• ここでパフォーマンスを最適化する必要がある場合は、それを実現するために何をする必要がありますか？

私はPHPとPHPMailerで本当に新しいです。小さな答えは私に役立ちます、ありがとう。

Answer 1

あなたのコードはうまくいくようですが、より良い方法はvariablesと呼ぶことができるため、クラスを呼び出すたびに設定する必要はありません。

class phpmailer { 
    public function sendMail($email, $message, $subject) 
    { 
     require_once('../phpmailer/class.phpmailer.php'); 
     require_once('../phpmailer/class.smtp.php'); 
     require_once('../phpmailer/class.pop3.php'); 
     $mail = new PHPMailer(); 
     $mail->isSMTP(); 
     $mail->SMTPDebug = 0; 
     $mail->SMTPAuth = true; 
     $mail->SMTPSecure = "ssl"; 
     $mail->Host = "smtp.gmail.com"; 
     $mail->Port = 465; 
     $mail->addAddress($email); 
     $mail->Username = "email@gmail.com"; 
     $mail->Password = "email_password"; 
     $mail->setFrom('email_Sent_from@gmail.com', 'Alias'); 
     $mail->addReplyTo("email_to@gmail.com", "Alias"); 
     $mail->Subject = $subject; 
     $mail->msgHTML($message); 
     $mail->send(); 
    } 
} 

その後、あなたはそれが好きで呼び出すことができます。

$email_send = new phpmailer(); 
$email_send->sendMail($user_email,$message,$subject);