V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
推荐工具
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 · 8642 views
    This topic created in 2825 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
        1
    Perry  
       Aug 13, 2018 via iPhone
    官方文档写的很清楚:

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

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