Discuz 7.2坑爹集锦-js篇

Discuz 7.2坑爹集锦-js篇

在调试DZ过程中firebug经常提示common.js出错，什么“Error: s is null”，“$ is not defined”，“BROWSER not defined”。其实多半和XML有关：当ajax操作时后台PHP出现错误（包括notice）导致前台JS在解析期望中XML格式返回值时出错。也许是DZ的ajax调用对返回值处理不够完善。解决办法是编辑php.ini设置display_error = off。
DZ的js文件没细看，主要是解决几个调试过程中遇到兼容性问题。

-------------------------------------------------------------------------------------------------------

类型：        过时函数
坑爹指数：    ★
代码：        include/js/common.js 10-14
BROWSER.ie = window.ActiveXObject && USERAGENT.indexOf('msie') != -1 && USERAGENT.substr(USERAGENT.indexOf('msie') + 5, 3);
BROWSER.firefox = document.getBoxObjectFor && USERAGENT.indexOf('firefox') != -1 && USERAGENT.substr(USERAGENT.indexOf('firefox') + 8, 3);
BROWSER.chrome = window.MessageEvent && !document.getBoxObjectFor && USERAGENT.indexOf('chrome') != -1 && USERAGENT.substr(USERAGENT.indexOf('chrome') + 7, 10);
BROWSER.opera = window.opera && opera.version();
BROWSER.safari = window.openDatabase && USERAGENT.indexOf('safari') != -1 && USERAGENT.substr(USERAGENT.indexOf('safari') + 7, 8);
点评：        测试过程中某天发现FF下登录窗口账户名只能选择用户名，而在chrome下面可选UID及email。当时一直没找到原因，某天测试时修改到此处，顺便用slice()替换substr()发现FF的登录窗问题正常了。猛然想起以前看过对于substr的一个注解：“ECMAscript 没有对该方法进行标准化，因此反对使用它”。在Firefox8以前都没这个问题，看来FF开始放弃对此函数的兼容了。另外Firefox9也不支持document.getBoxObjectFor对象了。
FIX：   
BROWSER.ie = window.ActiveXObject && USERAGENT.indexOf('msie') != -1 && USERAGENT.slice(USERAGENT.indexOf('msie') + 5, USERAGENT.indexOf('msie') + 8) || false;
BROWSER.firefox = USERAGENT.indexOf('firefox') != -1 && USERAGENT.slice(USERAGENT.indexOf('firefox') + 8, USERAGENT.indexOf('firefox') + 11) || false;
BROWSER.chrome = window.MessageEvent && !document.getBoxObjectFor && USERAGENT.indexOf('chrome') != -1 && USERAGENT.slice(USERAGENT.indexOf('chrome') + 7, USERAGENT.indexOf('chrome') + 17) || false;
BROWSER.opera = window.opera && opera.version() || false;
BROWSER.safari = window.openDatabase && USERAGENT.indexOf('safari') != -1 && USERAGENT.slice(USERAGENT.indexOf('safari') + 7, USERAGENT.indexOf('safari') + 15) || false;

-------------------------------------------------------------------------------------------------------

类型：        操作符顺序
坑爹指数：    ★
代码：        include/js/common.js=1112
function ajaxget(url, showid, waitid, loading, display, recall) {
    waitid = typeof waitid == 'undefined' || waitid === null ? showid : waitid;
    ...
}
点评：        怎么看这个赋值表达式也没问题，可是当minify之后就时不时会在这儿出错，提示“Uncaught ReferenceError: $ is not defined”，用小括号包裹之后就正常了。也蛮奇怪的……
FIX：        waitid = (typeof waitid == 'undefined' || waitid === null) ? showid : waitid;

-------------------------------------------------------------------------------------------------------

类型：        代码规范
坑爹指数：    ☆
代码：        include/js/common.js=834
    if($('fwin_dialog_submit')) $('fwin_dialog_submit').onclick = function() {
        if(typeof func == 'function') func();
        else eval(func);
        hideMenu(menuid, 'dialog')
    };
点评：        缺少分号结尾，跑到if判断后去了。不过js还是很宽容的，这两个小毛病不会导致啥问题。

 

ADD：有朋友说修正分号 后 反而会导致发帖编辑器上按钮（图片、附件等）全部失效了。此处请谨慎修改

-------------------------------------------------------------------------------------------------------

类型：        代码规范
坑爹指数：    ☆
代码：       
include/js/common.js=1140
     if(x.autogoto) scroll(0, x.showId.offsetTop);
include/js/common.js=1150
    if(!evaled) evalscript(s);
include/js/common.js=1199
    if(!evaled) evalscript(s);
include/js/common.js=1234
     if(isUndefined(timeout)) timeout = 3000;
    if(isUndefined(cache)) cache = 1;
    if(isUndefined(pos)) pos = '43';
    if(isUndefined(duration)) duration = timeout > 0 ? 0 : 3;
include/js/common.js=1551
    if(tagname == 'INPUT' || tagname == 'TEXTAREA') return;
include/js/common.js=1651
    if(lasttop == -1) lasttop = 0;
include/js/common.js=1841
     if(isUndefined(parsecode)) parsecode = true;
                if(parsecode) str= str.replace(/\s*\[code\]([\s\S]+?)\[\/code\]\s*/ig, function($1, $2) {return codetag($2);});
include/js/common.js=1860
    if(typeof wysiwyg != 'undefined' && wysiwyg) text = text.replace(/<br[^\>]*>/ig, '\n').replace(/<(\/|)[A-Za-z].*?>/ig, '');
include/js/common.js=835
    if($('fwin_dialog_submit')) $('fwin_dialog_submit').onclick = function() {
        if(typeof func == 'function') func();        // <----
        else eval(func);                    // <----
        hideMenu(menuid, 'dialog');
    }       
点评：        看来common.js是同一位大猿写的，代码不喜欢用大括号包裹条件句。虽然没有错误，minify之后也正常，但在后期维护时很容易导致问题。习惯成自然，养成的习惯都不好改，不一定害己但一定害人～



版权曾经拥有，欢迎网上分享

评论

2 楼 waiting 2012-02-10  

liu1084 写道

点评：        看来common.js是同一位大猿写的，代码不喜欢用大括号包裹条件句。虽然没有错误，minify之后也正常，但在后期维护时很容易导致问题。习惯成自然，养成的习惯都不好改，不一定害己但一定害人～

未必啊，也可能是刚刚毕业的人写的东西

新人往往按照规范来写，以前那时候规范不严格也不做minify啥的不会出问题所以容易养成在现在看来不规范的代码风格

1 楼 liu1084 2012-02-07  

点评：        看来common.js是同一位大猿写的，代码不喜欢用大括号包裹条件句。虽然没有错误，minify之后也正常，但在后期维护时很容易导致问题。习惯成自然，养成的习惯都不好改，不一定害己但一定害人～

未必啊，也可能是刚刚毕业的人写的东西