vovvkka
@vovvkka

Как сократить код?

как можно сократить код?
Может есть более простое решение?
router.delete('/:id', [auth, permit('admin')], async (req, res) => {
    const artist = await Artist.findById(req.params.id);

    if (!artist) {
        return res.status(404).send({error: 'Artist not found!'});
    }

    const album = await Album.find({artist: artist._id});

    if (!album) {
        return res.status(404).send({error: 'Album not found!'});
    }

    let allTracks = [];

    await Promise.all(album.map(async el => {
        const track = await Track.find({album: el._id});

        allTracks = [...allTracks, ...track];
    }));


    try {
        allTracks.map(async track => {
            await TrackHistory.deleteMany({track: track._id});
        });

        album.map(async el => {
            await Track.deleteMany({album: el._id});
        });

        await Album.deleteMany({artist: artist._id});
        await Artist.deleteOne({_id: artist._id});

        res.send('Delete successful!');
    } catch (e) {
        res.status(500);
    }
});
  • Вопрос задан
  • 138 просмотров
Решения вопроса 1
@hovsepyann
Software Development Engineer at Softconstruct
const { id: artistId } = req.params;

const albums = await Album.find({artist: artist._id});

if (!albums?.length) {
  return res.status(404).send({error: 'Album or artist not found!'});
}

const albumsIds = albums.map(album => album._id);

const allTracks = await Track.find({ album: { $in: albumsIds } });

if (!allTracks?.length) {
  return res.status(404).send({error: 'Tracks not found!'});
}

const tracksIds = allTracks .map(track=> track._id);

await TrackHistory.deleteMany({ track: { $in: tracksIds } });
await Track.deleteMany({ album: { $in: albumsIds } });

await Album.deleteMany({artist: artistId});
await Artist.deleteOne({ _id: artistId});

res.send('Delete successful!');
Ответ написан
Комментировать
Пригласить эксперта
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы