欢迎任何建议! 我对 php 类的了解非常有限，但下面是我想走的路线的起点。该代码反射(reflect)了我在脑海中看到的内容以及我希望如何开展业务。我的代码看起来还好吗，还是我离题太远了？

您有什么想法，您将如何完成表单->验证->插入查询->发送邮件->返回消息和错误这样的任务？

请尽量让你的回答足够简单，让我能够消化，对我来说，这是关于理解正在发生的事情，而不仅仅是复制/粘贴工作。

最诚挚的问候， 菲尔。

注意:这只是一个基础结构，没有添加完整的代码。

<?php
//=======================================
    //class.logging.php
//========================================
    class logging
    {

        public $data = array();
        public $errors = array();



        function __construct()
        {
            array_pop($_POST);
            $this->data =($this->_logging)? is_isset(filterStr($_POST) : '';

            foreach($this->data as $key=> $value)
            {
                $this->data[$key] = $value; 
            }
            //print_r($this->data); de-bugging
        }

        public function is_isset($str)
        {
            if(isset($str)) ? true: false;
        }

        public function filterStr($str)
        {
            return preg_match(do somthing, $str);
        }

        public function validate_post()
        {
            try
            {
                if(!is_numeric($data['cardID'])) ? throw new Exception('CardID must be numeric!') : continue;   
            }
            catch (Exception $e)
            {

                return $errors = $e->getCode();
            }
        }

        public function showErrors()
        {
            foreach($errors as $error => $err)
            {
                print('<div class="notok"></div><br />');
            }
        }

        public function insertQ()
        {
            $query = "";
        }

    }


//=======================================
    //Usercp.php
//========================================
if(isset($_GET['mode']))
     {
     $mode = $_GET['mode'];
     }
     else
     {
     $mode = 'usercp';
     }


     switch($mode)
    {
        case 'usercp':
        echo 'Welcome to the User Control Panel';
        break;

        case 'logging':
            require_once 'class.logging.php';
            $logger = new logging();

            if(isset($_POST['submit'])
            {
                if($logger->validate_post === true)
                {
                    $logger->insertQ();
                    require_once '/scripts/PHPMailer/class.phpmailer.php';
                    $mailer = new PHPMailer();
                    $mailer->PHPMailer();
                }
                else
                {
                    echo ''.$logger->showErrors.''; 
                }
            }
            else
            {
                echo 
                    '
                    <form action="'.$_SERVER['PHP_SELF'].'?mode=logging" method="post">

                    </form>
                    ';  
            }
        break;

        case 'user_logout':
        // do somthing
        break;

        case 'user_settings':
        // do somthing
        break;






?>

我决定使用此方法返回错误而不是在方法中打印它们，感谢 Igor 的建议!

catch (Exception $e)
            {
                $this->errors[] = $e->getMessage();

            #ERROR DE_BUGGING ONLY================================
            #print('<pre>');
                    #print_r($this->errors); 
            #print('</pre>');
            #=====================================================
            }
            if($this->errors)
            {
                return false;
            }
            else
            {
                return true;    
            }

最佳答案

看来您对 OOP 代码有一定的了解。我看到声明的公共(public)变量甚至 try/catches，虽然我会说不要忘记“function __construct()”前面的“public”可见性关键字——这不是绝对必要的，但它符合良好的编码习惯。

此外，我要说的是，您在这里所做的一切都已经过编写、调试和修复，并且已被数十个 PHP 框架中的每一个框架证明具有生产值(value)。您提到的具体任务，“表单->验证->插入查询->发送邮件->返回消息和错误”对于我选择的框架 Zend Framework 来说非常容易。我想 Symphony、Solar、Cake 等也是如此。

帮自己一个忙，停止编码已经编码的内容。学习具有社区、定期更新和精心编写的详尽文档的框架。再次，我推荐Zend Framework .

关于php - 对我的第一个项目 PHP 类的建议/建议，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/2826973/