2018-07-02

Objective:事实

写了几个小票,暂时没有遇到什么问题。倒是改了几个上周的 code review,有好几个地方需要改正。关于性能的:判断记录是否存在,使用 !user.orders.exists? 避免使用 user.orders.blank?,因为 exists? 的 sql 只返回 true false,而 blank 这个会加载数据集。

image.png

使用 += 操作字符串,不如 . concat 方法,因为前面会产生两个 object,concat 只会产生一个 object。

另外很多是命名问题,还有写测试的描述写的不明确的问题。

Reflective:感受

上面这些其实之前都是知道的,但是写代码时并没有形成习惯,习惯性的写成更好的解法。以后这方面的尽量不再被 code review 才发现,自己就写好。

Interpretive:想法

写代码加入更多的思考或许可以增进习惯的形成。

Decisional : 决定

之后再写代码时,review 过的尽量以后不再犯。

©著作权归作者所有,转载或内容合作请联系作者
【社区内容提示】社区部分内容疑似由AI辅助生成,浏览时请结合常识与多方信息审慎甄别。
平台声明:文章内容(如有图片或视频亦包括在内)由作者上传并发布,文章内容仅代表作者本人观点,简书系信息发布平台,仅提供信息存储服务。

相关阅读更多精彩内容

  • 1.数组 1.1 数组构造1.直接创建的方式 --> var arr = [1,2,3];2.构造函数方式 -->...
    无聊的凡人阅读 323评论 0 0
  • 1. 器官分割 HyperDense-Net: A hyper-densely connected CNN for...
    EdwardMa阅读 2,349评论 0 9
  • 1. tf.Variable与tf.get_variable tensorflow提供了通过变量名称来创建或者获取...
    鲁大帅阅读 327评论 0 0
  • 自从和爸爸的谈话之后,我有了很深的感悟,知道自己应该怎么做了,可是也知道人会反弹,现在我就感觉我的行为又反弹...
    秦子轩阅读 161评论 2 1
  • 什么是用户运营? 之前在专注于陌生人的社交软件陌陌待了两年,也算是我毕业后真正意义上的工作,作为一个之前从来没有接...
    大班尼banny阅读 1,315评论 0 12

友情链接更多精彩内容