💻 代码思考

    💻 代码思考 存在多个指针类型 out 函数参数,特别小心的是当其中一个out参数取值错误,其他out参数的赋值情况。

    比如

    TestOutClass* out1 = nullptr;
    TestOutClass2* out2 = nullptr;
    bool Getxxx(TestOutClass** ptr, TestOutClass2** ptr2){
    xxx
    }
    auto ret = Getxxx(&out1, &out2);

    当返回值为false的时候,理论上out1 和 out1 仍然是nullptr,外部不需要释放任何资源。

    因此在 Getxxx 函数内部实现的时候,可能会出现ptr 赋值成功,ptr2 赋值失败,此时返回值为false,导致外部没有意识到需要释放out1 的空间导致内存泄露。因此在内部实现的时候,应该等ptr 和ptr2 的内容都获取到后,再给两个指针赋值。

    💻 代码思考 好的注释应该是解释性语言,而不是动作性语言。具体例子:

      // |json_file_path_| is the path of a file that will be source of the
      // deserialization. |options| is a bitmask of JSONParserOptions.
      explicit JSONFileValueDeserializer(
          const base::FilePath& json_file_path,
          int options = base::JSON_PARSE_CHROMIUM_EXTENSIONS);

    上面代码是chromium base 库中关于json 解析工具的注释,它解释了两个参数的作用。

    一个不好的写法如下,对参数的注释是这个函数在做什么,而不是参数本身的说明:

    // We parse the file with the |json_file_path| path, passing in the 
    // bitmask of  JSONParserOptionstype |options| in the process
     explicit JSONFileValueDeserializer(
          const base::FilePath& json_file_path,
          int options = base::JSON_PARSE_CHROMIUM_EXTENSIONS);

    💻 代码思考 以往工作中常用的设计模式只有代理模式、观察者两种,似乎不太需要太复杂的设计模式。这一周工作主要是重构了之前写的一部分代码来支持更通用化的需求。因为这个需求中设计到很多状态的转换,每个状态都会有几个分支,尝试了状态设计模式,设计状态,设计每个状态会执行的动作,设计每个状态执行相应动作应该转移到的下一个状态以及副作用行为。Xstate 项目的可视化非常方便提前先构建好整个状态机的模型。

    写完之后,能很明显的感觉代码思路清晰很多。之前是在很多基础模块中去判断状态,然后进行对应的行为,这次所有的状态管理全部收敛到一个controller中,状态的转移过程对外是不可见的,外部基础模块只要调用controller的对应动作方法,controller会将工作委托给对应的状态处理。不再需要像之前那样,先获取当前状态,再判断当前状态,最后执行某个行为转移到新的状态。

    同时一个独立的功能需求的代码设计是不应该嵌入到一些基础类型的模块中的,而是应该写一个controller,将需要使用的基础模块作为该controller的成员指针来使用。

    💼 从工作中学到的 💻 代码思考在正式工作之前,我没有写过单测,主要是平时自己写的代码通常耦合度较高,没有单独抽象设计模块的概念。

    最近开始写一些通用能力的基础库需要写单测。第一次单测的编写是在代码基本完成后开始写的,完成单测后会发现一些bug,于是修改bug。但这个过程中会发现代码设计的一些不合理之处,比如多个接口的返回值是否更统一,对于接口可能出现错误时的返回值,应该如何处理(这个可以多参考chromium base库中的代码实现)。如果一个模块的多个接口设计不统一、那么单测同样也会非常复杂。

    其实上面的过程就有点TDD那味了,但是测试驱动开发,需要先写一个无法通过的测试,通过修改代码来使得测试通过,再重构代码。在这个过程中不断的开发代码,而不是先写完代码,再写单测试。

    单测的重要性是毋庸置疑,因为代码复杂性总可能出现某个分支逻辑错误。因此TDD将单测与开发结合,减轻了写单测的难度和压力,同时也能通过单测进一步发现代码的设计问题。

    参考文章:

联系方式

关于我

  • 来自南部的一个小城市,个性不张扬,讨厌随波逐流。

那年今日
3 月前

📝 每日记录 口腔溃疡下前两周长了两个,好的差不多了。接着左边和右边又各长一个... 感觉最近上火有点厉害。今天看了两个电影 🎬 观影 。一部是《哆啦A梦:大雄的猫狗时空》,这是一部2004年的剧场版。就算放到现在也没有感觉到违和。小狗狗太可爱了!一个时空穿梭带来的猫、狗大战的故事。另一部电影是《非常主播》。一直会看到的表情包,终于看完这部电影。虽然没有什么深刻的立意,不过喜剧片就不要要求那么严格了。女一号和女二号都好漂亮,表情包中的萌娃当然就是很萌的。一个30多岁的大叔有一个20多岁的女儿和孙子。而这个大叔还是有点小有名气的电台主播。剧情围绕大叔不想被曝光这件事情,从而产生的各种剧情冲突。总体还是不错的。

7 月前

📝 每日记录 虽然现在云文档工具越来越多,今天工作的时候需要写一个对外的文档,习惯了用飞书云文档的排版(如多栏、复杂表格内容等),但是如果对外发布(如公众号、社交平台、博客等)还是markdown的格式更通用,因此准备对外的文档还是写纯markdown吧。

9 月前

我的劲椎感觉好痛,甚至连我的左半边手就开始痛了....

12 月前

HomePod mini真是智障音响,家里没有好网千万别买...

13 月前

接近5个小时调试一个问题,在睡觉前终于fix了,太开心了,不用重编3w多个文件了

17 月前

我自己的脖子因为成天坐在电脑前有很酸痛,所以各位一定要保护好自己的颈椎吧,颈椎病这种慢性病实在是不容小觑。