MongoDB重复键错误

时间:2017-10-05 07:17:41

标签: javascript node.js mongodb

所以现在,我正在开发一项服务,允许多个事件在MongoDB上存储数据。它们通过每次新事件发生时在MongoDB上创建新集合来存储事件数据。如果同一事件需要存储不同的数据集,则应创建MongoDB中的新文档。

以下代码是我为处理此问题而创建的服务。

import WhiteBoardEvent from '../model/event.model';
import IEventStore from '../interface/eventStore.interface';
import * as MongoClient from 'mongodb';

export class EventStore implements IEventStore {
    private mongoDBEndpoint = "mongodb://192.168.10.10:27017";


    public insert(event: WhiteBoardEvent, callback: (err: any) => void): void {
        MongoClient.connect(this.mongoDBEndpoint, { connectTimeoutMS: 1000 }, (connErr, db) => {
            if (connErr) { db.close(); callback(connErr); return; }
            this.getNextSequence(db, event, (err, sequence) => {
                if (err) { db.close(); callback(err); return; }
                event.sequence = sequence;
                db.collection(event.roomId).insert(event, (err) => {
                    db.close();
                    callback(err);
                });
            });
        });
    }

    private createCounterCollection(db: MongoClient.Db, event: WhiteBoardEvent, callback: (err: any) => void): void {
        db.collection("counters").insert({
            roomId: event.roomId,
            sequence: 0
        }, callback);
    }

    private getNextSequence(db: MongoClient.Db, event: WhiteBoardEvent, callback: (err: any, sequence: number) => void): void {
        var collection = db.collection("counters");
        collection.findOneAndUpdate(
            { roomID: event.roomId },
            {
                $inc: { sequence: 1 },
                // new: true
            },
            {
                upsert: true,
                returnOriginal: false
            },
            (err, r) => {
                if (err) {
                    this.createCounterCollection(db, event, (err) => {
                        if (err) { callback(err, -1); return; }
                        callback(null, 0);
                    });
                    return;
                }
                callback(null, r.value.sequence);
                console.log("counter : " + r.value.sequence);
            }
        );
    }
}

以下代码是我创建的测试文件,以便我可以看到MongoDB中的更改。

import * as timers from 'timers';
import WhiteBoardEvent from './data/model/event.model';
import { EventStore } from './data/service/eventStore.service';

var model = new WhiteBoardEvent();
model.name = "t2";
model.roomId = "testRoom";
model.timestamp = new Date();
model.userId = "";


var model2 = new WhiteBoardEvent();
model2.name = "t1";
model2.roomId = "testRoom2";
model2.timestamp = new Date();
model2.userId = "";

var eventStore = new EventStore();

var timer1 = timers.setInterval(()=>{
    eventStore.insert(model, (err)=>{
        if(err){
            console.log(err);
        }else{
            console.log("Test Completed!");

        }
    });
}, 1000);

var timer2 = timers.setInterval(()=>{
    eventStore.insert(model2, (err)=>{
        if(err){
            console.log(err);
        }else{
            console.log("Test Completed!");

        }
    });
}, 1000);

这是我得到的输出片段。在这里,"测试完成"在第一个实例中显示,之后,我收到了重复的错误。

counter : 1
counter : 1
Test Completed!
Test Completed!
counter : 2
{ MongoError: E11000 duplicate key error collection: admin.testRoom index: 
_id_ dup key: { : ObjectId('59d5da14cedd6f28a5db8c93') }

任何人都可以帮我吗?先感谢您!

2 个答案:

答案 0 :(得分:2)

您正在创建WhiteBoardEvent的两个实例,而没有明确设置ID(这很好,但相关)。看看上面代码中的这段摘录:

db.collection(event.roomId).insert(event, (err) => {
    db.close();
    callback(err);
});

event交给MongoDB的insert后,会检查它是否有ID - 但事实并非如此。因此,MongoDB代码为您生成ID(请参阅here)。这一切都很棒 - 这就是你想要的。

然而,调用setInterval回调的 next 时间会发生什么?好吧,modelmodel2现在有一个ID集 - 它是根据我刚才描述的规则设置的。在这种情况下,现在model上有一个ID设置进入insert,您尝试重用与MongoDB代码相同的ID。

在测试代码中,您只需清除eventStore.insert回调中的ID,以确保每次都生成新ID。 e.g:

eventStore.insert(model, (err)=>{
    model._id = null;

    if(err){
        console.log(err);
    }else{
        console.log("Test Completed!");
    }
});

答案 1 :(得分:-1)

在您的计划中,您可能在unique: true上设置了密钥。 使用相同的键或未填写的键添加另一个对象将导致重复键错误。因为,如果未填写字段,则将填入null。所以2次null是重复键错误。确保不会发生这种情况。

使用sparse: true代替unique: true。另请注意,unique: true字段永远不能有两个相同的键。稀疏内部只能有多个空值(未定义),并且与unique: true的内容相同。

在您的情况下,您必须将用户ID设置为“”,如果设置为唯一,则可能会导致错误。 model.userId = "";

希望这能解决你的问题。另外请告诉我们你的模特。

斯文