Modifying query string to support parameter

Question

I have the route,

app.get("/employees", (req, res) => {
    data.getAllEmployees().then((data) => {
        res.json(data);
    }).catch(function(err) {
        console.log("An error was encountered: " + err);
    });
});

And I need to support queries such as /employees?status=value and /employees?department=value

I've created these functions in data-service.js

module.exports.getEmployeesByStatus = function(status) {
    return new Promise((resolve, reject) => {
        if (employees.length == 0) {
            reject("No results");
        } else {
            resolve(employees.status == status);
        }
    });
}
module.exports.getEmployeesByDepartment = function(department) {
    return new Promise((resolve, reject) => {
        if (employees.length == 0) {
            reject("No results");
        } else {
            resolve(employees.department == department);
        }
    });
}

How do I properly call these functions from within my GET/employees route? Im not sure where I should be specifying those queries. I reckon there must be an IF/ELSE statement somewhere, but my app.get route specifies app.get("/employees", .... Where do I add the IF/ELSE?

It seems so, I found this resource just now: stackoverflow.com/questions/6912584/… — user9399000
– user9399000, Commented Feb 23, 2018 at 18:06
But how do I modify my already-existing /employees route to support this functionality? — user9399000
– user9399000, Commented Feb 23, 2018 at 18:06

Osvaldo Maria · Accepted Answer · 2018-02-23 18:24:59Z

1

I think you need to create another handler for the "employees" path and then, you would need to check if a query string is available by using request.query, that returns an object with params from query string, and check what action should be taken from there.

something similar to:

app.get("/employees", (req, res) => {

    if(req.query.department)
         getEmployeesByStatus(req.query.department);
    else if(req.query.status)
        //and so on

});

also, you could set up a function called filter for instance, pass req.query as parameter and filter accordingly.. something like:

app.get("/employees", (req, res) => {

    if(!_.isEmpty(req.query))
        myService.filter(req.query);    
});

answered Feb 23, 2018 at 18:24

Osvaldo Maria

3715 silver badges14 bronze badges

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

3 Comments

user9399000 Over a year ago

My getEmployeesByManager() for reference: pastebin.com/4UDW5PLS

Osvaldo Maria Over a year ago

it says your link has been removed

user9399000 Over a year ago

Apologies: pastebin.com/0MGjDzAJ (disregard the err || on lines 25 and 34)

Collectives™ on Stack Overflow

Modifying query string to support parameter

1 Answer 1

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related