【问题标题】:Would you write this code the same way using OOP PHP?您会使用 OOP PHP 以同样的方式编写此代码吗?
【发布时间】:2010-08-27 21:57:49
【问题描述】:

我正在尝试使用我的一个脚本进行 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 的方式重写代码后,我现在可以在不破坏代码的情况下以多种顺序使用方法,这就是为什么我很困惑何时何地使用它的最佳位置。

【问题讨论】:

    标签: php oop coding-style


    【解决方案1】:

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

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

    【讨论】:

    • 谢谢乔希,非常感谢您的意见。
    【解决方案2】:

    正如您询问是否将此代码视为 OOP php 代码一样,我想分享一下我使用 OOP php 的方式。

    我创建带有 .class 扩展名的单独类文件。这个类文件大致有如下结构。

        class classname
        {
        var $var1;  //defining variables
        var $var2;
        .
        .
        var $varn;
    
        public function __construct($parm1,$param2,....$paramn)
        {
        //intialise variables
    
        $this->var1=$param1;
        .
        .
        $this->varn=$paramn;
        }
    
        //give definitions of all the functions here
    
        public function functionname($param1,$param2...$paramn)
    
        $city=$param1;
        $country=$param2;
        {
        /*include database connection file(created as db_conf) or explicitly write connection code(not preferable)*/
    
        //give function definition, manipulation, expresstions and sql queries
    
        $query1="INSERT INTO city(name,country) VALUES('$city','$country')";
        $result=mysql_query($query1);
    
    return value;
        }
    
        //similarly define other functions too
    
    
        }
    

    现在,如果您想对两个或多个表单使用相同的脚本,您需要在其中包含类文件并调用适当的方法。例如

    <?php
    include_once './classname.class.php'  //give complete path of the class file.
    $obj=new classname($par1,...,$parn); //create object
    
    $ret=$obj->functionname($par1,...,$parn);
    
    ?>
    

    我希望这会有所帮助:)

    【讨论】:

    • 谢谢Anurag,我实际上打算按照您建议我的方式使用我的课程。我将为每个表单创建一个新对象,它会有自己的选项,以解决很多问题。
    猜你喜欢
    • 2023-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 1970-01-01
    • 2013-04-07
    • 1970-01-01
    • 1970-01-01
    相关资源
    最近更新 更多