代码顺序的重要性

发表于 | 阅读 128

背景

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

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

错误模拟

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
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，抛出错误导致程序中断，赋值语句没有执行
执行结果

正确的写法

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
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. 如果出现这种傻瓜式错误，是该考虑重构代码了。