Objective:事实
写了几个小票,暂时没有遇到什么问题。倒是改了几个上周的 code review,有好几个地方需要改正。关于性能的:判断记录是否存在,使用 !user.orders.exists? 避免使用 user.orders.blank?,因为 exists? 的 sql 只返回 true false,而 blank 这个会加载数据集。
使用 += 操作字符串,不如 . concat 方法,因为前面会产生两个 object,concat 只会产生一个 object。
另外很多是命名问题,还有写测试的描述写的不明确的问题。
Reflective:感受
上面这些其实之前都是知道的,但是写代码时并没有形成习惯,习惯性的写成更好的解法。以后这方面的尽量不再被 code review 才发现,自己就写好。
Interpretive:想法
写代码加入更多的思考或许可以增进习惯的形成。
Decisional : 决定
之后再写代码时,review 过的尽量以后不再犯。