JS 的5个不良编码习惯

匿名 (未验证) 提交于 2019-12-02 23:57:01

1.不要使用隐式类型转换

JavaScript是一种松散类型的语言。如果使用得当,这是一个好处,因为它给你带来了灵活性。

+-*/=====)在处理不同类型的操作数时会进行隐式转换。

ifcondition){...}whilecondition){...}隐式地将条件转换为布尔值。

下面的示例依赖于类型的隐式转换,这种有时候会让人感到很困惑:

  1. console.log("2" + "1"); // => "21"

  2. console.log("2" - "1"); // => 1

  3. console.log('' == 0); // => true

  4. console.log(true == []); // -> false

  5. console.log(true == ![]); // -> false

过度依赖隐式类型转换是一个坏习惯。首先,它使你的代码在边缘情况下不太稳定。其次,增加了引入难以重现和修复的bug的机会。

现在咱们实现一个获取对象属性的函数。如果属性不存在,函数返回一个默认值

  1. function getProp(object, propertyName, defaultValue) {

  2. if (!object[propertyName]) {

  3. return defaultValue;

  4. }

  5. return object[propertyName];

  6. }

  7. const hero = {

  8. name: 'Batman',

  9. isVillian: false

  10. };

  11. console.log(getProp(hero, 'name', 'Unknown')); // => 'Batman'

getProp()name'Batman'

isVillian属性:

  1. console.log(getProp(hero, 'isVillian', true)); // => true

heroisVillianfalsegetProp()true

if(!object[propertyName]){...}隐式转换的布尔值。

这些错误很难发现,要修复该函数,就要明确验证值的类型:

  1. function getPropFixed(object, propertyName, defaultValue) {

  2. if (object[propertyName] === undefined) {

  3. return defaultValue;

  4. }

  5. return object[propertyName];

  6. }

  7. const hero = {

  8. name: 'Batman',

  9. isVillian: false

  10. };

  11. console.log(getPropFixed(hero, 'isVillian', true)); // => false

object[propertyName]===undefinedundefined

undefined。因此,上述解决方案可以进一步改进:

  1. function getPropFixedBetter(object, propertyName, defaultValue) {

  2. if (!(propertyName in object)) {

  3. return defaultValue;

  4. }

  5. return object[propertyName]

  6. }

原谅作者建议是:尽可能不要使用隐式类型转换。相反,请确保变量和函数参数始终具有相同的类型,必要时使用显式类型转换。

最佳实践列表:

  • ===进行比较

  • ==

  • operand1+operand2:两个操作数应该是数字或字符串

  • -*/%**:两个操作数都应该是数字

  • ifcondition){...}whilecondition){...}condition

你可能会说这种方式需要编写更多代码......你是对的!但是通过明确的方法,可以控制代码的行为。此外,显性提高了可读性。

2. 不要使用早期的JavaScript技巧

JavaScript的有趣之处在于,它的创建者没有料到这种语言会如此流行。

基于JavaScript构建的应用程序的复杂性比语言发展的速度还要快。这种情况迫使开发人员使用JavaScript技巧和变通方法,只是为了让事情正常运行。

array.indexOfitem)!==-1来检查。

ES6 及以后版本的功能要强大得多,可以使用新的语言特性安全地重构许多技巧。

array.includes(item)array.indexOf(item)!==-1

3. 不要污染函数作用域

在ES2015之前,你可能会养成了将所有变量声明在函数作用域里面。

来看看一个例子:

  1. function someFunc(array) {

  2. var index, item, length = array.length;

  3. /*

  4. * Lots of code

  5. */

  6. for (index = 0; index < length; index++) {

  7. item = array[index];

  8. // Use `item`

  9. }

  10. return someResult;

  11. }

indexitemlengthfor()块作用域内才被需要。

letconst,应该尽可能地限制变量的生命周期。

  1. function someFunc(array) {

  2. /*

  3. * Lots of code

  4. */

  5. const length = array.length;

  6. for (let index = 0; index < length; index++) {

  7. const item = array[index];

  8. // Use `item`

  9. }

  10. return someResult;

  11. }

