首页
学习
活动
专区
圈层
工具
发布
首页
学习
活动
专区
圈层
工具
MCP广场
社区首页 >专栏 >代码顺序的重要性

代码顺序的重要性

作者头像
超超不会飞
发布2020-09-18 10:14:08
发布2020-09-18 10:14:08
57900
代码可运行
举报
文章被收录于专栏:超超不会飞超超不会飞
运行总次数:0
代码可运行

背景

在修改前辈留下来的代码时踩到一个坑。简单讲一下代码做了什么事:在 try/catch 里实例化多个类,并调用类的方法为其赋值,catch 到错误也不会中断,而是通过打日志的形式收集报错信息。

我需要在里面新实例化一个类,有样学样,上来就是一顿copy改,可是新增的类的某个属性总是不对,排查了半天才发现一个坑。

错误模拟

代码语言:javascript
代码运行次数:0
运行
复制
class Point {
  constructor(x, y) {
    this.x = x
    this.y = y
  }

  getX() {
    return this.x
  }

  setX(x) {
    this.x = x
  }
}

let p, cat
try {
  p = new Point(1, 2)

  cat = new Cat('胖虎')  // 这里实例化了一个不存在的类,用来模拟报错,实际情况比这个复杂

  p.setX(66)  //  这一行应该写在 `cat` 的前面,不然 `cat` 报错后直接中断下面语句的执行,导致这行语句没有执行

} catch (e) {
  console.log(e)
}

console.log(p.getX())

执行结果,x 没有被赋值成66,抛出错误导致程序中断,赋值语句没有执行

正确的写法

代码语言:javascript
代码运行次数:0
运行
复制
class Point {
  constructor(x, y) {
    this.x = x
    this.y = y
  }

  getX() {
    return this.x
  }

  setX(x) {
    this.x = x
  }
}

let p, cat
try {
  p = new Point(1, 2)

  p.setX(66)  // 方法调用紧随实例后面,不受下面代码的影响

  cat = new Cat('胖虎')

} catch (e) {
  console.log(e)
}

console.log(p.getX())

执行结果,x 正确被赋值,错误也正常抛出

总结

  1. 如果 catch 了错误又不抛出,可以在调试的时候把错误抛出,方便发现问题。
  2. 代码结构顺序和整洁性很重要。
  3. 如果出现这种傻瓜式错误,是该考虑重构代码了。
本文参与 腾讯云自媒体同步曝光计划,分享自作者个人站点/博客。
原始发表:2018-11-07 ,如有侵权请联系 cloudcommunity@tencent.com 删除

本文分享自 作者个人站点/博客 前往查看

如有侵权,请联系 cloudcommunity@tencent.com 删除。

本文参与 腾讯云自媒体同步曝光计划  ,欢迎热爱写作的你一起参与!

评论
登录后参与评论
0 条评论
热度
最新
推荐阅读
目录
  • 背景
  • 错误模拟
  • 正确的写法
  • 总结
领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档