andimiya / articles-products-and-express Goto Github PK

For better code organization, instead of those loops in the API requests, you can refactor data retrieval into the Data Store itself, and have the routes call the function. For example, you can create the following functions in the Data Store files:

1. getAllProducts()
2. getProductById(id)
3. createProduct()
4. updateProductById(id)
5. deleteProductById(id)

Then your express routes just calls those functions on the data store file. That way all the logic is abstracted away into the data store file. This is a common pattern.

Just a code readability tip:

You can refactor something like:
db.editArticle(req.params.title, req.body.body, req.body.author);

into:
`let title = req.params.title;
let body = req.body.body;
let author = req.body.author

db.editArticle(title, body, author)`

This is a common pattern to make it a bit more readable.