MongoDB-GO向同类型数组追加元素时遇类型转换Panic问题

错误原因 #

panic的直接原因是result.UpsertedID为nil时强行做类型转换。UpdateOne方法只有在**实际执行了upsert操作（即原文档不存在，新插入了文档）**时，UpsertedID才会有值；如果是更新已存在的文档，UpsertedID是nil，此时直接result.UpsertedID.(primitive.ObjectID)就会触发类型转换panic。

另外你代码注释里写了"if it doesn't exists it will be created"，但默认UpdateOne不会执行upsert，必须显式开启该选项，否则原文档不存在时不会创建新文档，更新操作会无效果。

修复步骤 #

1. 开启Upsert选项：调用UpdateOne时传入options.Update().SetUpsert(true)，确保文档不存在时自动创建。
2. 安全获取更新后的文档ID：判断UpsertedID是否为nil，如果不为nil则用它，否则用传入的原organizationID。
3. 处理GetOne方法的错误：原代码忽略了GetOne的错误，需要返回该错误避免潜在问题。

修改后的代码 #

func (o *Organization) AddMembers(organizationID string, members []Member) (*Organization, error) {
    collection := client.Database("app-data").Collection("organizations")

    log.Println("[APP-DATA-DB:ADD-MEMBERS] Requested new add members: ", members, organizationID)

    change := bson.M{
        "$push": bson.M{
            "members": bson.M{"$each": members},
        },
        "$setOnInsert": bson.M{ // 可选：upsert时设置创建相关的默认字段
            "owner":      "", // 根据业务逻辑传入实际默认值或参数
            "created_at": time.Now(),
        },
    }

    id, err := primitive.ObjectIDFromHex(organizationID)
    if err != nil {
        log.Println("[APP-DATA-DB:ADD-MEMBERS] cannot convert to primitive: ", organizationID)
        return nil, err
    }

    filter := bson.D{{"_id", id}}
    // 开启Upsert选项
    opts := options.Update().SetUpsert(true)
    result, err := collection.UpdateOne(context.TODO(), filter, change, opts)
    if err != nil {
        return nil, err
    }

    // 确定查询用的ID
    var queryID string
    if result.UpsertedID != nil {
        // 执行了upsert，使用新插入的文档ID
        queryID = result.UpsertedID.(primitive.ObjectID).Hex()
    } else {
        // 执行了更新，使用原组织ID
        queryID = organizationID
    }

    // 获取更新后的文档并处理错误
    updatedOrg, err := o.GetOne(queryID)
    if err != nil {
        log.Println("[APP-DATA-DB:ADD-MEMBERS] failed to get updated organization: ", err)
        return nil, err
    }

    return updatedOrg, nil
}

额外说明 #

• 如果业务逻辑不允许upsert（仅允许给已存在的组织添加成员），可以去掉SetUpsert(true)选项，直接用原organizationID查询即可，还可以在更新前先检查文档是否存在，避免无效操作。
• $setOnInsert是可选配置，仅在upsert创建新文档时生效，可用于设置创建时间、默认所有者等初始化字段，不需要的话可以删除该部分。

内容的提问来源于stack exchange，提问作者yieniggu