Question

这里是我的项目的示例代码。我有一个个人作品集，主要是静态页面，运行这个简单的PHP脚本来验证表单并将电子邮件发送给管理员。

if ($_SERVER['REQUEST_METHOD'] === 'POST') {

    $flag = $_POST['FLAG'];
    $name = trim(stripcslashes($_POST['name']));
    $email = trim(stripcslashes($_POST['email']));
    $message = trim(stripcslashes($_POST['message']));
    $success = false;

    $mail = new Mail($flag, $name, $email, $message, $config);   

    $result = $mail->validateFields();

    if($result) {
        $errors = $result;
    } elseif ($result == false) {
        $mail->sendEmail();
        $success = true;
    }    
}

有什么想法可以改善我的代码吗？
我正在使用swiftmailer和第3方MTA（mailgun）。
MAIL类具有2种方法：Validate和Send。
validate方法用于验证字段，如果未发现错误，则返回false，否则返回包含错误消息的数组。
发送邮件正在使用swiftmailer发送电子邮件。任何想法如何使一切变得更好？我应该更改某些东西还是可以吗？

class Mail
{
public $flag = null;
public $name = null;
public $email = null;
public $message = null;
public $config = null;
public $errors = [];

function __construct($flag, $name, $email, $message, $config)
{
    $this->flag = $flag;
    $this->name = $name;
    $this->email = $email;
    $this->message = $message;
    $this->config = $config;

}

public function validateFields()
{   
    if (!empty($this->flag)) {
    $this->errors[] = 'SPAM FILTER ERROR =)';
    }

    if ( empty($this->name) | !preg_match('/^[a-zA-Z\s]+$/', $this->name)) {
        $this->errors[] = 'Enter valid name';
    }

    if ( empty($this->email) | !filter_var($this->email, FILTER_VALIDATE_EMAIL)) { 
        $this->errors[] = 'Enter valid email address';
    }

    if ( empty($this->message) ) {
        $this->errors[] = 'Please enter a message';
    }

    if( empty($this->errors) ) {
        return false;

    } else {
        return $this->errors;
    }
}

public function sendEmail()
{
    $transport = (new Swift_SmtpTransport("{$this->config['SMTP']}", 25))
        ->setUsername("{$this->config['USERNAME']}")
        ->setPassword("{$this->config['PASSWORD']}");

    /// rest of email logic
}
}

任何建议将不胜感激！

谢谢。

表单验证PHP清理

0 个答案: