Как запушить в массив модели объект, только если его там нет?

Question

[VegasChickiChicki]

Есть вот такая вот схема:

const chat = new schema({
  name: {
    type: String,
    required: true,
    unique: true,
  },
  users: [{
    name: {
      type: String,
      required: true,
      unique: true,
    }
  }],
  messages: [{
    user: {
      type: String,
      required: true,
    },
    message: {
      type: String,
      required: true,
    },
    date: {
      type: String,
      required: true,
    }
  }]
}, {
  versionKey: false,
  collation: 'chat'
});

Есть запрос, пользователь ищет чаты, если он их находит, то он может к ним присоединиться(к 1 за раз(за запрос)).

Выглядит это примерно так:

router.post('/join', async (request, response) => {
  models.chat.findOneAndUpdate({
    name: request.body.data.chat.name
  }, {
    $push: {
      users: {
        name: request.body.data.user.name,
      }
    }
  }, {
    new: true,
    upsert: true,
  }, (error, chat) => {
    if (!error){
      response.json({
        status: true,
        body: {
          chat
        },
      });

      //console.log('Chat update! chat: ', chat);
    } else {
      //console.log('error: ', error);
    }
  });
});

Не могу понять, могу ли я как то указать, что если в массиве users уже есть объект с именем, которое приходит в запросе, то ничего делать не нужно?

Answer 1

[Алексей Ярков]

Меняем $push на $addToSet и вуаля ))

Comments

[VegasChickiChicki]

Странно... Может я что-то не так делаю. Заменил, документацию почитал, то что нужно... Но не работает, все равно добавляет... Заметил, что добавляет с разными _id, может в этом дело?...

Алексей Ярков

[Алексей Ярков]

VegasChickiChicki, а если в схеме так указать?

users: [{
    name: {
      type: String,
      required: true,
      unique: true,
    },
    _id: false
  }]

Тогда по идее в users[0] будет только свойство name, но это не точно ))

[VegasChickiChicki]

Алексей Ярков, Если так указать, то mongo начинает ругаться, при создании новой модели с таким же пользователем... То есть если создать 2 разных чата, в которых будет один и тот же пользователь... На второй уже начнет ругаться, скажет, что модель(чат) с таким пользователем уже существует...

[Алексей Ярков]

VegasChickiChicki, ну тогда видимо придётся указывать ещё и _id при вставке:

$addToSet: {
      users: {
        name: request.body.data.user.name,
        _id: "321321321321321321"
      }
    }

[VegasChickiChicki]

Алексей Ярков, Смог решить проблему, скорее всего это было из-за айди. Нужен совет) Случится ли что нибудь страшное, если у пользователей не будет айди уникального, только имена? Чисто теоретически, существовать одинаковых пользователей не может...

[Алексей Ярков]

VegasChickiChicki, если name уникальный, то по идее ничего

[VegasChickiChicki]

const chat = new schema({
  name: {
    type: String,
    required: true,
    unique: true,
  },
  users: [{
    name: {
      type: String,
      required: true,
    },
    _id: false
  }],
  messages: [{
    user: {
      type: String,
      required: true,
    },
    message: {
      type: String,
      required: true,
    },
    date: {
      type: String,
      required: true,
    }
  }]
}, {
  versionKey: false,
  collation: 'chat'
});

Решение проблемы.

[VegasChickiChicki]

Алексей Ярков, Отлично, спасибо, вы очень помогли)

[Алексей Ярков]

VegasChickiChicki, да не за что )) Я про $addToSet сам узнал час назад )) Сижу, пишу бэкенд для сайта и вот понадобилось.