Trying to limit my results and sort my data, from MongoDB using NodeJS

Question

I am using MongoDB and Node.JS, I am trying to get data out of my MongoDB and show into my html page which I have working with the below code however this just brings back ALL entries in no particular order:

server.js

// This is for getting the list of all players from my DB
app.get("/getPlayers", function(request, response) {
    db.getPlayers().then(function(players){
        console.log(players);
        response.send(players);
    });
});

leadership.html

<script>
    $(function() {
        $.get("http://localhost:9000/getPlayers", {}, function (res) {
        let data = res;
        console.log(res);

        for (i = 0; i < data.length; i++) {
            let name = data[i].name;
            let score = data[i].score;

            console.log(data[i].name);


            $("#leadership").append("<tr><td class=\"name\">"
                + data[i].name + "</td><td class=\"score\">" 
                + data[i].score + "</td></tr>");
         }
     });
});
</script>

After looking at W3 Schools I tried to alter the code to this:

db.getPlayers().sort().limit(10).then(function(players)

However my Chrome console brings back an internal server error 500. Can someone point out how I can sort by LARGEST NUMBER first, then LIMIT the results to say 10? Within the database there is a collection called players, which holds name and score

db.js

var Player = mongoose.model("Player", {name: String, score: Number}); 

    module.exports.Player = Player;

"my Chrome console brings back an internal server error 500" - what does your server log say? — Sergio Tulentsev
– Sergio Tulentsev, Commented Jan 1, 2020 at 14:47
@SergioTulentsev It says Reference error: score is not defined — Sunny0101
– Sunny0101, Commented Jan 1, 2020 at 14:50
@prasad_ I need to sort on the score field and then have highest at the top, and then limit those results to say 10? — Sunny0101
– Sunny0101, Commented Jan 1, 2020 at 14:51

Daniel Skoroś · Accepted Answer · 2020-01-01 14:49:45Z

2

Try something like this. Order, sort and limit can be passed from front end or change default values after : mark.

Players is imported model, you can do it this way or use method in the model itself.

app.post('/api/players',(req,res)=>{

    let order = req.body.order ? req.body.order : "-1";
    let sortBy = req.body.sortBy ? req.body.sortBy : "_id";
    let limit = req.body.limit ? parseInt(req.body.limit) : 100;      

    Players.
    find().
    sort([[sortBy,order]]).
    limit(limit).
    exec((err,players)=>{
        if(err) return res.status(400).send(err);
        res.status(200).json({
            size: players.length,
            players
        })
    })
})

answered Jan 1, 2020 at 14:49

Daniel Skoroś

604 bronze badges

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

1 Comment

Sergio Tulentsev Over a year ago

Can be passed, but shouldn't (not in this unchecked manner, at least). With this, you yourself are opening the app for a DoS attack.

Collectives™ on Stack Overflow

Trying to limit my results and sort my data, from MongoDB using NodeJS

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related