你会用OOP PHP以相同的方式编写这段代码吗？

Question

我正试着用我的一个脚本去oop.这是一个联系脚本,通过jQuery的ajax函数发送电子邮件时处理编码.

我想让用户能够在同一页面中使用相同的脚本和两个表单,并使其成为一项简单的工作.

现在我已经制作了一个原型,它将如何用oop重写它.

这对我来说真的很困惑,但我每天都在学习.对我来说最困难的部分是我应该在哪里放置我的方法以及如何制作脚本流程.

为了演示我的意思,这里是我现在使用的代码的一些部分:

/*
 * Start defining some vars at the runtime
 */
public function __construct($data, $config = array()) {
    $lang = isset($config['language']) ? $config['language'] : 'en';
    $this->_getPhrases($lang);
    $this->_recieverEmail = isset ($config['reciever_email']) ? filter_var($config['reciever_email'], FILTER_SANITIZE_EMAIL) : die($this->_phrase['noRecieverEmail']);
    $this->_ajax = ($this->_enableAjax($config['ajax_enabled'])) ? true : false;
    $this->_data = isset($data) ? (is_array($data) ? $data : die($this->_phrase['errors']['dataNotArray'])) : $_POST;
}

/*
 * Send the message
 */
public function send() {
    if(!$this->isDataVaild($this->_data)) {
        return false;
    }

    $this->_data = $this->_cleanData($this->_data);
    $this->setSenderName($this->_data['name']);
    $this->setSenderEmail($this->_data['email']);
    $this->_message = $this->_generateMsg($this->data);

    $PHPMailer = new PHPMailerLite();
    $this->_sendUsing($PHPMailer, $this->_message);

    return true;
}

我特意选择了这两种方法,因为它们正在为我的脚本完成大部分工作.我用这种方式:

$config = array(
    'language' => 'en',
    'ajax_enabled' => false,
    'reciever_email' => 'recieve@localhost'
);

$contact = new coolContact($_POST, $config);

if($contact->send()) {
    echo 'Message sent with No problems';
} else {
    echo $contact->getErrors();
}

毕竟,这是我的问题:

我的问题

1. 我应该在send()方法内部还是在内部进行验证_generateMsg()？
2. 这段代码可以考虑oop php吗？

问题1对某些人来说可能很奇怪,所以让我解释一下:

在用我认为oop重写代码之后,我现在可以在不破坏代码的情况下使用许多顺序中的方法,这就是为什么我很困惑何时何地最好的地方使用它.

Answer 1

如果我是你，我会将验证放在它自己的方法中validate()。这样您就可以从任何函数调用该代码。尝试使每个方法尽可能简短和简洁，并尽可能将方法组合成更大的方法。看起来您可能已经对其自己的函数进行了验证，在这种情况下，您调用它的位置取决于逻辑中需要验证数据的位置。如果 acoolContact可以存在无效数据，但在数据更正之前无法发送，则在 中进行验证send()。如果一个coolContact必须有有效数据，期间，它永远不能存在无效数据，那么_generateMsg()应该调用validate()

至于#2，我会考虑这个 OOP，是的。但我在这里的回答都是意见......