mongodb 里 remove(None) 竟然是删库(＃°Д°)

[JCZ2MkKb5S8ZX9pq]

pymongo
手贱先取了个

old = db.find_one({conditions})

然后想更新 _id，就想删旧的再 insert，感觉旧的应该含 _id ，于是：

db.remove(old)
db.insert(new)

大意了 old 会取到 None，然后再 remove，竟然是删库……
还好损失不算太大，但第一次碰到，吐槽下。

[Perry]

官方文档写的很清楚：

To delete all documents in a collection, pass an empty document ({}).

[Trim21]

一直在用 delete_one, 而且这种情况一直写成{_id:old['_id']}, 从来不把变量直接传进去…

[mooncakejs]

sql 不加 where?

[boywang004]

又一次触发了吃瓜系人民群众喜闻乐见的事件…… 2333

[yulitian888]

做这种操作难道不是应该用 upsert：true 参数的？？

[reus]

虽然说楼主不看文档算有错
但当初设计这种接口的人就是蠢到爆炸

[g079708]

感谢入坑提醒

[tulongtou]

@reus #6 remove 就是删除，参数是条件，你删除不加条件跟 SQL 删除不加 where 有什么区别

[lrh3321]

好蠢，还好我只在测试环境下删记录

[reus]

@tulongtou 没区别啊，SQL 也一样蠢啊。

[zhicheng]

SQL 的 WHERE 参数里传 null 并不会删除全部数据，至少大多数情况不会。

你们觉得是数据库 Model 里生成不带 WHERE 的 DELETE 容易出现，还是程序里有个变量变成 null 容易出现？

[reus]

@tulongtou 我错了，SQL 的 DELETE 传了 id = null，一条数据都不会删除，SQL 没有 mongodb 这么蠢。这里根本不是不加条件，而是 id = old，而 old 为 null，也就是 id = null。

[shintendo]

@reus 你又错了，这里不是 id = null， 而是楼主自信认为 old 里面包含 id = XXX

[glues]

mongodb 你也敢用？

[reus]

@shintendo 这样…

[ilaipi]

奇怪了，都没有人觉得 删集合和删库不一个概念？

[noNOno]

太危险了.
感觉这接口开发者对 null 的理解有点弱.

[BOYPT]

尽然.find()返回全库，.remove()删除全库挺合理的。

[batter]

只 insert，update，不 delete，remove

[JCZ2MkKb5S8ZX9pq]

@ilaipi 对的 我说笼统了 是删了个 collection

[JCZ2MkKb5S8ZX9pq]

@yulitian888
@batter
想更新 _id，这样省掉一个标记更新时间的字段。

[JCZ2MkKb5S8ZX9pq]

@Perry
@BOYPT
pymongo 在大部分情况下，类型稍有不对就会报错的。如果它能考虑，区分一下 {} 和 None。
db.find() 返全部，db.find({}) 也可以理解（不设条件），但 db.find(None) 还是有点意外。

[460881773]

你如果你 没有条件，就是 删除全部。所以一定要加条件

[Trumeet]

然后再跑路？（雾

[poorcai]

同意六楼😂

[wngwei]

@glues mongodb 咋的了……

[leafiy]

还好不会 mongo，只用 mongoose

[wdlth]

论伪删除的重要性

[glues]

@wngwei 坑太多，很容易导致你跑路

[wngwei]

@glues 😂原来是这样

[pabupa]

🤣