Discuz 7.2坑爹集锦-js篇_PHP_编程开发_程序员俱乐部

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;

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

类型：??? ??? caozuofu.html" target="_blank">操作符顺序
坑爹指数：??? ★
代码：??? ??? 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还是很宽容的，这两个小毛病不会导致啥问题。

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

类型：??? ??? 代码规范
坑爹指数：??? ☆
代码：??? ???
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之后也正常，但在后期维护时很容易导致问题。习惯成自然，养成的习惯都不好改，不一定害己但一定害人～



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