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

Question

[вовчек]

как можно сократить код?
Может есть более простое решение?

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);
    }
});

Comments

[WbICHA]

Для начала тут стоило бы разобраться с асинхронностью.

[Yagher]

WbICHA, намекаете на то, что код больше похож на синхронный?

[WbICHA]

Yagher,

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

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

[Yagher]

WbICHA, кажется, что map надо заменить на фурыч.

[WbICHA]

Yagher, ты ведь шутишь, да?

[Yagher]

WbICHA, а что вы хотели сказать кодом?

[Grish Poghosyan]

Он о том, что нужно из .map возвращать массив промисов и обработать их через Promise.all.
Но ещё лучше будет разобраться с запросами mongodb.

[Yagher]

Grish Poghosyan, да, похоже на то, но может имелось в виду что-то еще.

Answer 1

[Grish Poghosyan]

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!');