放牧代码和思想
专注自然语言处理、机器学习算法
    This thing called love. Know I would've. Thrown it all away. Wouldn't hesitate.

论ECMall的粗糙

只举一个例子,说明ECMall表面的光鲜之后,隐藏着多么粗糙的内核。

ECMall中除了一个BaseApp类之外所有的App类都是ECBaseApp的子类,ECBaseApp是ECMall中举足轻重的一个类,它的父类是BaseApp,BaseApp往上在没有类了,也就是说ECBaseApp类是二把手。但是这个最核心的类的构造函数是这样写的:

function ECBaseApp()
    {
        echo "ECBaseApp".'<br>';
        parent::__construct();

        if (!defined('MODULE')) // 临时处理方案,此处不应对模块进行特殊处理
        {
            /* GZIP */
            if ($this->gzip_enabled())
            {
                ob_start('ob_gzhandler');
            }
            else
            {
                ob_start();
            }

            /* 非utf8转码 */
            if (CHARSET != 'utf-8' && isset($_REQUEST['ajax']))
            {
                $_FILES = ecm_iconv_deep('utf-8', CHARSET, $_FILES);
                $_GET = ecm_iconv_deep('utf-8', CHARSET, $_GET);
                $_POST = ecm_iconv_deep('utf-8', CHARSET, $_POST);
            }

            /* 载入配置项 */
            $setting =& af('settings');
            Conf::load($setting->getAll());
            echo "conf".'<br>';

            /* 初始化访问者(放在此可能产生问题) */
            $this->_init_visitor();

            /* 计划任务守护进程 */
            $this->_run_cron();
        }
    }

两处echo是我为了调试加上去的,看到那句// 临时处理方案,此处不应对模块进行特殊处理的时候,我简直惊呆了!在正式发布的程序里居然还有临时处理方案。好吧,算你行。

知识共享许可协议 知识共享署名-非商业性使用-相同方式共享码农场 » 论ECMall的粗糙

评论 欢迎留言

  • 昵称 (必填)
  • 邮箱 (必填)
  • 网址

我的作品

HanLP自然语言处理包《自然语言处理入门》