zoukankan      html  css  js  c++  java
  • 代码顺序的重要性

    背景

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

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

    错误模拟

    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,抛出错误导致程序中断,赋值语句没有执行
    执行结果

    正确的写法

    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. 如果出现这种傻瓜式错误,是该考虑重构代码了。

    博客首发地址:https://www.jianshu.com/u/13cd86311525

  • 相关阅读:
    android之StrictMode介绍
    m3u8介绍
    Spring笔记3
    android之常用命令(未完待续)
    JAVA核心技术
    Struts2笔记2
    android之lint警告This Handler class should be static or leaks might occur
    [Algorithm]01分数规划——Update:2012年7月27日
    asp生成html静态
    图片放大缩小
  • 原文地址:https://www.cnblogs.com/chaohangz/p/9924663.html
Copyright © 2011-2022 走看看