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之后也正常，但在后期维护时很容易导致问题。习惯成自然，养成的习惯都不好改，不一定害己但一定害人～

当你能梦的时候就不要放弃梦

相关文章：

你感兴趣的文章：

标签云：