Проверка формы PHP очищает - PullRequest
Новая
       9

Проверка формы PHP очищает

0 голосов
/ 10 февраля 2019

Вот пример кода моего проекта.У меня есть личное портфолио, которое в основном представляет собой статическую страницу, на которой выполняется этот простой 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 и сторонний 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
}
}

Любые предложения будут очень благодарны!

Спасибо.

...