Как передать идентификаторы объектов в другую схему

У меня есть такая схема:

var MusicSchema = new Schema({
    music_genres:[{
        type:Schema.Types.ObjectId,
        ref:'music_genres',
    }],
    music: {
        type:String,
        required:true,
    },
});

теперь во внешнем интерфейсе у меня есть множественный выбор, где пользователь выбирает один или несколько разных музыкальных жанров.

so when clicked submit, sometimes I get (if user chose only one genre) - 5cab466ed076761558a76148 or if multiple - [ '5cab466ed076761558a76148', '5cab4915d076761558a7614a' ].

Итак, если пользователь выбрал только 1 жанр, это строка, но если пользователь выбрал несколько, это массив.

router.post('/',async (req,res)=>{
    const newMusic = new Music();
    if (typeof req.body.music_genres === "string") req.body.music_genres = [req.body.music_genres];

    for(var i in req.body.music_genres) newMusic.music_genres.push(req.body.music_genres[i]);  
    await newMusic.save();

Вопрос: Я ненавижу, когда я пишу оператор if и проверяя, является ли он строкой, делаю его массивом. Я также ненавижу заявления. Есть ли способ сделать этот код лучше без цикла if и for?

Стоит ли изучать PHP в 2023-2024 годах?
Стоит ли изучать PHP в 2023-2024 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
0
0
71
2
Перейти к ответу Данный вопрос помечен как решенный

Ответы 2

Если вы хотите удалить цикл for из кода, вы должны использовать функцию мангуста insertMany, как вы можете видеть в ее официальном документе здесь документ, вы можете передать массив этой функции, тогда она будет обрабатывать массовое создание.

но для вашей первой проблемы я думаю, что лучший подход - сделать тело запроса интегрированным со стороны клиента, чтобы вы могли легко сделать строку массивом с одним элементом в нем, я думаю, что это будет лучше для согласованности кода на стороне сервера.

вы можете предоставить код, как вы это сделаете? потому что я сохраняю insertMany не в своей музыкальной модели, а в моем поддокументе music.music_genres

Nika Kurashvili 08.04.2019 15:58
Ответ принят как подходящий

Вы можете использовать тройку с спред оператор вместо цикла с push()

 router.post('/',async (req,res)=>{
    const newMusic = new Music();
    typeof req.body.music_genres === "string" ? newMusic.music_genres = 
      [req.body.music_genres]: newMusic.music_genres = [...req.body.music_genres];
    await newMusic.save();

Пример

let foo = 'foo';
let arr = ['a', 'b'];
let bar = [];
a = foo;
typeof a == 'string' ? bar = [a] : bar = [...a];
console.info(bar);

Другие вопросы по теме