Loop over a nested array of objects

Question

I am trying to load the following array with 2 object in the below league model.

The array that I want to load:

module.exports = [
    {
        title: "Osdorp StraatVoetbal",
        location: "Osdorp",
        city: "Amsterdam",
        lat: 52.3599,
        lon: 4.7891,
    },
    {
       title: "Slotervaart StraatVoetbal",
       location: "Slotervaart",
       city: "Amsterdam",
       lat: 52.3562,
       lon: 4.8273,
    }
];

The League model:

const mongoose = require('mongoose');
const Schema = mongoose.Schema;

const LeagueSchema = new Schema ({
    title: String,
    location: String,
    city: String,
    lat: Number,
    lon: Number
});

module.exports = mongoose.model('League', LeagueSchema);

I am using the following function to load the array, but getting TypeError.

const mongoose = require('mongoose');
const League = require('../models/league');
const leagues = require('./leagues');

mongoose.connect('mongodb://localhost:27017/footballeagues', {
    useNewUrlParser: true,
    useUnifiedTopology: true,
});

const db = mongoose.connection
db.on('error', console.error.bind(console, 'connection error:'));
db.once('open', ()=>{
    console.log("Mongoose Database connected")
});

const seedDB =  async() => {
    leagues.forEach( async (i)=>{
        const seedLeague = new League({
            title : `${leagues[i].title}`,
            location : `${leagues[i].location}`,
            city: `${leagues[i].city}`, 
            lat: `${leagues[i].lat}`,
            lon: `${leagues[i].lon}`
        })
        await seedLeague.save();
    })
}

seedDB().then( ()=>{
    mongoose.connection.close;
    //console.log('connection closed')
})

Could someone please help and advise me how to fix the below error:

Aaron Cicali · Accepted Answer · 2021-10-22 22:45:53Z

1

The issue is that your code expects an array index as the parameter to your forEach() callback, but what it actually receives is the array value.

Here is working code that will solve your issue:

    const seedDB =  async() => {
        leagues.forEach( async (element)=>{
            const seedLeague = new League({
                title : `${element.title}`,
                location : `${element.location}`,
                city: `${element.city}`, 
                lat: `${element.lat}`,
                lon: `${element.lon}`
            })
            await seedLeague.save();
        })
    }

edited Oct 22, 2021 at 22:45

Aaron Cicali

1,55615 silver badges30 bronze badges

answered Oct 22, 2021 at 22:31

Hussein Mohamed

1691 silver badge10 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

Zizo Over a year ago

Thank you that worked :)

Henry Ecker Over a year ago

@Zizo If this answer resolved your problem, you are able to accept it as the answer to your question, by clicking the checkmark to the left. (See What does it mean to accept an answer? for information) This will let others know this question has been solved.

Collectives™ on Stack Overflow

Loop over a nested array of objects

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related