推荐工具
› RoboMongo
推荐书目
› 50 Tips and Tricks for MongoDB Developers
Related Blogs
› Snail in a Turtleneck
JCZ2MkKb5S8ZX9pq
V2EX  ›  MongoDB

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

  •  1
     
  •   JCZ2MkKb5S8ZX9pq · Aug 13, 2018 · 9205 views
    This topic created in 3008 days ago, the information mentioned may be changed or developed.

    pymongo
    手贱先取了个

    old = db.find_one({conditions})
    

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

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

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

    31 replies  •  2018-08-14 11:00:07 +08:00
    Perry
    Perry
    Aug 13, 2018 via iPhone
    1
    官方文档写的很清楚:

    To delete all documents in a collection, pass an empty document ({}).
    Trim21
    Trim21
    Aug 13, 2018 via Android
    2
    一直在用 delete_one, 而且这种情况一直写成{_id:old['_id']}, 从来不把变量直接传进去…
    mooncakejs
    mooncakejs
    Aug 13, 2018 via iPhone❤️1
    3
    sql 不加 where?
    vela
    vela
    Aug 13, 2018
    4
    又一次触发了吃瓜系人民群众喜闻乐见的事件…… 2333
    yulitian888
    yulitian888
    Aug 13, 2018
    5
    做这种操作难道不是应该用 upsert:true 参数的??
    reus
    reus
    Aug 13, 2018❤️11
    6
    虽然说楼主不看文档算有错
    但当初设计这种接口的人就是蠢到爆炸
    g079708
    g079708
    Aug 13, 2018 via iPhone
    7
    感谢入坑提醒
    DonaidTrump
    DonaidTrump
    Aug 13, 2018 via iPhone❤️1
    8
    @reus #6 remove 就是删除,参数是条件,你删除不加条件跟 SQL 删除不加 where 有什么区别
    lrh3321
    lrh3321
    Aug 13, 2018
    9
    好蠢,还好我只在测试环境下删记录
    reus
    reus
    Aug 13, 2018
    10
    @tulongtou 没区别啊,SQL 也一样蠢啊。
    zhicheng
    zhicheng
    Aug 13, 2018 via iPhone
    11
    SQL 的 WHERE 参数里传 null 并不会删除全部数据,至少大多数情况不会。

    你们觉得是数据库 Model 里生成不带 WHERE 的 DELETE 容易出现,还是程序里有个变量变成 null 容易出现?
    reus
    reus
    Aug 13, 2018
    12
    @tulongtou 我错了,SQL 的 DELETE 传了 id = null,一条数据都不会删除,SQL 没有 mongodb 这么蠢。这里根本不是不加条件,而是 id = old,而 old 为 null,也就是 id = null。
    shintendo
    shintendo
    Aug 13, 2018
    13
    @reus 你又错了,这里不是 id = null, 而是楼主自信认为 old 里面包含 id = XXX
    glues
    glues
    Aug 13, 2018
    14
    mongodb 你也敢用?
    reus
    reus
    Aug 13, 2018
    15
    @shintendo 这样…
    ilaipi
    ilaipi
    Aug 13, 2018
    16
    奇怪了,都没有人觉得 删集合和删库不一个概念?
    noNOno
    noNOno
    Aug 13, 2018
    17
    太危险了.
    感觉这接口开发者对 null 的理解有点弱.
    BOYPT
    BOYPT
    Aug 13, 2018
    18
    尽然.find()返回全库,.remove()删除全库挺合理的。
    batter
    batter
    Aug 13, 2018
    19
    只 insert,update,不 delete,remove
    JCZ2MkKb5S8ZX9pq
    JCZ2MkKb5S8ZX9pq
    OP
    Aug 13, 2018
    20
    @ilaipi 对的 我说笼统了 是删了个 collection
    JCZ2MkKb5S8ZX9pq
    JCZ2MkKb5S8ZX9pq
    OP
    Aug 13, 2018
    21
    @yulitian888
    @batter
    想更新 _id,这样省掉一个标记更新时间的字段。
    JCZ2MkKb5S8ZX9pq
    JCZ2MkKb5S8ZX9pq
    OP
    Aug 13, 2018
    22
    @Perry
    @BOYPT
    pymongo 在大部分情况下,类型稍有不对就会报错的。如果它能考虑,区分一下 {} 和 None。
    db.find() 返全部,db.find({}) 也可以理解(不设条件),但 db.find(None) 还是有点意外。
    460881773
    460881773
    Aug 13, 2018
    23
    你如果你 没有条件,就是 删除全部。所以一定要加条件
    Trumeet
    Trumeet
    Aug 13, 2018 via Android
    24
    然后再跑路?(雾
    bytewalk
    bytewalk
    Aug 13, 2018 via iPhone
    25
    同意六楼😂
    wngwei
    wngwei
    Aug 13, 2018
    26
    @glues mongodb 咋的了……
    schemesAlot
    schemesAlot
    Aug 13, 2018
    27
    还好不会 mongo,只用 mongoose
    wdlth
    wdlth
    Aug 13, 2018 via Android
    28
    论伪删除的重要性
    glues
    glues
    Aug 13, 2018 via iPhone
    29
    @wngwei 坑太多,很容易导致你跑路
    wngwei
    wngwei
    Aug 14, 2018
    30
    @glues 😂原来是这样
    pabupa
    pabupa
    Aug 14, 2018
    31
    🤣
    About   ·   Help   ·   Advertise   ·   Blog   ·   API   ·   FAQ   ·   Privacy   ·   Solana   ·   1116 Online   Highest 6679   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 62ms · UTC 23:52 · PVG 07:52 · LAX 15:52 · JFK 18:52
    ♥ Do have faith in what you're doing.