indexitemfor()循环块作用域。length 被移动到使用地方的附近。

重构后的代码更容易理解,因为变量不会分散在整个函数作用域内,它们存在于使用地方的附近。

在使用的块作用域定义变量

if 块作用域

  1. // 不好

  2. let message;

  3. // ...

  4. if (notFound) {

  5. message = 'Item not found';

  6. // Use `message`

  7. }

  8. // 好

  9. if (notFound) {

  10. const message = 'Item not found';

  11. // Use `message`

  12. }

for 块作用域

  1. // 不好

  2. let item;

  3. for (item of array) {

  4. // Use `item`

  5. }

  6. // 好

  7. for (const item of array) {

  8. // Use `item`

  9. }

4.尽量避免 undefined 和 null

undefined。例如

  1. let count;

  2. console.log(count); // => undefined

  3. const hero = {

  4. name: 'Batman'

  5. };

  6. console.log(hero.city); // => undefined

countundefined

hero.cityundefined

undefinedundefined进行比较时,你正在处理未初始化状态的变量。

变量、对象属性和数组在使用前必须用值初始化

undefined进行比较方式。

判断属性是否存在

  1. // 不好

  2. const object = {

  3. prop: 'value'

  4. };

  5. if (object.nonExistingProp === undefined) {

  6. // ...

  7. }

  8. // 好

  9. const object = {

  10. prop: 'value'

  11. };

  12. if ('nonExistingProp' in object) {

  13. // ...

  14. }

对象的默认属性

  1. // 不好

  2. function foo(options) {

  3. if (object.optionalProp1 === undefined) {

  4. object.optionalProp1 = 'Default value 1';

  5. }

  6. // ...

  7. }

  8. // 好

  9. function foo(options) {

  10. const defaultProps = {

  11. optionalProp1: 'Default value 1'

  12. };

  13. options = {

  14. ...defaultProps,

  15. ...options

  16. }

  17. }

默认函数参数

  1. // 不好

  2. function foo(param1, param2) {

  3. if (param2 === undefined) {

  4. param2 = 'Some default value';

  5. }

  6. // ...

  7. }

  8. // 好

  9. function foo(param1, param2 = 'Some default value') {

  10. // ...

  11. }

nullnullnull作为参数调用函数。

nullnull的函数中检查它的存在,这很容易出错。

  1. function bar(something) {

  2. if (something) {

  3. return foo({ value: 'Some value' });

  4. } else {

  5. return foo(null);

  6. }

  7. }

  8. function foo(options) {

  9. let value = null;

  10. if (options !== null) {

  11. value = options.value;

  12. // ...

  13. }

  14. return value;

  15. }

nulltry/catch机制,默认对象的使用。

5. 不要使用随意的编码风格,执行一个标准

有什么比阅读具有随机编码风格的代码更令人生畏的事情?你永远不知道会发生什么!

如果代码库包含许多开发人员的不同编码风格,该怎么办?,这种就像各色人物涂鸦墙。

整个团队和应用程序代码库都需要相同的编码风格,它提高了代码的可读性。

一些有用的编码风格的例子:

  • Airbnb JS 风格指南

  • 谷歌 JS 风格指南

老实说,当我在回家前准备提交时,我可能会忘记设计代码的样式。

我自己总说:保持代码不变,以后再更新它,但是“以后”意味着永远不会。

这里建议使用 eslint 来规范编码风格。

  1. 安装eslint

  2. 使用最适合自己的编码风格配置 eslint

  3. 设置一个预提交钩子,在提交之前运行eslint验证。

总结

编写高质量和干净的代码需要纪律,克服不好的编码习惯。

undefinednull

现在这种语言发展得相当快。找出复杂的代码,并使用最新 JS 特性来重构。

整个代码库的一致编码风格有益于可读性。良好的编程技能总是一个双赢的解决方案。

易学教程内所有资源均来自网络或用户发布的内容,如有违反法律规定的内容欢迎反馈
该文章没有解决你所遇到的问题?点击提问,说说你的问题,让更多的人一起探讨吧!