Inserting data into MongoDB with mgo

Question

I'm trying to insert some data in MongoDB using Go.

Here is the data struct:

type Entry struct {
    Id          string `json:"id",bson:"_id,omitempty"`
    ResourceId  int    `json:"resource_id,bson:"resource_id"`
    Word        string `json:"word",bson:"word"`
    Meaning     string `json:"meaning",bson:"meaning"`
    Example     string `json:"example",bson:"example"`
}

This is my insert function:

func insertEntry(db *mgo.Session, entry *Entry) error {
    c := db.DB(*mongoDB).C("entries")
    count, err := c.Find(bson.M{"resourceid": entry.ResourceId}).Limit(1).Count()
    if err != nil {
        return err
    }
    if count > 0 {
        return fmt.Errorf("resource %s already exists", entry.ResourceId)
    }
    return c.Insert(entry)
}

And finally, this is how I call it:

entry := &Entry{
    ResourceId:  resourceId,
    Word:        word,
    Meaning:     meaning,
    Example:     example,
}
err = insertEntry(db, entry)
if err != nil {
    log.Println("Could not save the entry to MongoDB:", err)
}

The trouble is, I was expecting my bson tags to magically work, but they don't. Instead of data being saved as:

{ "_id" : ObjectId("53700d9cd83e146623e6bfb4"), "resource_id" : 7660708, "word" : "Foo" ...}

It gets saved as:

{ "_id" : ObjectId("53700d9cd83e146623e6bfb4"), "id" : "", "resourceid" : 7660708, "word" : "Foo"...}

How can I fix this?

TrevorSStone · Accepted Answer · 2014-05-12T02:56:13.617

15

Change entry to:

type Entry struct {
    Id          string `json:"id" bson:"_id,omitempty"`
    ResourceId  int    `json:"resource_id" bson:"resource_id"`
    Word        string `json:"word" bson:"word"`
    Meaning     string `json:"meaning" bson:"meaning"`
    Example     string `json:"example" bson:"example"`
}

The syntax for Struct Tags does not use commas between tags. I believe this should fix it.

edited May 12 '14 at 02:56

answered May 12 '14 at 02:40

TrevorSStone

421
3
8

this is wrong. not only does Id need to be of type bson.ObjectId but the bson tags aren't necessary except for ResourceId which does not get autoconverted to "resource_id". Uppercase struct fields get lowercased by mgo. Only the json lowercase field name is needed. – May 12 '14 at 11:47
1

@dalu The _id can actually be of any type. Edit: Well, unless the OP did expect an ObjectId, which isn't entirely clear from the question. – Gustavo Niemeyer May 12 '14 at 13:24
@dalu as Gustavo said, the _id does not need to be an ObjectId, That is the default, but not always the required choice. Also, nothing is wrong. I personally think that including the bson struct tag is better practice in the long run even if it is the same. Imagine if a future dev comes along and decides to change the variable "Word" to "Title". Without defining the bson tag, they would be altering the datastore without realizing it. Keeping a bson tag helps maintenance down the line IMO and at worse, makes you type a few more characters for clarity. – TrevorSStone May 12 '14 at 20:07

score 8 · Answer 2 · 2014-05-12T02:55:50.773

8

type Entry struct {
    Id          bson.ObjectId `bson:"_id,omitempty" json:"id"`
    ResourceId  int           `json:"resource_id" bson:"resource_id"`
    Word        string        `json:"word"`
    Meaning     string        `json:"meaning"`
    Example     string        `json:"example"`
}

Instead of Count() and Insert() you can use UpsertId which does just that, if an Id exists the record is replaced if not it's inserted.

Insert() with an empty ObjectId lets MongoDB handle Id assignment.

Edit: Misread your Count query. You also have an error in there. It should be "resource_id" not "resourceid" because you declared that the bson field is named "resource_id"

edited May 12 '14 at 02:55

answered May 12 '14 at 02:49

The way how `insertEntry` works may seem counter-intuitive, but it makes sense in the way the program works. – if __name__ is None May 12 '14 at 02:54
yes, corrected that. I thought you were checking if an objectId existed then insert a new record. – May 12 '14 at 03:00
1

wtf that's the correct answer. Id needs to be bson.ObjectId – May 12 '14 at 11:44
If we dont insert object id mongo will create a default _id parameter – Himanshu Jan 03 '18 at 11:02

score 0 · Answer 3 · answered May 11 '19 at 23:19

Update your Entry struct as:

type Entry struct {
    Id          string `bson:"_id"`
    ResourceId  int    `json:"resource_id,omitempty"`
    Word        string `json:"word,omitempty"`
    Meaning     string `json:"meaning,omitempty"`
    Example     string `json:"example,omitempty"`
}

This should work!

Inserting data into MongoDB with mgo

3 Answers3