可能被你忽略的 JavaScript 代码陷阱
来源:互联网 发布:创投公司 知乎 编辑:程序博客网 时间:2024/05/17 16:00
下面这段代码,你知道有哪些错误吗:
请仔细思考后再往下阅读。
—- 帮助你思考的刷屏线 开始 —-
—- 帮助你思考的刷屏线 结束 —-
1. isLive = config.isLive || true, 当传入的值有可能就是0, undefined, null, false, "", NaN这六个 falsy 值时,用 || 来设定默认值不妥当。更保险的做法是:
isLive = "isLive" in config ? config.isLive : true;
如果是独立变量,可以采用:
someVar = typeof someVar !== "undefined" ? someVar : defaultValue;
注意:大部分情况下,用 || 已经够用,比如:
container = container || document
name = config.name || "无名氏"
一切皆权衡。
2. var g_bar = g_bar || "", 原意是取全局变量 g_bar 的值给内部变量 g_bar, 默认为空字符串。然而,实际情况等价为:
var g_bar;
g_bar = g_bar || "";
很明显,|| 号左边的 g_bar 也是内部变量,并且为 undefined, 因此var g_bar = g_bar || ""实际上是var g_bar = "", 没有满足代码的原始意图。
思考:代码中的var container = container || document有无问题?为什么?
3. if(g_foo) { /* code */ }, 这段代码在执行时会报错。我们都知道在 JS 里,变量不定义就可以用。但一定要清楚,未定义的变量,仅仅是可写,但不可读。比如:
g_foo = 2; // 等价 window.g_foo = 2
var t = g_foo2; // 不等价为 var t = window.g_foo2, 会报错
具体原因可以参见 JavaScript 运行机制浅探:
未定义变量意味着在 scriptObject 的变量表中找不到,JS 引擎会沿着 scriptObject 的 upvalue 往上寻找,如果都没找到,对于写操作 i = 1; 最后就会等价为 window.i = 1; 给 window 对象新增了一个属性。对于读操作,如果一直追溯到全局执行环境的 scriptObject 上都找不到,就会产生运行期错误。
因此严谨的写法是:
if(window.g_foo) {
/* your code */
}
不要小看这些细微之处,有时会让人抓狂的。但这些细微之处又很容易被忽略或滥用。比如 YUI 2.8r4 里,有一个遗传了很久的 bug:
var NOTHING = [];
// ....
later: function(when, o, fn, data, periodic) {
when = when || 0;
o = o || {};
var m = fn, d = data, f, r;
// ...
if (d && !L.isArray(d)) {
d = [data];
}
f = function() {
m.apply(o, d || NOTHING);
};
// ...
}
当你的调用代码类似Lang.later(delay[0], o, "show", index)时,如果 index 不幸是 base-0 的,那么取 0 时,m.apply(o, d || NOTHING)会让你得到“惊喜”。更妥的做法是类似 YUI3 中的修正:
// ...
if (!L.isArray(d)) {
d = [data];
}?
f = function() {
m.apply(o, d);
};
//...
对于 || 和 && 的用法,很多 JS 书籍(无论中外),都用来片面强调 JS 的灵活性,包括 Douglas 的《JavaScript The Good Parts》中也存在误导。
最后,有感于 NCZ 今天写的 Writing Maintainable Code, 再举一例(和本文主题关系不明显,但的确又有关系,交给你去思考啰):
var isBoy = true;
isBoy = typeof isGirl !== "undefined" ? !isGirl : true;
或者来个耍酷的代码:
var isBoy = true;
(typeof isGirl !== "undefined") && (isBoy = !isGirl);
然而,以上两种写法,无论从代码长度还是性能上讲,都不如更直白的写法:
var isBoy = true;
if(typeof isGirl !== "undefined") isBoy = !isGirl;
简单质朴,往往是最好的。
转自:http://lifesinger.org/blog/2009/12/easy-ignorable-js-code/
- 可能被你忽略的 JavaScript 代码陷阱
- 可能被你忽略的TextView属性
- 细数 javascript 容易被忽略的语法陷阱
- 那些可能被你忽略的MySQL优化技巧
- java--你可能忽略的细节(一)
- java--你可能忽略的细节(二)
- php你可能忽略的小姿势
- Android 你可能忽略的提高敲代码效率的方式
- [乐意黎转载]细数 javascript 容易被忽略的语法陷阱
- VS 一些你可能忽略但是用着很方便的快捷键
- 关于推送,你可能忽略的那些事儿
- 数据库的使用你可能忽略了这些
- Duilib 一些可能被忽略的问题
- PHP一些可能被忽略的基础知识
- php,你可能不知道的“死亡”陷阱
- 你可能不知道的陷阱, IEnumerable接口
- scanf可能遇到的陷阱
- 制作flash可能用到的javascript代码
- ubuntu安装手柄驱动玩游戏
- 笔试题小结
- 导致sendto执行的时间过长的原因
- 工作两年的一点点感想
- Android操作系统第三季在美市场份额达44%
- 可能被你忽略的 JavaScript 代码陷阱
- 类的关系
- 超市VIP卡管理系统
- 几个分页存储过程
- CSDN上一篇值得深思的文章
- jsp表头固定
- ORG指令理解
- 工具vba
- PHP返回JS是JSON格式的字符串,转换为对象