程序犯了好些小错误，出事了，现在坐立不安

[kankana]

有个feature是要求用户suspended 30天后,将此用户的所有数据给删掉. 30天内是可以恢复的。

我直接队列，将该任务延迟到一个月后。

现在才发现用户提前被删除了。

原来，延迟参数，我传递的是30*24*60， 实际上，时间单位是秒。。。

现在用户要恢复，结果数据全没了。。。

很对不起老板。。。搞砸了。。。

---

lessons learned：

1, 加强测试。

2, 定时备份数据库。

[evlos]

>> 只是，有些敏感数据，是属于我们的，不希望用户退款后,能够再利用到这些数据.

为什么不直接禁止用户访问呢？

[aaronlam]

过个好年，有错就改

[873681136]

我自己也在维护一个OJ，我的做法是添加一个锁定字段，删除操作并不实际删除，而是在锁定字段打个标记，其它数据查询都优先查询非锁定的标记，这样到时恢复数据就好办多了。

[overflow]

永远不要轻易删除

[chinafeng]

@kankana 如果按照你这种心理，那我觉得我都能天台见了，最高纪录10分钟编译正式生产3次...调整好自己的心态，就算是国家的天河计算机，前几天都还有漏洞能控制大部分超算，更何况这些小产品小公司(相对国家来说，这些应该都算小了吧)

[sb]

摸摸…学习下各种教训

[zungmou]

对于运行的系统，从不去删除数据，只有空间不足时通过数据维护的功能对距离时间最久的数据进行删除。

[Felldeadbird]

尽量逻辑删除。免得到时候出问题就麻烦大了。

[surewen]

在我司，这种错误犯两次就得走人。

[Phariel]

learned => learnt

[kankana]

@Phariel 欢迎歪楼,欢迎google

[kankana]

@chinafeng 标题说"好些",但内容我只写了一个. 唉,太马虎了老实说. 完全没经验. 代码部署前, 生产环境得做调整,完全没考虑到...

[Elethom]

想問問 po 主在哪家公司⋯⋯

[realityone]

我是把记录移到另一个表里，专门记录被标记删除的数据

[juneszh]

*个毛线啊 3600 86400 这些“常量”都不记还怎么当程序员的 回去打屁股吧

[kmvan]

楼主是无证编程还是酒后编程？

[palytoxin]

以前公司自己的框架，数据库curd操作都做了封装，所有表都有个is_deleted字段
服务器的数据不可能也千万不要真的删除。
老板人挺好，新年加油

[sharpnk]

"30*24*60" code review is for shit like this. If senior members of your team failed to spot errors like this during the code review, they should be partially responsible too.

[qw7692336]

@sumhat 或许比较机密的数据，真的要删除

[msg7086]

@Phariel
‘Learnt’ or ‘learned’?
These are alternative forms of the past tense and past participle of the verb learn. Both are acceptable, but learned is often used in both British English and American English, while learnt is much more common in British English than in American English. [1]

[1]: http://www.oxforddictionaries.com/words/learnt-vs-learned

@kankana
这就是测试不到位啦。
每次完成一个功能就要写对应的测试。
比如写完这个「30天后删除」，就必须要测试它。

Timecop.travel(29.days.ago) do
@u = User.new
@u.suspend
end
User.clean_suspended
expect(User.find(@u).count).to eq(1)

Timecop.travel(31.days.ago) do
@u = User.new
@u.suspend
end
User.clean_suspended
expect(User.find(@u).count).to eq(0)

另外，如果mysql有做binlog的话，是可以重放还原的。