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 过的尽量以后不再犯。

你可能感兴趣的:(2018-07-02)